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

SLES15 enablement #2007

Merged
merged 29 commits into from
Jan 19, 2022
Merged

SLES15 enablement #2007

merged 29 commits into from
Jan 19, 2022

Conversation

jlind23
Copy link
Contributor

@jlind23 jlind23 commented Jan 17, 2022

Closes #1733

@mergify
Copy link
Contributor

mergify bot commented Jan 17, 2022

This pull request does not have a backport label. Could you fix it @jlind23? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-v./d./d./d is the label to automatically backport to the 7./d branch. /d is the digit
    NOTE: backport-skip has been added to this pull request.

@mergify mergify bot added the backport-skip Skip notification from the automated backport with mergify label Jan 17, 2022
@jlind23 jlind23 added the v8.1.0 label Jan 17, 2022
@mdelapenya
Copy link
Contributor

/test

@elasticmachine
Copy link
Contributor

elasticmachine commented Jan 17, 2022

💔 Tests Failed

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

Expand to view the summary

Build stats

  • Start Time: 2022-01-18T21:36:49.731+0000

  • Duration: 62 min 8 sec

  • Commit: a956858

Test stats 🧪

Test Results
Failed 1
Passed 217
Skipped 0
Total 218

Test errors 1

Expand to view the tests failures

Initializing / End-To-End Tests / kubernetes_autodiscover_elastic-agent / [empty] – TEST-x86_64-kubernetes-autodiscover-c7a820d1-2022-01-18-21:56:11.xml
  • no error details
  • Expand to view the stacktrace

     Test report file /var/lib/jenkins/workspace/PR-2007-30-8fbd9186-5026-48d1-acae-3636f8011bee/outputs/18.117.220.102/TEST-x86_64-kubernetes-autodiscover-c7a820d1-2022-01-18-21:56:11.xml was length 0 
    

🐛 Flaky test report

❕ There are test failures but not known flaky tests.

Expand to view the summary

Genuine test errors 1

💔 There are test failures but not known flaky tests, most likely a genuine test failure.

  • Name: Initializing / End-To-End Tests / kubernetes_autodiscover_elastic-agent / [empty] – TEST-x86_64-kubernetes-autodiscover-c7a820d1-2022-01-18-21:56:11.xml

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@jlind23 jlind23 requested a review from adam-stokes January 17, 2022 16:53
@jlind23
Copy link
Contributor Author

jlind23 commented Jan 18, 2022

/test

Co-authored-by: Victor Martinez <[email protected]>
jlind23 and others added 2 commits January 18, 2022 14:09
@adam-stokes
Copy link
Contributor

Also, we need to pull the ansible_user var into this as well:

https://github.com/elastic/e2e-testing/blob/main/.ci/ansible/tasks/runners.yml#L39

@jlind23
Copy link
Contributor Author

jlind23 commented Jan 18, 2022

@mdelapenya will you take care of this?

@mdelapenya
Copy link
Contributor

Yes, assigning to me

@mdelapenya mdelapenya self-assigned this Jan 18, 2022
@adam-stokes
Copy link
Contributor

/test

@adam-stokes
Copy link
Contributor

Ok, this should do it, the last error was inside the Jenkinsfile wrt directory permission, but the actual tests on SLES itself passed.

@jlind23
Copy link
Contributor Author

jlind23 commented Jan 19, 2022

@adam-stokes if the test succeed can we merge it then?

Copy link
Contributor

@mdelapenya mdelapenya left a comment

Choose a reason for hiding this comment

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

Awesome! LGTM

image

@mdelapenya
Copy link
Contributor

@adam-stokes if the test succeed can we merge it then?

@jlind23 I'm going to merge this little one, as the SLES tests passed. The only test error is related to k8s-autodiscover and the elastic-agent test suite, which is something reported in #1992

@mdelapenya mdelapenya added backport-v7.16.0 Automated backport with mergify backport-v7.17.0 Automated backport with mergify backport-v8.0.0 Automated backport with mergify and removed backport-skip Skip notification from the automated backport with mergify labels Jan 19, 2022
@@ -3,21 +3,31 @@ PLATFORMS:
stack:
image: "ami-0d90bed76900e679a"
instance_type: "c5.4xlarge"
username: admin
Copy link
Contributor

Choose a reason for hiding this comment

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

Well done here! As this is the descriptor of the machine, it makes total sense to have the configuration here

@mdelapenya mdelapenya removed the backport-v7.16.0 Automated backport with mergify label Jan 19, 2022
@mdelapenya mdelapenya merged commit 764116f into main Jan 19, 2022
mergify bot pushed a commit that referenced this pull request Jan 19, 2022
* SLES15 enablement

* fix: set ansible_user depending on OS

* fix: proper vars path

* fix: rename SLES distribution file

* fix: read distribution vars dynamically with include_vars

* fix: keep original behaviour for installing the stack on Debian

* fix: set vars correctly

* chore: debug ansible user

* Update .ci/.e2e-tests.yaml

Co-authored-by: Victor Martinez <[email protected]>

* Update .ci/ansible/vars/SLES.yml

Co-authored-by: Adam Stokes <[email protected]>

* Update .ci/ansible/playbook.yml

Co-authored-by: Victor Martinez <[email protected]>

* Fix package install for distro, update include_vars for all tasks

Signed-off-by: Adam Stokes <[email protected]>

* fix path to vars

Signed-off-by: Adam Stokes <[email protected]>

* try with ansible_playbook_vars_root

Signed-off-by: Adam Stokes <[email protected]>

* try var_files

Signed-off-by: Adam Stokes <[email protected]>

* typo

Signed-off-by: Adam Stokes <[email protected]>

* use full path and fix quoting

Signed-off-by: Adam Stokes <[email protected]>

* use include_vars

Signed-off-by: Adam Stokes <[email protected]>

* make include_vars first in task list for each block

Signed-off-by: Adam Stokes <[email protected]>

* dont include_vars on localhost execution

Signed-off-by: Adam Stokes <[email protected]>

* remove conflicting statements

Signed-off-by: Adam Stokes <[email protected]>

* have e2e-tests.yaml drive the login information

Signed-off-by: Adam Stokes <[email protected]>

* fix update cache on debian based systems

Signed-off-by: Adam Stokes <[email protected]>

* fix permission on output directory

Signed-off-by: Adam Stokes <[email protected]>

* fix group ownership in create test script

Signed-off-by: Adam Stokes <[email protected]>

* better os detection in ansible

Signed-off-by: Adam Stokes <[email protected]>

* fix chown in jenkinsfile

Signed-off-by: Adam Stokes <[email protected]>

Co-authored-by: Manuel de la Peña <[email protected]>
Co-authored-by: Adam Stokes <[email protected]>
Co-authored-by: Victor Martinez <[email protected]>
(cherry picked from commit 764116f)
mergify bot pushed a commit that referenced this pull request Jan 19, 2022
* SLES15 enablement

* fix: set ansible_user depending on OS

* fix: proper vars path

* fix: rename SLES distribution file

* fix: read distribution vars dynamically with include_vars

* fix: keep original behaviour for installing the stack on Debian

* fix: set vars correctly

* chore: debug ansible user

* Update .ci/.e2e-tests.yaml

Co-authored-by: Victor Martinez <[email protected]>

* Update .ci/ansible/vars/SLES.yml

Co-authored-by: Adam Stokes <[email protected]>

* Update .ci/ansible/playbook.yml

Co-authored-by: Victor Martinez <[email protected]>

* Fix package install for distro, update include_vars for all tasks

Signed-off-by: Adam Stokes <[email protected]>

* fix path to vars

Signed-off-by: Adam Stokes <[email protected]>

* try with ansible_playbook_vars_root

Signed-off-by: Adam Stokes <[email protected]>

* try var_files

Signed-off-by: Adam Stokes <[email protected]>

* typo

Signed-off-by: Adam Stokes <[email protected]>

* use full path and fix quoting

Signed-off-by: Adam Stokes <[email protected]>

* use include_vars

Signed-off-by: Adam Stokes <[email protected]>

* make include_vars first in task list for each block

Signed-off-by: Adam Stokes <[email protected]>

* dont include_vars on localhost execution

Signed-off-by: Adam Stokes <[email protected]>

* remove conflicting statements

Signed-off-by: Adam Stokes <[email protected]>

* have e2e-tests.yaml drive the login information

Signed-off-by: Adam Stokes <[email protected]>

* fix update cache on debian based systems

Signed-off-by: Adam Stokes <[email protected]>

* fix permission on output directory

Signed-off-by: Adam Stokes <[email protected]>

* fix group ownership in create test script

Signed-off-by: Adam Stokes <[email protected]>

* better os detection in ansible

Signed-off-by: Adam Stokes <[email protected]>

* fix chown in jenkinsfile

Signed-off-by: Adam Stokes <[email protected]>

Co-authored-by: Manuel de la Peña <[email protected]>
Co-authored-by: Adam Stokes <[email protected]>
Co-authored-by: Victor Martinez <[email protected]>
(cherry picked from commit 764116f)
@mergify mergify bot deleted the jlind23/SLES15-tests branch January 19, 2022 09:38
mdelapenya pushed a commit that referenced this pull request Jan 19, 2022
* SLES15 enablement

* fix: set ansible_user depending on OS

* fix: proper vars path

* fix: rename SLES distribution file

* fix: read distribution vars dynamically with include_vars

* fix: keep original behaviour for installing the stack on Debian

* fix: set vars correctly

* chore: debug ansible user

* Update .ci/.e2e-tests.yaml

Co-authored-by: Victor Martinez <[email protected]>

* Update .ci/ansible/vars/SLES.yml

Co-authored-by: Adam Stokes <[email protected]>

* Update .ci/ansible/playbook.yml

Co-authored-by: Victor Martinez <[email protected]>

* Fix package install for distro, update include_vars for all tasks

Signed-off-by: Adam Stokes <[email protected]>

* fix path to vars

Signed-off-by: Adam Stokes <[email protected]>

* try with ansible_playbook_vars_root

Signed-off-by: Adam Stokes <[email protected]>

* try var_files

Signed-off-by: Adam Stokes <[email protected]>

* typo

Signed-off-by: Adam Stokes <[email protected]>

* use full path and fix quoting

Signed-off-by: Adam Stokes <[email protected]>

* use include_vars

Signed-off-by: Adam Stokes <[email protected]>

* make include_vars first in task list for each block

Signed-off-by: Adam Stokes <[email protected]>

* dont include_vars on localhost execution

Signed-off-by: Adam Stokes <[email protected]>

* remove conflicting statements

Signed-off-by: Adam Stokes <[email protected]>

* have e2e-tests.yaml drive the login information

Signed-off-by: Adam Stokes <[email protected]>

* fix update cache on debian based systems

Signed-off-by: Adam Stokes <[email protected]>

* fix permission on output directory

Signed-off-by: Adam Stokes <[email protected]>

* fix group ownership in create test script

Signed-off-by: Adam Stokes <[email protected]>

* better os detection in ansible

Signed-off-by: Adam Stokes <[email protected]>

* fix chown in jenkinsfile

Signed-off-by: Adam Stokes <[email protected]>

Co-authored-by: Manuel de la Peña <[email protected]>
Co-authored-by: Adam Stokes <[email protected]>
Co-authored-by: Victor Martinez <[email protected]>
(cherry picked from commit 764116f)

Co-authored-by: Julien Lind <[email protected]>
mdelapenya added a commit to mdelapenya/e2e-testing that referenced this pull request Jan 19, 2022
* SLES15 enablement

* fix: set ansible_user depending on OS

* fix: proper vars path

* fix: rename SLES distribution file

* fix: read distribution vars dynamically with include_vars

* fix: keep original behaviour for installing the stack on Debian

* fix: set vars correctly

* chore: debug ansible user

* Update .ci/.e2e-tests.yaml

Co-authored-by: Victor Martinez <[email protected]>

* Update .ci/ansible/vars/SLES.yml

Co-authored-by: Adam Stokes <[email protected]>

* Update .ci/ansible/playbook.yml

Co-authored-by: Victor Martinez <[email protected]>

* Fix package install for distro, update include_vars for all tasks

Signed-off-by: Adam Stokes <[email protected]>

* fix path to vars

Signed-off-by: Adam Stokes <[email protected]>

* try with ansible_playbook_vars_root

Signed-off-by: Adam Stokes <[email protected]>

* try var_files

Signed-off-by: Adam Stokes <[email protected]>

* typo

Signed-off-by: Adam Stokes <[email protected]>

* use full path and fix quoting

Signed-off-by: Adam Stokes <[email protected]>

* use include_vars

Signed-off-by: Adam Stokes <[email protected]>

* make include_vars first in task list for each block

Signed-off-by: Adam Stokes <[email protected]>

* dont include_vars on localhost execution

Signed-off-by: Adam Stokes <[email protected]>

* remove conflicting statements

Signed-off-by: Adam Stokes <[email protected]>

* have e2e-tests.yaml drive the login information

Signed-off-by: Adam Stokes <[email protected]>

* fix update cache on debian based systems

Signed-off-by: Adam Stokes <[email protected]>

* fix permission on output directory

Signed-off-by: Adam Stokes <[email protected]>

* fix group ownership in create test script

Signed-off-by: Adam Stokes <[email protected]>

* better os detection in ansible

Signed-off-by: Adam Stokes <[email protected]>

* fix chown in jenkinsfile

Signed-off-by: Adam Stokes <[email protected]>

Co-authored-by: Manuel de la Peña <[email protected]>
Co-authored-by: Adam Stokes <[email protected]>
Co-authored-by: Victor Martinez <[email protected]>
mdelapenya added a commit that referenced this pull request Jan 19, 2022
* chore: add resiliency when provisioning the stack (#1990) (#1993)

(cherry picked from commit 54e882b)

Co-authored-by: Manuel de la Peña <[email protected]>

* SLES15 enablement (#2007)

* SLES15 enablement

* fix: set ansible_user depending on OS

* fix: proper vars path

* fix: rename SLES distribution file

* fix: read distribution vars dynamically with include_vars

* fix: keep original behaviour for installing the stack on Debian

* fix: set vars correctly

* chore: debug ansible user

* Update .ci/.e2e-tests.yaml

Co-authored-by: Victor Martinez <[email protected]>

* Update .ci/ansible/vars/SLES.yml

Co-authored-by: Adam Stokes <[email protected]>

* Update .ci/ansible/playbook.yml

Co-authored-by: Victor Martinez <[email protected]>

* Fix package install for distro, update include_vars for all tasks

Signed-off-by: Adam Stokes <[email protected]>

* fix path to vars

Signed-off-by: Adam Stokes <[email protected]>

* try with ansible_playbook_vars_root

Signed-off-by: Adam Stokes <[email protected]>

* try var_files

Signed-off-by: Adam Stokes <[email protected]>

* typo

Signed-off-by: Adam Stokes <[email protected]>

* use full path and fix quoting

Signed-off-by: Adam Stokes <[email protected]>

* use include_vars

Signed-off-by: Adam Stokes <[email protected]>

* make include_vars first in task list for each block

Signed-off-by: Adam Stokes <[email protected]>

* dont include_vars on localhost execution

Signed-off-by: Adam Stokes <[email protected]>

* remove conflicting statements

Signed-off-by: Adam Stokes <[email protected]>

* have e2e-tests.yaml drive the login information

Signed-off-by: Adam Stokes <[email protected]>

* fix update cache on debian based systems

Signed-off-by: Adam Stokes <[email protected]>

* fix permission on output directory

Signed-off-by: Adam Stokes <[email protected]>

* fix group ownership in create test script

Signed-off-by: Adam Stokes <[email protected]>

* better os detection in ansible

Signed-off-by: Adam Stokes <[email protected]>

* fix chown in jenkinsfile

Signed-off-by: Adam Stokes <[email protected]>

Co-authored-by: Manuel de la Peña <[email protected]>
Co-authored-by: Adam Stokes <[email protected]>
Co-authored-by: Victor Martinez <[email protected]>

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: Julien Lind <[email protected]>
Co-authored-by: Adam Stokes <[email protected]>
Co-authored-by: Victor Martinez <[email protected]>
mdelapenya added a commit to mdelapenya/e2e-testing that referenced this pull request Jan 19, 2022
mdelapenya added a commit to mdelapenya/e2e-testing that referenced this pull request Jan 26, 2022
* main: (45 commits)
  feat: add CentOS 8 support (elastic#2034)
  fix: set default region for AWS cli (elastic#2053)
  chore: use Ansible's built-in replace instead of sed (elastic#2048)
  chore: split stack configuration and start into two tasks (elastic#2044)
  feat: enable SSH access to users for debugging cloud instances (elastic#2001)
  fix: use the right branch for 7.17 backports (elastic#2025)
  SLES15 enablement (elastic#2007)
  chore: bump stale agent for main (elastic#2014)
  Update `fetchBeatsBinary` to be reused in elastic-agent-poc (elastic#1984)
  chore: add resiliency when provisioning the stack (elastic#1990)
  chore: bump elastic-package to v0.32.1 (elastic#1959)
  feat: export Fetch&Download methods in the /pkg directory (elastic#1943)
  bump stack version 8.1.0-dbc834fd (elastic#1948)
  bump stack version 8.1.0-76902d39 (elastic#1946)
  chore: retire 7.15 adding 7.17 (elastic#1938)
  ci: use withAPMEnv (elastic#1917)
  Update main branch (elastic#1928)
  bump stack version 8.1.0-befff95a (elastic#1929)
  chore: properly evaluate how tests are skipped on CI when checking modified files (elastic#1924)
  bump stack version 8.1.0-60bffc32 (elastic#1921)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport-v7.17.0 Automated backport with mergify backport-v8.0.0 Automated backport with mergify v8.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add SLES15 to the e2e-testing matrix of linux vms we test against
5 participants