-
Notifications
You must be signed in to change notification settings - Fork 11
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
compatible: Add arm64 support to build_charm.yaml and release_charm.yaml #141
Conversation
5b5c331
to
d7defa7
Compare
testing on https://github.com/canonical-test4/postgresql-k8s-operator/actions/runs/8050255568 (cannot test on canonical org since waiting for IS to add Azure runners) |
|
||
|
||
class Architecture(str, enum.Enum): | ||
X64 = "amd64" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be better to name it AMD64
to keep it consistent.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
X64
matches github naming
…e format https://discourse.charmhub.io/t/charmcraft-bases-provider-support/4713 charmcraft.yaml bases with build-on are a singleton list of one dict, not a dict Follow up to #141 Encountered on https://github.com/canonical/postgresql-k8s-operator/actions/runs/8073677258/job/22057758552?pr=403
…e format (#146) https://discourse.charmhub.io/t/charmcraft-bases-provider-support/4713 charmcraft.yaml bases with build-on are a singleton list of one dict, not a dict Follow up to #141 Encountered on https://github.com/canonical/postgresql-k8s-operator/actions/runs/8073677258/job/22057758552?pr=403
Uses spec (pending review) ST111 - Multi-architecture
upstream-source
in charm OCI resourceshttps://docs.google.com/document/d/19pzpza7zj7qswDRSHBlpqdBrA7Ndcnyh6_75cCxMKSo/edit