Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PR #1764/2fe39baa backport][stable-5] Fix SASL/SCRAM + add option for SASL IAM & add option to disable unauthenticated clients #1780

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Apr 19, 2023

This is a backport of PR #1764 as merged into main (2fe39ba).

SUMMARY

Many thanx to @markuman for throwing me into this issue.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

msk_cluster

ADDITIONAL INFORMATION

I will probably add more tests after working w/ this.

…thenticated clients (#1764)

Fix SASL/SCRAM + add option for SASL IAM & add option to disable unauthenticated clients

SUMMARY

fix SASL/SCRAM - Fixes #1761
add IAM authentication
add option to disable unauthenticated clients

Many thanx to @markuman for throwing me into this issue.
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
msk_cluster
ADDITIONAL INFORMATION
I will probably add more tests after working w/ this.

Reviewed-by: Mark Chappell
Reviewed-by: Gabriel PREDA
Reviewed-by: Markus Bergholz <[email protected]>
(cherry picked from commit 2fe39ba)
@github-actions
Copy link

github-actions bot commented Apr 19, 2023

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/3843db9f6d4d4a718a038bd922961969

✔️ ansible-galaxy-importer SUCCESS in 3m 54s
✔️ build-ansible-collection SUCCESS in 12m 34s
ansible-test-sanity-docker-devel FAILURE in 10m 27s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 31s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 12m 50s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 06s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 10m 17s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 5m 50s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 11s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 33s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 6m 09s
✔️ ansible-test-changelog SUCCESS in 4m 25s
✔️ ansible-test-splitter SUCCESS in 4m 56s
✔️ integration-community.aws-1 SUCCESS in 35m 36s
✔️ integration-community.aws-2 SUCCESS in 41m 50s
Skipped 20 jobs

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Apr 19, 2023
@softwarefactory-project-zuul
Copy link
Contributor

Build failed (gate pipeline). For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

https://ansible.softwarefactory-project.io/zuul/buildset/2cb8b6c1272f4e419618c1534da18cc7

ansible-galaxy-importer FAILURE in 3m 53s
✔️ build-ansible-collection SUCCESS in 12m 36s
ansible-test-sanity-docker-devel FAILURE in 12m 46s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 42s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 41s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 13m 07s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 10m 18s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 02s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 04s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 8m 59s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 6m 27s
✔️ ansible-test-changelog SUCCESS in 4m 23s
✔️ ansible-test-splitter SUCCESS in 4m 39s
✔️ integration-community.aws-1 SUCCESS in 33m 07s
✔️ integration-community.aws-2 SUCCESS in 39m 36s
Skipped 20 jobs

@tremble
Copy link
Contributor

tremble commented Apr 19, 2023

regate

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/0d5663884f0040a8882f4c5d8c371ecd

✔️ ansible-galaxy-importer SUCCESS in 4m 37s
✔️ build-ansible-collection SUCCESS in 12m 43s
ansible-test-sanity-docker-devel FAILURE in 13m 18s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 12m 58s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 13m 14s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 12m 58s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 12m 46s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 8m 29s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 8m 10s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 32s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 8m 55s
✔️ ansible-test-changelog SUCCESS in 4m 20s
✔️ ansible-test-splitter SUCCESS in 4m 47s
✔️ integration-community.aws-1 SUCCESS in 36m 31s
✔️ integration-community.aws-2 SUCCESS in 37m 38s
Skipped 20 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 496ad7b into stable-5 Apr 19, 2023
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-5/2fe39baaa98a3f488303e473c611a65244fc4034/pr-1764 branch April 19, 2023 15:08
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants