Integrate a new ci framework for travis #463
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Early WIP. Want to see travis run this.
|
rthallisey
added
the
do-not-merge
rthallisey
force-pushed
the
new-ci-framework
branch
8 times, most recently
from
88dd8a6
to
553757f
Compare
openshift-ci-robot
added
the
size/L
rthallisey
force-pushed
the
new-ci-framework
branch
2 times, most recently
from
3266ff6
to
4c12f65
Compare
openshift-ci-robot
added
size/XL
rthallisey
force-pushed
the
new-ci-framework
branch
2 times, most recently
from
fededf9
to
f54d0e3
Compare
1 similar comment
rthallisey
force-pushed
the
new-ci-framework
branch
2 times, most recently
from
fb24cbe
to
f182529
Compare
rthallisey
removed
the
do-not-merge
rthallisey
force-pushed
the
new-ci-framework
branch
from
f182529
to
a6366d4
Compare
rthallisey
force-pushed
the
new-ci-framework
branch
from
a6366d4
to
c2fc08a
Compare
rthallisey
force-pushed
the
new-ci-framework
branch
from
c2fc08a
to
4bef642
Compare
openshift-ci-robot
added
size/L
jmrodri
added
3.8
do-not-merge
|
We have decided per review to wait until Wednesday November 8th, 2017 before merging. At that point it will be ready to go. |
rthallisey
removed
the
do-not-merge
djzager
approved these changes
Nov 10, 2017
rthallisey
force-pushed
the
new-ci-framework
branch
from
c960ff4
to
77a2ee7
Compare
|
Let's merge the ansibleplaybookbundle CI first so I can reference those templates in config.yaml depends-on: ansibleplaybookbundle/postgresql-apb#7 |
added 18 commits
November 14, 2017 14:04
This repo should carry it's own config.yaml since it's a consumer of the new framework. Remove the local ci scripts and move the templates to the templates directory.
- Add folded up broker an catalog logs to the end of a run - Error properly when setup.sh fails
Verify the secret is created instead of the podpreset
We don't need the broker scripts anymore. We're moving to the new framework.
Make these scripts individually consumable so the APB gates can use them.
The gate job for the broker runs the asb with shell scripts. Make sure we use catasb for non asb jobs.
|
Changes Unknown when pulling 77a2ee7 on rthallisey:new-ci-framework into ** on openshift:master**. |
|
Ok both dependent PRs were merged. ansibleplaybookbundle/mediawiki-apb#8 Merging this now. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Describe what this PR does and why we need it:
New ci framework will move away from have a set of bash scripts to run only mediawiki and postgresql. This framework will allow us to test any apb.
Changes proposed in this pull request