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

setup: pin macaroonbakery<1.3.3 for python3.6 #671

Merged
merged 1 commit into from
Jan 12, 2024
Merged

Conversation

lathiat
Copy link

@lathiat lathiat commented Jan 5, 2024

macaroonbakery in v1.3.4 added a constraint of protobuf>=3.20.0[0] which makes it incompatible with python 3.6 and v1.3.3 was released in a broken state[1].

macaroonbakery in v1.3.4 added a constraint of protobuf>=3.20.0[0] which
makes it incompatible with python 3.6 and v1.3.3 was released in a
broken state[1].
@lathiat
Copy link
Author

lathiat commented Jan 5, 2024

I'm still having trouble with octavia/ussuri failing to build due to macaroonbakery/protobuf (https://review.opendev.org/c/openstack/charm-octavia/+/902746).

It's only broken for Python3.6 (Bionic) which only effects ussuri, victoria is fine since it only needs Python3.8 (Focal).

The specific issue is charm-tools < 3.0.0 -> libcharmstore -> theblues -> macaroonbakery

charm-tools 3.x actually installs and works fine on python3.6 (and works to build the charm) because libcharmstore was dropped, but the stable charms currently tend to pin charm-tools to <3.0.0 (or == 2.8.x)

Copy link
Contributor

@coreycb coreycb left a comment

Choose a reason for hiding this comment

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

This looks good to me. I can't merge it though.

Copy link
Contributor

@freyes freyes left a comment

Choose a reason for hiding this comment

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

LGTM. This is the approach we have taken elsewhere.

@ajkavanagh ajkavanagh merged commit bef504f into juju:2.8 Jan 12, 2024
1 check passed
openstack-mirroring pushed a commit to openstack/charm-octavia that referenced this pull request Mar 6, 2024
Switch to using the latest charm-tools <3.0.0 to fix three build errors:
- An error related to 'pytz' fixed in charm-tools 2.8.6:
  juju/charm-tools#654
- An error related to ruamel.yaml fixed in charm-tools 2.8.8:
  here: juju/charm-tools#668
- An error related to macaroonbakery fixed in charm-tools 2.8.9:
  juju/charm-tools#671

Additionally merge bindep.txt to install libpq-dev required for
psycopg2-binary as wheels no longer exist for py36.

Additional fix for ussuri only:
- Switch bionic test bundles from vault 1.7/edge to 1.6/edge as the 1.7
  channel no longer supports bionic.

Change-Id: Idc16b84dc3dcdf001c73c7d2fdd57b1d08ebfb23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants