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

first release #43

Merged
merged 17 commits into from
Jun 5, 2023
Merged

first release #43

merged 17 commits into from
Jun 5, 2023

Conversation

maiconrocha
Copy link
Collaborator

No description provided.

@aws-ia-automator-gamma
Copy link

Hi there,

I am commenting because your project has not yet been published and you have opened a pr to the main branch.
If we proceed with reviewing this pr and it is merged this will result in the publication of your project. If you are
not ready to publish yet, we recommend you close this PR and develop against a different branch.
Once you are ready to publish you can open a PR from that branch to main.

If you are ready to publish, please issue the following comment on this PR and I will initiate the publication review
process:

/confirm-publication

If you require more information please reach out to your AWS PoC who is assisting you with this project.

@maiconrocha maiconrocha changed the title [Draft] first release first release May 11, 2023
@maiconrocha
Copy link
Collaborator Author

functional tests are running fine now

TestExamplesBasic 2023-05-11T06:03:34Z logger.go:66: Destroy complete! Resources: 52 destroyed.
TestExamplesBasic 2023-05-11T06:03:34Z logger.go:66: 
PASS
ok      github.com/aws-ia/terraform-project-ephemeral   2349.661s
End of Functional Tests
===========================================================================
TASK LOGS END
===========================================================================
[15:39](https://amzn-aws.slack.com/archives/D054UP6MBGV/p1683785391545639)

static-test as well

No problems detected!

Starting markdown lint
Success - markdown lint found no linting issues!
Starting terraform-docs
Success - Terraform Docs creation verified!
End of Static Tests
===========================================================================
TASK LOGS END
===========================================================================

@maiconrocha
Copy link
Collaborator Author

/confirm-publication

Copy link

@aws-ia-automator-gamma aws-ia-automator-gamma bot left a comment

Choose a reason for hiding this comment

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

Review hs been requested from required contributors, once they have approved I will remove my request for changes. When this PR is merged your project will be published

@tbulding
Copy link
Contributor

/do-e2e-tests

@aws-ia-automator-gamma
Copy link

Checks must all be completed successfully before end to end testing can be run.

@tbulding tbulding marked this pull request as draft May 11, 2023 20:30
@tbulding tbulding marked this pull request as ready for review May 11, 2023 20:30
@tlindsay42
Copy link
Member

Per our DMs, the error in static analysis is valid. It's because resource "aws_iam_role_policy_attachment" "custom_policy" is not conditionally created, but the variable is implemented as optional with a default of "", which that resource correctly won't accept for the policy_arn property. To fix it, make that resource conditional on var.custom_policy_arn having a value, or better yet, a length greater than 20 to match the rule, or even better than that- add the resource condition and a regex validation on the variable to ensure that it's either a properly formatted ARN or empty (default).

main.tf Show resolved Hide resolved
@maiconrocha
Copy link
Collaborator Author

/do-e2e-tests

@aws-ia-automator-gamma
Copy link

End to end test has been scheduled

@aws-ia-automator-gamma
Copy link

e2e test in progress

Copy link

@aws-ia-automator-gamma aws-ia-automator-gamma bot left a comment

Choose a reason for hiding this comment

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

e2e test has completed with errors. If you are an external contributor, please contact the project maintainers for more information.

@maiconrocha
Copy link
Collaborator Author

@tlindsay42 how I can check the logs for the e2e tests?

@maiconrocha
Copy link
Collaborator Author

/do-e2e-tests

@maiconrocha
Copy link
Collaborator Author

can someone please run the e2e tests again? I believe I have fixed the issue, local tests are ok.

@aws-ia-automator-gamma
Copy link

Checks must all be completed successfully before end to end testing can be run.

@maiconrocha
Copy link
Collaborator Author

static tests are failing now, weird
Failure - Terraform Docs creation failed, ensure you have precommit installed and running before submitting the Pull Request
it was working previously

@maiconrocha
Copy link
Collaborator Author

/do-e2e-tests

@aws-ia-automator-gamma
Copy link

End to end test has been scheduled

@aws-ia-automator-gamma
Copy link

e2e test in progress

Copy link

@aws-ia-automator-gamma aws-ia-automator-gamma bot left a comment

Choose a reason for hiding this comment

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

e2e test has completed with errors. If you are an external contributor, please contact the project maintainers for more information.

@maiconrocha
Copy link
Collaborator Author

/do-e2e-tests

1 similar comment
@maiconrocha
Copy link
Collaborator Author

/do-e2e-tests

@aws-ia-automator-gamma
Copy link

End to end test has been scheduled

@aws-ia-automator-gamma
Copy link

e2e test in progress

Copy link

@aws-ia-automator-gamma aws-ia-automator-gamma bot left a comment

Choose a reason for hiding this comment

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

e2e test has completed with errors. If you are an external contributor, please contact the project maintainers for more information.

@maiconrocha
Copy link
Collaborator Author

/do-e2e-tests

@aws-ia-automator-gamma
Copy link

Checks must all be completed successfully before end to end testing can be run.

@maiconrocha
Copy link
Collaborator Author

/do-e2e-tests

@aws-ia-automator-gamma
Copy link

End to end test has been scheduled

@aws-ia-automator-gamma
Copy link

e2e test in progress

Copy link

@aws-ia-automator-gamma aws-ia-automator-gamma bot left a comment

Choose a reason for hiding this comment

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

e2e test has completed with errors. If you are an external contributor, please contact the project maintainers for more information.

@maiconrocha
Copy link
Collaborator Author

/do-e2e-tests

@aws-ia-automator-gamma
Copy link

Checks must all be completed successfully before end to end testing can be run.

@maiconrocha
Copy link
Collaborator Author

/do-e2e-tests

@aws-ia-automator-gamma
Copy link

End to end test has been scheduled

@aws-ia-automator-gamma
Copy link

e2e test in progress

Copy link

@aws-ia-automator-gamma aws-ia-automator-gamma bot left a comment

Choose a reason for hiding this comment

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

e2e tests completed successfully

@tbulding tbulding self-requested a review June 5, 2023 14:18
@tbulding tbulding merged commit a57771d into aws-ia:main Jun 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants