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

elb_application_lb: enable the functional test #348

Conversation

goneri
Copy link
Member

@goneri goneri commented Jan 7, 2021

Remove the unsupported aliases for the elb_application_lb test.

Use HTTP instead of HTTPS to avoid the dependency on iam:ListServerCertificates and the other Certificate related operations.

Depends on:

@goneri goneri force-pushed the elb_application_lb-enable-the-functional-test_24328 branch 2 times, most recently from 22ad598 to e22cd3e Compare January 7, 2021 18:00
@goneri
Copy link
Member Author

goneri commented Jan 7, 2021

PR on hold until mattclay/aws-terminator#121 is merged and deployed.

Remove the `unsupported` aliases for the `elb_application_lb` test.

Use HTTP instead of HTTPS to avoid the dependency on
`iam:ListServerCertificates` and the other Certificate related operations.
@goneri goneri force-pushed the elb_application_lb-enable-the-functional-test_24328 branch from e22cd3e to 6a3a0f6 Compare January 12, 2021 17:48
@ansibullbot
Copy link

@goneri: Greetings! Thanks for taking the time to open this pullrequest. In order for the community to handle your pullrequest effectively, we need a bit more information.

Here are the items we could not find in your description:

  • issue type

Please set the description of this pullrequest with this template:
https://raw.githubusercontent.com/ansible/ansible/devel/.github/PULL_REQUEST_TEMPLATE.md

click here for bot help

@ansibullbot
Copy link

@ansibullbot ansibullbot added integration tests/integration needs_info This issue requires further information. Please answer any outstanding questions needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR needs_template This issue/PR has an incomplete description. Please fill in the proposed template correctly needs_triage plugins plugin (any type) tests tests labels Jan 13, 2021
@goneri goneri merged commit 5f3c86c into ansible-collections:main Jan 18, 2021
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
Remove the `unsupported` aliases for the `elb_application_lb` test.

Use HTTP instead of HTTPS to avoid the dependency on
`iam:ListServerCertificates` and the other Certificate related operations.
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
Remove the `unsupported` aliases for the `elb_application_lb` test.

Use HTTP instead of HTTPS to avoid the dependency on
`iam:ListServerCertificates` and the other Certificate related operations.
danielcotton pushed a commit to danielcotton/community.aws that referenced this pull request Nov 23, 2021
Remove the `unsupported` aliases for the `elb_application_lb` test.

Use HTTP instead of HTTPS to avoid the dependency on
`iam:ListServerCertificates` and the other Certificate related operations.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
integration tests/integration needs_info This issue requires further information. Please answer any outstanding questions needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR needs_template This issue/PR has an incomplete description. Please fill in the proposed template correctly needs_triage plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants