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

Add arm64, remove i386 compatibility from gluster::repo:apt #189

Merged
merged 1 commit into from
Dec 29, 2018

Conversation

jacksgt
Copy link
Contributor

@jacksgt jacksgt commented Dec 4, 2018

Looking at the glusterfs repository at
https://download.gluster.org/pub/gluster/glusterfs/, it appears
that (at least going back to 3.10):
a) arm64 packages are available
b) i386 packages are not available

Therefore, update the puppet module accordingly

Looking at the glusterfs repository at
https://download.gluster.org/pub/gluster/glusterfs/, it appears
that (at least going back to 3.10):
a) arm64 packages are available
b) i386 packages are not available

Therefore, update the puppet module accordingly
@ekohl
Copy link
Member

ekohl commented Dec 4, 2018

I'd like to get #170 in. Doesn't have to be before this, but looks like you're getting into the apt code and wanted to bring it to your attention.

@jacksgt
Copy link
Contributor Author

jacksgt commented Dec 4, 2018

@ekohl Sure thing! I'd be happy to rebase / edit once your code is merged.

@jacksgt
Copy link
Contributor Author

jacksgt commented Dec 4, 2018

IMO that class needs a major overhaul anyway, e.g to support Gluster 5, remove obsolete / update documentation, ...

@bastelfreak
Copy link
Member

@jacksgt can you please rebase?

@jacksgt
Copy link
Contributor Author

jacksgt commented Dec 28, 2018

@bastelfreak Sure, done!

@bastelfreak
Copy link
Member

Thanks!

@bastelfreak bastelfreak merged commit d0c701c into voxpupuli:master Dec 29, 2018
@smortex smortex changed the title Add arm64, remove i386 compatibility from ::gluster::repo:apt Add arm64, remove i386 compatibility from gluster::repo:apt Nov 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants