-
Notifications
You must be signed in to change notification settings - Fork 13
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
feat(cg-1342): support opsworks services #131
base: alpha
Are you sure you want to change the base?
Conversation
@tyler-dunkel @m-pizarro I have a comment in the [opsworksApp/format.ts] Also, the code is not fully tested. I just ran it without data. I didn't create any opsworks resources in AWS. Don't know what set of services need to create and afraid this would cost a lot. Could someone create a set of test services? |
@ckoning whats the cost for setting up opswork? |
@tyler-dunkel test the data and updated the arn builder to use the appId |
@tyler-dunkel it's ready to go. We only have a opsWorksStack to test but I think we can go ahead. Unless you want to test the other services also... |
Issue tracker links
Add links to any relevant tasks/stories/bugs/pagerduty/etc
Example - dummy TODO project
TODO-123
Changes/solution
How does this change address the problem?
Testing
Describe how the testing was done, plus evidence, if not covered by automated tests
Notes and considerations
Add any additional notes and/or considerations
Dependencies
_Add dependencies on any other PRs, if applicable