SLES15 enablement by jlind23 · Pull Request #2007 · elastic/e2e-testing

jlind23 · 2022-01-17T14:53:05Z

mergify · 2022-01-17T14:53:22Z

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.

mdelapenya · 2022-01-17T15:18:04Z

/test

elasticmachine · 2022-01-17T16:21:39Z

💔 Tests Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS

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

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

💔 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 · 2022-01-18T10:50:48Z

/test

.ci/.e2e-tests.yaml

Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>

.ci/ansible/vars/SLES.yml

.ci/ansible/playbook.yml

Co-authored-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com>

Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>

adam-stokes · 2022-01-18T13:30:07Z

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 · 2022-01-18T13:33:42Z

@mdelapenya will you take care of this?

mdelapenya · 2022-01-18T13:39:42Z

Yes, assigning to me

Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com>

adam-stokes · 2022-01-18T20:08:08Z

/test

Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com>

adam-stokes · 2022-01-18T21:37:07Z

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

jlind23 · 2022-01-19T08:50:12Z

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

mdelapenya

Awesome! LGTM

mdelapenya · 2022-01-19T08:57:47Z

@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 · 2022-01-19T08:59:47Z

.ci/.e2e-tests-for-elastic-agent.yaml

  stack:
    image: "ami-0d90bed76900e679a"
    instance_type: "c5.4xlarge"
+    username: admin


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

* 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 <victormartinezrubio@gmail.com> * Update .ci/ansible/vars/SLES.yml Co-authored-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * Update .ci/ansible/playbook.yml Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com> * Fix package install for distro, update include_vars for all tasks Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix path to vars Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * try with ansible_playbook_vars_root Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * try var_files Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * typo Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * use full path and fix quoting Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * use include_vars Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * make include_vars first in task list for each block Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * dont include_vars on localhost execution Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * remove conflicting statements Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * have e2e-tests.yaml drive the login information Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix update cache on debian based systems Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix permission on output directory Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix group ownership in create test script Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * better os detection in ansible Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix chown in jenkinsfile Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> Co-authored-by: Manuel de la Peña <mdelapenya@gmail.com> Co-authored-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com> (cherry picked from commit 764116f)

* 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 <victormartinezrubio@gmail.com> * Update .ci/ansible/vars/SLES.yml Co-authored-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * Update .ci/ansible/playbook.yml Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com> * Fix package install for distro, update include_vars for all tasks Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix path to vars Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * try with ansible_playbook_vars_root Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * try var_files Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * typo Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * use full path and fix quoting Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * use include_vars Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * make include_vars first in task list for each block Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * dont include_vars on localhost execution Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * remove conflicting statements Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * have e2e-tests.yaml drive the login information Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix update cache on debian based systems Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix permission on output directory Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix group ownership in create test script Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * better os detection in ansible Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix chown in jenkinsfile Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> Co-authored-by: Manuel de la Peña <mdelapenya@gmail.com> Co-authored-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com> (cherry picked from commit 764116f) Co-authored-by: Julien Lind <julien.lind@elastic.co>

* 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 <victormartinezrubio@gmail.com> * Update .ci/ansible/vars/SLES.yml Co-authored-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * Update .ci/ansible/playbook.yml Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com> * Fix package install for distro, update include_vars for all tasks Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix path to vars Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * try with ansible_playbook_vars_root Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * try var_files Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * typo Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * use full path and fix quoting Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * use include_vars Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * make include_vars first in task list for each block Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * dont include_vars on localhost execution Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * remove conflicting statements Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * have e2e-tests.yaml drive the login information Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix update cache on debian based systems Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix permission on output directory Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix group ownership in create test script Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * better os detection in ansible Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix chown in jenkinsfile Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> Co-authored-by: Manuel de la Peña <mdelapenya@gmail.com> Co-authored-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>

* chore: add resiliency when provisioning the stack (#1990) (#1993) (cherry picked from commit 54e882b) Co-authored-by: Manuel de la Peña <mdelapenya@gmail.com> * 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 <victormartinezrubio@gmail.com> * Update .ci/ansible/vars/SLES.yml Co-authored-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * Update .ci/ansible/playbook.yml Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com> * Fix package install for distro, update include_vars for all tasks Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix path to vars Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * try with ansible_playbook_vars_root Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * try var_files Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * typo Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * use full path and fix quoting Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * use include_vars Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * make include_vars first in task list for each block Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * dont include_vars on localhost execution Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * remove conflicting statements Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * have e2e-tests.yaml drive the login information Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix update cache on debian based systems Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix permission on output directory Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix group ownership in create test script Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * better os detection in ansible Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> * fix chown in jenkinsfile Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> Co-authored-by: Manuel de la Peña <mdelapenya@gmail.com> Co-authored-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> Co-authored-by: Julien Lind <julien.lind@elastic.co> Co-authored-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com> Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>

* main: SLES15 enablement (elastic#2007)

* 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) ...

SLES15 enablement

408c8ae

mergify bot added the backport-skip Skip notification from the automated backport with mergify label Jan 17, 2022

jlind23 added the v8.1.0 label Jan 17, 2022

jlind23 requested a review from adam-stokes January 17, 2022 16:53

mdelapenya and others added 4 commits January 18, 2022 10:27

fix: set ansible_user depending on OS

93d3f85

Merge branch 'main' into jlind23/SLES15-tests

aeb1cc6

fix: proper vars path

8ce0a67

Merge branch 'main' into jlind23/SLES15-tests

14682ed

mdelapenya added 5 commits January 18, 2022 12:28

fix: rename SLES distribution file

b71e24a

fix: read distribution vars dynamically with include_vars

fa7d2f3

fix: keep original behaviour for installing the stack on Debian

f4c4f15

fix: set vars correctly

3a87a2c

chore: debug ansible user

0d16624

v1v reviewed Jan 18, 2022

View reviewed changes

.ci/.e2e-tests.yaml Outdated Show resolved Hide resolved

Update .ci/.e2e-tests.yaml

7cc7710

Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>

adam-stokes reviewed Jan 18, 2022

View reviewed changes

.ci/ansible/vars/SLES.yml Outdated Show resolved Hide resolved

v1v reviewed Jan 18, 2022

View reviewed changes

.ci/ansible/playbook.yml Outdated Show resolved Hide resolved

jlind23 and others added 2 commits January 18, 2022 14:09

Update .ci/ansible/vars/SLES.yml

c8e8e38

Co-authored-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com>

Update .ci/ansible/playbook.yml

7e464d8

Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>

mdelapenya self-assigned this Jan 18, 2022

adam-stokes added 3 commits January 18, 2022 08:48

Fix package install for distro, update include_vars for all tasks

bf607c7

Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com>

fix path to vars

0d5edfc

Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com>

try with ansible_playbook_vars_root

aff3487

Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com>

adam-stokes added 4 commits January 18, 2022 13:33

remove conflicting statements

c0af3d4

Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com>

have e2e-tests.yaml drive the login information

6910108

Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com>

fix update cache on debian based systems

ae435ca

Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com>

fix permission on output directory

29ebfb0

Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com>

adam-stokes added 3 commits January 18, 2022 15:31

fix group ownership in create test script

abfe61c

Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com>

better os detection in ansible

7920201

Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com>

fix chown in jenkinsfile

a956858

Signed-off-by: Adam Stokes <51892+adam-stokes@users.noreply.github.com>

mdelapenya approved these changes Jan 19, 2022

View reviewed changes

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

mdelapenya reviewed Jan 19, 2022

View reviewed changes

mdelapenya removed the backport-v7.16.0 Automated backport with mergify label Jan 19, 2022

mdelapenya merged commit 764116f into main Jan 19, 2022

mergify bot mentioned this pull request Jan 19, 2022

[8.0](backport #2007) SLES15 enablement #2021

Merged

mergify bot mentioned this pull request Jan 19, 2022

[7.16](backport #2007) SLES15 enablement #2022

Closed

mergify bot deleted the jlind23/SLES15-tests branch January 19, 2022 09:38

mdelapenya mentioned this pull request Jan 19, 2022

[7.17](backport #2007) SLES15 enablement #2024

Merged

mdelapenya added a commit to mdelapenya/e2e-testing that referenced this pull request Jan 19, 2022

Merge branch 'main' into ssh-access-to-cloud-vms

f0967c8

* main: SLES15 enablement (elastic#2007)

Conversation

jlind23 commented Jan 17, 2022

Uh oh!

mergify bot commented Jan 17, 2022

Uh oh!

mdelapenya commented Jan 17, 2022

Uh oh!

elasticmachine commented Jan 17, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

💔 Tests Failed

Build stats

Test stats 🧪

Test errors

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

🐛 Flaky test report

Genuine test errors

🤖 GitHub comments

Uh oh!

jlind23 commented Jan 18, 2022

Uh oh!

Uh oh!

Uh oh!

Uh oh!

adam-stokes commented Jan 18, 2022

Uh oh!

jlind23 commented Jan 18, 2022

Uh oh!

mdelapenya commented Jan 18, 2022

Uh oh!

adam-stokes commented Jan 18, 2022

Uh oh!

adam-stokes commented Jan 18, 2022

Uh oh!

jlind23 commented Jan 19, 2022

Uh oh!

mdelapenya left a comment

Choose a reason for hiding this comment

Uh oh!

mdelapenya commented Jan 19, 2022

Uh oh!

mdelapenya Jan 19, 2022

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

elasticmachine commented Jan 17, 2022 •

edited

Loading

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