Skip to content

test: build docker image#17905

Merged
kuisathaverat merged 5 commits intoelastic:masterfrom
kuisathaverat:test-docker
May 4, 2020
Merged

test: build docker image#17905
kuisathaverat merged 5 commits intoelastic:masterfrom
kuisathaverat:test-docker

Conversation

@kuisathaverat
Copy link
Copy Markdown
Contributor

@kuisathaverat kuisathaverat commented Apr 22, 2020

What does this PR do?

  • Push the Docker images generated to the observability-ci namespace in our Docker registry.
  • Limit the changes that trigger a full build on the main pipeline.

Why is it important?

After this PR we will have fresh snapshots every time we run the packaging job.
Also, changes in the packaging job will no trigger a full build of the main pipeline.

@kuisathaverat
Copy link
Copy Markdown
Contributor Author

/packaging

2 similar comments
@kuisathaverat
Copy link
Copy Markdown
Contributor Author

/packaging

@kuisathaverat
Copy link
Copy Markdown
Contributor Author

/packaging

@kuisathaverat
Copy link
Copy Markdown
Contributor Author

/packaging

@kuisathaverat
Copy link
Copy Markdown
Contributor Author

/packaging

@kuisathaverat
Copy link
Copy Markdown
Contributor Author

/packaging

@kuisathaverat kuisathaverat marked this pull request as ready for review April 24, 2020 07:32
@kuisathaverat kuisathaverat requested review from a team, andrewkroh and urso April 24, 2020 07:32
@kuisathaverat
Copy link
Copy Markdown
Contributor Author

/packaging

@kuisathaverat kuisathaverat requested a review from jsoriano April 28, 2020 11:47
@kuisathaverat kuisathaverat requested review from abellavia-em, mtojek and ruflin and removed request for abellavia-em April 28, 2020 11:48
@ruflin ruflin requested a review from ph April 28, 2020 14:37
@ph ph requested a review from michalpristas April 28, 2020 14:40
@ph
Copy link
Copy Markdown
Contributor

ph commented Apr 28, 2020

@kuisathaverat How do I test that PR?

@kuisathaverat
Copy link
Copy Markdown
Contributor Author

the comment /packaging trigger the job related to this PR, we are pending of a change in the job to notify a GitHub check, so right now the only way to see the result it is to go to the Job PR https://beats-ci.elastic.co/job/Beats/job/packaging/view/change-requests/job/PR-17905/

@kuisathaverat
Copy link
Copy Markdown
Contributor Author

@andrewkroh any thoughts?

@kuisathaverat kuisathaverat merged commit 7d0a45a into elastic:master May 4, 2020
kuisathaverat added a commit to kuisathaverat/beats that referenced this pull request May 4, 2020
* feat: push docker images

* fix: retrict a little the changes that trigger a full build

* fix: restore use of gitcheckout

* Update .ci/packaging.groovy
kuisathaverat added a commit to kuisathaverat/beats that referenced this pull request May 4, 2020
* feat: push docker images

* fix: retrict a little the changes that trigger a full build

* fix: restore use of gitcheckout

* Update .ci/packaging.groovy
kuisathaverat added a commit that referenced this pull request May 5, 2020
* feat: push docker images

* fix: retrict a little the changes that trigger a full build

* fix: restore use of gitcheckout

* Update .ci/packaging.groovy
kuisathaverat added a commit that referenced this pull request May 5, 2020
* feat: push docker images

* fix: retrict a little the changes that trigger a full build

* fix: restore use of gitcheckout

* Update .ci/packaging.groovy
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label May 25, 2020
@mdelapenya mdelapenya added the Team:Automation Label for the Observability productivity team label May 25, 2020
@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label May 25, 2020
@kuisathaverat kuisathaverat deleted the test-docker branch December 24, 2020 11:08
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
* feat: push docker images

* fix: retrict a little the changes that trigger a full build

* fix: restore use of gitcheckout

* Update .ci/packaging.groovy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automation ci enhancement Team:Automation Label for the Observability productivity team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants