Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add main script #1

Merged
merged 1 commit into from
Sep 22, 2021
Merged

Add main script #1

merged 1 commit into from
Sep 22, 2021

Conversation

antoniotorresm
Copy link
Collaborator

Add initial script with two tests implemented:

  • EnrollmentTest: try to enroll a number of clients simultaneously
  • APITest: run a IPA CLI command a number of times simultaneously

This requires the host to have vagrant, libvirt, and ansible available
in order to run.

Signed-off-by: Antonio Torres antorres@redhat.com

@antoniotorresm
Copy link
Collaborator Author

@rcritten @fcami

@rcritten
Copy link
Contributor

rcritten commented Sep 14, 2021

Can you add this as the script header:

#
# Copyright (C) 2021 FreeIPA Contributors see COPYING for license
#

I think this can be GPLv3 so add that COPYING file to the root as well.

@rcritten
Copy link
Contributor

Can you also update the README with information on how to execute the tests?

@antoniotorresm
Copy link
Collaborator Author

Updated PR:

  • Added README
  • Added COPYING
  • Added gitignore
  • Added IP address generator for private network
  • Added support for passing private keys (some Vagrant images need a certain key to log in)
  • Increased size of management network (so that we can have more than 255 clients)

@antoniotorresm antoniotorresm force-pushed the initial-pr branch 2 times, most recently from c88396c to b16435d Compare September 22, 2021 09:42
Add initial script with two tests implemented:

* EnrollmentTest: try to enroll a number of clients simultaneously
* APITest: run a IPA CLI command a number of times simultaneously

This requires the host to have vagrant, libvirt, and ansible available
in order to run.

Signed-off-by: Antonio Torres <antorres@redhat.com>
@rcritten
Copy link
Contributor

ack, looks good to me.

@rcritten rcritten merged commit 5963af5 into freeipa:master Sep 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants