-
-
Notifications
You must be signed in to change notification settings - Fork 502
Configuration options discovered from source (documentation change suggestions) #9110
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
base: main
Are you sure you want to change the base?
Conversation
| ## Configuring a database via config files | ||
|
|
||
| The database dialect is configured in **`/WEB-INF/config-node/srv.xml`**. Uncomment the dialect to use. | ||
| The database dialect is configured in **`/WEB-INF/config-node/srv.xml`**. Uncomment the dialect to use or use Java System property to configure it with `-Dgeonetwork.db.type=postgres` as described in https://github.com/geonetwork/core-geonetwork/blob/4.2.13/web/src/main/webResources/WEB-INF/config-node/srv.xml#L36. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| The database dialect is configured in **`/WEB-INF/config-node/srv.xml`**. Uncomment the dialect to use or use Java System property to configure it with `-Dgeonetwork.db.type=postgres` as described in https://github.com/geonetwork/core-geonetwork/blob/4.2.13/web/src/main/webResources/WEB-INF/config-node/srv.xml#L36. | |
| The database dialect is configured in **`/WEB-INF/config-node/srv.xml`**. Uncomment the dialect to use or use Java System property to configure it with `-Dgeonetwork.db.type=postgres` as described in https://github.com/geonetwork/core-geonetwork/blob/e8cee088ca714789788d9de0cb8f143d09a8e9ba/web/src/main/webResources/WEB-INF/config-node/srv.xml#L36. |
Use a link to main branch.
| Alternative environment variables are also supported for non-container (WAR-file) environments: | ||
|
|
||
| ``` text | ||
| GEONETWORK_DB_USERNAME=example |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would document also GEONETWORK_DB_TYPE
| GEONETWORK_DB_PORT=5432 | ||
| ``` | ||
|
|
||
| Based on compile-time default settings: https://github.com/geonetwork/core-geonetwork/blob/4.2.13/pom.xml#L1515-L1520 that are used to set these https://github.com/geonetwork/core-geonetwork/blob/4.2.13/web/src/main/webResources/WEB-INF/config-db/jdbc.properties#L24-L28. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Based on compile-time default settings: https://github.com/geonetwork/core-geonetwork/blob/4.2.13/pom.xml#L1515-L1520 that are used to set these https://github.com/geonetwork/core-geonetwork/blob/4.2.13/web/src/main/webResources/WEB-INF/config-db/jdbc.properties#L24-L28. | |
| Based on compile-time default settings: https://github.com/geonetwork/core-geonetwork/blob/e8cee088ca714789788d9de0cb8f143d09a8e9ba/pom.xml#L1556-L1562 that are used to set these https://github.com/geonetwork/core-geonetwork/blob/e8cee088ca714789788d9de0cb8f143d09a8e9ba/web/src/main/webResources/WEB-INF/config-db/jdbc.properties#L24-L28 |
Use links to main branch.
Apparently documentation has moved to the main repository so I'm suggesting the changes made in geonetwork/doc#248 to be migrated to this documentation. We've been tinkering with GeoNetwork migration from 3.x to 4.x for some time now and looking at the code I noticed these configuration options as well. Also noticed that the indexes are by default prefixed with
gn-(https://github.com/geonetwork/core-geonetwork/blob/4.2.13/pom.xml#L1565-L1569).I'm not sure if you want to have links to source code in the documentation or if the defaults are different in the prepackaged versions of WAR-file or the Containerized setup. These are the ones that are defaulted when building from source. As we are migrating to 4.2.x before continuing to go for 4.4.x things might have changed, but at least raising awareness for someone more experienced with GN to take a look at the docs.
Maybe also take a look at this: geonetwork/doc#249 as it seems the path to
encryptor.propertieson the docs was wrong (in our experience).Checklist
mainbranch, backports managed with labelREADME.mdfilespom.xmldependency management. Update build documentation with intended library use and library tutorials or documentation