Skip to content
This repository was archived by the owner on Sep 17, 2024. It is now read-only.

chore: abstract image pulling#1137

Merged
adam-stokes merged 1 commit intoelastic:masterfrom
mdelapenya:skip-pull
May 5, 2021
Merged

chore: abstract image pulling#1137
adam-stokes merged 1 commit intoelastic:masterfrom
mdelapenya:skip-pull

Conversation

@mdelapenya
Copy link
Copy Markdown
Contributor

What does this PR do?

It abstract the pull images method so that the consumer is passing what images to pull

Why is it important?

To make the pull method reusable by other test suites.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have run the Unit tests for the CLI, and they are passing locally
  • I have run the End-2-End tests for the suite I'm working on, and they are passing locally
  • I have noticed new Go dependencies (run make notice in the proper directory)

Related issues

@mdelapenya mdelapenya self-assigned this May 5, 2021
@mdelapenya mdelapenya requested a review from a team May 5, 2021 15:26
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 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 #1137 opened

  • Start Time: 2021-05-05T15:26:57.710+0000

  • Duration: 22 min 29 sec

  • Commit: 107cf23

Test stats 🧪

Test Results
Failed 0
Passed 152
Skipped 0
Total 152

Trends 🧪

Image of Build Times

Image of Tests

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 152
Skipped 0
Total 152

@adam-stokes adam-stokes marked this pull request as ready for review May 5, 2021 15:52
@adam-stokes adam-stokes merged commit 4c0075f into elastic:master May 5, 2021
mergify bot pushed a commit that referenced this pull request May 5, 2021
(cherry picked from commit 4c0075f)

# Conflicts:
#	e2e/_suites/fleet/ingest_manager_test.go
#	internal/docker/docker.go
mergify bot pushed a commit that referenced this pull request May 5, 2021
(cherry picked from commit 4c0075f)

# Conflicts:
#	e2e/_suites/fleet/ingest_manager_test.go
#	internal/docker/docker.go
@mdelapenya mdelapenya deleted the skip-pull branch May 6, 2021 06:25
mdelapenya added a commit that referenced this pull request May 6, 2021
(cherry picked from commit 4c0075f)

# Conflicts:
#	e2e/_suites/fleet/ingest_manager_test.go
#	internal/docker/docker.go

Co-authored-by: Manuel de la Peña <mdelapenya@gmail.com>
mdelapenya added a commit that referenced this pull request May 6, 2021
(cherry picked from commit 4c0075f)

# Conflicts:
#	e2e/_suites/fleet/ingest_manager_test.go
#	internal/docker/docker.go

Co-authored-by: Manuel de la Peña <mdelapenya@gmail.com>
mdelapenya added a commit that referenced this pull request May 6, 2021
* 7.x:
  chore: abstract image pulling (#1137) (#1138)
  Pull fresh docker images before suite (#1123) (#1140)
mdelapenya added a commit that referenced this pull request May 6, 2021
* 7.13.x:
  chore: abstract image pulling (#1137) (#1139)
  Pull fresh docker images before suite (#1123) (#1141)
mdelapenya added a commit to mdelapenya/e2e-testing that referenced this pull request May 6, 2021
* master:
  chore: update mergify's titles (elastic#1142)
  chore: remove unused pipelines (elastic#1143)
  feat: use Docker copy to transfer binaries to containers (elastic#1136)
  chore: abstract image pulling (elastic#1137)
  Support multiple deployment backends (elastic#1130)
  chore: remove unused code (elastic#1119)
  Unify fleet and stand-alone suites (elastic#1112)
  Pull fresh docker images before suite (elastic#1123)
  chore: initialise timeout factor next to the declaration (elastic#1118)
  chore: match Go version with Beats (elastic#1120)
  feat: support running k8s autodiscover suite for Beats PRs and local repositories (elastic#1115)
  Auto bootstrap fleet during initialize scenario (elastic#1116)
  Kubernetes Deployment (elastic#1110)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants