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

Bump fedora #203

Merged
merged 2 commits into from
Aug 25, 2021
Merged

Bump fedora #203

merged 2 commits into from
Aug 25, 2021

Conversation

NathanTP
Copy link
Contributor

This bumps fedora to a more minimal image (and more recent build). We had previously been working off an arbitrary full-sized nightly from when they were first porting Fedora. This one seems a bit more stable and definitely much smaller (800MB vs 6GB). Despite the name, it's not all that minimal. For example, it includes Python.

I'm not sure if there are any packages we think should be included by default, but I can add a guest-init for that. I'm also considering adding a guest-init that updates the dnf repository caches to make installing stuff faster for everyone. Since the caches are old it takes a few minutes the first time you install something. Also, this is a good opportunity to update other stuff if we think there's something with Fedora we really need.

BTW: This is still experiencing ucb-bar/chipyard#950 but it boots in normal disk mode.

@NathanTP NathanTP requested a review from alonamid August 24, 2021 00:26
Copy link
Contributor

@alonamid alonamid left a comment

Choose a reason for hiding this comment

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

Is the networking part related to the original intent of this PR?
Otherwise, LGTM. Tagging @jerryz123 and @a0u who occasionally use the fedora image in case they have any other package requirements.

800MB vs 6GB for the previous image. Fedora-Minimal-Rawhide-20191123
@NathanTP
Copy link
Contributor Author

lol, no. Somehow I got my branches crossed. It should only be the changes to Fedora's makefile. I fixed it.

@NathanTP NathanTP requested a review from abejgonzalez August 24, 2021 20:57
from dealing with a few minutes of cache updates.
@NathanTP NathanTP merged commit d25efe5 into dev Aug 25, 2021
@NathanTP NathanTP deleted the bumpFedora branch August 25, 2021 16:15
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.

2 participants