awx-operator
awx-operator copied to clipboard
Implement extra sidecar support
SUMMARY
This PR adds a new property that allows you to specify sidecars to add to the AWX pod as blocks within the config.
Fixes https://github.com/ansible/awx-operator/issues/1235
ISSUE TYPE
- New or Enhanced Feature
ADDITIONAL INFORMATION
This is useful for people who might need to add a VPN or other extra functionality to awx while using the awx-operator.
@Yomanz Could you add some documentation in the README.md on how to use this in this PR? In that section, could you also add some potential use cases?
Otherwise, I think this looks good. Feel free to join our next community meeting to share this with others in the community!
- https://github.com/ansible/awx/issues/13563
@Yomanz Could you add some documentation in the README.md on how to use this in this PR? In that section, could you also add some potential use cases?
Otherwise, I think this looks good. Feel free to join our next community meeting to share this with others in the community!
* [AWX Office Hours Agenda - Mar 14 2023 awx#13563](https://github.com/ansible/awx/issues/13563)
I will add some documentation, yeah.
Hey @Yomanz , are you still interested in adding a README.md entry here?
Are there any update with this feature? We were looking for the same feature. It will be good if it can be implemented
@WenboDu1228 would you be interested in adding docs for this feature with some use case(s)? Then we can consider this feature complete and merge (code looks good and works as is).
It could be on a separate PR.
Hi @Yomanz thanks for your contribution and welcome to the AWX community.
Recently there's been some big changes in the AWX operator and AWX, we split up the task and web deployment
for this feature to go forward can you implement the sidecar for both web and task container following how we handled the replicas
parameters
What is the delay with getting this merged and publishing a new release with this feature? I and several others need this functionality. Can we please dispense with the foot-dragging and get it done?
Hi @Yomanz thanks for your contribution and welcome to the AWX community.
Recently there's been some big changes in the AWX operator and AWX, we split up the task and web deployment
for this feature to go forward can you implement the sidecar for both web and task container following how we handled the
replicas
parameters
I can look into doing this 👍
Hey @TheRealHaoLiu and @Yomanz, I am willing to takeover if it is ok with you. I can continue on @Yomanz 's work and make the changes for the separation between web and task pods. I think I would need to create a new PR for this and close this one.
Hey @TheRealHaoLiu and @Yomanz, I am willing to takeover if it is ok with you. I can continue on @Yomanz 's work and make the changes for the separation between web and task pods.
I think it would need to create a new PR for this and close this one.
Yeah crack on that'd be great, I did plan to finish this but I just ran out of time for it unfortunately.
If we implement and overrides feature like describe in https://github.com/ansible/awx-operator/issues/479, that would allow for injection sidecar containers as well.
hey @Yomanz https://github.com/ansible/awx-operator/pull/1695 try this out it should allow you to inject extra sidecars