Add automated Schema File publish action #1858
Conversation
|
@open-telemetry/docs-approvers please review. |
The action should be triggered manually after every spec release and will copy/update schema files from this repository to the website repository so that they become available publicly. The publish-schemas action after manual triggering will create a PR in https://github.com/open-telemetry/opentelemetry.io repo, and the PR needs to be approved and merged manually. This is similar to how we publish doc updates from language repos to the website repo. Resolves open-telemetry/opentelemetry.io#577 Resolves open-telemetry#1693
tigrannajaryan
force-pushed
the
feature/tigran/schema-publish-action
branch
from
2af3b9e
to
1f37e9c
Compare
| SOURCE_PATH: schemas | ||
| run: | | ||
| TARGET_DIR=$(mktemp -d) | ||
| export GITHUB_TOKEN=$API_TOKEN_GITHUB |
There was a problem hiding this comment.
@austinlparker are you ok with the token reuse here?
| git clone "https://$API_TOKEN_GITHUB@github.com/$DESTINATION_REPO.git" "$TARGET_DIR" | ||
| rsync -av --delete "$SOURCE_PATH/" "$TARGET_DIR/$DESTINATION_PATH/" | ||
| cd "$TARGET_DIR" | ||
| git checkout -b docs-$GITHUB_REPOSITORY-$GITHUB_SHA |
There was a problem hiding this comment.
GITHUB_SHA this is SHA of specs repo, not protos, correct?
There was a problem hiding this comment.
Yes, I think so. (This is code is copied from Java repo, I didn't write it, I believe is @austinlparker the author).
|
@austinlparker Is on paternity leave, so we need to proceed without him. Anybody we could poke for the time being? Maybe @tedsuo ? Else let's just go ahead and merge it. |
|
@SergeyKanzhelev @tigrannajaryan Let's merge this? I think it deserves a run ;) |
|
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
Yes, I think we need to merge it. If anything goes wrong with it I can take a look after it is merged. It needs to be run manually anyway so won't affect CI. |
|
Assuming Austin is fine with the token reuse. |
* Add automated Schema File publish action The action should be triggered manually after every spec release and will copy/update schema files from this repository to the website repository so that they become available publicly. The publish-schemas action after manual triggering will create a PR in https://github.com/open-telemetry/opentelemetry.io repo, and the PR needs to be approved and merged manually. This is similar to how we publish doc updates from language repos to the website repo. Resolves open-telemetry/opentelemetry.io#577 Resolves open-telemetry#1693 * Make branch name and commit message more relevant Co-authored-by: Sergey Kanzhelev <S.Kanzhelev@live.com>
The action should be triggered manually after every spec release
and will copy/update schema files from this repository to the
website repository so that they become available publicly.
The publish-schemas action after manual triggering will create a PR
in https://github.com/open-telemetry/opentelemetry.io repo, and the
PR needs to be approved and merged manually.
This is similar to how we publish doc updates from language repos
to the website repo.
Resolves open-telemetry/opentelemetry.io#577
Resolves #1693