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

ref(docs): update to deploy/run via oci #185

Merged
merged 1 commit into from
Nov 29, 2023
Merged

ref(docs): update to deploy/run via oci #185

merged 1 commit into from
Nov 29, 2023

Conversation

vdice
Copy link
Member

@vdice vdice commented Nov 28, 2023

  • Updates the deployment mechanism of this website (GH workflow and nomad job) to use OCI

TODO:

  • Test run from this branch

@vdice vdice marked this pull request as ready for review November 28, 2023 19:26
@vdice
Copy link
Member Author

vdice commented Nov 28, 2023

This is ready for review. Website is successfully deployed from this branch (eg https://github.com/fermyon/bartholomew/actions/runs/7023277511) and I toggled DNS to the LB hosting this oci-based deployment. (I can toggle back if there happen to be other PRs in this repo affecting the docs site, but I'm expecting there won't be any.)

Note that this PR represents the pattern that'll be used to convert the other Fermyon websites, so definitely want to get things right/address nits/etc.

After LGTMs and prior to merge, I'll squash/rebase commits and remove my testing branch from the workflow trigger list.

@vdice vdice changed the title ref(*): update to deploy/run via oci ref(docs): update to deploy/run via oci Nov 28, 2023
deploy/bartholomew-docs.nomad Outdated Show resolved Hide resolved
Signed-off-by: Vaughn Dice <vaughn.dice@fermyon.com>
@vdice
Copy link
Member Author

vdice commented Nov 29, 2023

(As mentioned, 93e7e82 represents commits squashed down to one and removal of the vdice* testing branch config from deploy.yml)

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.

2 participants