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

Use the tudorg/xgo-base image from docker hub #6448

Merged
merged 1 commit into from
Feb 22, 2018

Conversation

tsg
Copy link
Contributor

@tsg tsg commented Feb 22, 2018

Another attempt to fix the transient packaging error. I pushed the
tudorg/xgo-base image to Docker Hub and have the build script pull it
from there, rather than building it locally.

This should reduce the amount of network dependencies and speed up the
build as well.

Same could be done for xgo-base-deb, but this part is currently more
critical.

Another attempt to fix the transient packaging error. I pushed the
tudorg/xgo-base image to Docker Hub and have the build script pull it
from there, rather than building it locally.

This should reduce the amount of network dependencies and speed up the
build as well.

Same could be done for xgo-base-deb, but this part is currently more
critical.
Copy link
Member

@andrewkroh andrewkroh left a comment

Choose a reason for hiding this comment

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

I'm good with the temporary fix.

@andrewkroh andrewkroh merged commit 3e408ca into elastic:master Feb 22, 2018
tsg added a commit to tsg/beats that referenced this pull request Feb 23, 2018
Another attempt to fix the transient packaging error. I pushed the
tudorg/xgo-base image to Docker Hub and have the build script pull it
from there, rather than building it locally.

This should reduce the amount of network dependencies and speed up the
build as well.

Same could be done for xgo-base-deb, but this part is currently more
critical.
@tsg tsg mentioned this pull request Feb 23, 2018
tsg added a commit to tsg/beats that referenced this pull request Feb 23, 2018
Another attempt to fix the transient packaging error. I pushed the
tudorg/xgo-base image to Docker Hub and have the build script pull it
from there, rather than building it locally.

This should reduce the amount of network dependencies and speed up the
build as well.

Same could be done for xgo-base-deb, but this part is currently more
critical.
graphaelli pushed a commit that referenced this pull request Feb 23, 2018
* Use the tudorg/xgo-base image from docker hub (#6448)

Another attempt to fix the transient packaging error. I pushed the
tudorg/xgo-base image to Docker Hub and have the build script pull it
from there, rather than building it locally.

This should reduce the amount of network dependencies and speed up the
build as well.

Same could be done for xgo-base-deb, but this part is currently more
critical.

* Avoid git cloning from the deb6 image (#6459)

It seems that by now Debian 6 is so old that `git clone` from Github
repositories no longer works. We only needed git clone from it to
get gotpl running, which is a relatively small program so I included it
and its dependency (go-yaml) as vendored files.

Tested `make package` locally.

* Remove dependency on gopkg.in (#6461)

This removes the remaining `go get` for gotpl, by using the vendored
copy. This should get rid of our dependency on the gopkg.in service.
ruflin pushed a commit that referenced this pull request Feb 27, 2018
Another attempt to fix the transient packaging error. I pushed the
tudorg/xgo-base image to Docker Hub and have the build script pull it
from there, rather than building it locally.

This should reduce the amount of network dependencies and speed up the
build as well.

Same could be done for xgo-base-deb, but this part is currently more
critical.
andrewkroh pushed a commit to andrewkroh/beats that referenced this pull request Mar 11, 2018
Another attempt to fix the transient packaging error. I pushed the
tudorg/xgo-base image to Docker Hub and have the build script pull it
from there, rather than building it locally.

This should reduce the amount of network dependencies and speed up the
build as well.

Same could be done for xgo-base-deb, but this part is currently more
critical.
ruflin pushed a commit that referenced this pull request Mar 12, 2018
Another attempt to fix the transient packaging error. I pushed the
tudorg/xgo-base image to Docker Hub and have the build script pull it
from there, rather than building it locally.

This should reduce the amount of network dependencies and speed up the
build as well.

Same could be done for xgo-base-deb, but this part is currently more
critical.
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
Another attempt to fix the transient packaging error. I pushed the
tudorg/xgo-base image to Docker Hub and have the build script pull it
from there, rather than building it locally.

This should reduce the amount of network dependencies and speed up the
build as well.

Same could be done for xgo-base-deb, but this part is currently more
critical.
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
* Use the tudorg/xgo-base image from docker hub (elastic#6448)

Another attempt to fix the transient packaging error. I pushed the
tudorg/xgo-base image to Docker Hub and have the build script pull it
from there, rather than building it locally.

This should reduce the amount of network dependencies and speed up the
build as well.

Same could be done for xgo-base-deb, but this part is currently more
critical.

* Avoid git cloning from the deb6 image (elastic#6459)

It seems that by now Debian 6 is so old that `git clone` from Github
repositories no longer works. We only needed git clone from it to
get gotpl running, which is a relatively small program so I included it
and its dependency (go-yaml) as vendored files.

Tested `make package` locally.

* Remove dependency on gopkg.in (elastic#6461)

This removes the remaining `go get` for gotpl, by using the vendored
copy. This should get rid of our dependency on the gopkg.in service.
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.

2 participants