Skip to content

[bitnami/neo4j] Add chart - #27616

Merged
migruiz4 merged 25 commits into
mainfrom
neo4j-chart
Jul 4, 2024
Merged

[bitnami/neo4j] Add chart#27616
migruiz4 merged 25 commits into
mainfrom
neo4j-chart

Conversation

@migruiz4

@migruiz4 migruiz4 commented Jul 1, 2024

Copy link
Copy Markdown
Contributor

Description of the change

Adds the new chart 'bitnami/neo4j' to the catalog.

This PR is currently in draft, the following items are still pending:

  • Release and use bitnami/neo4j image (currently using a custom image)

Checklist

  • Chart version bumped in Chart.yaml according to semver. This is not necessary when the changes only affect README.md files.
  • Variables are documented in the values.yaml and added to the README.md using readme-generator-for-helm
  • Title of the pull request follows this pattern [bitnami/<name_of_the_chart>] Descriptive title
  • All commits signed off and in agreement of Developer Certificate of Origin (DCO)

Miguel Ruiz added 2 commits June 27, 2024 12:32
Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
@github-actions github-actions Bot added the neo4j label Jul 1, 2024
@bitnami-bot bitnami-bot added the verify Execute verification workflow for these changes label Jul 1, 2024
@github-actions
github-actions Bot requested a review from alemorcuq July 1, 2024 14:59
@migruiz4
migruiz4 requested review from fmulero, javsalgar and jotamartos and removed request for alemorcuq July 1, 2024 15:01
@migruiz4 migruiz4 assigned javsalgar, jotamartos and fmulero and unassigned alemorcuq Jul 1, 2024
Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
@carrodher carrodher added the new-product Request new product to be added into the catalog label Jul 1, 2024
@carrodher
carrodher removed the request for review from javsalgar July 1, 2024 15:04
Miguel Ruiz and others added 7 commits July 2, 2024 14:09
Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
@migruiz4
migruiz4 marked this pull request as ready for review July 2, 2024 15:09
Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
Miguel Ruiz and others added 8 commits July 2, 2024 17:13
Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
jotamartos
jotamartos previously approved these changes Jul 3, 2024
@jotamartos

jotamartos commented Jul 3, 2024

Copy link
Copy Markdown
Contributor

LGTM! I know you are still making changes and can review them when needed but the current code looks fine 👍

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

@fmulero fmulero left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Brilliant!! thanks a lot @migruiz4

Everything looks great, I have only some improvements.

NOTE: Recent charts (for instance: seaweed) can use cert manager for automated certificates. I think is worth to give it a glance.

Comment thread bitnami/neo4j/templates/statefulset.yaml
Comment thread bitnami/neo4j/templates/tls-secret.yaml Outdated
Miguel Ruiz and others added 4 commits July 3, 2024 14:49
Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

@fmulero fmulero left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@carrodher
carrodher requested a review from rafariossaa July 4, 2024 11:08
@migruiz4
migruiz4 merged commit e74b9ca into main Jul 4, 2024
@migruiz4
migruiz4 deleted the neo4j-chart branch July 4, 2024 11:49
@github-actions github-actions Bot added the solved label Jul 4, 2024

@rafariossaa rafariossaa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

jclab-joseph pushed a commit to bitmoa/charts that referenced this pull request Jul 24, 2026
* [bitnami/neo4j] Add chart

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

* Add support for readOnlyRootFilesystem, use development image

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

* Remove serviceMonitor

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

* Add VIB tests and image improvements

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

* Add neo4j host

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

* Update chart

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

* Fix values metadata

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

* Update CHANGELOG.md

Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>

* Update README.md with readme-generator-for-helm

Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>

* Fix test inputs

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

* Add TLS support

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

* Change readiness

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

* Update README metadata

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

* Update README.md with readme-generator-for-helm

Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>

* Update test service ports

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

* Add goss https test

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

* Fix goss vars file

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

* Update dev image

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

* Update CHANGELOG.md

Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>

* Increase timeout

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

* Change bolt port

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

* Use bitnami/neo4j:5.20.0-debian-12-r2

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

* Modify TLS secrets

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

* Update CHANGELOG.md

Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>

* Add ingress-tls-secret.yaml

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>

---------

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
Co-authored-by: Bitnami Containers <bitnami-bot@vmware.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bitnami neo4j new-product Request new product to be added into the catalog solved verify Execute verification workflow for these changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants