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

ARGO_SECURE is not taken into account #5607

Closed
sylock opened this issue Apr 7, 2021 · 1 comment · Fixed by #5626
Closed

ARGO_SECURE is not taken into account #5607

sylock opened this issue Apr 7, 2021 · 1 comment · Fixed by #5626
Assignees
Labels
solution/workaround There's a workaround, might not be great, but exists type/bug

Comments

@sylock
Copy link
Contributor

sylock commented Apr 7, 2021

Summary

When trying to disable the TLS functionnality of argo server, the documentation says you have 2 options:

  • pass an argument --secure=false
  • set an environment variable ARGO_SECURE=false

Since argo v3.0.0 the defaults is to enable TLS by default. I can revert to plain text using --secure=false but setting the ARGO_SECURE environement variable has absolutely no effect.

Diagnostics

$ oc version
Client Version: 4.7.2
Server Version: 4.5.5
Kubernetes Version: v1.18.3+08c38ef
argo v3.0.1 (manifest and image)

To reproduce: set the environement variable ARGO_SECURE=false to the deployment and check the log of the server POD. You'll see that secure option is still to "true".

Expected behavior

ARGO_SECURE environement variable is taken into account and set the secure option accordingly.


Message from the maintainers:

Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

@sylock sylock added the type/bug label Apr 7, 2021
@alexec alexec self-assigned this Apr 7, 2021
@alexec alexec added the solution/workaround There's a workaround, might not be great, but exists label Apr 7, 2021
@alexec
Copy link
Contributor

alexec commented Apr 7, 2021

workaround: --secure=false

patrit added a commit to baloise-incubator/okd4-cluster-infra-apps that referenced this issue Apr 11, 2021
patrit added a commit to baloise-incubator/okd4-cluster-infra-apps that referenced this issue Apr 11, 2021
alexec added a commit that referenced this issue Apr 13, 2021
Signed-off-by: Alex Collins <alex_collins@intuit.com>
@simster7 simster7 mentioned this issue Apr 19, 2021
50 tasks
simster7 pushed a commit that referenced this issue Apr 19, 2021
Signed-off-by: Alex Collins <alex_collins@intuit.com>
niiku pushed a commit to baloise-incubator/okd4-cluster-infra-apps that referenced this issue Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
solution/workaround There's a workaround, might not be great, but exists type/bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants