e2e-testing icon indicating copy to clipboard operation
e2e-testing copied to clipboard
verified publisher icon elastic

Igor api key validation

Open IgorGuz2000 opened this issue 4 years ago • 13 comments

What does this PR do?

This PR test communication API keys are invalidated during unenrollment

Why is it important?

Checklist

  • [x] 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 (make unit-test), and they are passing locally
  • [x] 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)

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

https://github.com/elastic/e2e-testing/issues/1192

IgorGuz2000 avatar Nov 05 '21 20:11 IgorGuz2000

This pull request does not have a backport label. Could you fix it @IgorGuz2000? 🙏 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[bot] avatar Nov 05 '21 20:11 mergify[bot]

:broken_heart: 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-20T19:14:05.289+0000

  • Duration: 62 min 18 sec

  • Commit: b60ad262b8522cb74578d81213b3ce70260d0c98

Test stats :test_tube:

Test Results
Failed 5
Passed 219
Skipped 0
Total 224

Test errors 5

Expand to view the tests failures

Initializing / End-To-End Tests / fleet_amd64_fleet_mode_agent / Un-enrolling with revoke agent deactivates the agent – Fleet Mode Agent
    Expand to view the error details

     Step the agent Api key invalidated "true": The agent Api key invalidated is should be: true
  • no stacktrace
Initializing / End-To-End Tests / kubernetes_autodiscover_elastic-agent / [empty] – TEST-x86_64-kubernetes-autodiscover-79e77bf2-2022-01-20-19:34:32.xml
  • no error details
    • Expand to view the stacktrace

     Test report file /var/lib/jenkins/workspace/PR-1783-28-6b7ada2c-9a89-4900-a9c1-664d262ec71e/outputs/18.219.45.37/TEST-x86_64-kubernetes-autodiscover-79e77bf2-2022-01-20-19:34:32.xml was length 0
Initializing / End-To-End Tests / fleet_arm64_fleet_mode_agent / Deploying the agent with enroll and then run on rpm and deb – Fleet Mode Agent
    Expand to view the error details

     Step an agent is deployed to Fleet
  • no stacktrace
Initializing / End-To-End Tests / fleet_sles15_fleet_mode_agent / Deploying the agent with enroll and then run on rpm and deb – Fleet Mode Agent
    Expand to view the error details

     Step an agent is deployed to Fleet
  • no stacktrace
Initializing / End-To-End Tests / fleet_amd64_fleet_mode_agent / Deploying the agent with enroll and then run on rpm and deb – Fleet Mode Agent
    Expand to view the error details

     Step an agent is deployed to Fleet
  • no stacktrace

Steps errors 2

Expand to view the steps failures

Shell Script
  • Took 21 min 12 sec . View more details here
  • Description: ssh -tt -o TCPKeepAlive=yes -o ServerAliveInterval=60 -o ServerAliveCountMax=10 -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -i /var/lib/jenkins/workspace/PR-1783-28-301e8c3e-560b-4392-87e5-60b0a23831fc/e2essh [email protected] -- 'sudo bash /home/admin/e2e-testing/.ci/scripts/functional-test.sh "fleet_mode_agent && ~@nightly && ~@skip:amd64" '
Archive the artifacts
  • Took 0 min 0 sec . View more details here
  • Description: [2022-01-20T19:54:48.168Z] Archiving artifacts script returned exit code 2

:bug: Flaky test report

:grey_exclamation: There are test failures but not known flaky tests.

Expand to view the summary

Genuine test errors 5

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

  • Name: Initializing / End-To-End Tests / fleet_amd64_fleet_mode_agent / Un-enrolling with revoke agent deactivates the agent – Fleet Mode Agent
  • Name: Initializing / End-To-End Tests / kubernetes_autodiscover_elastic-agent / [empty] – TEST-x86_64-kubernetes-autodiscover-79e77bf2-2022-01-20-19:34:32.xml
  • Name: Initializing / End-To-End Tests / fleet_arm64_fleet_mode_agent / Deploying the agent with enroll and then run on rpm and deb – Fleet Mode Agent
  • Name: Initializing / End-To-End Tests / fleet_sles15_fleet_mode_agent / Deploying the agent with enroll and then run on rpm and deb – Fleet Mode Agent
  • Name: Initializing / End-To-End Tests / fleet_amd64_fleet_mode_agent / Deploying the agent with enroll and then run on rpm and deb – Fleet Mode Agent

:robot: GitHub comments

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

  • /test : Re-trigger the build.

elasticmachine avatar Nov 05 '21 20:11 elasticmachine

Just a note to hold off on merging this until https://github.com/elastic/e2e-testing/pull/1740 is merged as this PR will need some cleanup on the feature file @IgorGuz2000 @mdelapenya

adam-stokes avatar Nov 11 '21 18:11 adam-stokes

/test

IgorGuz2000 avatar Nov 16 '21 19:11 IgorGuz2000

Waiting for the https://github.com/elastic/e2e-testing/pull/1740 to be merged

IgorGuz2000 avatar Nov 18 '21 23:11 IgorGuz2000

This pull request is now in conflict. Could you fix it @IgorGuz2000? 🙏 To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b Igor_API_Key_validation upstream/Igor_API_Key_validation
git merge upstream/master
git push upstream Igor_API_Key_validation

mergify[bot] avatar Nov 30 '21 19:11 mergify[bot]

This pull request is now in conflict. Could you fix it @IgorGuz2000? 🙏 To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b Igor_API_Key_validation upstream/Igor_API_Key_validation
git merge upstream/main
git push upstream Igor_API_Key_validation

mergify[bot] avatar Mar 29 '22 11:03 mergify[bot]

This pull request is now in conflict. Could you fix it @IgorGuz2000? 🙏 To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b Igor_API_Key_validation upstream/Igor_API_Key_validation
git merge upstream/main
git push upstream Igor_API_Key_validation

mergify[bot] avatar Sep 27 '22 14:09 mergify[bot]

This pull request is now in conflict. Could you fix it @IgorGuz2000? 🙏 To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b Igor_API_Key_validation upstream/Igor_API_Key_validation
git merge upstream/main
git push upstream Igor_API_Key_validation

mergify[bot] avatar Apr 03 '23 07:04 mergify[bot]

This pull request is now in conflict. Could you fix it @IgorGuz2000? 🙏 To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b Igor_API_Key_validation upstream/Igor_API_Key_validation
git merge upstream/main
git push upstream Igor_API_Key_validation

mergify[bot] avatar Aug 21 '23 22:08 mergify[bot]

This pull request is now in conflict. Could you fix it @IgorGuz2000? 🙏 To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b Igor_API_Key_validation upstream/Igor_API_Key_validation
git merge upstream/main
git push upstream Igor_API_Key_validation

mergify[bot] avatar Nov 01 '23 08:11 mergify[bot]

This pull request is now in conflict. Could you fix it @IgorGuz2000? 🙏 To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b Igor_API_Key_validation upstream/Igor_API_Key_validation
git merge upstream/main
git push upstream Igor_API_Key_validation

mergify[bot] avatar Dec 14 '23 11:12 mergify[bot]

This pull request is now in conflict. Could you fix it @IgorGuz2000? 🙏 To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b Igor_API_Key_validation upstream/Igor_API_Key_validation
git merge upstream/main
git push upstream Igor_API_Key_validation

mergify[bot] avatar Feb 20 '24 11:02 mergify[bot]