Skip to content
This repository has been archived by the owner on Feb 5, 2020. It is now read-only.

config.tf: bump versions for 1.7.9-tectonic.4 #2653

Merged
merged 1 commit into from
Jan 4, 2018
Merged

config.tf: bump versions for 1.7.9-tectonic.4 #2653

merged 1 commit into from
Jan 4, 2018

Conversation

diegs
Copy link
Contributor

@diegs diegs commented Jan 4, 2018

  • console
  • kvo
  • kubernetes
  • tectonic

- console
- kvo
- kubernetes
- tectonic
@diegs
Copy link
Contributor Author

diegs commented Jan 4, 2018

test upgrade from 1.7.5-tectonic.1 1.7.5-tectonic.1

@sudhaponnaganti
Copy link
Contributor

@yifan-gu @Quentin-M - can you review this

@diegs
Copy link
Contributor Author

diegs commented Jan 4, 2018

@cpanato any ideas about the azure failures? Also seeing them in #2654

@cpanato
Copy link
Contributor

cpanato commented Jan 4, 2018

retest this please

@cpanato
Copy link
Contributor

cpanato commented Jan 4, 2018

re-running only on Azure because AWS and Bare Metal already passed.

@cpanato
Copy link
Contributor

cpanato commented Jan 4, 2018

I ran manually the conformance tests on AWS and it passed:

Ran 124 of 699 Specs in 3115.225 seconds
SUCCESS! -- 124 Passed | 0 Failed | 0 Pending | 575 Skipped PASS

@cpanato
Copy link
Contributor

cpanato commented Jan 4, 2018

All tests are green, jenkins did not report back the status to github.

@sudhaponnaganti
Copy link
Contributor

@cpanato - why status is not being exiting - looks like all checks are done.

@cpanato
Copy link
Contributor

cpanato commented Jan 4, 2018

for this branch the github/jenkins sometimes don't report properly. @sudhaponnaganti
All tests are green. good to go.

@diegs
Copy link
Contributor Author

diegs commented Jan 4, 2018

@cpanato @yifan-gu will need an approval to merge

Copy link
Contributor

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

From my side lgtm

@diegs
Copy link
Contributor Author

diegs commented Jan 4, 2018

Thanks, I will also need @cpanato or @yifan-gu to merge since I do not have privileges to override the stuck status check.

@cpanato cpanato merged commit 736a62f into coreos:1.7.9 Jan 4, 2018
@cpanato
Copy link
Contributor

cpanato commented Jan 4, 2018

@diegs done

@diegs diegs deleted the kvo branch January 4, 2018 19:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants