Skip to content

chore(deps): update plugins #177

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

Merged
merged 1 commit into from
May 4, 2021
Merged

chore(deps): update plugins #177

merged 1 commit into from
May 4, 2021

Conversation

updatecli-gha-in-jenkins-infra-org[bot]
Copy link
Contributor

Automated changes by create-pull-request GitHub action

Copy link
Contributor

@MarkEWaite MarkEWaite left a comment

Choose a reason for hiding this comment

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

Working for me in my installation

@dduportal
Copy link
Contributor

Sounds like the pod in charge of building this image are OOM-killed. To be diagnosed by the team.

@garethjevans
Copy link
Contributor

any changes to the pipeline library? or infra.ci config that could have caused this? i think it started happening on friday.

@dduportal
Copy link
Contributor

dduportal commented May 4, 2021

I've merged the PR allowing to provide custom Docker images 5 days ago: jenkins-infra/pipeline-library@7d3a5f3
It might have cause the change.

One of the changes is the switch from the podtemplate yml to a scripted version of the container. I now realize that the yml template had an indentation issue around the resources/limits: it means that the builder container was not limited in resource, and it is since friday 😭 Let me open a PR to increase the resources and let's try here

@dduportal
Copy link
Contributor

Ping @garethjevans do you minde adding the following lineat the beginning of the Jenkinsfile of this PR:

@Library('pipeline-library@pull/207/head') _

to be able to validate the resources/limit increased proposed in jenkins-infra/pipeline-library#207 ?

@dduportal
Copy link
Contributor

Shared Library fixed: this PR should be OK

@garethjevans garethjevans merged commit df9bfea into main May 4, 2021
@garethjevans garethjevans deleted the update-plugins branch May 4, 2021 11:15
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.

3 participants