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

[NEW]: Implement opentelekomcloud_config for rancher2_machine_config_v2 #5

Merged
merged 6 commits into from
Apr 3, 2024

Conversation

artem-lifshits
Copy link
Member

@artem-lifshits artem-lifshits commented Apr 2, 2024

Summary of the Pull Request

Added support for OpenTelekomCloud for rancher2_machine_config_v2 resource.

PR Checklist

Acceptance tests setup

Following minimal env vars are needed to run tests:

RANCHER_INSECURE=false;
RANCHER_TOKEN_KEY=token-XXXXXXXXXX;
RANCHER_URL=https://rancher-test.XXXXXX.in;
TF_ACC=1

Acceptance Steps Performed

2024/04/02 14:42:11 Getting from https://rancher-test.XXXXXXXin/ping
2024/04/02 14:42:11 Time to get req: 219 ms
=== RUN TestAccRancher2MachineConfigV2Basic
--- PASS: TestAccRancher2MachineConfigV2Basic (7.68s)
PASS

Process finished with the exit code 0

@artem-lifshits artem-lifshits self-assigned this Apr 2, 2024
Copy link
Member

@anton-sidelnikov anton-sidelnikov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job

@artem-lifshits artem-lifshits added the gate Merge PR label Apr 3, 2024
Copy link

otc-zuul bot commented Apr 3, 2024

Build succeeded.
https://zuul.otc-service.com/t/eco/buildset/ae2c6de5999d4e9fa490ddb117e46eb5

✔️ lint SUCCESS in 2m 59s
✔️ vet SUCCESS in 2m 40s
✔️ test SUCCESS in 2m 55s
✔️ goreleaser-build SUCCESS in 11m 33s

@otc-zuul otc-zuul bot merged commit 6823dea into master Apr 3, 2024
2 checks passed
@otc-zuul otc-zuul bot deleted the machine_config branch April 3, 2024 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gate Merge PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants