Skip to content

website/docs: include OS-specific docker-compose install instructions + minor fixes#8975

Merged
fheisler merged 10 commits intomainfrom
website/docs/improve-grammar
Mar 25, 2024
Merged

website/docs: include OS-specific docker-compose install instructions + minor fixes#8975
fheisler merged 10 commits intomainfrom
website/docs/improve-grammar

Conversation

@fheisler
Copy link
Member

Checklist

  • Local tests pass (ak test authentik/)
  • The code has been formatted (make lint-fix)

If changes to the frontend have been made

  • The code has been formatted (make web)

If applicable

  • The documentation has been updated
  • The documentation has been formatted (make website)

@fheisler fheisler requested a review from a team as a code owner March 20, 2024 12:57
@netlify
Copy link

netlify bot commented Mar 20, 2024

Deploy Preview for authentik-storybook canceled.

Name Link
🔨 Latest commit 41d46b7
🔍 Latest deploy log https://app.netlify.com/sites/authentik-storybook/deploys/6601831e2f632500084094a7

@netlify
Copy link

netlify bot commented Mar 20, 2024

Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit 41d46b7
🔍 Latest deploy log https://app.netlify.com/sites/authentik-docs/deploys/6601831e5bdd3c000886347b
😎 Deploy Preview https://deploy-preview-8975--authentik-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@tanberry tanberry left a comment

Choose a reason for hiding this comment

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

Looks awesome, thanks so much!

Copy link
Contributor

@kensternberg-authentik kensternberg-authentik left a comment

Choose a reason for hiding this comment

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

Just being nitpicky.


If running in Kubernetes, the default value is set to 2 and should in most cases not be changed, as scaling can be done with multiple pods running the web server.
Otherwise, authentik will use 1 worker for each 4 CPU cores + 1 as a value below 2 workers is not recommended.
Otherwise, authentik will use 1 worker for each 4 CPU cores + 1, because a value below 2 workers is not recommended.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggestion: "Otherwise, authentik will use 1 worker for each 4 CPU cores + 1. A value below 2 workers is not recommended."

Also suggestion: Say why.

@BeryJu BeryJu changed the title docs: include OS-specific docker-compose install instructions + minor fixes website/docs: include OS-specific docker-compose install instructions + minor fixes Mar 25, 2024
fheisler and others added 9 commits March 25, 2024 09:54
Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com>
Signed-off-by: Fletcher Heisler <fheisler@users.noreply.github.com>
Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com>
Signed-off-by: Fletcher Heisler <fheisler@users.noreply.github.com>
Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com>
Signed-off-by: Fletcher Heisler <fheisler@users.noreply.github.com>
Signed-off-by: Fletcher Heisler <fheisler@users.noreply.github.com>
Signed-off-by: Fletcher Heisler <fheisler@users.noreply.github.com>
Signed-off-by: Fletcher Heisler <fheisler@users.noreply.github.com>
Signed-off-by: Fletcher Heisler <fheisler@users.noreply.github.com>
@fheisler fheisler force-pushed the website/docs/improve-grammar branch from f05e352 to 720840c Compare March 25, 2024 13:55
@fheisler fheisler merged commit d506e5d into main Mar 25, 2024
@fheisler fheisler deleted the website/docs/improve-grammar branch March 25, 2024 15:33
kensternberg-authentik added a commit that referenced this pull request Mar 26, 2024
* main:
  web: bump API Client version (#9035)
  website/docs: maintenance, re-add system settings (#9026)
  core: bump duo-client from 5.2.0 to 5.3.0 (#9029)
  website: bump express from 4.18.2 to 4.19.2 in /website (#9027)
  web: bump express from 4.18.3 to 4.19.2 in /web (#9028)
  web: bump the eslint group in /web with 2 updates (#9030)
  core: bump goauthentik.io/api/v3 from 3.2024022.3 to 3.2024022.5 (#9031)
  website: bump @types/react from 18.2.69 to 18.2.70 in /website (#9032)
  web: bump the eslint group in /tests/wdio with 2 updates (#9033)
  web: bump katex from 0.16.9 to 0.16.10 in /web (#9025)
  translate: Updates for file locale/en/LC_MESSAGES/django.po in fr (#9023)
  website/docs: include OS-specific docker-compose install instructions + minor fixes (#8975)
kensternberg-authentik added a commit that referenced this pull request Mar 26, 2024
* main: (27 commits)
  web: bump API Client version (#9035)
  website/docs: maintenance, re-add system settings (#9026)
  core: bump duo-client from 5.2.0 to 5.3.0 (#9029)
  website: bump express from 4.18.2 to 4.19.2 in /website (#9027)
  web: bump express from 4.18.3 to 4.19.2 in /web (#9028)
  web: bump the eslint group in /web with 2 updates (#9030)
  core: bump goauthentik.io/api/v3 from 3.2024022.3 to 3.2024022.5 (#9031)
  website: bump @types/react from 18.2.69 to 18.2.70 in /website (#9032)
  web: bump the eslint group in /tests/wdio with 2 updates (#9033)
  web: bump katex from 0.16.9 to 0.16.10 in /web (#9025)
  translate: Updates for file locale/en/LC_MESSAGES/django.po in fr (#9023)
  website/docs: include OS-specific docker-compose install instructions + minor fixes (#8975)
  web: bump API Client version (#9021)
  sources/ldap: add ability to disable password write on login (#8377)
  web: bump API Client version (#9020)
  lifecycle: migrate: ensure template schema exists before migrating (#8952)
  website/integrations: Update nextcloud Admin Group Expression (#7314)
  web/flow: general ux improvements (#8558)
  website: bump @types/react from 18.2.67 to 18.2.69 in /website (#9016)
  core: bump requests-oauthlib from 1.4.0 to 2.0.0 (#9018)
  ...
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.

5 participants