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

charming-actions/upload-charm action is uploading images incorrectly and affecting charms' functionality #962

Closed
DnPlas opened this issue Jul 1, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@DnPlas
Copy link
Contributor

DnPlas commented Jul 1, 2024

Bug Description

It appears that canonical/charming-actions#139 is affecting some charms' functionality:

We need to investigate how this issue is affecting us:

  • How images get attached to charms?
  • How is this affecting tracks?
  • How can we be sure about which image is attached to a certain charm revision?

To Reproduce

canonical/charming-actions#139 contains all the necessary information.

Environment

CI environment.

Relevant Log Output

https://github.com/canonical/charming-actions/issues/139 contains all the necessary information.

Additional Context

No response

@DnPlas DnPlas added the bug Something isn't working label Jul 1, 2024
Copy link

Thank you for reporting us your feedback!

The internal ticket has been created: https://warthogs.atlassian.net/browse/KF-5952.

This message was autogenerated

@orfeas-k
Copy link
Contributor

orfeas-k commented Jul 4, 2024

Doing some investigation after canonical/charming-actions#139 (comment) comment where they suggest that this change took place back in November 2023, I think I 've found another place where this has bitten us (and probably there is more)

Kserve-controller

Looking at repo's releases, I see the following:

Revision 561

resources:

  • kube-rbac-proxy-image: gcr.io/kubebuilder/kube-rbac-proxy:v0.13.1
    resource-revision: 441
  • kserve-controller-image: charmedkubeflow/kserve-controller:0.13.0-e151c86
    resource-revision: 446

Revision 558

resources:

  • kube-rbac-proxy-image: gcr.io/kubebuilder/kube-rbac-proxy:v0.13.1
    resource-revision: 441
  • kserve-controller-image: kserve/kserve-controller:v0.13.0
    resource-revision: 446

and taking a look at the CI runs:

they have images with different hash but they are being released with the same resource-revision.

@DnPlas
Copy link
Contributor Author

DnPlas commented Jul 5, 2024

It looks like this issue is also affecting the PR tracks and the "stable" tracks (e.g. track/2.0 -> 2.0/stable)

orfeas-k added a commit to canonical/charming-actions that referenced this issue Jul 10, 2024
…#147)

Use resource revision number returned by upload-resource command when
uploading a charm with resources. This replaces the previous behaviour
when the action was listing resources and was using the latest
available.

Closes #139
Ref canonical/bundle-kubeflow#962
This was referenced Jul 10, 2024
orfeas-k added a commit to canonical/admission-webhook-operator that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/argo-operators that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/argo-operators that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/dex-auth-operator that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/envoy-operator that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/katib-operators that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/kfp-operators that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/knative-operators that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/kserve-operators that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/kubeflow-dashboard-operator that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/kubeflow-profiles-operator that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/kubeflow-roles-operator that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/kubeflow-tensorboards-operator that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/kubeflow-volumes-operator that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/metacontroller-operator that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/minio-operator that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/mlmd-operator that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/notebook-operators that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/oidc-gatekeeper-operator that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/resource-dispatcher that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/seldon-core-operator that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/argo-operators that referenced this issue Jul 15, 2024
orfeas-k added a commit to canonical/istio-operators that referenced this issue Jul 25, 2024
orfeas-k added a commit to canonical/istio-operators that referenced this issue Jul 25, 2024
orfeas-k added a commit to canonical/istio-operators that referenced this issue Jul 25, 2024
orfeas-k added a commit to canonical/istio-operators that referenced this issue Jul 25, 2024
orfeas-k added a commit to canonical/istio-operators that referenced this issue Jul 25, 2024
orfeas-k added a commit to canonical/istio-operators that referenced this issue Jul 26, 2024
orfeas-k added a commit to canonical/istio-operators that referenced this issue Jul 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants