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 version of trend-cloudone-onboard. #2

Merged
merged 29 commits into from
May 2, 2023
Merged

Conversation

raphabot
Copy link
Collaborator

@raphabot raphabot commented Apr 6, 2023

⚠️ Please do not merge

Altough I was able to run taskcat to test it locally, I wasn't able to properly run ash, and I still have multiple questions.

Questions

  • How do I properly run ash?
  • How do I configure secrets in the remote taskcat execution? There are two parameters that are secrets unique to a customer, I wonder how we can then test this in GitHub Actions.
  • This stack uses CloudTrail so I am imported the the abi-cloudtrail submodule. I have no idea, however, how to use the submodule instead of creating a trail myself (current behavior)
  • All my lambda functions are small enough that they are built-in into the templates. Is that accepted? Do I need to copy them over the lambdas/ folder as well?

@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.

@raphabot raphabot marked this pull request as draft April 6, 2023 18:59
@raphabot raphabot marked this pull request as ready for review April 14, 2023 13:53
@kkvinjam
Copy link
Collaborator

/do-e2e-tests

@aws-ia-automator-gamma
Copy link

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

@kkvinjam
Copy link
Collaborator

/confirm-publication

@sshvans sshvans marked this pull request as draft April 28, 2023 03:59
@sshvans sshvans marked this pull request as ready for review April 28, 2023 03:59
@kkvinjam
Copy link
Collaborator

kkvinjam commented May 2, 2023

/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.

@kkvinjam
Copy link
Collaborator

kkvinjam commented May 2, 2023

/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

@kkvinjam kkvinjam merged commit 4d5a232 into aws-ia:main May 2, 2023
@raphabot raphabot deleted the devel branch May 2, 2023 19:50
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.

2 participants