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

CI: add preview deployments for PRs #94

Open
vdice opened this issue Mar 11, 2024 · 8 comments
Open

CI: add preview deployments for PRs #94

vdice opened this issue Mar 11, 2024 · 8 comments
Assignees
Labels
area: CI Improvements or additions to the CI/CD pipeline. enhancement New feature request.

Comments

@vdice
Copy link
Contributor

vdice commented Mar 11, 2024

No description provided.

@ThorstenHans
Copy link
Contributor

I was doing some research on this one.

Although actions/deploy-pages has a preview input (https://github.com/actions/deploy-pages/tree/b580d214b4e13b2a70d0e04376a86ed862ebb558?tab=readme-ov-file#inputs-), it's marked alpha features which is not available to the public.

See actions/deploy-pages#180 and https://github.com/orgs/community/discussions/7730 for further details

@vdice
Copy link
Contributor Author

vdice commented Mar 18, 2024

I'm noticing some community actions like this one... though have only glanced at it on the surface. Would want to test out if it looked promising.

@ThorstenHans
Copy link
Contributor

ThorstenHans commented Mar 18, 2024

Stumbled upon that one as well.

I wasn't sure if that could somehow impact search experience as we index all the things on spinkube.dev maybe we can instruct it to deploy previews to another origin as well and prevent the search engine from indexing that one

@vdice
Copy link
Contributor Author

vdice commented Mar 18, 2024

I wasn't sure if that could somehow impact search experience as we index all the things on spinkube.dev maybe we can instruct it to deploy previews to another origin as well and prevent the search engine from indexing that one

Oh good point. Based on the examples, it looks like the default preview URLs would be something like spinkube.github.io/documentation/pr-preview/pr-1. Yea, definitely don't want to affect the search indexing...

@vdice
Copy link
Contributor Author

vdice commented Mar 18, 2024

Oops, reading the fine print:

First, ensure that your repository is configured to have its GitHub Pages site deployed from a branch, by setting the source for the deployment under Settings > Pages of your repository to Deploy from branch:

That won't work for us. Still, perhaps worth a try to experiment on a fork with this action...

@vdice
Copy link
Contributor Author

vdice commented May 14, 2024

Circling back to this. With features/fixes that have landed in Spin and the Cloud plugin, we can utilize Fermyon Cloud to host PR preview deployments for this documentation site (in conjunction with the preview deploy action). Two things we would need: A Fermyon Cloud account and PAT as well as a GitHub PAT for with correct perms for posting a comment on PRs with the preview URL.

Is this something we'd like to add? The main website deployment could remain via GH Pages as it is today. There still doesn't seem to be a solution for GH Pages-based previews that would fit our needs (ref #94 (comment)).

@bacongobbler
Copy link
Contributor

I can take a closer look at this.

@bacongobbler bacongobbler self-assigned this Jul 8, 2024
@bacongobbler bacongobbler added the enhancement New feature request. label Jul 8, 2024
@bacongobbler bacongobbler linked a pull request Jul 8, 2024 that will close this issue
@bacongobbler bacongobbler added area: CI Improvements or additions to the CI/CD pipeline. and removed enhancement New feature request. labels Jul 8, 2024
@bacongobbler
Copy link
Contributor

Quick summary of my learnings (see #187):

My suggested feedback would be to take a look at alternate hosting providers for deploy previews. As @vdice suggested, perhaps we look inwards at Fermyon Cloud? I'd be more than happy to offer up my own account, or perhaps we already have an account for hosting internal infra on there.

@bacongobbler bacongobbler added the enhancement New feature request. label Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: CI Improvements or additions to the CI/CD pipeline. enhancement New feature request.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants