-
Notifications
You must be signed in to change notification settings - Fork 27
ci(travis): include statik dependency #201
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
Conversation
# upload packages to artifactory | ||
- $TRAVIS_BUILD_DIR/scripts/ci_upload_packages.sh | ||
# now we're going to create packages & upload packages | ||
- $TRAVIS_BUILD_DIR/scripts/ci_create_packages.sh && $TRAVIS_BUILD_DIR/scripts/ci_upload_packages.sh |
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.
this ought to stop it from uploading if packages failed to be created
@@ -3,6 +3,7 @@ FROM golang:$GO_VERSION as builder | |||
|
|||
WORKDIR /go/src/github.com/optimizely/agent | |||
COPY . . | |||
RUN make install |
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.
this drops in the new dependency on statik
Codecov Report
@@ Coverage Diff @@
## master #201 +/- ##
==========================================
- Coverage 82.11% 82.03% -0.09%
==========================================
Files 27 27
Lines 1230 1230
==========================================
- Hits 1010 1009 -1
- Misses 165 166 +1
Partials 55 55
Continue to review full report at Codecov.
|
- export APP_VERSION=$(git describe --abbrev=0 --tags | tr -d '^v') | ||
- $TRAVIS_BUILD_DIR/scripts/ci_create_packages.sh | ||
|
||
- stage: Build and Upload | ||
if: type = push AND tag IS present |
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 a new stage to test building docker images without uploading them... less clunky debugging
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.
I will recommend to use wait-for-it
to check port is opened.
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.
agent is not being run, its just built and packaged so there's no port involved here
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.
LGTM
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.
lgtm
Summary
make install
to the build Dockerfiles