search

conjurdemos / conjur-intro

Miscellaneous utilities that make it easier to make, manage, and run demos
Apache License 2.0
10 stars 11 forks source link

Add step to upgrade test to include restart #145

Closed codihuston closed 6 months ago

codihuston commented 7 months ago

Add step to upgrade test to include restart. This is intended to catch an issue discovered in CNJR-4747.

codihuston commented 6 months ago

@codihuston this looks good to me..

qq: is the possum.conf update issue only for backups earlier than 13.3 - for example - would a 13.3 backup have had an issue with possum.conf before this fix/change? If not, we should test with 13.2 or earlier when we run this..

Hey @adamouamani, in my testing of this issue locally when we discovered it, the possum.conf file appears to be included in the backup process. So whatever is in that file during the backup should also appear in the back up. That is, a 13.3 image would be expected to have this configuration option (as it ships with it), thus a restore from such a backup should preserve that setting. However, in the case that it does not exist in a 13.3 backup, the cinc recipe change in PR#254 and PR#257 the Appliance should create that setting, as it now does from older backups.