Skip to content
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

release 1.0.0 final changes [geonode 4.1.2] #74

Conversation

mwallschlaeger
Copy link
Contributor

@mwallschlaeger mwallschlaeger commented Sep 8, 2023

Description

This PR fixes minor issues with the chart and prepares for the new chart release 1.0.0.

Type of Change

Please select the relevant option:

  • Bug fix
  • New feature
  • Documentation update
  • Refactoring
  • Other (please describe)

Related Issue

If there is an existing issue related to this pull request, please reference it here.

closes #72 #73 #66 #52

Checklist

Please ensure that your pull request meets the following requirements:

  • The pull request is limited to one type (docs, feature, bug fix, etc.)
  • The pull request is as small as possible. Consider opening multiple pull requests instead of one large one.
  • The feature or bug fix has been discussed and documented in an issue beforehand.

Additional Notes

Any additional information or context regarding the pull request can be provided here.

Thank you for creating this pull request

@mwallschlaeger mwallschlaeger added bug Something isn't working enhancement New feature or request labels Sep 8, 2023
@mwallschlaeger mwallschlaeger self-assigned this Sep 8, 2023
Copy link
Contributor

@ridoo ridoo left a comment

Choose a reason for hiding this comment

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

What about the secret separation? Do you plan to include this in another release?

charts/geonode/README.md Outdated Show resolved Hide resolved
charts/geonode/README.md Outdated Show resolved Hide resolved
@mwallschlaeger
Copy link
Contributor Author

What about the secret separation? Do you plan to include this in another release?

no i want to wait until the secret separation is merged and add it to the release.

Copy link
Contributor

@ridoo ridoo left a comment

Choose a reason for hiding this comment

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

Looks good

@mwallschlaeger mwallschlaeger merged commit e989c20 into main Sep 11, 2023
2 checks passed
@mwallschlaeger mwallschlaeger deleted the issue#72_Bug_Celery_container_is_not_starting_properly_and_exiting_right_away branch September 11, 2023 10:20
mwallschlaeger added a commit that referenced this pull request Sep 11, 2023
* Issue#59 task prepare this repository to be available on artifact hub (#65)

* Fixes [#59,#58]

* Issue#27 py csw single microservice (#68)

issue#27_csw_single_microsservice

* Issue#67 separate site url related settings from ingress config (#69)

issue#67_separate_site_url_related_settings_from_ingress_config

* Provide confidential values as secret (#71)

* WIP: Provide confidential values as secret

* Fix parameter names in comment

* Provide demo credentials as base64

* Add named secret in values.yaml with default content

* Remove secret boilerplate in values.yaml

* Configure secret creation via flag

* Enhance secret configuration

- Use existingSecretName to configure secret via external secret
- rabbitmq uses its own naming syntax. We show them for convenience
- fixing external database property

Postgres secrets for external databases has to be added still

* Fix yaml syntax

* Fix yaml and property refs

* Adds configurable secret to external database

* Adjust minikube values

* Resolve go template issues

* Update documentation

* Fix wording

* release 1.0.0 final changes [geonode 4.1.2] (#74)

release 1.0.0

* Issue#72 bug celery container is not starting properly and exiting right away (#76)

* release 1.0.0

* fixing release build (#78)

* added github token to install helm

---------

Co-authored-by: Henning Bredel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Celery container is not starting properly and exiting right away
2 participants