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
CONSOLE-2494: Bump node and yarn versions in docker builder file #7306
Conversation
/hold Waiting for PR7304 to merge |
Let's update kubectl as well. |
I don't think we need to wait for the ART change although #7304 is in the merge queue. |
@spadgett |
Wouldn't the yarn tarball name in Dockerfile.product also need to be updated for this? |
/retest |
Isn't that part of #7304? I don't think it needs to be in one PR. It shouldn't break anything if done separately. |
#7304 doesn't touch yarn, only nodejs. |
/retest |
it seems that the new node/yarn may have broken the analyze script: openshift/release#13715 |
/hold cancel |
@spadgett - I think all the other dependencies for this update has been merged. Can we get an LGTM? |
@suomiy Sorry, missed this. It looks like we're running into this problem? webpack/webpack#4629 |
/hold |
@spadgett seems like this issue got fixed recently. Should we update webpack-cli to the newest version to proceed with this one? |
Sounds good to me. We're working on updating our dependencies anyway. Thanks! |
|
/lgtm |
/approve |
1 similar comment
/approve |
@bparees can you lgtm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@zherman0 I think we should go ahead and move to the latest kubectl as well, 1.20.4.
@zherman0 Can you update the |
openshift/release#13715 has merged |
I just built and pushed the latest image to quay: https://quay.io/repository/coreos/tectonic-console-builder?tab=tags |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: glekner, spadgett, vojtechszocs, zherman0 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@zherman0: The following test failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
/retest |
/hold cancel |
Need to wait for [ART-2456] to be complete and for PR7304 to merge.
This bumps dependencies for nodejs and yarn in the build file.