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

Support http proxy #185

Open
mzhidovinov opened this issue Apr 8, 2024 · 6 comments
Open

Support http proxy #185

mzhidovinov opened this issue Apr 8, 2024 · 6 comments
Assignees

Comments

@mzhidovinov
Copy link

mzhidovinov commented Apr 8, 2024

Patterns Operator needs to support deployments that require the use of HTTP proxy to access the git repo (e.g. for initial cloning).

Two scenarios should be recognized:

  1. There is a cluster-wide proxy configuration.
    Patterns operator needs to recognize and use resulting environment variables (e.g. HTTP_PROXY/HTTPS_PROXY/NO_PROXY).
  2. The use of proxy is configured specifically for the Patterns Operator (e.g. for given repo)
    In that case, Patterns Operator may need to propagate the proxy configuration to the appropriate ArgoCD instances
    This may require support from validatedpatterns/common
@mbaldessari
Copy link
Contributor

So part 1. is done via #190 . It will be part of the upcoming 0.0.45 release (will be a few days, because the community CI is currently down and it is required to release a new version).

@mbaldessari
Copy link
Contributor

So I managed to cram it in to 0.0.44 which is out now. If you also update common/ in your project, cluster-wide proxy support should just work out of the box. Can you let me know if it all works for you as well? (I tested http proxy, https + custom ca and without proxy and they all worked okay for me). Thanks

@mzhidovinov
Copy link
Author

mzhidovinov commented Apr 17, 2024 via email

@mbaldessari
Copy link
Contributor

Thanks Michael! So I was wondering what the use case for 2. is because it feels like we'd be adding quite a bit of code all over the place to support that (changes in the operator + common). It kinda feels very niche? I mean, if you have a proxy you must use then the whole cluster usually has to use it, I would expect.

@mzhidovinov
Copy link
Author

mzhidovinov commented Apr 19, 2024 via email

@mbaldessari
Copy link
Contributor

Thanks Michael, this is helpful! Btw. Custom CA is somewhat already there, just make sure that in your pattern CR you add extraCapabilities: initcontainers in the spec field and make sure you have the operator version at >= 0.0.44. Once you do that it will magically pick up the trusted-ca-bundle in all argo instances. The reason for it not being the default is that we're still testing a few scenarios.

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

No branches or pull requests

2 participants