Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

skip euleros build due to timeout reason #56

Merged
merged 1 commit into from
Mar 1, 2018
Merged

skip euleros build due to timeout reason #56

merged 1 commit into from
Mar 1, 2018

Conversation

liangchenye
Copy link
Contributor

@liangchenye liangchenye commented Feb 1, 2018

Fixes #46.

Signed-off-by: Liang Chenye [email protected]

@jcvenegas
Copy link
Member

jcvenegas commented Feb 1, 2018

Hi, @liangchenye if we still had some timeouts due to not output given the travis timeout, could you try to edit the line
https://github.com/kata-containers/osbuilder/blob/master/.travis.yml#L19
to

script: 
- "travis_wait 30 .ci/run.sh"

Acording to travis docs that will increase the wait time to 30 min (the current is 20 according to travis)
https://docs.travis-ci.com/user/common-build-problems/#Build-times-out-because-no-output-was-received

@jodh-intel
Copy link
Contributor

Random idea: you could try contacting https://www.mirrorservice.org/ (or some other mirroring site) and see if they would mirror euleros. They already mirror centos and scientific linux (centos based):

@liangchenye
Copy link
Contributor Author

@jodh-intel I got feedback from the maintainer of 'mirrorservice.org', they prefer to host non-commercial product. So they mirror centos/opensuse and don't mirror RHEL/SLE.
EulerOS is more like a 'commercial product', so I agree it is not suitable to mirror it there.

We do have a community brand 'iSula' which is a container OS, but it is quite new. (hopefully one day it will be well-known.)

@liangchenye
Copy link
Contributor Author

liangchenye commented Feb 3, 2018

@jcvenegas I contact our host server maintainer, he said there are some traffic issues on 1.31st. The builder job failed on the same day. https://travis-ci.org/kata-containers/osbuilder/builds/335289441

So maybe I can close this PR?

@jodh-intel
Copy link
Contributor

jodh-intel commented Feb 5, 2018

Hi @liangchenye - thanks for the update.

lgtm

Approved with PullApprove

Copy link
Contributor

@grahamwhaley grahamwhaley left a comment

Choose a reason for hiding this comment

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

lgtm
but I think this now needs a rebase?

@jcvenegas
Copy link
Member

jcvenegas commented Feb 9, 2018

So maybe I can close this PR?

@liangchenye sure we can close it, feel free to increase the time out as well if you think will need it in the future.

@jodh-intel
Copy link
Contributor

Hi @liangchenye - do you plan to close this then, or do you want to rebase so we can merge it?

@liangchenye
Copy link
Contributor Author

The build failed again,I'll rebase it. I'm on vacation,I‘ll do it the day after tomorrow.

@liangchenye
Copy link
Contributor Author

rebased, PTAL @jodh-intel

PS,
I built euleros locally by running build_rootfs_image_initrd euleros, it works.

@jodh-intel
Copy link
Contributor

Thanks @liangchenye!

I built euleros locally by running build_rootfs_image_initrd euleros, it works.

Great! 😄

lgtm

@jodh-intel
Copy link
Contributor

ping @kata-containers/builder

@jodh-intel jodh-intel mentioned this pull request Feb 22, 2018
@jodh-intel
Copy link
Contributor

Force merging since:

  • we've waited long enough for this to be nacked.
  • we need this to make the CI more stable...

@jodh-intel jodh-intel merged commit 56eb88b into kata-containers:master Mar 1, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants