Skip to content

Commit

Permalink
Remove jenkins_authentication::username_password smoke test
Browse files Browse the repository at this point in the history
This is no longer supported upstream and does not work currently.

Signed-off-by: Lance Albertson <[email protected]>
  • Loading branch information
ramereth committed Feb 4, 2021
1 parent c48ab84 commit 6c11dc8
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 53 deletions.
1 change: 0 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,6 @@ jobs:
- 'smoke-war-stable'
- 'smoke-war-latest'
- 'authentication-private-key'
- 'authentication-username-password'
- 'jenkins-proxy-config'
- 'jenkins-proxy-remove'
fail-fast: false
Expand Down
7 changes: 0 additions & 7 deletions kitchen.yml
Original file line number Diff line number Diff line change
Expand Up @@ -105,13 +105,6 @@ suites:
- name: authentication_private_key # Tested in smoke suite
run_list: jenkins_authentication::private_key

- name: authentication_username_password
run_list: jenkins_authentication::username_password
excludes:
- windows-2012r2
- windows-2016
- windows-2019

#
# Proxy suites
# Cannot be tested together
Expand Down

This file was deleted.

2 comments on commit 6c11dc8

@davidsainty
Copy link
Contributor

@davidsainty davidsainty commented on 6c11dc8 Feb 4, 2021

Choose a reason for hiding this comment

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

There is an open issue about this (#762) . When this test fails it is demonstrating a regression, it isn't the test that is broken...

@ramereth
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah thanks! I'll look at reverting this commit and using that fix. Sorry I missed that issue.

Please sign in to comment.