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

Install arm64 kolet #30

Closed
wants to merge 2 commits into from
Closed

Conversation

glevand
Copy link
Contributor

@glevand glevand commented Aug 30, 2018

This gets the coreos-assembler docker image to (almost) build on arm64 machines. It also adds an install of the arm64 mantle kolet for all architectures.

@cgwalters I only saw x86_64 builds of rpmdistro-gitoverlay. build.sh would fail on that. Commenting out that install would get a successful build.

@cgwalters
Copy link
Member

It doesn't look to me like copr even supports aarch64? That said, I'd like to deprecate it, see this issue.

Short term, we could potentially just rebuild it here? I forget even why it's archful.

@cgwalters
Copy link
Member

Also, thanks a ton for working on this!

@glevand
Copy link
Contributor Author

glevand commented Sep 4, 2018

@cgwalters I could build rpmdistro-gitoverlay on my arm64 test machine. Were you thinking to add its build to build.sh, or to have it available for download somewhere?

@cgwalters
Copy link
Member

Were you thinking to add its build to build.sh

That - it's Python so it doesn't really have a build anyways.

@Promaethius
Copy link

Really glad to see arm64 support for CoreOS!

cgwalters added a commit to cgwalters/coreos-assembler that referenced this pull request Oct 1, 2018
We want the ability to make breaking changes to how we build
RPMs; having it as a submodule will allow us to break things.

Further, today COPR doesn't support e.g. aarch64, which imposes
an artificial limitation on building this container.
(Today we're using the Ignition from COPR, but actually for
 validating configs we could probably just use the main repo)

Obsoletes: coreos#30
@cgwalters
Copy link
Member

Thanks for submitting this btw!

OK, at the time - we probably should have merged this PR and continued on top. But anyways, a whole lot has changed since then. I believe it's just the last hunk in your patch here for the kolet that still applies.

How about doing that upstream? Although a tricky thing is that today coreos-assembler follows Fedora in not doing cross-compilation; there's no real point to installing all kolet architectures. (Though I can imagine changing to support cross build/testing here like CL does, but there's immense complexity there)

@cgwalters cgwalters changed the title Fixups for arm64 Install arm64 kolet Nov 9, 2018
@nhorman
Copy link

nhorman commented Mar 20, 2019

sorry, I've not been paying attention to this, but currently the build.sh file needs updates. Can you resolve them, and I'll integrate this

@cgwalters
Copy link
Member

Ah...wait, are you reading github notifications for coreos-assembler and thinking it's coreos-installer or something?

@dustymabe
Copy link
Member

Ah...wait, are you reading github notifications for coreos-assembler and thinking it's coreos-installer or something?

Yeah I'm a bit confused.. In case you are looking for coreos-installer the work is now happening in https://github.com/coreos/coreos-installer

@glevand glevand closed this Oct 14, 2019
@glevand glevand deleted the for-merge-arm64 branch October 14, 2019 15:49
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.

5 participants