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

Image refresh for fedora-coreos #2008

Merged

Conversation

cockpituous
Copy link
Contributor

@cockpituous cockpituous commented May 13, 2021

Image refresh for fedora-coreos

@github-actions github-actions bot added the bot label May 13, 2021
@cockpituous
Copy link
Contributor

image-refresh in progress on 3-ci-srv-03.
Log: https://logs.cockpit-project.org/logs/image-refresh-2008-20210513-023737/

@cockpituous cockpituous changed the title Image refresh for fedora-coreos WIP: 3-ci-srv-03: [no-test] Image refresh for fedora-coreos May 13, 2021
@cockpituous
Copy link
Contributor

cockpituous pushed a commit that referenced this pull request May 13, 2021
@cockpituous cockpituous force-pushed the image-refresh-fedora-coreos-20210513-024444 branch from 1b33c9c to 863d04a Compare May 13, 2021 02:44
@cockpituous cockpituous changed the title WIP: 3-ci-srv-03: [no-test] Image refresh for fedora-coreos Image refresh for fedora-coreos May 13, 2021
@martinpitt
Copy link
Member

Most tests fail with "invalid or unusable locale: en_US.UTF-8", so some change in the image lost locales? If that was on purpose, we need to switch our default locale to C.UTF-8 now. This won't work on RHEL 7, but we don't test that from master any more, so we can finally do that.

@allisonkarlitskaya
Copy link
Member

I mean, that's kinda obvious:

Removed:
  glibc-all-langpacks (2.32-4.fc33)

We could just add back glibc-langpack-en. That's what I did in my toolboxes, anyway...

@allisonkarlitskaya
Copy link
Member

To be honest, I'd almost rather add "invalid or unusable locale" to the allowed journal messages, or to stop printing this message entirely. All that it means is that the browser sent a language header for which we have no system locale installed. That's not really an error, but we always treat it as if it were.

Otherwise, the locale is set according to the brower's language string, and I don't know of a way to force a browser to send "C" as its language, so I think the only other fix is indeed to add the langpack.

@allisonkarlitskaya
Copy link
Member

Third and forth option, assuming the change was introduced accidentally upstream:

  • naughty all the things
  • wait

@martinpitt
Copy link
Member

We could just add back glibc-langpack-en

Please not -- we want to test the actual image, not a "hacked" one specifically for cockpit.

I'd almost rather add "invalid or unusable locale" to the allowed journal messages

That's my preferred solution. We could start with conditionally adding it on fcos.

@martinpitt
Copy link
Member

After cockpit-project/cockpit#15837 there are six remaining failures due to a new SELinux violation:

audit: type=1400 audit(1621337254.332:217): avc: denied { write } for pid=672 comm="systemd-hostnam" name="systemd" dev="tmpfs" ino=2 scontext=system_u:system_r:systemd_hostnamed_t:s0 tcontext=system_u:object_r:init_var_run_t:s0 tclass=dir permissive=0

This looks new, and we don't have a naughty for it yet, but it's already reported downstream: https://bugzilla.redhat.com/show_bug.cgi?id=1953060

@allisonkarlitskaya allisonkarlitskaya force-pushed the image-refresh-fedora-coreos-20210513-024444 branch from 863d04a to 8574ca8 Compare May 18, 2021 15:48
Copy link
Member

@martinpitt martinpitt left a comment

Choose a reason for hiding this comment

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

Thanks! Assuming green -- you still need to trigger the tests.

@allisonkarlitskaya
Copy link
Member

Thanks! Assuming green -- you still need to trigger the tests.

already did that :)

@allisonkarlitskaya allisonkarlitskaya merged commit 8ecdb46 into master May 18, 2021
@allisonkarlitskaya allisonkarlitskaya deleted the image-refresh-fedora-coreos-20210513-024444 branch May 18, 2021 16:19
@martinpitt
Copy link
Member

Interesting, this PR moved the FCOS image from Fedora 33 to 34. That means that we should also switch the build image.

martinpitt added a commit to martinpitt/bots that referenced this pull request May 19, 2021
The image refresh in cockpit-project#2008
lifted the CoreOS image to Fedora 34.
@martinpitt
Copy link
Member

Done in PR #2033

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