Skip to content

Conversation

@emmanuelmathot
Copy link
Contributor

@emmanuelmathot emmanuelmathot commented May 16, 2025

Description

This PR introduces several improvements and fixes:

Breaking Changes

  • Remove pathType and pathSuffix configurations in favor of controller-specific defaults

Added

  • Add upgrade job to handle database permissions for migrations from pre-0.7.0 versions
  • Add separate ingress configuration for STAC browser
  • Support custom cluster naming via postgrescluster.name
  • Add comprehensive upgrade documentation

Changed

  • Improve Nginx and Traefik support with controller-specific rewrites
  • Increase bootstrap job retry limit to 3 attempts
  • Enhance secret handling with custom PostgreSQL cluster names
  • Update documentation with detailed configuration options

Documentation Updates

  • Added upgrade.md for version migration guidance
  • Updated unified-ingress.md with new configuration details
  • Updated configuration.md with comprehensive options
  • Added upgrade notice to README.md

Migration Guide

Users upgrading from pre-0.7.0 versions should:

  1. Read the upgrade guide (docs/upgrade.md)
  2. Specify previousVersion during upgrade
  3. Remove pathType/pathSuffix from ingress config

Testing Done

  • Tested upgrade from 0.6.0
  • Verified ingress configurations with both NGINX and Traefik
  • Tested PostgreSQL cluster with custom names
  • Validated all documentation changes

Co-authored-by: Jonas <j08lue@gmail.com>
@batpad
Copy link
Member

batpad commented May 16, 2025

@emmanuelmathot the Ingress changes look really good to me! Unfortunately, I won't have the time to test these changes on the WRI eoapi-k8s deploy before I leave for vacation, so will only be able to test that mid-next week. But all the changes look great to me, thank you!

Copy link
Contributor

@pantierra pantierra left a comment

Choose a reason for hiding this comment

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

LGTM

@emmanuelmathot emmanuelmathot merged commit b7b0cf9 into main May 16, 2025
2 checks passed
@emmanuelmathot emmanuelmathot deleted the fixes-refac branch May 16, 2025 14:37
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