-
Notifications
You must be signed in to change notification settings - Fork 96
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
fix(_rotate_kms_key): remove SkipPerIssues since the ticket is resolved #9153
fix(_rotate_kms_key): remove SkipPerIssues since the ticket is resolved #9153
Conversation
As per discussion, this clause can be removed as the ticket is resolved and doesn't have sct skip label (what could have made it being skipped). At the same, it led to unexpected failures when self.params appeared to be dict type instance instead of SCTConfiguration as expected. refs: - scylladb#9134
yes we can remove this code, but I would suggest going to the root cause of why params argument wasn't set creating the cluster, we shouldn't let it happen. |
I think the actual fix was suppose to be:
|
Yes, totally agree with you and I raised a bug about it (#9134) that remains open. |
okay, but still it looks like this peace of code I'm removing is not relevant anymore since the original issue is closed and not labeled as |
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
As per discussion in #9134, this clause can be removed as the ticket is resolved and doesn't have sct skip label (what could have made it being skipped). At the same, it led to unexpected failures when self.params appeared to be dict type instance instead of SCTConfiguration as expected.
Testing
PR pre-checks (self review)
backport
labels