Skip to content

feat(RAIN-94027): add terraform permissions for AWS service compute-optimizer #113

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

Closed
wants to merge 4 commits into from

Conversation

yingxinl
Copy link
Contributor

@yingxinl yingxinl commented Dec 2, 2024

Summary

Add terraform permissions for AWS service compute-optimizer

How did you test this change?

Tested in tilt. Permissions are added only for dev8-rm-test account (249446771485). By running the queries below, we can see AccessDeniedException in other accounts except account_id of 249446771485.

-- Collection completed for envGuid: DEV8_B895FFBB65A0D5A2E543F62A2CB3CD9343293637C75D5C9BA80 
-- with startTime: 2024-12-04T08:10:00 and endTime: 2024-12-04T08:20:00

use database DEV8_CDB_DEV8_B895FFBB65A0D5A2E543F62A2CB3CD9343293637C75D5C9BA80;

select request_guid, start_time, end_time from aws_cfg_internal.config_summary_t
where start_time = '2024-12-04T08:10:00-00:00' and end_time = '2024-12-04T08:20:00-00:00';

SELECT DISTINCT account_id, service, api_key, status
FROM aws_cfg_internal.config_preview_details_t
WHERE REQUEST_GUID = 'c5578dc6-cefd-41fe-a081-209c0619458a';

Issue

https://lacework.atlassian.net/browse/RAIN-94027

@yingxinl yingxinl marked this pull request as ready for review December 4, 2024 18:43
@yingxinl
Copy link
Contributor Author

yingxinl commented Dec 5, 2024

Cannot be merged due to unsigned commits, so another PR was created here.

@yingxinl yingxinl closed this Dec 5, 2024
@yingxinl yingxinl deleted the RAIN-94027-compute-optimizer-permissions branch December 5, 2024 00:49
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