Skip to content

credentialsSecret is a required field, but also has the empty string as a default value #349

Closed

Description

@fhennig

While working on something else I noticed that the credentialsSecret field in the clusterConfig is not optional (i.e. it is required), but we also derive Default for the whole struct, so the field gets a default value, which is the empty string for the string type.

This seems like a bug to me.

Not urgent.

Activity

  1. moved this to Selected for Development in Stackable End-to-End Coordinationon Dec 10, 2025
  2. moved this from Selected for Development to In Refinement in Stackable End-to-End Coordinationon Jan 14, 2026
  3. moved this from In Refinement to In Progress in Stackable End-to-End Coordinationon Feb 3, 2026
  4. moved this to Development: In Progress in Stackable Engineeringon Feb 17, 2026
  5. sbernauer commented on Feb 19, 2026

    @sbernauer
    Member

    Looks like this was already fixed, the field is now mandatory:

    credentialsSecret:
    description: |-
    The name of the Secret object containing the admin user credentials and database connection details.
    Read the
    [getting started guide first steps](https://docs.stackable.tech/home/nightly/airflow/getting_started/first_steps)
    to find out more.
    type: string

  6. adwk67 commented on Feb 19, 2026

    @adwk67
    Member

    Closing: see comment #747 (comment)

  7. moved this from In Progress to Done in Stackable End-to-End Coordinationon Feb 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions