Skip to content

bump(*)#86

Merged
openshift-merge-robot merged 2 commits intoopenshift:masterfrom
coreydaley:bump_deps
Apr 18, 2019
Merged

bump(*)#86
openshift-merge-robot merged 2 commits intoopenshift:masterfrom
coreydaley:bump_deps

Conversation

@coreydaley
Copy link
Copy Markdown

@coreydaley coreydaley commented Apr 10, 2019

Includes bump due to openshift/api#285
changes to use go-bindata/go-bindata
updates to use klog

@openshift-ci-robot openshift-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Apr 10, 2019
@coreydaley
Copy link
Copy Markdown
Author

/assign @adambkaplan

New pull request to bump deps

Copy link
Copy Markdown
Contributor

@adambkaplan adambkaplan left a comment

Choose a reason for hiding this comment

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

Looks like api and client-go bumps weren't clean.

If we're upgrading api and client-go to master, we must also update the kube pins to point to 1.13 tags/branches.

Comment thread glide.lock Outdated
@coreydaley
Copy link
Copy Markdown
Author

/retest

aws_vpc.new_vpc: Error creating VPC: VpcLimitExceeded: The maximum number of VPCs has been reached.

@coreydaley
Copy link
Copy Markdown
Author

@adambkaplan ptal

1 similar comment
@coreydaley
Copy link
Copy Markdown
Author

@adambkaplan ptal

@coreydaley
Copy link
Copy Markdown
Author

/retest

failed to initialize the cluster: Working towards 0.0.1-2019-04-11-160713: 91% complete: timed out waiting for the condition

@coreydaley
Copy link
Copy Markdown
Author

/retest

4 similar comments
@coreydaley
Copy link
Copy Markdown
Author

/retest

@coreydaley
Copy link
Copy Markdown
Author

/retest

@coreydaley
Copy link
Copy Markdown
Author

/retest

@coreydaley
Copy link
Copy Markdown
Author

/retest

Comment thread glide.yaml
Comment thread glide.yaml Outdated
@coreydaley
Copy link
Copy Markdown
Author

/retest

2 similar comments
@coreydaley
Copy link
Copy Markdown
Author

/retest

@coreydaley
Copy link
Copy Markdown
Author

/retest

Copy link
Copy Markdown
Contributor

@adambkaplan adambkaplan left a comment

Choose a reason for hiding this comment

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

looks good, but want @deads2k to take another pass before signing off.

/approve

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 12, 2019
@coreydaley
Copy link
Copy Markdown
Author

/retest

@adambkaplan
Copy link
Copy Markdown
Contributor

/hold

Other control plane operators are not on golang 1.11

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 12, 2019
@coreydaley
Copy link
Copy Markdown
Author

/retest

@coreydaley
Copy link
Copy Markdown
Author

@adambkaplan ptal, dropped the go1.11 commit

@adambkaplan
Copy link
Copy Markdown
Contributor

/hold cancel

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 15, 2019
@adambkaplan
Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Apr 15, 2019
@coreydaley
Copy link
Copy Markdown
Author

/retest

@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

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

@coreydaley
Copy link
Copy Markdown
Author

/retest

@coreydaley
Copy link
Copy Markdown
Author

/retest

could not copy stable imagestreamtag: Timeout: request did not complete within allowed duration

@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

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

@coreydaley
Copy link
Copy Markdown
Author

/retest

3 similar comments
@coreydaley
Copy link
Copy Markdown
Author

/retest

@coreydaley
Copy link
Copy Markdown
Author

/retest

@coreydaley
Copy link
Copy Markdown
Author

/retest

@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.

@coreydaley
Copy link
Copy Markdown
Author

/hold
needs another dependency bump when library-go pr has merged openshift/library-go#354

@openshift-ci-robot openshift-ci-robot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. and removed lgtm Indicates that a PR is ready to be merged. labels Apr 17, 2019
@coreydaley
Copy link
Copy Markdown
Author

/hold cancel

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 18, 2019
@coreydaley
Copy link
Copy Markdown
Author

coreydaley commented Apr 18, 2019

/retest

level=error msg="\t* module.vpc.aws_route.to_nat_gw[1]: 1 error occurred:"
level=error msg="\t* aws_route.to_nat_gw.1: Error creating route: timeout while waiting for state to become 'success' (timeout: 2m0s)"

@coreydaley
Copy link
Copy Markdown
Author

@adambkaplan ptal, I think I got it this time, the last failure was a cluster installation issue

@adambkaplan
Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Apr 18, 2019
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adambkaplan, coreydaley

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-merge-robot openshift-merge-robot merged commit 04a593c into openshift:master Apr 18, 2019
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.

6 participants