Skip to content

Add Data Encryption at Rest#155

Merged
vanxa merged 1 commit into
mainfrom
GDB-14591-encryption-at-rest
Jun 24, 2026
Merged

Add Data Encryption at Rest#155
vanxa merged 1 commit into
mainfrom
GDB-14591-encryption-at-rest

Conversation

@vanxa

@vanxa vanxa commented May 22, 2026

Copy link
Copy Markdown
Contributor

Add Data Encryption at Rest

Description

Add support for data encryption at rest configuration

  • Adds variables and user_data assignments for the new properties required to configure Encryption at rest
  • Adds support for file-based and pkcs12-based encryption at rest configuration
  • Adds keystore as ssm parameter, if pkcs12 encryption at rest is selected
  • Adds masterkey as ssm parameter, if file encryption at rest mode is selected
  • Updates 04_gdb_conf_overrides.sh.tpl to configure encryption at rest based on mode selected
  • Updated Changelog

Related Issues

Screenshots (if applicable)

Checklist

  • I have tested these changes thoroughly.
  • My code follows the project's coding style.
  • I have added appropriate comments to my code, especially in complex areas.
  • All new and existing tests passed locally.

@vanxa
vanxa requested review from mihailradkov, simonzhekoff and viktor-ribchev and removed request for simonzhekoff and viktor-ribchev May 22, 2026 13:47
Comment thread modules/graphdb/templates/04_gdb_conf_overrides.sh.tpl Outdated
Comment thread modules/graphdb/templates/04_gdb_conf_overrides.sh.tpl Outdated
Comment thread README.md Outdated
Comment thread modules/graphdb/variables.tf Outdated
Comment thread modules/graphdb/variables.tf Outdated
Comment thread CHANGELOG.md
@vanxa
vanxa force-pushed the GDB-14591-encryption-at-rest branch 3 times, most recently from 334eeab to 5cb739f Compare June 10, 2026 13:40
@mihailradkov

Copy link
Copy Markdown
Contributor

GraphDB 13.4.0? That's at least 2-3 years too yearly 😀

@vanxa vanxa changed the title Update GDB to 13.4.0. Add Data Encryption at Rest Update GDB to 13.1.0. Add Data Encryption at Rest Jun 17, 2026
@vanxa
vanxa force-pushed the GDB-14591-encryption-at-rest branch from 325cd27 to 2476445 Compare June 19, 2026 11:31
@mihailradkov

Copy link
Copy Markdown
Contributor

Update changelog.md

vanxa pushed a commit to Ontotext-AD/graphdb-helm that referenced this pull request Jun 23, 2026
Adds configuration parameters under `encryption` to configure encryption at rest
(see here: Ontotext-AD/terraform-aws-graphdb#155)
Add examples
vanxa pushed a commit to Ontotext-AD/graphdb-helm that referenced this pull request Jun 23, 2026
- Adds configuration parameters under `encryption` to configure encryption at rest
(see here: Ontotext-AD/terraform-aws-graphdb#155)
- Currently, file-based and pkcs12-based encryption at rest
  configurations are supported
- Add examples
Comment thread variables.tf Outdated
@vanxa vanxa changed the title Update GDB to 13.1.0. Add Data Encryption at Rest Add Data Encryption at Rest Jun 23, 2026
Comment thread variables.tf Outdated
Comment thread variables.tf Outdated
@vanxa
vanxa force-pushed the GDB-14591-encryption-at-rest branch 2 times, most recently from e3811a2 to 7adee21 Compare June 23, 2026 10:12
Comment thread CHANGELOG.md Outdated
@vanxa
vanxa force-pushed the GDB-14591-encryption-at-rest branch from 7adee21 to 072343d Compare June 23, 2026 10:56
- Adds variables and user_data assignments for the new properties
  required to configure Encryption at rest
- Adds support for file-based and pkcs12-based encryption at rest
  configuration
- Adds keystore as ssm parameter, if `pkcs12` encryption at rest is selected
- Adds masterkey as ssm parameter, if `file` encryption at rest mode is
selected
- Updates `04_gdb_conf_overrides.sh.tpl` to configure encryption at rest
  based on mode selected
- Updated Changelog
@vanxa
vanxa force-pushed the GDB-14591-encryption-at-rest branch from 072343d to 198380a Compare June 24, 2026 07:08
@vanxa
vanxa requested a review from tonyKunchev June 24, 2026 07:08

@tonyKunchev tonyKunchev 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.

It looks good, but I would wait for another approval before merging. I am not too familiar with the code in the repository, so someone else needs to verify the changes as well.

@vanxa
vanxa merged commit 678c0b6 into main Jun 24, 2026
3 checks passed
@vanxa
vanxa deleted the GDB-14591-encryption-at-rest branch June 24, 2026 09:05
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