Christian Adams

Results 109 comments of Christian Adams

@JonTheNiceGuy if you need to add tolerations and pod_affinity to the awx application deployments, is there a reason why you wouldn't just do so on the AWX.spec? Docs here: https://github.com/ansible/awx-operator/tree/devel/.helm/starter#awx...

@Raptus1 I am a bit confused. You mentioned that you have an external database configured for you AWX instance, but I do not see a custom `postgres_configuration_secret` configured on the...

@alexiseichst I think we should add these same changes to the task deployment before merging. And CI is currently failing. This PR needs more attention before it can be merged.

Other people in the community who use helm, what do you think about this approach? @gfa can you point out the problem you are solving by splitting the lifecycle of...

Isn't it the case that when you upgrade with helm it does not affect your CRD's and that it is a separate manual step for upgrade/apply the CRD's? * https://github.com/ansible/awx-operator/blob/devel/.helm/starter/README.md#caveats-on-upgrading-existing-installation...

@kurokobo we have reviewed this PR and are ready to merge it, but it looks like there are some conflicts. Could you please rebase? I think that will resolve the...

@akkaba23 you are configuring the ca cert used by postgres via `bundle_ca_crt` right?

@akkaba23 could you please these the changes in this PR out? * https://github.com/ansible/awx-operator/pull/1800

Could you give this PR a try and see if it solves your issue? https://github.com/ansible/awx-operator/pull/1799

@gdasson Looks like this PR follows the plan laid out in this comment: * https://github.com/ansible/awx-operator/pull/1425#issuecomment-1854578532 Continuing the work there. Changes suggested before merge: * I think `_settings` at the end...