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

Added test runner for all examples if live credentials are present #357

Merged
1 commit merged into from May 16, 2022
Merged

Added test runner for all examples if live credentials are present #357

1 commit merged into from May 16, 2022

Conversation

ghost
Copy link

@ghost ghost commented May 14, 2022

Please describe the change you are making

This PR adds a test runner that iterates over all example directories and runs them against the oVirt Terraform provider if the OVIRT_URL, OVIRT_USERNAME, and OVIRT_PASSWORD environment variables are set.

Some of these don't pass right now, these need to be investigated.

Are you the owner of the code you are sending in, or do you have permission of the owner?

Yes

The code will be published under the BSD 3 clause license. Have you read and understood this license?

Yes

@ghost ghost requested review from engelmi and eslutsky May 14, 2022 14:29
@ovirt-infra
Copy link

All tests passed

7 similar comments
@ovirt-infra
Copy link

All tests passed

@ovirt-infra
Copy link

All tests passed

@ovirt-infra
Copy link

All tests passed

@ovirt-infra
Copy link

All tests passed

@ovirt-infra
Copy link

All tests passed

@ovirt-infra
Copy link

All tests passed

@ovirt-infra
Copy link

All tests passed

@ghost ghost marked this pull request as ready for review May 16, 2022 09:54
@ovirt-infra
Copy link

All tests passed

@ghost ghost merged commit 26c6773 into oVirt:main May 16, 2022
@ghost ghost deleted the test-examples branch May 16, 2022 10:19
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants