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

Change signoff check image to slim-buster [skip ci] #4479

Merged
merged 1 commit into from
Jan 10, 2022

Conversation

pxLi
Copy link
Collaborator

@pxLi pxLi commented Jan 10, 2022

Signed-off-by: Peixin Li pxli@nyu.edu

the alpine image became unavailable (missing several build envs) to correctly install signoff check dependencies, e.g. https://github.com/NVIDIA/spark-rapids/runs/4756311717?check_suite_focus=true

change it to use slim-buster to simply fix signoff check

Signed-off-by: Peixin Li <pxli@nyu.edu>
@pxLi pxLi added the build Related to CI / CD or cleanly building label Jan 10, 2022
@pxLi
Copy link
Collaborator Author

pxLi commented Jan 10, 2022

build

@pxLi
Copy link
Collaborator Author

pxLi commented Jan 10, 2022

Going to force merge this to unblock signoff check, and will start a dummy PR to check

@pxLi pxLi merged commit 9417ee5 into NVIDIA:branch-22.02 Jan 10, 2022
@tgravescs
Copy link
Collaborator

Please add description as to why we changed it

@pxLi
Copy link
Collaborator Author

pxLi commented Jan 11, 2022

Please add description as to why we changed it

updated PR desc

pxLi added a commit to pxLi/spark-rapids that referenced this pull request Jan 19, 2022
Signed-off-by: Peixin Li <pxli@nyu.edu>
pxLi added a commit to pxLi/spark-rapids that referenced this pull request Jan 19, 2022
Signed-off-by: Peixin Li <pxli@nyu.edu>
pxLi added a commit that referenced this pull request Jan 19, 2022
Signed-off-by: Peixin Li <pxli@nyu.edu>
pxLi added a commit that referenced this pull request Jan 19, 2022
Signed-off-by: Peixin Li <pxli@nyu.edu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Related to CI / CD or cleanly building
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants