Skip to content

Fix manifests.txt invalidation#94

Merged
emilyalbini merged 1 commit into
rust-lang:masterfrom
Noratrieb:manifests-invalidation-v2
Jul 6, 2025
Merged

Fix manifests.txt invalidation#94
emilyalbini merged 1 commit into
rust-lang:masterfrom
Noratrieb:manifests-invalidation-v2

Conversation

@Noratrieb

Copy link
Copy Markdown
Member

The run yesterday failed

running "aws" "cloudfront" "create-invalidation" "--invalidation-batch" "file:///codebuild/output/src2735315778/src/release/payload.json" "--distribution-id" "E3NZU1LCBHH4A4"
An error occurred (InvalidArgument) when calling the CreateInvalidation operation: Your request contains one or more invalid invalidation paths.

And the docs say

In the CloudFront console, you can omit the leading slash in the path, like this: images/image2.jpg. When you use the CloudFront API directly, invalidation paths must begin with a leading slash.

So looks like that's the bug.

The run yesterday failed
```
running "aws" "cloudfront" "create-invalidation" "--invalidation-batch" "file:///codebuild/output/src2735315778/src/release/payload.json" "--distribution-id" "E3NZU1LCBHH4A4"
An error occurred (InvalidArgument) when calling the CreateInvalidation operation: Your request contains one or more invalid invalidation paths.
```

And the docs say
> In the CloudFront console, you can omit the leading slash in the path, like this: images/image2.jpg. When you use the CloudFront API directly, invalidation paths must begin with a leading slash.

So looks like that's the bug.

@emilyalbini emilyalbini left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good (I gave Nora the failure log)

@emilyalbini emilyalbini merged commit 061b45a into rust-lang:master Jul 6, 2025
6 checks passed
@Noratrieb Noratrieb deleted the manifests-invalidation-v2 branch July 6, 2025 14:35
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