Skip to content

Commit

Permalink
Bump Redis Persistence To Also Update Command (#178)
Browse files Browse the repository at this point in the history
* Add ConfigMap for Persistent Redis Configuration

* Add initial changes needed

* Update snapshots

* i am but a servant to the great ol' Code Coverage

* Cleanup

* Snapshott

* Failing chart test :D

* Woooo

* the anti-power of eunsoo shin

* bet

* lol

* Better testing

* Abstract away configmap interface

* Update snapshot and naming conventions

* Update host path

* Update host path

* Oops :'D

* Bump version again
  • Loading branch information
joyliu-q authored Oct 27, 2023
1 parent d014744 commit bccca58
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
2 changes: 1 addition & 1 deletion cdk/kittyhawk/.projenrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ const project = new TypeScriptProject({
},
});

project.addFields({ ["version"]: "1.1.9" });
project.addFields({ ["version"]: "1.1.10" });
project.prettier?.ignoreFile?.addPatterns("src/imports");
project.testTask.steps.forEach((step) => {
if (step.exec) {
Expand Down
2 changes: 1 addition & 1 deletion cdk/kittyhawk/package.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 3 additions & 0 deletions cdk/kittyhawk/src/application/redis.ts
Original file line number Diff line number Diff line change
Expand Up @@ -102,6 +102,9 @@ export class RedisApplication extends Application {
deployment: {
image: redisProps.deployment?.image ?? "redis",
tag: redisProps.deployment?.tag ?? "6.0",
...(redisProps.persistData
? { cmd: ["redis-server", "/redis-master/redis.conf"] }
: {}),
volumeMounts: [
...(redisProps.persistData
? [
Expand Down

0 comments on commit bccca58

Please sign in to comment.