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

v3.4.0 release assets: quick-start-* manifests include invalid references to httpbin that results in a failed deployment #9659

Closed
2 of 3 tasks
hype-eater opened this issue Sep 22, 2022 · 1 comment · Fixed by #9759
Labels
P1 High priority. All bugs with >=5 thumbs up that aren’t P0, plus: Any other bugs deemed high priority type/bug type/regression Regression from previous behavior (a specific type of bug)

Comments

@hype-eater
Copy link

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)

What happened/what you expected to happen?

I regularly use the quick-start-minimal.yaml produced as a release asset to install are usable (and useful!) Argo Workflows in small or ephemeral test/development/demo environments. The quick-start manifests from the latest release include an additional Deployment for httpbin (presumably for testing) which is doomed to fail due to an invalid image tag. Argo Workflows works as expected (as far as I have tested), and it seems fine to just delete the httpbin Deployment. I'm not sure httpbin really belongs in the quick-start manifests, but if it's going to be there, the image tag should be valid.

The "Quick Start" documentation includes instructions to use these assets to install Argo Workflows.


I expected: 1. A minimal installation of components that enable use of Argo Workflows. 2. Those components should be deploy successfully.

I experienced: Starting with 3.4.0, I see the installation includes a Deployment of httpbin, but the Deployment uses the image tag corresponding to the Argo Workflows release version rather than a tag that is valid for the httpbin image.


The problem appears to come from the fact that the latest tag is specified in the quick-start base configuration:

containers:
- name: main
image: kennethreitz/httpbin:latest

When producing dist artifacts, the string "latest" is replacement by the (Argo Workflows) version:

argo-workflows/Makefile

Lines 389 to 391 in e556fe3

dist/manifests/%: manifests/%
@mkdir -p dist/manifests
sed 's/:latest/:$(VERSION)/' manifests/$* > $@

Version

v3.4.0

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.

N/A No workflow - this is solely about the quick-start-* manifests in the release assets.

Logs from the workflow controller

N/A not the workflow-controller

Logs from in your workflow's wait container

N/A not a workflow


To reproduce

Install using quick-start manifest:

kubectl apply -n argo -f https://github.com/argoproj/argo-workflows/releases/latest/download/quick-start-minimal.yaml

or

kubectl apply -n argo -f https://github.com/argoproj/argo-workflows/releases/download/v3.4.0/quick-start-minimal.yaml

Outcome:

$ kubectl get deploy -n argo
NAME                  READY   UP-TO-DATE   AVAILABLE   AGE
argo-server           1/1     1            1           38m
httpbin               0/1     1            0           38m
minio                 1/1     1            1           38m
workflow-controller   1/1     1            1           38m
$ kubectl get events --field-selector=type!=Normal
LAST SEEN   TYPE      REASON   OBJECT                        MESSAGE
36m         Warning   Failed   pod/httpbin-7c5dfc9db-m9cb7   Failed to pull image "kennethreitz/httpbin:v3.4.0": rpc error: code = Unknown desc = Error response from daemon: manifest for kennethreitz/httpbin:v3.4.0 not found: manifest unknown: manifest unknown
36m         Warning   Failed   pod/httpbin-7c5dfc9db-m9cb7   Error: ErrImagePull
36m         Warning   Failed   pod/httpbin-7c5dfc9db-m9cb7   Error: ImagePullBackOff
@hype-eater hype-eater changed the title v3.4.0 release assets: quick-start-* manifests include invalid references to httpbin that results in a failed deployments v3.4.0 release assets: quick-start-* manifests include invalid references to httpbin that results in a failed deployment Sep 22, 2022
@sarabala1979 sarabala1979 added type/regression Regression from previous behavior (a specific type of bug) P1 High priority. All bugs with >=5 thumbs up that aren’t P0, plus: Any other bugs deemed high priority labels Sep 23, 2022
@alexec
Copy link
Contributor

alexec commented Sep 30, 2022

To fix, prefix image with quay.io/?

terrytangyuan pushed a commit that referenced this issue Oct 7, 2022
…ixes #9659 (#9759)

fix: quick-start-* manifests pointing to invalid httpbin image tag

Signed-off-by: Rohan Kumar <[email protected]>

Signed-off-by: Rohan Kumar <[email protected]>
juchaosong pushed a commit to juchaosong/argo-workflows that referenced this issue Nov 3, 2022
…ixes argoproj#9659 (argoproj#9759)

fix: quick-start-* manifests pointing to invalid httpbin image tag

Signed-off-by: Rohan Kumar <[email protected]>

Signed-off-by: Rohan Kumar <[email protected]>
Signed-off-by: juchao <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 High priority. All bugs with >=5 thumbs up that aren’t P0, plus: Any other bugs deemed high priority type/bug type/regression Regression from previous behavior (a specific type of bug)
Projects
None yet
3 participants