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

Drop Go 1.4 conditions, switch to 1.6 by default #8975

Merged
merged 2 commits into from
May 24, 2016

Conversation

smarterclayton
Copy link
Contributor

[test]

@smarterclayton
Copy link
Contributor Author

@csrwng can you fix these govet errors so I can kick this over?

@smarterclayton
Copy link
Contributor Author

@sdodson @brenton FYI I'm preparing to drop Go 1.4.2 support from Origin, so the next master branch pick up for OSE will probably need to switch as well.

@csrwng
Copy link
Contributor

csrwng commented May 23, 2016

Submitted #8984

@smarterclayton
Copy link
Contributor Author

[test]

1 similar comment
@smarterclayton
Copy link
Contributor Author

[test]

@csrwng
Copy link
Contributor

csrwng commented May 24, 2016

Hitting #8946
and extended test failure on readiness probe.

[test]

@sdodson
Copy link
Member

sdodson commented May 24, 2016

It'd be nice if we handled rpm builds in this same pr. smarterclayton#19 will take care of that

* [RPMs] Refactor golang BuildRequires

* [RPMs] Bump buildrequires to 1.6.2, drop other arch deps
@smarterclayton
Copy link
Contributor Author

[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 633f5c6

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/4031/)

@smarterclayton
Copy link
Contributor Author

[merge]

@openshift-bot
Copy link
Contributor

openshift-bot commented May 24, 2016

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/4037/) (Image: devenv-rhel7_4274)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 633f5c6

@openshift-bot openshift-bot merged commit c12d753 into openshift:master May 24, 2016
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.

4 participants