CORS-4054: Migrate from SDK v1 to v2 in installconfig#9760
CORS-4054: Migrate from SDK v1 to v2 in installconfig#9760openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
Conversation
|
@barbacbd: This pull request references CORS-4054 which is a valid jira issue. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
/cc @tthvo |
tthvo
left a comment
There was a problem hiding this comment.
Looks good! I have some small comments.
I guess ideally we would want to gather these error check into a shared place (not in this PR tho 🤣).
|
/label platform/aws |
|
/approve This one also needs a rebase... |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: patrickdillon The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
** Use the sdk v2 to check whether the error is a throttle error.
|
@barbacbd: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
/lgtm |
a1bfb1c
into
openshift:main
|
[ART PR BUILD NOTIFIER] Distgit: ose-installer |
|
[ART PR BUILD NOTIFIER] Distgit: ose-baremetal-installer |
|
[ART PR BUILD NOTIFIER] Distgit: ose-installer-artifacts |
** Update the vendor code to include aws/retry. This package will be
used for throttling error checks.
pkg/asset/installconfig/basedomain.go:
** Use the sdk v2 to check whether the error is a throttle error.