This repository was archived by the owner on Sep 17, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 39
feat: add support for running the tests in ARM #707
Merged
Merged
Changes from 2 commits
Commits
Show all changes
86 commits
Select commit
Hold shift + click to select a range
b618488
feat: add support for running the tests in ARM
mdelapenya 17c836f
fix: arm workers are not immutable yet
mdelapenya 9eb92ea
Merge branch 'master' into arm-suppport
mdelapenya e47600e
chore: calculate artifact from the GOARCH variable
mdelapenya c009e8d
chore: run elasticsearch in the right platform
mdelapenya a880fcd
chore: add log for architecture
mdelapenya 34bcaf3
Merge branch 'master' into arm-suppport
mdelapenya f8a0d51
chore: print env variable for GOARCH
mdelapenya 0cd42e0
chore: set GOARCH based in the worker
mdelapenya d42766d
fix: wrong copy&paste
mdelapenya 6663c23
chore: install k8s dependencies on ARM
mdelapenya 33bc88e
chore: install godog using GOARCH
mdelapenya c7193a2
fix: proper values for GO env in Makefile
mdelapenya 8e7bf49
chore: add log when requesting a node
mdelapenya 05ea8ce
fix: define Groovy variable
mdelapenya d51a013
chore: use proper platform for kibana
mdelapenya 5a8a4ae
chore: simplify platforms selection logic
mdelapenya 235e3bb
fix: use proper CI label for node selector
mdelapenya 8326a23
chore: remove running containers in clean-docker script
mdelapenya 7a8d36c
chore: clean up docker state on mutable workers
mdelapenya 30053ab
chore: use parallel branch name for filebeat logs
mdelapenya 04555b2
fix: add forgotten double quotes
mdelapenya 529fc73
chore: print Docker compose version On ARM
mdelapenya ac6d3a0
chore: use aarch64 architecture for RPM packages
mdelapenya fbcbdcf
chore: support ARM in tar installer
mdelapenya b4d6601
Merge branch 'master' into arm-suppport
mdelapenya 9a20396
fix: proper arch for TAR installer
mdelapenya 65d5010
chore: log when the artifact is found
mdelapenya 610d0b8
chore: be defensive about no running containers
mdelapenya e83517b
Merge branch 'master' into arm-suppport
mdelapenya 9d3dadb
chore: add tests for RPM artifact name
mdelapenya e6c65c9
chore: add tests for x86_64 TAR artifact
mdelapenya 2d1dc07
Merge branch 'master' into arm-suppport
mdelapenya 855f949
chore: remove docker label from CI workers
mdelapenya 5528a1b
Merge branch 'master' into arm-suppport
mdelapenya b3c33c0
chore: separate ARM tests in a second parallel stage
mdelapenya b8bdbe5
Merge branch 'master' into arm-suppport
mdelapenya f2e57c6
chore: update stage name
mdelapenya 99426de
fix: update arm target platform
mdelapenya 6c2f21b
chore: use GOARCH in the junit file
mdelapenya 4906816
chore: use GOARCH in filebeat logs
mdelapenya a14eebb
chore: run all stages in parallel
mdelapenya 1d3770d
chore: add platform to log when skipping tests
mdelapenya 4cb4afa
fix: use a filebeat version that supports multiplatform
mdelapenya 9f4522a
Merge branch 'master' into arm-suppport
mdelapenya e1d355b
Merge branch 'master' into arm-suppport
mdelapenya d045f10
Merge branch 'master' into arm-suppport
mdelapenya 7142ab8
chore: do not run integrations on ARM
mdelapenya 496b5ef
chore: move test resources for ARM
mdelapenya 6bdf785
Merge branch 'master' into arm-suppport
mdelapenya bae9a66
chore: use staging environment for package-registry
mdelapenya 33deffa
fix: remove clean because arm workers are now immutable
mdelapenya 73fb39f
Merge branch 'master' into arm-suppport
mdelapenya a755f6f
chore: use new multiplatform images
mdelapenya 7883dbb
chore: use new multiplatform base images for fleet-server
mdelapenya 63ca5f1
Merge branch 'master' into arm-suppport
mdelapenya 11f2d22
Merge branch 'master' into arm-suppport
mdelapenya f8632dd
chore: add platform to elastic-agent cloud flavour
mdelapenya 6749590
fix: use proper variable for platform
mdelapenya fbe8cd6
Merge branch 'master' into arm-suppport
mdelapenya 9104501
chore: do not clean up installer's workspace
mdelapenya ac300e2
Revert "chore: do not clean up installer's workspace"
mdelapenya 01cc441
chore(ci): skip pull of images from code
mdelapenya 2d9e29f
Merge branch 'master' into arm-suppport
mdelapenya b31a67b
Revert "chore(ci): skip pull of images from code"
mdelapenya 4e46490
chore: set platform when pulling images
mdelapenya 43e9e6d
Merge branch 'master' into arm-suppport
mdelapenya 84b93b4
Merge branch 'master' into arm-suppport
mdelapenya 339a052
chore: update packer cache with our systemd images
mdelapenya b4cd209
fix: request proper platform image for the agent
mdelapenya e9e39ff
Merge branch 'master' into arm-suppport
mdelapenya 52161a6
feat: exclude scenarios per platform
mdelapenya 5be1485
chore: remove tags for endpoint, as they are not used at all
mdelapenya 6ed5d55
fix: skip endpoint integration on ARM64
mdelapenya 080da73
feat: provide supported platforms at the scenario level on CI
mdelapenya ce81c7c
fix: use proper length method for groovy arrays
mdelapenya 72130dd
fix: groovy arrays use size
mdelapenya 9f347d5
fix: override suite platforms per scenario
mdelapenya 9ef81ca
chore: update Jenkins label for arm workers
mdelapenya 0128c1b
chore: simplify platforms, reusing stack
mdelapenya bb7eea0
chore: explicitly pass GOARCH to CI script
mdelapenya 752594d
Revert "chore: update Jenkins label for arm workers"
mdelapenya bd198ee
Merge branch 'master' into arm-suppport
mdelapenya 95532aa
Merge branch 'master' into arm-suppport
mdelapenya 0ece4da
Merge branch 'master' into arm-suppport
mdelapenya aa09c0e
Merge branch 'master' into arm-suppport
mdelapenya File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -316,8 +316,15 @@ def generateFunctionalTestStep(Map args = [:]){ | |
| } | ||
| tags += excludeNightlyTag | ||
|
|
||
| workerLabels = "${platform}" | ||
| if (platform != "arm") { | ||
| // ARM is not ready as immutable workers | ||
| workerLabels += " && immutable" | ||
| } | ||
| workerLabels += " && docker" | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Will add this!
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Mmm... does it mean that we cannot use |
||
|
|
||
| return { | ||
| node("${platform} && immutable && docker") { | ||
| node("${workerLabels}") { | ||
| try { | ||
| deleteDir() | ||
| unstash 'source' | ||
|
|
||
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you describe the intent of the branching? I see it has to do with immutable workers, but don't have context - maybe that is what you could comment better on?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exactly, this is how Infra's workers are labelled: arm are not immutable yet