Skip to content

Switch to go.mod and bump to k8s 1.17.1#16

Merged
openshift-merge-robot merged 11 commits intoopenshift:masterfrom
soltysh:bump_k8s
Jan 21, 2020
Merged

Switch to go.mod and bump to k8s 1.17.1#16
openshift-merge-robot merged 11 commits intoopenshift:masterfrom
soltysh:bump_k8s

Conversation

@soltysh
Copy link
Copy Markdown
Contributor

@soltysh soltysh commented Jan 20, 2020

Picks some of the changes from #10 but I started with go.mod and bump first. More will follow up to land something along the lines like #10.
/assign @deads2k

Comment thread go.mod
k8s.io/kubernetes v1.17.1
)

replace (
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

later try to remove replaces. My guess is that k/k is requiring several of these replaces. If so, just comment to that effect.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I already tried, the ones required are the top 4, although I might try getting rid of glog now for sure, probably ginkgo too the other two has to stay. The k8s needs to be there since go.mod can't parse them out from k/k since it points to staging 😞

@deads2k
Copy link
Copy Markdown
Contributor

deads2k commented Jan 20, 2020

/approve

@soltysh
Copy link
Copy Markdown
Contributor Author

soltysh commented Jan 20, 2020

Fixed deps, let's wait for the tests now.

@openshift-ci-robot openshift-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Jan 20, 2020
@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 20, 2020
@soltysh
Copy link
Copy Markdown
Contributor Author

soltysh commented Jan 20, 2020

/retest

@deads2k
Copy link
Copy Markdown
Contributor

deads2k commented Jan 21, 2020

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jan 21, 2020
@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: deads2k, soltysh

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

2 similar comments
@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-merge-robot openshift-merge-robot merged commit 2ef9b85 into openshift:master Jan 21, 2020
@soltysh soltysh deleted the bump_k8s branch January 22, 2020 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants