Drop the machine-type memory reads from the WIF roles [murmur:murmuration/record-actual-vm-shape-costs] - #9
Open
maerlyn5 wants to merge 1 commit into
Conversation
murmuration no longer asks a cloud for a machine type's RAM. The shape fixes it and the catalog records it, so the read settled nothing a customer's IAM role had to be widened for. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
ApprovabilityVerdict: Not approved Macroscope's review found this PR not approvable — This PR changes customer-facing AWS and GCP WIF IAM policies by revoking machine-type metadata permissions from VM roles. Although the diff is small and focused, authorization changes are security-sensitive and require human review. You can add or adjust custom eligibility rules. Learn more. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
compute.machineTypes.getandec2:DescribeInstanceTypeswere granted so murmuration could read a machine type's RAM at VM creation, sizing GCE's suspend snapshot and AWS's hibernation swap volume. It no longer asks: the chosen machine type fixes its RAM andCatalogMachineType.memory_gbalready records it, so the cloud read was a second answer to a question with one — bought at the price of a permission on every customer's role.Both are now unused, and this removes them from
modules/gcp-wifandmodules/aws-wif.The boot-disk read stays and needs nothing new:
disk_size_gbmay be left unset to mean "take the boot image's own size", so only the created disk knows the GiB a customer is charged for. That runs oncompute.instances.getandec2:DescribeImages, which these roles already hold.Pairs with prassoai/murmuration#6231, which removes the calls. Merge order does not matter — removing a permission nothing requests, and removing a request for a permission, are independently safe.
Supersedes #8, which was closed. That branch carried the pre-squash version of #7's grant commit followed by this removal, so a reviewer diffing it against its own base saw a grant added and taken away and read the whole PR as a no-op. Against
mainit was not:maincarries #7's grants today. This branch is cut frommainand contains only the removal, so the diff says what it does.Created via Murmur
Note
Drop machine-type memory read permissions from AWS and GCP WIF roles
ec2:DescribeInstanceTypesaction from thevm_creatorinline policy in modules/aws-wif/main.tfcompute.machineTypes.getpermission from thevm_lifecyclecustom role in modules/gcp-wif/main.tfaws_iam_role.vm_creatoror granted themurmurVmCreatorcustom role can no longer read machine-type memory informationMacroscope summarized 3ba8583.