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.

Sign up for GitHub

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

Jump to bottom

Add a second job that runs the broker on k8s #643

Merged
merged 2 commits into from
Jan 22, 2018
Merged

Add a second job that runs the broker on k8s #643

merged 2 commits into from
Jan 22, 2018

Conversation

rthallisey
Copy link
Contributor

@rthallisey rthallisey commented Jan 16, 2018 •
edited

Describe what this PR does and why we need it:
Add a second job to the travis gate the runs the broker on k8s against freshly built service-catalog images.

note: The job time doesn't increase since they run in parallel even though travis reports the total time.

Changes proposed in this pull request

  • The script run_latest_k8s_build.sh will run the broker on a k8s cluster.

depends-on: ansibleplaybookbundle/postgresql-apb#29

@openshift-ci-robot openshift-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jan 16, 2018
@rthallisey rthallisey force-pushed the k8s-scripts branch 5 times, most recently from 8e947ba to 8f8a42a Compare January 16, 2018 02:55
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 8e947ba on rthallisey:k8s-scripts into ** on openshift:master**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 9b8adbf on rthallisey:k8s-scripts into ** on openshift:master**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 49463b8 on rthallisey:k8s-scripts into ** on openshift:master**.

@rthallisey
Copy link
Contributor Author

broker running on kubernetes CI job

@rthallisey rthallisey force-pushed the k8s-scripts branch 2 times, most recently from c7826a2 to b974755 Compare January 16, 2018 14:45
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling c7826a2 on rthallisey:k8s-scripts into ** on openshift:master**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling b974755 on rthallisey:k8s-scripts into ** on openshift:master**.

@rthallisey rthallisey force-pushed the k8s-scripts branch 2 times, most recently from 8802a36 to 73b17ed Compare January 16, 2018 16:32
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 8802a36 on rthallisey:k8s-scripts into ** on openshift:master**.

@rthallisey rthallisey force-pushed the k8s-scripts branch 2 times, most recently from dbc7aed to 7cf2575 Compare January 16, 2018 17:43
@coveralls
Copy link

coveralls commented Jan 16, 2018 •
edited

Coverage Status

Changes Unknown when pulling dbc7aed on rthallisey:k8s-scripts into ** on openshift:master**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 49a6a99 on rthallisey:k8s-scripts into ** on openshift:master**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 49a6a99 on rthallisey:k8s-scripts into ** on openshift:master**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling addc97d on rthallisey:k8s-scripts into ** on openshift:master**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling ce1e91e on rthallisey:k8s-scripts into ** on openshift:master**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling a4583b6 on rthallisey:k8s-scripts into ** on openshift:master**.

@rthallisey rthallisey force-pushed the k8s-scripts branch 2 times, most recently from addc97d to 21a4180 Compare January 19, 2018 19:00
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 21a4180 on rthallisey:k8s-scripts into ** on openshift:master**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 71821be on rthallisey:k8s-scripts into ** on openshift:master**.

@rthallisey rthallisey changed the title [WIP] Add a second job that runs the broker on k8s Add a second job that runs the broker on k8s Jan 22, 2018
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 22, 2018
jmrodri
jmrodri approved these changes Jan 22, 2018
View reviewed changes
Copy link
Contributor

@jmrodri jmrodri left a comment

Choose a reason for hiding this comment

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

ACK

djzager
djzager approved these changes Jan 22, 2018
View reviewed changes
Copy link
Member

@djzager djzager left a comment

Choose a reason for hiding this comment

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

ACK. Looks like you have a conflict on .travis.yml

@djzager djzager merged commit de6d98a into openshift:master Jan 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmrodri jmrodri jmrodri approved these changes

@djzager djzager djzager approved these changes

@shawn-hurley shawn-hurley Awaiting requested review from shawn-hurley

Assignees
No one assigned
Labels
size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@rthallisey @coveralls @jmrodri @djzager @openshift-ci-robot