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

Delete CHANGELOG_PENDING.md #10703

Merged
merged 1 commit into from Sep 15, 2022
Merged

Delete CHANGELOG_PENDING.md #10703

merged 1 commit into from Sep 15, 2022

Conversation

AaronFriel
Copy link
Member

No description provided.

@pulumi-bot
Copy link
Contributor

Changelog

[uncommitted] (2022-09-14)

@AaronFriel
Copy link
Member Author

bors merge

@AaronFriel
Copy link
Member Author

bors retry

@bors
Copy link
Contributor

bors bot commented Sep 14, 2022

Already running a review

bors bot added a commit that referenced this pull request Sep 14, 2022
10702: Prep for real 3.40.0 release r=AaronFriel a=AaronFriel

This turns the "safeties off" after validating the bors workflow and ensuring everything up to the dispatch commands succeeds.

It also fixes an error in `get-next-version` that resulted in the "vNext" PR #10701 to calculate the wrong "next version", which was corrected by hand. (The branch name and commit history is evidence of the error.)

10703: Delete CHANGELOG_PENDING.md r=AaronFriel a=AaronFriel



10708: ci: Re-enable checksums r=AaronFriel a=AaronFriel

In the absence of key distribution or attestation of a "proper" signing key for now, this enables the "sign" job which adds checksum artifacts, sans signatures.

This leaves open the possibility of using sigstore.dev & tooling, or some other tooling to sign artifacts.

Fixes #10707.

Co-authored-by: Aaron Friel <mayreply@aaronfriel.com>
@bors
Copy link
Contributor

bors bot commented Sep 14, 2022

Build failed (retrying...):

bors bot added a commit that referenced this pull request Sep 14, 2022
10703: Delete CHANGELOG_PENDING.md r=AaronFriel a=AaronFriel



10708: ci: Re-enable checksums r=AaronFriel a=AaronFriel

In the absence of key distribution or attestation of a "proper" signing key for now, this enables the "sign" job which adds checksum artifacts, sans signatures.

This leaves open the possibility of using sigstore.dev & tooling, or some other tooling to sign artifacts.

Fixes #10707.

Co-authored-by: Aaron Friel <mayreply@aaronfriel.com>
@bors
Copy link
Contributor

bors bot commented Sep 14, 2022

Build failed (retrying...):

@bors
Copy link
Contributor

bors bot commented Sep 14, 2022

Canceled.

@AaronFriel
Copy link
Member Author

bors retry

@AaronFriel
Copy link
Member Author

bors merge

@bors
Copy link
Contributor

bors bot commented Sep 14, 2022

Already running a review

bors bot added a commit that referenced this pull request Sep 14, 2022
10703: Delete CHANGELOG_PENDING.md r=AaronFriel a=AaronFriel



10717: ci: Skip Python service tests when access token is absent r=AaronFriel a=AaronFriel

These tests fail loudly and hard when `PULUMI_ACCESS_TOKEN` is not set, making local dev experience more brittle. Along with changes made in a future PR to enable easier local dev testing without a Pulumi service account, this will improve developer experience when running `make` in the repo.

Co-authored-by: Aaron Friel <mayreply@aaronfriel.com>
@bors
Copy link
Contributor

bors bot commented Sep 14, 2022

Build failed (retrying...):

bors bot added a commit that referenced this pull request Sep 15, 2022
10703: Delete CHANGELOG_PENDING.md r=AaronFriel a=AaronFriel



Co-authored-by: Aaron Friel <mayreply@aaronfriel.com>
@bors
Copy link
Contributor

bors bot commented Sep 15, 2022

Build failed:

@AaronFriel
Copy link
Member Author

bors retry

@bors
Copy link
Contributor

bors bot commented Sep 15, 2022

Build succeeded:

@bors bors bot merged commit bc704af into master Sep 15, 2022
@bors bors bot deleted the friel/rm-changelog-pending branch September 15, 2022 06:14
bors bot added a commit that referenced this pull request Nov 8, 2022
11277: ci: block non-linear merges r=AaronFriel a=AaronFriel

This ensures commit history is linear, enabling customer-owned forks of the pulumi CLI to more easily maintain their fork. Reverse merges into PR branches result in a more complex process for them and for us.

To test this PR, I based my PR branch off an older commit from the target branch and added a merge commit. That resulted in the check failing:
https://github.com/pulumi/pulumi/actions/runs/3416559764/jobs/5686840393

> Checking merge commit efb7be0  for non-linear history
> Main branch parent is: 83c9dfc Merge #11262
> PR branch parents are d9460b9
> Checking: d9460b9 Merge remote-tracking branch 'origin/master' into friel/block-reverse-merge
> Error: Non-linear history, PR contains a merge d9460b9. Remove this by rebasing on the target.
> Error: Detected non-linear history.
> Error: Process completed with exit code 1.

Fixes #10903

The script used by this lint contains several "tests" and useful diagnostics. Example outputs:

```shell
$ ./scripts/git-linear-history-check.sh f033d9d
Checking merge commit f033d9d for non-linear history
Main branch parent is: 0797f29 Merge #10817
PR branch parents are 110dd76 ffbb03c cdf8f20 396650a
Checking: 110dd76 ci: Pin yarn lockfile for security & dependency scanning
Checking: ffbb03c ci: Build binary with .exe extension on Windows
Checking: cdf8f20 ci: Remove several test skips, check if unnecessary
Checking: 396650a ci: Re-enable Windows tests with temp dir
✅ Commit history is linear.
```

```shell
$ ./scripts/git-linear-history-check.sh 0f3e536 
Checking merge commit 0f3e536 for non-linear history
Main branch parent is: bc704af Merge #10703 #10717
PR branch parents are 9065d7c 22f2989 9f5ec4a
Checking: 9065d7c refactored defaultServiceLoop into its own method
::error::Non-linear history, PR contains a merge fa09da6. Remove this by rebasing on the target.
::error::Non-linear history, PR contains a merge 536f3d6. Remove this by rebasing on the target.
Checking: 22f2989 ci: Fix package parallelism assignment
Checking: 9f5ec4a Add missing `ProgramTestOptions` overrides in `With`
::error::Detected non-linear history.
```

The `::error::` messages should appear in GitHub Actions logs at the top level (the workflow level) as well as in the detailed output of the action. 

Co-authored-by: Aaron Friel <mayreply@aaronfriel.com>
bors bot added a commit that referenced this pull request Nov 8, 2022
11277: ci: block non-linear merges r=AaronFriel a=AaronFriel

This ensures commit history is linear, enabling customer-owned forks of the pulumi CLI to more easily maintain their fork. Reverse merges into PR branches result in a more complex process for them and for us.

To test this PR, I based my PR branch off an older commit from the target branch and added a merge commit. That resulted in the check failing:
https://github.com/pulumi/pulumi/actions/runs/3416559764/jobs/5686840393

> Checking merge commit efb7be0  for non-linear history
> Main branch parent is: 83c9dfc Merge #11262
> PR branch parents are d9460b9
> Checking: d9460b9 Merge remote-tracking branch 'origin/master' into friel/block-reverse-merge
> Error: Non-linear history, PR contains a merge d9460b9. Remove this by rebasing on the target.
> Error: Detected non-linear history.
> Error: Process completed with exit code 1.

Fixes #10903

The script used by this lint contains several "tests" and useful diagnostics. Example outputs:

```shell
$ ./scripts/git-linear-history-check.sh f033d9d
Checking merge commit f033d9d for non-linear history
Main branch parent is: 0797f29 Merge #10817
PR branch parents are 110dd76 ffbb03c cdf8f20 396650a
Checking: 110dd76 ci: Pin yarn lockfile for security & dependency scanning
Checking: ffbb03c ci: Build binary with .exe extension on Windows
Checking: cdf8f20 ci: Remove several test skips, check if unnecessary
Checking: 396650a ci: Re-enable Windows tests with temp dir
✅ Commit history is linear.
```

```shell
$ ./scripts/git-linear-history-check.sh 0f3e536 
Checking merge commit 0f3e536 for non-linear history
Main branch parent is: bc704af Merge #10703 #10717
PR branch parents are 9065d7c 22f2989 9f5ec4a
Checking: 9065d7c refactored defaultServiceLoop into its own method
::error::Non-linear history, PR contains a merge fa09da6. Remove this by rebasing on the target.
::error::Non-linear history, PR contains a merge 536f3d6. Remove this by rebasing on the target.
Checking: 22f2989 ci: Fix package parallelism assignment
Checking: 9f5ec4a Add missing `ProgramTestOptions` overrides in `With`
::error::Detected non-linear history.
```

The `::error::` messages should appear in GitHub Actions logs at the top level (the workflow level) as well as in the detailed output of the action. 

Co-authored-by: Aaron Friel <mayreply@aaronfriel.com>
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.

None yet

4 participants