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

Add Release Workflow #59

Merged
merged 1 commit into from
Aug 29, 2024
Merged

Add Release Workflow #59

merged 1 commit into from
Aug 29, 2024

Conversation

aws-donocl
Copy link
Contributor

@aws-donocl aws-donocl commented Aug 27, 2024

Add a workflow to build and publish a release with tarball of Code Editor.

This workflow is triggered whenever a tag is pushed that begins with v (ex. v10.0.1 or v10.0.1-dev).
Manual approval is required for the tarball to be created and published as a release.

Testing
An example of a successful run of a workflow extremely similar to this: https://github.com/aws-donocl/sagemaker-code-editor/actions/runs/10564424011

Notes
This demonstrates updating the meta.yaml file in sagemaker-code-editor-feedstock. However, the feedstock repository does not get actually updated by this workflow, due to time constraints. This was decided together with @aws-zuoyuan.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Add a workflow to build and publish a release with tarball of Code Editor.

**Testing**
An example of a successful run of a workflow extremely similar to this:
https://github.com/aws-donocl/sagemaker-code-editor/actions/runs/10564424011

**Notes**
This demonstrates updating the meta.yaml file in sagemaker-code-editor-feedstock. However, the feedstock repository does not get actually updated by this workflow, due to time constraints. This was decided together with @aws-zuoyuan.
aws-pangestu
aws-pangestu previously approved these changes Aug 28, 2024
ref: 'dev'
- name: Create a new dev branch
run: |
git checkout -b ${{ env.branch-name }}
Copy link
Contributor

Choose a reason for hiding this comment

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

where does env.branch-name come from? what if a branchw ith the same name already exists?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This comes from lines 16-17. The branch is called 1automated-dev-update`. I believe if there is already a branch with this name, it will still checkout that branch.

SHA256=${{ needs.release.outputs.sha256_hash }}
sed -i "s|sha256: [0-9a-f]*|sha256: $SHA256|" recipe/meta.yaml
- name: Run diff
run: git diff
Copy link
Contributor

Choose a reason for hiding this comment

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

Is the updated meta.yaml not pushed to the new branch?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Correct. From the merge request description:

This demonstrates updating the meta.yaml file in sagemaker-code-editor-feedstock. However, the feedstock repository does not get actually updated by this workflow, due to time constraints. This was decided together with @aws-zuoyuan.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We can discuss this during standup if you feel its necessary

Copy link
Contributor

Choose a reason for hiding this comment

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

Okay, that's fine. If you don't plan to implement this can you create a backlog issue for this and link it here

# Triggers the workflow on updates to the "main" branch which include a version tag
push:
tags:
- '**' # Push events to every tag including hierarchical tags like v1.0/beta
Copy link
Contributor

Choose a reason for hiding this comment

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

should we only check tags with prefix v*?

@aws-donocl aws-donocl merged commit feadcc6 into main Aug 29, 2024
1 check passed
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.

3 participants