Skip to content
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

Fix travis script #1067

Merged
merged 3 commits into from
Feb 2, 2017
Merged

Fix travis script #1067

merged 3 commits into from
Feb 2, 2017

Conversation

emilevauge
Copy link
Member

@emilevauge emilevauge commented Jan 25, 2017

This PR fixes Travis script issues from #843:

  • add skip_cleanup in pages & releases
  • deploy traefik* in releases (in order to deploy traefik default linux binary)
  • fix provider script instead of scripts
  • fix how yarn is installed
  • remove docker 1.13 from tests (really, really, really unstable)
  • how do i pronounce this damn project ?

There are still some strange issue on this new script. For example, this job is green but there is one integration test failing https://travis-ci.org/containous/traefik/jobs/194860155#L2078.

/cc @guilhem @vdemeester

Signed-off-by: Emile Vauge [email protected]

Copy link
Contributor

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

Step 3 : COPY dist/traefik /

lstat dist/traefik: no such file or directory

.travis.yml Outdated
install:
- docker version
- pip install --user -r requirements.txt
- 'if [ "$TRAVIS_EVENT_TYPE" = "cron" ]; then
Copy link
Contributor

Choose a reason for hiding this comment

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

We do have a travis cron job ? 🤔

Copy link
Member Author

Choose a reason for hiding this comment

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

We do :)

Copy link
Contributor

Choose a reason for hiding this comment

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

Nice, I didn't know 😅

@emilevauge emilevauge force-pushed the fix-travis-script branch 5 times, most recently from 5f06c89 to 868cb8e Compare January 30, 2017 11:48
@Russell-IO
Copy link
Contributor

@emilevauge build seems to be stuck here

@emilevauge emilevauge force-pushed the fix-travis-script branch 3 times, most recently from 79ae76b to 0a4c724 Compare February 1, 2017 20:10
@emilevauge
Copy link
Member Author

This PR should have been great.
I tried to pre-build base images (including glide and yarn dependencies) using Travis cron in order to get faster builds. But in the end, NOTHING changed. Jobs were still slow as hell.

So I reverted everything and just added few fixes...

Copy link
Contributor

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

LGTM 🐸

@vdemeester vdemeester merged commit c118237 into master Feb 2, 2017
@vdemeester vdemeester deleted the fix-travis-script branch February 2, 2017 09:58
jangie pushed a commit to jangie/traefik that referenced this pull request Feb 23, 2017
* Fix travis script

Signed-off-by: Emile Vauge <[email protected]>

* how do i pronounce this damn project

Signed-off-by: Emile Vauge <[email protected]>

* Remove unstable Docker 1.13 tests

Signed-off-by: Emile Vauge <[email protected]>
@ldez ldez added this to the 1.2 milestone Oct 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants