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

[AutoScaling] refactoring resource/opentelekomcloud_as_* #1963

Merged
merged 3 commits into from Oct 17, 2022
Merged

Conversation

anton-sidelnikov
Copy link
Member

@anton-sidelnikov anton-sidelnikov commented Oct 14, 2022

Summary of the Pull Request

PR Checklist

  • Tests added/passed.
  • Documentation updated.
  • Schema updated.
  • Release notes added.

Acceptance Steps Performed

=== RUN   TestAccASPolicyV2_basic
=== PAUSE TestAccASPolicyV2_basic
=== CONT  TestAccASPolicyV2_basic
--- PASS: TestAccASPolicyV2_basic (326.88s)
=== RUN   TestAccASPolicyV2_withSize
=== PAUSE TestAccASPolicyV2_withSize
=== CONT  TestAccASPolicyV2_withSize
--- PASS: TestAccASPolicyV2_withSize (246.98s)
=== RUN   TestAccASPolicyV2_conflictsAction
--- PASS: TestAccASPolicyV2_conflictsAction (99.42s)
PASS

=== RUN   TestAccASV1Policy_basic
=== PAUSE TestAccASV1Policy_basic
=== CONT  TestAccASV1Policy_basic
--- PASS: TestAccASV1Policy_basic (100.68s)
PASS

=== RUN   TestAccASV1Group_basic
=== PAUSE TestAccASV1Group_basic
=== CONT  TestAccASV1Group_basic
--- PASS: TestAccASV1Group_basic (254.86s)
=== RUN   TestAccASV1Group_RemoveWithSetMinNumber
=== PAUSE TestAccASV1Group_RemoveWithSetMinNumber
=== CONT  TestAccASV1Group_RemoveWithSetMinNumber
--- PASS: TestAccASV1Group_RemoveWithSetMinNumber (210.36s)
=== RUN   TestAccASV1Group_WithoutSecurityGroups
=== PAUSE TestAccASV1Group_WithoutSecurityGroups
=== CONT  TestAccASV1Group_WithoutSecurityGroups
--- PASS: TestAccASV1Group_WithoutSecurityGroups (210.36s)
PASS


=== RUN   TestAccASV1Configuration_basic
=== PAUSE TestAccASV1Configuration_basic
=== CONT  TestAccASV1Configuration_basic
--- PASS: TestAccASV1Configuration_basic (137.26s)
=== RUN   TestAccASV1Configuration_publicIP
=== PAUSE TestAccASV1Configuration_publicIP
=== CONT  TestAccASV1Configuration_publicIP
--- PASS: TestAccASV1Configuration_publicIP (145.24s)
=== RUN   TestAccASV1Configuration_invalidDiskSize
=== PAUSE TestAccASV1Configuration_invalidDiskSize
=== CONT  TestAccASV1Configuration_invalidDiskSize
--- PASS: TestAccASV1Configuration_invalidDiskSize (84.72s)
=== RUN   TestAccASV1Configuration_multipleSecurityGroups
=== PAUSE TestAccASV1Configuration_multipleSecurityGroups
=== CONT  TestAccASV1Configuration_multipleSecurityGroups
--- PASS: TestAccASV1Configuration_multipleSecurityGroups (160.15s)
PASS

Process finished with the exit code 0

@anton-sidelnikov anton-sidelnikov self-assigned this Oct 14, 2022
@anton-sidelnikov anton-sidelnikov marked this pull request as ready for review October 14, 2022 11:57
Aloento
Aloento previously approved these changes Oct 14, 2022
@lego963
Copy link
Contributor

lego963 commented Oct 14, 2022

@anton-sidelnikov AutoScaling] refactoring resource/opentelekomcloud_cbr_* ? cbr? :D

@anton-sidelnikov anton-sidelnikov changed the title [AutoScaling] refactoring resource/opentelekomcloud_cbr_* [AutoScaling] refactoring resource/opentelekomcloud_as_* Oct 14, 2022
@anton-sidelnikov
Copy link
Member Author

@anton-sidelnikov AutoScaling] refactoring resource/opentelekomcloud_cbr_* ? cbr? :D

love cbr so much ^-^

@vladimirvshivkov vladimirvshivkov added the gate Merge PR label Oct 17, 2022
@otc-zuul
Copy link

otc-zuul bot commented Oct 17, 2022

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

✔️ build-otc-releasenotes SUCCESS in 3m 25s
✔️ otc-golangci-lint SUCCESS in 3m 05s
✔️ golang-make-test SUCCESS in 3m 24s
✔️ golang-make-vet SUCCESS in 2m 50s
✔️ tflint SUCCESS in 1m 43s
✔️ goreleaser-build SUCCESS in 9m 10s

@otc-zuul otc-zuul bot merged commit c8a9342 into devel Oct 17, 2022
@otc-zuul otc-zuul bot deleted the as_ref branch October 17, 2022 08:33
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.

None yet

4 participants