-
Notifications
You must be signed in to change notification settings - Fork 67
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
✨ Adding support for adding aws tags #471
✨ Adding support for adding aws tags #471
Conversation
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.
/approve
the test fails..
Thanks @qiujian16 do you know why tests fail? It seems to be related to go version, not related to our change. |
hrm that is strange... @zhujian7 and idea? |
@jaswalkiranavtar could you rebase this PR to have a try after it is merged? |
121376c
to
ace9b55
Compare
5d4b153
to
cfe3db0
Compare
Signed-off-by: Jeffrey Wong <[email protected]>
cfe3db0
to
6d7e5d4
Compare
this looks like a flaky test.. /approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jaswalkiranavtar, qiujian16 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
1f52a8c
into
open-cluster-management-io:main
Summary
Related issue(s)
Fixes open-cluster-management-io/ocm#514