Skip to content

root: use correct default schema for install_id#13006

Merged
BeryJu merged 1 commit intomainfrom
root/use-correct-schema-for-install_id
Feb 13, 2025
Merged

root: use correct default schema for install_id#13006
BeryJu merged 1 commit intomainfrom
root/use-correct-schema-for-install_id

Conversation

@BeryJu
Copy link
Member

@BeryJu BeryJu commented Feb 12, 2025

Details

follow up to #12949

closes #12154


Checklist

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

If an API change has been made

  • The API schema has been updated (make gen-build)

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)

Signed-off-by: Jens Langhammer <jens@goauthentik.io>
@BeryJu BeryJu requested a review from a team as a code owner February 12, 2025 23:48
@netlify
Copy link

netlify bot commented Feb 12, 2025

Deploy Preview for authentik-storybook canceled.

Name Link
🔨 Latest commit 479517f
🔍 Latest deploy log https://app.netlify.com/sites/authentik-storybook/deploys/67ad3341df196f0008bcbefa

@netlify
Copy link

netlify bot commented Feb 12, 2025

Deploy Preview for authentik-docs canceled.

Name Link
🔨 Latest commit 479517f
🔍 Latest deploy log https://app.netlify.com/sites/authentik-docs/deploys/67ad334129829a0008c40f93

@codecov
Copy link

codecov bot commented Feb 12, 2025

Codecov Report

Attention: Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.

Project coverage is 92.69%. Comparing base (98f3b9a) to head (479517f).
Report is 2 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
authentik/root/install_id.py 33.33% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #13006      +/-   ##
==========================================
- Coverage   92.75%   92.69%   -0.07%     
==========================================
  Files         785      785              
  Lines       39623    39623              
==========================================
- Hits        36754    36729      -25     
- Misses       2869     2894      +25     
Flag Coverage Δ
e2e 48.39% <33.33%> (-0.10%) ⬇️
integration 24.54% <33.33%> (ø)
unit 90.43% <33.33%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-actions
Copy link
Contributor

authentik PR Installation instructions

Instructions for docker-compose

Add the following block to your .env file:

AUTHENTIK_IMAGE=ghcr.io/goauthentik/dev-server
AUTHENTIK_TAG=gh-479517fa8d37d20c1d9a0d1179cb8e7704ca9310
AUTHENTIK_OUTPOSTS__CONTAINER_IMAGE_BASE=ghcr.io/goauthentik/dev-%(type)s:gh-%(build_hash)s

Afterwards, run the upgrade commands from the latest release notes.

Instructions for Kubernetes

Add the following block to your values.yml file:

authentik:
    outposts:
        container_image_base: ghcr.io/goauthentik/dev-%(type)s:gh-%(build_hash)s
global:
    image:
        repository: ghcr.io/goauthentik/dev-server
        tag: gh-479517fa8d37d20c1d9a0d1179cb8e7704ca9310

Afterwards, run the upgrade commands from the latest release notes.

@BeryJu BeryJu merged commit 82d4e8a into main Feb 13, 2025
72 of 73 checks passed
@BeryJu BeryJu deleted the root/use-correct-schema-for-install_id branch February 13, 2025 00:25
@sentry
Copy link

sentry bot commented Feb 13, 2025

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ TypeError: query parameters should be a sequence or a mapping, got str authentik.blueprints.v1.tasks.apply_blueprint View Issue
  • ‼️ TypeError: query parameters should be a sequence or a mapping, got str authentik.flows.views.executor.FlowExecutorView View Issue
  • ‼️ TypeError: query parameters should be a sequence or a mapping, got str authentik.core.api.users.UserViewSet View Issue
  • ‼️ TypeError: query parameters should be a sequence or a mapping, got str authentik.enterprise.tasks.enterprise_update_usage View Issue

Did you find this useful? React with a 👍 or 👎

BeryJu added a commit that referenced this pull request Feb 13, 2025
Signed-off-by: Jens Langhammer <jens@goauthentik.io>

#13006
BeryJu added a commit that referenced this pull request Feb 13, 2025
* root: correctly use correct schema for install_id

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

#13006

* format

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

---------

Signed-off-by: Jens Langhammer <jens@goauthentik.io>
rissson added a commit that referenced this pull request Feb 13, 2025
@rissson rissson mentioned this pull request Feb 13, 2025
6 tasks
kensternberg-authentik added a commit that referenced this pull request Feb 20, 2025
* main: (111 commits)
  root: correctly use correct schema for install_id (#13018)
  website: bump docusaurus-plugin-openapi-docs from 4.3.3 to 4.3.4 in /website (#13011)
  web: bump API Client version (#13017)
  core: bump aws-cdk-lib from 2.178.1 to 2.178.2 (#13013)
  core: bump oss/go/microsoft/golang from 1.23-fips-bookworm to 1.24-fips-bookworm (#13012)
  website: bump docusaurus-theme-openapi-docs from 4.3.3 to 4.3.4 in /website (#13010)
  lifecycle/aws: bump aws-cdk from 2.178.1 to 2.178.2 in /lifecycle/aws (#13009)
  core: bump github.com/sethvargo/go-envconfig from 1.1.0 to 1.1.1 (#13008)
  web/admin: fix source selection for identification stage (#13007)
  core: bump sentry-sdk from 2.20.0 to 2.21.0 (#13014)
  website/integrations: Open WebUI (#12939)
  root: use correct default schema for install_id (#13006)
  website/docs: fix a minor typo (#13004)
  enterprise/providers/ssf: fixes v2 (#13003)
  root: make default postgres schema configurable (#12949)
  providers/oauth2: cleanup tokens when user is deactivated (#12859)
  website/docs: fix Nginx redirection example (#12920)
  core: bump twilio from 9.4.4 to 9.4.5 (#12993)
  core: bump coverage from 7.6.11 to 7.6.12 (#12994)
  core: bump cryptography from 44.0.0 to 44.0.1 (#12992)
  ...
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.

Authentik requires public schema

1 participant