-
Notifications
You must be signed in to change notification settings - Fork 1.3k
chore: Update docs with new release workflow #2898
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
Changes from 1 commit
ecf99f0
54d9fbe
00651be
748213b
136258d
b779e51
1820ae9
cdb5005
4f1c410
ae4bc72
256bc63
a8ca125
8bc5b1c
bb2e4c7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
Signed-off-by: Kevin Zhang <kzhang@tecton.ai>
- Loading branch information
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8,15 +8,15 @@ For Feast maintainers, these are the concrete steps for making a new release. | |
|
|
||
| 1. Merge upstream master changes into your fork. | ||
| 2. Create a tag manually for the release on your fork. For example, if your release is on version 0.22.0, create a tag by doing the following. | ||
|
felixwang9817 marked this conversation as resolved.
Outdated
|
||
| - Checkout master branch and run `git tag v0.22.0`. | ||
| - Run `git push --tags` to push the tag to remote. | ||
| - Checkout master branch and run `git tag v0.22.0`. | ||
| - Run `git push --tags` to push the tag to remote. | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. To push the tag to your forks master branch.
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fixed |
||
| 3. Access the `Actions` tab on your github UI on your fork and click the `build_wheels` action. | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What does this workflow do? What does it build? What combinations?
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fixed. |
||
| 4. Look for the header `This workflow has a workflow_dispatch event trigger.` and click `Run Workflow` on the right. | ||
| 5. Run the branch off of the tag you just created(`v0.22.0`) in this case and verify that the workflow worked. | ||
| 5. Run the branch off of the tag you just created(`v0.22.0` in this case) and verify that the workflow worked. | ||
|
felixwang9817 marked this conversation as resolved.
Outdated
felixwang9817 marked this conversation as resolved.
Outdated
|
||
|
|
||
| ### Release | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What stuff does this workflow do? Python release? Java release? UI release? Helm charts? Docker images?
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Clarified |
||
| 6. Generate a [Personal Access Token](https://docs.github.com/en/authentication/keeping-your-account-and-data-secure/creating-a-personal-access-token) or retrieve your saved personal access token. | ||
| - The personal access token should have all of the permissions under the `repo` checkbox. | ||
| - The personal access token should have all of the permissions under the `repo` checkbox. | ||
| 7. Access the `Actions` tab on the main `feast-dev/feast` repo and find the `release` action. | ||
| 8. Look for the header `This workflow has a workflow_dispatch event trigger.` again and click `Run Workflow` on the right. | ||
| 9. Try the dry run first with your personal access token. If this succeeds, uncheck `Dry Run` and run the release workflow. | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe this should have two sections, one on how to run the
build_wheelsworkflow on personal forks with a manually applied tag to verify wheels are being created correctly, and a second on how to run thereleaseworkflow using your personal access token on master in the feast repo.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed