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

POL-1378 Linting Updates: Cost Policies: AWS #2734

Merged
merged 189 commits into from
Oct 16, 2024
Merged

Conversation

XOmniverse
Copy link
Contributor

@XOmniverse XOmniverse commented Oct 8, 2024

Description

Various small changes to Security policies to bring them up to current linting standards. Also removes known bad coding patterns to avoid their reuse.

@XOmniverse XOmniverse requested a review from a team as a code owner October 8, 2024 20:39
Copy link
Contributor

github-actions bot commented Oct 8, 2024

10 Warnings
⚠️

Important Files Modified

Please make sure these modifications were intentional and have been tested. These files are necessary for configuring the Github repository and managing automation.

.dangerfile/policy_tests.rb

⚠️

cost/aws/ec2_compute_optimizer/aws_ec2_compute_optimizer.pt

Policy will not be published in the public catalog. If this is not the intended behavior, remove the publish field from the policy's info metadata block.

⚠️

cost/aws/extended_support/aws_extended_support.pt

Recommendation policy has export that is missing recommended fields. These fields are scraped by the Flexera platform for dashboards:

Line 422: resourceName, tags

⚠️

cost/aws/instance_cost_per_hour/aws_instance_cost_per_hour.pt

Policy will not be published in the public catalog. If this is not the intended behavior, remove the publish field from the policy's info metadata block.


Possible invalid spacing between comma-separated items found:

Line 426: chartAxisFormat: encodeURI("chxs=0N*c" + ds_currency['code'] + "2z*|1,min40"),

Comma separated items should be organized as follows, with a single space following each comma: apple, banana, pear

⚠️

cost/aws/object_storage_optimization/aws_object_storage_optimization.pt

Detected new request datasource(s) in Policy Template file. Please verify the README.md has any new permissions that may be required.

⚠️

cost/aws/rightsize_elasticache/aws_rightsize_elasticache.pt

Policy template updated but associated README.md file has not been. Please verify that any necessary changes have been made to the README.

⚠️

cost/aws/rightsize_redshift/aws_rightsize_redshift.pt

Policy template updated but associated README.md file has not been. Please verify that any necessary changes have been made to the README.

⚠️

cost/aws/s3_bucket_size/aws_bucket_size.pt

Possible invalid spacing between comma-separated items found:

Line 40: description "The number of GiB of space a bucket needs to exceed to include it in the results. Binary gibibytes (e.g. 1,073,741,824 bytes in a gibibyte) are used to maintain parity with how AWS S3 storage is billed."

Comma separated items should be organized as follows, with a single space following each comma: apple, banana, pear

⚠️

cost/aws/s3_storage_policy/aws_s3_bucket_policy_check.pt

Detected new request datasource(s) in Policy Template file. Please verify the README.md has any new permissions that may be required.

⚠️

cost/aws/schedule_instance/aws_schedule_instance.pt

Policy template updated but associated README.md file has not been. Please verify that any necessary changes have been made to the README.

Generated by 🚫 Danger

taterdog590
taterdog590 previously approved these changes Oct 14, 2024
@nia-vf1 nia-vf1 dismissed taterdog590’s stale review October 14, 2024 16:28

Outstanding conversation to be resolved

Copy link
Contributor

@nia-vf1 nia-vf1 left a comment

Choose a reason for hiding this comment

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

LGTM

@XOmniverse XOmniverse merged commit bf5a168 into master Oct 16, 2024
11 checks passed
@XOmniverse XOmniverse deleted the POL-1378-linting-cost-aws branch October 16, 2024 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants