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

[Backport release-0.47] fix Secret/secretmanagers.aws update loop for replica field and custom diff logic when replica config is empty #1158

Conversation

turkenf
Copy link
Collaborator

@turkenf turkenf commented Feb 15, 2024

Signed-off-by: Erhan Cagirici <[email protected]>
(cherry picked from commit 456d8d3)
@turkenf turkenf changed the title [Backport release-0.47] [Backport release-0.47] fix Secret/secretmanagers.aws update loop for replica field Feb 15, 2024
@turkenf turkenf changed the title [Backport release-0.47] fix Secret/secretmanagers.aws update loop for replica field [Backport release-0.47] fix Secret/secretmanagers.aws update loop for replica field and custom diff logic when replica config is empty Feb 15, 2024
@turkenf
Copy link
Collaborator Author

turkenf commented Feb 15, 2024

/test-examples="examples/secretsmanager/secret.yaml"

@turkenf
Copy link
Collaborator Author

turkenf commented Feb 15, 2024

/test-examples="examples/secretsmanager/secret-withreplica.yaml"

@sergenyalcin
Copy link
Collaborator

sergenyalcin commented Feb 15, 2024

/test-examples="examples/secretsmanager/secret.yaml"

Uptest run: https://github.com/upbound/provider-aws/actions/runs/7917258418

@sergenyalcin
Copy link
Collaborator

sergenyalcin commented Feb 15, 2024

/test-examples="examples/secretsmanager/secret-withreplica.yaml"

Uptest run: https://github.com/upbound/provider-aws/actions/runs/7917600558

@sergenyalcin
Copy link
Collaborator

Because of the shared runner out-of-memory issues, for now, we ran the following jobs locally:

  • check-diff (locally passed) ✅
  • unit-tests (locally passed) ✅

Also, the uptest runs cover the local-deploy job. In other words, in uptest runs, we ensure that the packages are correctly deployed because the work done in local-deploy is done. lint failure is also a known issue because of memory issues.

Copy link
Collaborator

@ulucinar ulucinar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @sergenyalcin & @turkenf, lgtm.

@sergenyalcin sergenyalcin merged commit f0f1504 into crossplane-contrib:release-0.47 Feb 16, 2024
8 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants