Skip to content

ci(travis): include statik dependency - #201

Merged
juancarlostong merged 3 commits into
masterfrom
jctong/add_statik_dep
Mar 23, 2020
Merged

ci(travis): include statik dependency#201
juancarlostong merged 3 commits into
masterfrom
jctong/add_statik_dep

Conversation

@juancarlostong

@juancarlostongjuancarlostong commented Mar 23, 2020

Copy link
Copy Markdown
Contributor

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)

Comment thread.travis.yml
# 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
Copy Markdown
ContributorAuthor

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


WORKDIR /go/src/github.com/optimizely/agent
COPY . .
RUN make install

Copy link
Copy Markdown
ContributorAuthor

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

codecovBot commented Mar 23, 2020

Copy link
Copy Markdown

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 FilesCoverage Δ
pkg/server/server.go90.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.

Comment thread.travis.yml
- $TRAVIS_BUILD_DIR/scripts/ci_create_packages.sh

- stage: Build and Upload
if: type = push AND tag IS present

Copy link
Copy Markdown
ContributorAuthor

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
Copy Markdown
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
Copy Markdown
ContributorAuthor

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

@mikecdavismikecdavis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@msohailhussainmsohailhussain left a comment

Copy link
Copy Markdown
Contributor

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 masterMar 23, 2020
@juancarlostong
juancarlostong deleted the jctong/add_statik_dep branch March 23, 2020 21:01
Sign up for freeto 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

@juancarlostong@mikecdavis@msohailhussain