Scout24 / c-bastion

Jump-host functionality, in the cloud.
6 stars 3 forks source link

Ssh personal environment #7

Closed schlomo closed 8 years ago

schlomo commented 8 years ago
esc commented 8 years ago

Travis had a hickup, I suppose, restarting.

esc commented 8 years ago

it seems like the docker integration is broken.

schlomo commented 8 years ago

It hangs on docker push. IMHO doing a docker push on a PR build is wrong, that should only happen for builds from master.

esc commented 8 years ago

Yeah, see #6

esc commented 8 years ago

Last time I tried, the travis.yaml wasn't flexible enough to allow this.

esc commented 8 years ago

also, this has been working all along, so it is surprising that the upload is breaking now.

esc commented 8 years ago

Regarding the logging stuff, I'd like to reject this change for the time being, at least until the logging situation in Taupage has been cleaned up. This is just a temporary rejection. How do you feel about that?

esc commented 8 years ago

Since #6 has been fixed, please rebase against master and I'll merge it.

esc commented 8 years ago

Wahay, travis is green!

esc commented 8 years ago

Let me know, if you figure out where the logs go in Taupage.

esc commented 8 years ago

Also, you'll need to wait a few minutes before Travis builds the master and the image becomes available at DockerHub.

schlomo commented 8 years ago

/var/log/application.log seems to be the default place for the logs on Taupage.

On 22 March 2016 at 11:04, Valentin Haenel notifications@github.com wrote:

Also, you'll need to wait a few minutes before Travis builds the master and the image becomes available at DockerHub.

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/ImmobilienScout24/c-bastion/pull/7#issuecomment-199729093