Skip to content

add mkdocs - #63

Draft
kafu-densha wants to merge 1 commit into
mainfrom
mkdocs
Draft

kafu-densha wants to merge 1 commit into
mainfrom
mkdocs

Conversation

@kafu-densha

@kafu-densha kafu-densha commented Oct 2, 2026 •

Copy link
Copy Markdown

depends on ocf/mkdocs#94 (merged)

@kafu-densha
kafu-densha marked this pull request as ready for review October 3, 2026 03:52
@kafu-densha

Copy link
Copy Markdown
Author

tested and working at http://staging-c.ocf.berkeley.edu/

@kafu-densha
kafu-densha enabled auto-merge (squash) October 3, 2026 04:03
namespace: mkdocs
spec:
rules:
- host: bestdocs.ocf.berkeley.edu

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this should be docs.o.b.e, we should be 301ing bestdocs to that

alternatively it could be both

ocf.berkeley.edu/flux: mkdocs
spec:
containers:
- image: ghcr.io/ocf/ocf-docs:latest # {"$imagepolicy": "flux-system:mkdocs"}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

using the image update policy is probably not ideal if there will be a new build of the container every time the docs are updated (as opposed to just pinning latest)

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

latest will always point to the latest build, not exactly sure what the imagepolicy does

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

should I remove ':latest' and keep the policy?

@kafu-densha
kafu-densha disabled auto-merge October 4, 2026 02:36
@kafu-densha
kafu-densha marked this pull request as draft October 4, 2026 02:36
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