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

fix: pre-commit auto update commit author #102

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

maryamtahhan
Copy link
Contributor

@maryamtahhan maryamtahhan commented Jan 6, 2025

@maryamtahhan
Copy link
Contributor Author

@SamYuan1990 Can you please have a look? this should set the committer as github-actions[bot] moving forward

@maryamtahhan
Copy link
Contributor Author

The unit test failure seems related to authentication:

#9 [5/7] RUN sudo ./main.sh prerequisites
#9 0.086 sudo: PAM account management error: Authentication service cannot retrieve authentication info
#9 0.086 sudo: a password is required
#9 ERROR: process "/bin/sh -c sudo ./main.sh prerequisites" did not complete successfully: exit code: 1
------
 > [5/7] RUN sudo ./main.sh prerequisites:
0.086 sudo: PAM account management error: Authentication service cannot retrieve authentication info
0.086 sudo: a password is required
------
WARNING: No output specified with docker-container driver. Build result will only remain in the build cache. To push result image into registry use --push or to load image into docker use --load

 1 warning found (use docker --debug to expand):
 - ConsistentInstructionCasing: Command 'From' should match the case of the command majority (uppercase) (line 1)
ubi.Dockerfile:12

@SamYuan1990
Copy link
Contributor

@SamYuan1990 Can you please have a look? this should set the committer as github-actions[bot] moving forward

I suppose we need @rootfs, @sthaha or @dave-tucker 's help here.
ref
https://access.redhat.com/solutions/4060861

the error msg for current ci as sudo: PAM account management error: Authentication service cannot retrieve authentication info seems some ubi image/pem setting changes so that user 0 can't use sudo in container image test approach to run some scripts.
is there any suggestion?

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