Skip to content

cluster: add projected volume support #510

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 21 commits into
base: main
Choose a base branch
from

Conversation

plesoun-stein
Copy link

I added projected volume support in In cluster specification
tested.
my simple testing env:
namespace: aaa
clusterName: firstcnpg-cluster
values:

  projectedVolumeTemplate:
    sources:
      - secret:
          name: crl
          items:
            - key: crl.pem
              path: crl/crl-01.pem

secret successfully mounted in /projected/crl/crl-01.pem

@itay-grudev
Copy link
Collaborator

itay-grudev commented Feb 3, 2025

Could you please include an example in the non-default configuration test?

@plesoun-stein
Copy link
Author

Could you please include an example in the non-default configuration test?

to the file 01-non_default_configuration_cluster.yaml only or 01-non_default_configuration_cluster-assert.yaml too?

@plesoun-stein
Copy link
Author

plesoun-stein commented Feb 11, 2025

Could you please include an example in the non-default configuration test?

I have added projected volume definiton into values file 01-non_default_configuration_cluster.yaml

  projectedVolumeTemplate:
    sources:
      - secret:
          name: ca-secret 
          items:
          - key: ca.crt
            path: crt/ca-01.pem

I have reused a ca-secret from above and I expect, there is a ca.crt key in the secret.

@plesoun-stein
Copy link
Author

Ihave fixed trailing spaces error

plesoun-stein and others added 13 commits February 12, 2025 10:53
Signed-off-by: Tomas Steiner <[email protected]>
* Increased Scheduled Backups test cleanup timeout

Currently the scheduled backups test times out causing a test failure, simply because it take a bit longer to remove the backup objects.

Signed-off-by: Itay Grudev <[email protected]>

* Increased timeout to 5min

Signed-off-by: Itay Grudev <[email protected]>

---------

Signed-off-by: Itay Grudev <[email protected]>
Signed-off-by: Tomas Steiner <[email protected]>
* Running tests in parallel with a matrix strategy

---------

Signed-off-by: Itay Grudev <[email protected]>
Signed-off-by: Itay Grudev <[email protected]>
Signed-off-by: Tomas Steiner <[email protected]>
…-pg#450)

Signed-off-by: Tomas Tormo <[email protected]>
Co-authored-by: Itay Grudev <[email protected]>
Signed-off-by: Tomas Steiner <[email protected]>
…native-pg#513)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: Tomas Steiner <[email protected]>
…#514)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: Tomas Steiner <[email protected]>
…g#468)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: Tomas Steiner <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: Tomas Steiner <[email protected]>
@itay-grudev
Copy link
Collaborator

The test here is missing an assertion.

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Apr 16, 2025
@mercuryyy
Copy link

Can this be merged ?

@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:S This PR changes 10-29 lines, ignoring generated files. labels May 7, 2025
Signed-off-by: Tomas Steiner <[email protected]>
@plesoun-stein
Copy link
Author

The test here is missing an assertion.

I have added assertion in charts/cluster/test/postgresql-cluster-configuration/01-non_default_configuration_cluster-assert.yaml
I hope it's on the right place.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no tests size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants