Skip to content

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

Merged
merged 3 commits into from
Mar 23, 2020
Merged

Conversation

juancarlostong
Copy link
Contributor

@juancarlostong juancarlostong commented Mar 23, 2020

Summary

  • due to new statik depedency, added extra line to run make install to the build Dockerfiles
  • made uploading packages contingent on successful package creation (previously it was uploading regardless)

# 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
Copy link
Contributor Author

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
Copy link
Contributor Author

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
Copy link

codecov bot commented Mar 23, 2020

Codecov Report

Merging #201 into master will decrease coverage by 0.08%.
The diff coverage is n/a.

Impacted file tree graph

@@            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              
Impacted Files Coverage Δ
pkg/server/server.go 90.00% <0.00%> (-1.43%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c866570...4f69d63. Read the comment docs.

- 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
Copy link
Contributor Author

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

Copy link
Contributor

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.

Copy link
Contributor Author

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

Copy link
Contributor

@mikecdavis mikecdavis left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@msohailhussain msohailhussain left a comment

Choose a reason for hiding this comment

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

lgtm

@juancarlostong juancarlostong merged commit 754ed1a into master Mar 23, 2020
@juancarlostong juancarlostong deleted the jctong/add_statik_dep branch March 23, 2020 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants