ci: revert back to previous logic of adding ownership to crates.io #380
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.
I've done some experiments with
cargo owner --add
command and apparently that the bot (containerd-runwasi-release-bot
) is not able to addgithub:containerd:runwasi-committers
as an owner to the published crate by itself. See https://github.com/containerd/runwasi/actions/runs/6711072386/job/18237873816#step:8:13However, the bot was able to add the user who triggered the action as an owner to the published crate. See: https://github.com/containerd/runwasi/actions/runs/6714433126/job/18247647749#step:3:26
I want to revert the change back to previous logic to first examine if this published crate already has the desired owner. Otherwise, it adds the user who triggered the action as an owner and the user has then to add
github:containerd:runwasi-committers
as an owner. This follows the doc inRELEASE.md
.The branch
add-owner
serves the single purpose of adding myself as an owner to the publishedcontainerd-shim-wasm-test-modules
crate so that I can add runwasi-committes to its owners. I will delete this branch shortly after.