Skip to content

Use DeploymentRuntimeConfig for aws upbound family provider#18

Open
Strangerxxx wants to merge 2 commits into
gitops-bridge-dev:mainfrom
Strangerxxx:patch-1
Open

Use DeploymentRuntimeConfig for aws upbound family provider#18
Strangerxxx wants to merge 2 commits into
gitops-bridge-dev:mainfrom
Strangerxxx:patch-1

Conversation

@Strangerxxx

@Strangerxxx Strangerxxx commented Dec 16, 2024

Copy link
Copy Markdown

I really want to tweak tolerations for the family provider pod, so I decided to update provider definition to attach DeploymentRuntimeConfig

Signed-off-by: Daniel Chaplin <black.danek@gmail.com>
@Strangerxxx

Copy link
Copy Markdown
Author

@csantanapr, could you please check this out?
Thank you!

@csantanapr

Copy link
Copy Markdown
Member

Hi @Strangerxxx
The change looks looks good, go ahead and bump the version of the chart from v3.0.0 to v3.1.0 here https://github.com/gitops-bridge-dev/gitops-bridge-helm-charts/blob/main/charts/crossplane-aws-upbound/Chart.yaml#L18
in this PR. when we merge it it will create the chart release

@Strangerxxx

Strangerxxx commented Jul 8, 2025

Copy link
Copy Markdown
Author

@csantanapr thanks for the review!

It's a bit surprising for me, that version has to be bumped by contributor, not by maintainer.

Why you can't just merge the change and then commit a bump separately?...

No problem, I can obviously do the bump for ya :)

Signed-off-by: Daniel Chaplin <black.danek@gmail.com>
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.

2 participants