-
Notifications
You must be signed in to change notification settings - Fork 169
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
update SDK to track2 in ensureGatewayCreate #3616
Conversation
/azp run ci,e2e |
Azure Pipelines successfully started running 2 pipeline(s). |
Please rebase pull request. |
1037a78
to
6102c5d
Compare
/azp run ci,e2e |
Azure Pipelines successfully started running 2 pipeline(s). |
/azp run e2e |
Azure Pipelines successfully started running 1 pipeline(s). |
Please rebase pull request. |
6102c5d
to
23e0173
Compare
23e0173
to
a9c059d
Compare
/azp run ci,e2e |
Azure Pipelines successfully started running 2 pipeline(s). |
Waiting for #3638 to be released because it had a bug about tenant ID settings, and want to confirm it works first. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/azp run ci,e2e |
Azure Pipelines successfully started running 2 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thank you for updating these clients, Ayato.
I left some minor questions, but they can be tackled in a follow-up, if needed.
Which issue this PR addresses:
Fixes a part of https://issues.redhat.com/browse/ARO-4665 and https://issues.redhat.com/browse/ARO-7316
This is blocked until #3638 is released.
What this PR does / why we need it:
This PR makes
ensureGateway
action use track2 SDK.Test plan for issue:
unittest for ensureGateway.
e2e
Is there any documentation that needs to be updated for this PR?
tech debt cleanup N/A
How do you know this will function as expected in production?
cluster installation test