Skip to content
This repository has been archived by the owner on Aug 9, 2023. It is now read-only.

Add k8s container #39

Merged
merged 4 commits into from
Jan 20, 2022
Merged

Add k8s container #39

merged 4 commits into from
Jan 20, 2022

Conversation

maeve-fpf
Copy link
Contributor

This is another PR like freedomofpress/securedrop-docs#295 (please read that for an explanation of these changes).

It would be smaller than freedomofpress/securedrop-workstation-docs#96, except it moves everything applicable under a docs directory for consistency with the other two repos. I ran into a strange circular symlink problem without making that change, and it seemed like a good idea anyway. However, this may need more review than the other two for that reason. I've run make docs and make docs-lint and they seem correct.

This is deployed at: https://staging-support-docs.securedrop.org/

Like both other repos, CI is currently set up to trigger on a test branch of this repo, so nothing will happen immedately when this is merged into main.

@maeve-fpf maeve-fpf requested a review from eloquence December 22, 2021 16:31
@maeve-fpf maeve-fpf force-pushed the k8s-container branch 2 times, most recently from 923a724 to 98277e7 Compare January 5, 2022 16:40
@maeve-fpf maeve-fpf force-pushed the k8s-container branch 6 times, most recently from 2ea3b4c to 2c9e023 Compare January 11, 2022 18:33
Copy link
Member

@eloquence eloquence left a comment

Choose a reason for hiding this comment

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

Somehow Sphinx gets stuck in an infinite loop for me when building the container:

Your branch is up to date with 'origin/main'.
Running Sphinx v4.2.0
/roc/1/cwd/general_support.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/index.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/encrypted_email_setup.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/using_redmine.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/creating_your_account.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/troubleshooting.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/encrypted_email_troubleshooting.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/encrypted_email_overview.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/encrypted_email_do_not_want.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/proc/1/cwd/general_support.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/proc/1/cwd/index.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/proc/1/cwd/encrypted_email_setup.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/proc/1/cwd/using_redmine.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/proc/1/cwd/creating_your_account.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/proc/1/cwd/troubleshooting.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/proc/1/cwd/encrypted_email_troubleshooting.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/proc/1/cwd/encrypted_email_overview.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/proc/1/cwd/encrypted_email_do_not_want.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/proc/1/cwd/proc/1/cwd/general_support.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/proc/1/cwd/proc/1/cwd/index.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/proc/1/cwd/proc/1/cwd/encrypted_email_setup.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/proc/1/cwd/proc/1/cwd/using_redmine.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/proc/1/cwd/proc/1/cwd/creating_your_account.rst: WARNING: document not readable. Ignored.
/roc/1/cwd/proc/1/cwd/proc/1/cwd/troubleshooting.rst: WARNING: document not readable. Ignored.

and so on, with ever deeper nesting

Copy link
Member

@eloquence eloquence left a comment

Choose a reason for hiding this comment

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

Works as expected when testing with the temp commit on staging-ci-test.

@eloquence eloquence merged commit 24a3ef4 into main Jan 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants