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

Publish Docker images for ARM architecture #19

Closed
ngosang opened this issue Mar 6, 2022 · 11 comments
Closed

Publish Docker images for ARM architecture #19

ngosang opened this issue Mar 6, 2022 · 11 comments

Comments

@ngosang
Copy link
Collaborator

ngosang commented Mar 6, 2022

It's easy with Docker buildx. I'm doing it in some of my projects with GitHub actions => https://github.com/ngosang/docker-amule/blob/master/.github/workflows/release-docker.yml#L49

Requested archs:

  • linux/arm/v7 => arm32
  • linux/arm64/v8 => arm64
@PhilipMay
Copy link
Collaborator

Hi @ngosang - maybe you want to check this project here: https://github.com/PhilipMay/smart-prom-next

It provides these platforms: linux/386, linux/amd64, linux/arm/v5, linux/arm/v7, linux/arm64/v8

See here: https://github.com/PhilipMay/smart-prom-next/pkgs/container/smart-prom-next

There is a tab called "OS /Arch" where you can see the provided platform images:

image

@PhilipMay
Copy link
Collaborator

@ngosang if you tested it - feedback would be very welcome...

@ngosang
Copy link
Collaborator Author

ngosang commented Jul 27, 2022

@matusnovak If you don't have time to maintain this project you can make me or @PhilipMay maintainers. There is no point in having 2 forks with very few users each one.

@PhilipMay I tested you image but it's not working for me. I will open some issues in your project.

@PhilipMay
Copy link
Collaborator

@matusnovak If you don't have time to maintain this project you can make me or @PhilipMay maintainers. There is no point in having 2 forks with very few users each one.

I don't consider this a fork. It is a standalone development.

@PhilipMay I tested you image but it's not working for me. I will open some issues in your project.

Yes. Sure - many thanks for the issues. Please provide detailed infos.

@matusnovak
Copy link
Owner

Hi @ngosang and @PhilipMay

I apologise for such long delay. I had a time difficult few months and I really wish I had more time for GitHub.

Making you into a maintainer is a good idea. Are you both ok with you both being the repo maintainers?

@PhilipMay
Copy link
Collaborator

I generally agree.

However, to be honest:

I personally think that my own fundamentally new implementation is better. Otherwise I would not have started it.

So I'll really actively maintain my own implementation.

@matusnovak
Copy link
Owner

matusnovak commented Jul 27, 2022

I understand. This project started as something stupid simple that did the job. I have never planned that this would become more useful to other people.

I have added you both to this repo, you should have an invite.

You are free to change the implementation. Also feel free to change the readme to point anyone who stumbles upon this repo to link to your more better implementation.

And many thenks for all of your PRs and issues. And I am sorry I was not able to work on this project.

@PhilipMay
Copy link
Collaborator

PhilipMay commented Jul 27, 2022

@matusnovak ok cool. Many thanks for your trust.

@ngosang I would love to have a conversation with you about:
"How we can handle this repo, my repo, and the maintenance?"

Is it ok if I set up a 30 min MS Teams or WebEx Teams chat with us?
Your Mail is [removed for spam protection] - right?

@ngosang
Copy link
Collaborator Author

ngosang commented Jul 27, 2022

@matusnovak I don't have much time lately but I forked this Prometheus exporter to use it in my infrastructure. I prefer to maintain this repository than my fork, at least I can get contributions back. If you agree I can do the following:

  • I'm going to review and merge all open PRs/issues but I don't plan to implement new features, just bug fixing and security updates
  • At least 1 change breaks compatibility so I will release version 2.0.0 => Converted the metrics name into lower case #13

@PhilipMay Thank you for your work. I didn't see your project was a new implementation. Since it requires some changes to work in my infrastructure and I have some Grafana dashboards and Prometheus rules coupled to this project I prefer to keep using this project for now. If your project gains popularity I will move eventually... Please, remove my email (spam bots)

@PhilipMay
Copy link
Collaborator

Please, remove my email (spam bots)

I will do that.
Your mail is visible for all spam bots on your GitHub profile page by the way.

@ngosang
Copy link
Collaborator Author

ngosang commented Jul 28, 2022

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

No branches or pull requests

3 participants