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

packer-cache: cache the existing docker images on ARM and some more #25068

Merged
merged 3 commits into from
Apr 14, 2021

Conversation

v1v
Copy link
Member

@v1v v1v commented Apr 14, 2021

What does this PR do?

Cache more docker images to workaround the issue we see in the ARM ephemeral workers with the existing rate limit in the docker pull.

Why is it important?

Rate limit causes build failures

Test

docker pull is platform agnostic:

jenkins@beats-ci-immutable-ubuntu-1804-aarch64-1618391886589946761:~$ docker pull docker.elastic.co/elasticsearch/elasticsearch:8.0.0-SNAPSHOT
8.0.0-SNAPSHOT: Pulling from elasticsearch/elasticsearch
5cfb807c1c07: Pull complete 
232a829a49a0: Pull complete 
305e6cd2445d: Pull complete 
5f24f0e9a71f: Pull complete 
03ddbb0b2c42: Pull complete 
Digest: sha256:6279ee9ac8013ecbff4608d310a6e58ff544500a7a80cb68b5967d2d19607f42
Status: Downloaded newer image for docker.elastic.co/elasticsearch/elasticsearch:8.0.0-SNAPSHOT
docker.elastic.co/elasticsearch/elasticsearch:8.0.0-SNAPSHOT
jenkins@beats-ci-immutable-ubuntu-1804-aarch64-1618391886589946761:~$ docker inspect docker.elastic.co/elasticsearch/elasticsearch:8.0.0-SNAPSHOT

...
        "Architecture": "arm64",
        "Variant": "v8",
        "Os": "linux",
        "Size": 561893423,
        "VirtualSize": 561893423,
...

@v1v v1v requested a review from a team as a code owner April 14, 2021 09:53
@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label Team:Automation Label for the Observability productivity team and removed needs_team Indicates that the issue/PR needs a Team:* label labels Apr 14, 2021
@v1v v1v self-assigned this Apr 14, 2021
@v1v v1v added the automation label Apr 14, 2021
@elasticmachine
Copy link
Collaborator

elasticmachine commented Apr 14, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: Pull request #25068 updated

  • Start Time: 2021-04-14T11:12:25.467+0000

  • Duration: 14 min 56 sec

  • Commit: 541e8a5

Trends 🧪

Image of Build Times

❕ Flaky test report

No test was executed to be analysed.

.ci/packer_cache.sh Outdated Show resolved Hide resolved
Co-authored-by: cachedout <mike.place@elastic.co>
@v1v v1v merged commit 0dbc89e into elastic:master Apr 14, 2021
@v1v v1v deleted the feature/cache-packer-arm branch April 14, 2021 11:27
v1v added a commit to v1v/beats that referenced this pull request Apr 15, 2021
* upstream/master:
  packer cache support for the 7.x and 7.latestMinor branches (elastic#25091)
  Remove EventFetcher and EventsFetcher interface (elastic#25093)
  Update go-structform to 0.0.8 (elastic#25051)
  Update copy_fields.asciidoc (elastic#25053)
  [elastic-agent] ensure container is backwards compatible (elastic#25092)
  Add --fleet-server-service-token. Rename --fleet-server to --fleet-server-es. (elastic#25083)
  Add cgroup.cpuacct percentages (elastic#25057)
  Add tests for truncated and symlinked files in filestream input (elastic#24425)
  Fix panic when Hearbeat monitor initialization fails twice (elastic#25073)
  [Filebeat][httpjson] Change append transform to initiate new fields as a slice (elastic#25074)
  Osquerybeat: Result values type translation (elastic#25012)
  Update Osquerybeat spec to get it downloading from the correct artifactory path (elastic#25076)
  Fix changelog (elastic#25079)
  Strip Azure EventHub connection string in debug logs (elastic#25066)
  Change googlecloud to gcp in field names (elastic#25038)
  Bump stack version to 7.12.0 for testing (elastic#24957)
  packer-cache: cache the existing docker images on ARM and some more (elastic#25068)
  Disable logstash TestFetch flaky test (elastic#25044)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation Team:Automation Label for the Observability productivity team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants