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

Adjustments to enable building on launchpad #1376

Merged
merged 8 commits into from
Aug 1, 2023

Conversation

addyess
Copy link
Member

@addyess addyess commented Jul 21, 2023

In order to build charms targeting a certain architecture, the intent is to leverage a build farm of launchpad builders using the launchpadlib library within the charm builder job

By changing only the charm-support-matrix.inc to include the builder: 'launchpad' option within a specific charm, the charm will be built on launchpad rather than on the jenkins runner.

It is necessary only for the charm to be built using charmcraft.yaml (no reactive charms are supported there) and each builds-on entry in that charmcraft yaml will recieve a separate builder.

Incidentally, this allows for a single charm build invocation to also yield multiple charm build outputs -- one for each builds-on entry. Much of the rearchitecture in this PR is around handling a single charm build invocation creating multiple charm files (eg. one charm file per architecture)

In that event -- these changes support each architecture being uploaded, associated with its resources, and released to the same charmhub channel -- for the various architectures|bases it supports.

These changes also facilitate having a different architecture's OCI image attached to a charm so that juju can download the appropriate binary arch and binary oci-image for that charm deployment.

  • update build-charms / promote charms job jjb completed

@addyess addyess force-pushed the akd/charm-building/lp_builds branch from 96d4446 to f043213 Compare July 21, 2023 18:51
Copy link
Member

@mateoflorido mateoflorido left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 💯

@addyess addyess merged commit ee418fc into main Aug 1, 2023
6 checks passed
@addyess addyess deleted the akd/charm-building/lp_builds branch August 1, 2023 02:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants