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 debian-testing #7410

Merged
merged 2 commits into from
Feb 11, 2025

Conversation

cockpituous
Copy link
Contributor

@cockpituous cockpituous commented Feb 8, 2025

Image refresh for debian-testing

  • image-refresh debian-testing

@github-actions github-actions bot added the bot label Feb 8, 2025
@cockpituous cockpituous changed the title Image refresh for debian-testing WIP: e9bbbaa5ef1e: [no-test] Image refresh for debian-testing Feb 8, 2025
@cockpituous
Copy link
Contributor

cockpituous pushed a commit that referenced this pull request Feb 8, 2025
@cockpituous cockpituous force-pushed the image-refresh-debian-testing-20250208-225428 branch from 673ada8 to 593b818 Compare February 8, 2025 22:54
@cockpituous cockpituous changed the title WIP: e9bbbaa5ef1e: [no-test] Image refresh for debian-testing Image refresh for debian-testing Feb 8, 2025
@cockpituous
Copy link
Contributor

@martinpitt
Copy link
Member

retrying for comparison

@martinpitt
Copy link
Member

Failure repeats, this is real. Adding to pilot board.

@allisonkarlitskaya
Copy link
Member

    checkMotdForUser(string, expected=old_pam and expected, user='bob')
  libpam-lastlog2:amd64 (2.40.4-1 -> 2.40.4-2)
  libpam-modules-bin (1.5.3-7+b1 -> 1.7.0-2)
  libpam-modules:amd64 (1.5.3-7+b1 -> 1.7.0-2)
  libpam-runtime (1.5.3-7 -> 1.7.0-2)
  libpam-wtmpdb:amd64 (0.13.0-4 -> 0.13.0-5)
  libpam0g:amd64 (1.5.3-7+b1 -> 1.7.0-2)

Right. This should be pretty straight-forward.

@allisonkarlitskaya
Copy link
Member

Not so straight-forward. debian-testing is not on the old_pam list, and the comment notes that "old" means "before 1.5.0" but we're going from 1.5.x to 1.7. I guess PAM changed its behaviour again...

@allisonkarlitskaya allisonkarlitskaya self-assigned this Feb 10, 2025
@allisonkarlitskaya
Copy link
Member

This is starting to smell like a PAM regression... the user bob shouldn't be able to see that message, but he can.

@allisonkarlitskaya allisonkarlitskaya force-pushed the image-refresh-debian-testing-20250208-225428 branch from 593b818 to 0155f31 Compare February 11, 2025 08:49
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!

@allisonkarlitskaya allisonkarlitskaya merged commit c8c2b98 into main Feb 11, 2025
12 checks passed
allisonkarlitskaya pushed a commit that referenced this pull request Feb 11, 2025
@allisonkarlitskaya allisonkarlitskaya deleted the image-refresh-debian-testing-20250208-225428 branch February 11, 2025 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants