Skip to content

Conversation

raihankhan
Copy link
Contributor

No description provided.

Copy link

github-actions bot commented Jan 1, 2025

Visit the preview URL for this PR (updated for commit 3cc75a4):

https://kubedb-v2-hugo--pr732-recommendation-qb9yu3du.web.app

(expires Fri, 15 Aug 2025 12:02:03 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 0f29ae8ae0bd54a99bf2b223b6833be47acd5943

@raihankhan raihankhan force-pushed the recommendation branch 5 times, most recently from ef0ab9f to 06dc42a Compare January 30, 2025 11:43
@raihankhan raihankhan force-pushed the recommendation branch 2 times, most recently from 0719db5 to d24d3fd Compare February 27, 2025 13:00
@souravbiswassanto souravbiswassanto marked this pull request as draft March 27, 2025 11:03
Signed-off-by: raihankhan <[email protected]>
Signed-off-by: raihankhan <[email protected]>
Signed-off-by: raihankhan <[email protected]>
Signed-off-by: raihankhan <[email protected]>
Copy link

gitguardian bot commented Jul 1, 2025

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secret in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
17098337 Triggered Generic Password 2fda6b6 docs/examples/postgres/gitops/pg-rotate-auth.yaml View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

```bash
$ kubectl get elasticsearchopsrequest -n es elastic-1740475409-rotate-auth-auto -w
NAME TYPE STATUS AGE
elastic-1740475409-rotate-auth-auto UpdateVersion Progressing 3m12s
Copy link
Member

Choose a reason for hiding this comment

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

Type is UpdateVersion. should it be RotateAuth?

elastic xpack-8.15.0 Ready 10m
```

Since,duration for client certificate is set as `1h20min`, it is expected that the recommendation engine will generate a rotate-auth recommendation at least after 54 minutes (two-third of lifespan) of the client certificate creation. Once generated you will get a similar recommendation as follows.
Copy link
Member

Choose a reason for hiding this comment

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

should it be rotate-auth?

xpack-8.8.2 8.8.2 ElasticStack ghcr.io/appscode-images/elastic:8.8.2 17h
```

Let's deploy an Elasticsearch cluster with version `xpack-8.15.0`. We are going to create a cluster topology with 2 master nodes, 3 data nodes and 2 ingest node. We also have to provide an available storageclass for each of the node types. Make sure to have an issuer/clusterIssuer to refer in the manifest. Though KubeDB managed elasticsearch supports TLS in both cert-manager provisioned and Operator provisioned ways, rotate tls only works when certificates are provisioned via cert-manager.
Copy link
Member

Choose a reason for hiding this comment

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

rotate tls or reconfigure tls?

labels:
app.kubernetes.io/instance: elastic
app.kubernetes.io/managed-by: kubedb.com
app.kubernetes.io/type: rotate-tls
Copy link
Member

Choose a reason for hiding this comment

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

reconfigure tls?

@souravbiswassanto souravbiswassanto removed the request for review from pritamdas99 August 21, 2025 10:27
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.

4 participants