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 rhel-8-7 #3585

Merged
merged 2 commits into from
Jul 8, 2022
Merged

Conversation

cockpituous
Copy link
Contributor

@cockpituous cockpituous commented Jul 4, 2022

Image refresh for rhel-8-7

  • image-refresh rhel-8-7

@github-actions github-actions bot added the bot label Jul 4, 2022
@cockpituous
Copy link
Contributor

image-refresh in progress on 3-ci-srv-01.
Log: https://logs.cockpit-project.org/logs/image-refresh-3585-20220704-224141/

@cockpituous cockpituous changed the title Image refresh for rhel-8-7 WIP: 3-ci-srv-01: [no-test] Image refresh for rhel-8-7 Jul 4, 2022
@cockpituous cockpituous changed the title WIP: 3-ci-srv-01: [no-test] Image refresh for rhel-8-7 WIP: 4-ci-srv-04: [no-test] Image refresh for rhel-8-7 Jul 4, 2022
@cockpituous
Copy link
Contributor

image-refresh in progress on 4-ci-srv-04.
Log: https://logs.cockpit-project.org/logs/image-refresh-3585-20220704-224141/

@cockpituous
Copy link
Contributor

@cockpituous cockpituous force-pushed the image-refresh-rhel-8-7-20220704-231459 branch from c3f7236 to d759ef8 Compare July 4, 2022 23:15
@cockpituous cockpituous changed the title WIP: 4-ci-srv-04: [no-test] Image refresh for rhel-8-7 Image refresh for rhel-8-7 Jul 4, 2022
@jelly
Copy link
Member

jelly commented Jul 5, 2022

subscription manager issues:

audit: type=1400 audit(1656976966.994:7): avc:  denied  { write } for  pid=23541 comm="gpg" name="trustdb.gpg" dev="vda3" ino=8388855 scontext=system_u:system_r:insights_client_t:s0 tcontext=unconfined_u:object_r:admin_home_t:s0 tclass=file permissive=0
audit: type=1400 audit(1656976970.662:8): avc:  denied  { name_connect } for  pid=23578 comm="platform-python" dest=8888 scontext=system_u:system_r:insights_client_t:s0 tcontext=system_u:object_r:unreserved_port_t:s0 tclass=tcp_socket permissive=0

audit: type=1400 audit(1656977075.514:11): avc:  denied  { write } for  pid=23562 comm="gpg" name="trustdb.gpg" dev="vda3" ino=8388855 scontext=system_u:system_r:insights_client_t:s0 tcontext=unconfined_u:object_r:admin_home_t:s0 tclass=file permissive=0

And test/verify/check-system-services TestServices.testBasicSession which fails and that might have something to do with the systemd update?

Notable updates

  selinux-policy (3.14.3-100.el8 -> 3.14.3-104.el8)
  selinux-policy-targeted (3.14.3-100.el8 -> 3.14.3-104.el8)
  subscription-manager (1.28.30-1.el8 -> 1.28.31-1.el8)
  subscription-manager-cockpit (1.28.30-1.el8 -> 1.28.31-1.el8)
  subscription-manager-rhsm-certificates (1.28.30-1.el8 -> 1.28.31-1.el8)
  systemd (239-59.el8 -> 239-60.el8)
  systemd-container (239-59.el8 -> 239-60.el8)
  systemd-libs (239-59.el8 -> 239-60.el8)
  systemd-pam (239-59.el8 -> 239-60.el8)
  systemd-udev (239-59.el8 -> 239-60.el8)

@marusak
Copy link
Member

marusak commented Jul 7, 2022

@jelly Is this about Next run being one day earlier? If so, then I believe this is related to the fact when we do refreshes. We used to do them in early morning but then we moved to late evening. See 83086d7

There might be mismatch between system and browser time. In the test, we calculate system time, but it might be that browser time is different - or maybe not, see cockpit-project/cockpit#15626

But in any case, this test fails only when you trigger it late in night, if you retry it, it will succeed just fine. But indeed would be nice to fix this, as we need to retry a lot of tests now.

@jelly
Copy link
Member

jelly commented Jul 7, 2022

@jelly Is this about Next run being one day earlier? If so, then I believe this is related to the fact when we do refreshes. We used to do them in early morning but then we moved to late evening. See 83086d7

There might be mismatch between system and browser time. In the test, we calculate system time, but it might be that browser time is different - or maybe not, see cockpit-project/cockpit#15626

But in any case, this test fails only when you trigger it late in night, if you retry it, it will succeed just fine. But indeed would be nice to fix this, as we need to retry a lot of tests now.

Created an issue to track this in cockpit, as I don't want it to block the refresh from landing.

@jelly jelly force-pushed the image-refresh-rhel-8-7-20220704-231459 branch from 13a52bf to aba5334 Compare July 7, 2022 14:53
@jelly jelly requested a review from marusak July 8, 2022 06:50
Copy link
Member

@marusak marusak left a comment

Choose a reason for hiding this comment

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

Thank you!

@jelly jelly merged commit 45e01fc into main Jul 8, 2022
jelly pushed a commit that referenced this pull request Jul 8, 2022
@jelly jelly deleted the image-refresh-rhel-8-7-20220704-231459 branch July 8, 2022 07:28
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