-
Notifications
You must be signed in to change notification settings - Fork 9
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
Travis CI configuration #114
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you move all yml file to different directory, for example subdirectory templates
?
.cilibs/compile_code.sh
Outdated
done | ||
|
||
echo "Compile code" | ||
if [ -z "$TRAVIS_BRANCH" ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And what about "$CIRCLECI_BRANCH"
part ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, it was messed up, should work now.
FLOODGATE_EXTRA_PARAMS=$1 | ||
|
||
echo Test Floodgate against running Spinnaker instance | ||
/floodgate/bin/floodgate --version |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe you can add some info about what is happening before executing this commands?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added printing as was in .circleci/config.yml
.travis.yml
Outdated
go: | ||
- 1.14.1 | ||
env: | ||
- REALESE_BRANCHES="release-v[0-9]+\.[0-9]+\.x" RELEASE_TAGS="v[0-9]+\.[0-9]+\.[0-9]+(-rc[0-9]+)?" BUILD_OS="$([ $TRAVIS_EVENT_TYPE == cron ] || [[ $TRAVIS_BRANCH =~ $RELEASE_BRANCHES ]] || [[ $TRAVIS_TAG =~ $RELEASE_TAGS ]] && echo darwin) linux" BUILD_ARCH=amd64 GATE_API_BRANCH=release-1.20.x SEND_COVERITY=send |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you use here multiline syntax for example with |
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Improved readability but I don't see how I could make declaration of BUILD_ARCH more clear.
@@ -0,0 +1,19 @@ | |||
dist: xenial |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about kind of periodic workflow?
4e1a2e2
to
35cbdd5
Compare
PR includes configuration and scripts needed to use travis as CI. All code is moved to bash scripts that are used by both CircleCI and Travis.
What is missing in comparison with CircleCI: