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

CLI: HTTP client ignores proxy #10794

Open
2 of 3 tasks
Adam-D-Lewis opened this issue Mar 31, 2023 · 7 comments · May be fixed by #12527
Open
2 of 3 tasks

CLI: HTTP client ignores proxy #10794

Adam-D-Lewis opened this issue Mar 31, 2023 · 7 comments · May be fixed by #12527
Labels
area/cli The `argo` CLI P3 Low priority type/bug

Comments

@Adam-D-Lewis
Copy link

Adam-D-Lewis commented Mar 31, 2023

Pre-requisites

  • I have double-checked my configuration
  • I can confirm the issues exists when I tested with :latest
  • I'd like to contribute the fix myself (see contributing guide) - Sorry, I'm not a gopher :(

What happened/what you expected to happen?

Runnning
./argo-linux-amd64 submit -n dev hello-world.yaml --proxy-url http://localhost:8000 --argo-server <my-domain>:443 --argo-http1 --argo-base-href=/argo --token <my-token> --secure=true -v

outputs the curl command that is run which is
curl -X POST -H 'Authorization: ******' -d '{"namespace":"dev","workflow":{"kind":"Workflow","apiVersion":"argoproj.io/v1alpha1","metadata":{"generateName":"hello-world-","namespace":"dev","creationTimestamp":null,"labels":{"workflows.argoproj.io/archive-strategy":"false"},"annotations":{"workflows.argoproj.io/description":"This is a simple hello world example.\nYou can also run it in Python: https://couler-proj.github.io/couler/examples/#hello-world\n"}},"spec":{"templates":[{"name":"whalesay","inputs":{},"outputs":{},"metadata":{},"container":{"name":"","image":"docker/whalesay:latest","command":["cowsay"],"args":["hello world"],"resources":{}}}],"entrypoint":"whalesay","arguments":{}},"status":{"startedAt":null,"finishedAt":null}},"createOptions":{}}' 'https://<my-domain>:443/argo/api/v1/workflows/dev?'

As you can see the --proxy-url flag is ignored. I would have expected a -x http://localhost:8000 to be added to the curl command.

Version

v3.4.5 (latest binary I could find)

Paste a small workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.

apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: hello-world-
  labels:
    workflows.argoproj.io/archive-strategy: "false"
  annotations:
    workflows.argoproj.io/description: |
      This is a simple hello world example.
      You can also run it in Python: https://couler-proj.github.io/couler/examples/#hello-world
spec:
  entrypoint: whalesay
  templates:
  - name: whalesay
    container:
      image: docker/whalesay:latest
      command: [cowsay]
      args: ["hello world"]

Logs from the workflow controller

N/A

Logs from in your workflow's wait container

N/A
@sarabala1979
Copy link
Member

@Adam-D-Lewis Argo CLI will not support proxy-url. it might be coming from kubectl option. Do you like to implement it?

@sarabala1979 sarabala1979 added the P3 Low priority label Apr 6, 2023
@Adam-D-Lewis
Copy link
Author

Thanks for the response. Sorry, I can't commit to implement it right now. I've decided to pursue another option.

@agilgur5 agilgur5 added the area/cli The `argo` CLI label Sep 1, 2023
@stale

This comment was marked as resolved.

@stale stale bot added the problem/stale This has not had a response in some time label Sep 17, 2023
@terrytangyuan terrytangyuan removed the problem/stale This has not had a response in some time label Sep 20, 2023
@shimako55
Copy link

I have implemented the feature, but I am unsure how to write tests for submitting a PR. Could you please give me some advice?

@shimako55 shimako55 linked a pull request Jan 16, 2024 that will close this issue
@monaka
Copy link

monaka commented Apr 25, 2024

I suppose this has been fixed by #12827 ...?

@Joibel
Copy link
Member

Joibel commented Sep 9, 2024

As suggested by @monaka, closing as implemented by #12867.

@Joibel Joibel closed this as completed Sep 9, 2024
@Joibel
Copy link
Member

Joibel commented Sep 9, 2024

Ah, no, this is about the CLI tool, so not re-opening.

@Joibel Joibel reopened this Sep 9, 2024
@agilgur5 agilgur5 changed the title Argo CLI tool HTTP client ignores proxy CLI: HTTP client ignores proxy Oct 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/cli The `argo` CLI P3 Low priority type/bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants