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 tutorial deployment process #1154

Merged
merged 6 commits into from
Apr 16, 2024
Merged

Add tutorial deployment process #1154

merged 6 commits into from
Apr 16, 2024

Conversation

frankharkins
Copy link
Member

@frankharkins frankharkins commented Apr 10, 2024

Adds a workflow to deploy tutorials from main, and documentation on how to do it. This PR also switches to using environment secrets rather than actions secrets, which should be more secure as we can limit who can run them and which branches they can run on. See https://github.com/Qiskit/documentation/actions/runs/8647825547/job/23710189504 for an example of the action using the staging environment.

@frankharkins frankharkins marked this pull request as ready for review April 10, 2024 14:12
@frankharkins frankharkins linked an issue Apr 10, 2024 that may be closed by this pull request
@frankharkins frankharkins temporarily deployed to Learning platform (staging) April 11, 2024 13:42 — with GitHub Actions Inactive
@frankharkins frankharkins temporarily deployed to Learning platform (staging) April 11, 2024 13:44 — with GitHub Actions Inactive
I realised we can test this out by dispatching from this branch
@frankharkins frankharkins temporarily deployed to Learning platform (staging) April 11, 2024 13:47 — with GitHub Actions Inactive
Copy link
Collaborator

@Eric-Arellano Eric-Arellano left a comment

Choose a reason for hiding this comment

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

Thanks!

Co-authored-by: Eric Arellano <14852634+Eric-Arellano@users.noreply.github.com>
@frankharkins frankharkins enabled auto-merge April 16, 2024 12:17
@frankharkins frankharkins added this pull request to the merge queue Apr 16, 2024
Merged via the queue into main with commit 65bc7f6 Apr 16, 2024
2 checks passed
@frankharkins frankharkins deleted the FH/deploy-tutorials branch April 16, 2024 12:20
github-merge-queue bot pushed a commit that referenced this pull request Apr 23, 2024
I thought I'd tested this in #1154 but I actually only tested the
uploader test workflow. I'd forgotten to pass the environment from the
input to the job.

Something which wasn't my fault is that some tutorial UUIDs seem to have
changed, I'm not sure why this is but am investigating.

With the new UUIDs, the workflow is working ([see
run](https://github.com/Qiskit/documentation/actions/runs/8798350887/job/24145159885)).
frankharkins added a commit to frankharkins/documentation that referenced this pull request Jul 22, 2024
Adds a workflow to deploy tutorials from `main`, and documentation on
how to do it. This PR also switches to using environment secrets rather
than actions secrets, which should be more secure as we can limit who
can run them and which branches they can run on. See
https://github.com/Qiskit/documentation/actions/runs/8647825547/job/23710189504
for an example of the action using the staging environment.

---------

Co-authored-by: Eric Arellano <14852634+Eric-Arellano@users.noreply.github.com>
frankharkins added a commit to frankharkins/documentation that referenced this pull request Jul 22, 2024
I thought I'd tested this in Qiskit#1154 but I actually only tested the
uploader test workflow. I'd forgotten to pass the environment from the
input to the job.

Something which wasn't my fault is that some tutorial UUIDs seem to have
changed, I'm not sure why this is but am investigating.

With the new UUIDs, the workflow is working ([see
run](https://github.com/Qiskit/documentation/actions/runs/8798350887/job/24145159885)).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

set up testing and learning platform deployment for tutorials
2 participants