Derive the webapp host container id from stable data #865

Merged
dboreham merged 3 commits from dboreham/fix-webapp-container-id into main 2024-07-04 22:38:08 +00:00
Owner

Note that this change only affects the creation of new deployment directories. Any existing deployments are going to be unchanged, and hence the container id will be incorrect (we'll build a container with the new ID while the deployment data says to run the old one). This isn't as bad as it sounds because we have already broken stability of the ids (somehow). So we need to fix up any in-use deployments anyway.

Note that this change only affects the creation of new deployment directories. Any existing deployments are going to be unchanged, and hence the container id will be incorrect (we'll build a container with the new ID while the deployment data says to run the old one). This isn't as bad as it sounds because we have already broken stability of the ids (somehow). So we need to fix up any in-use deployments anyway.
dboreham added 1 commit 2024-07-04 21:40:47 +00:00
Derive the webapp host container id from stable data
All checks were successful
Lint Checks / Run linter (pull_request) Successful in 37s
Webapp Test / Run webapp test suite (pull_request) Successful in 4m44s
Smoke Test / Run basic test suite (pull_request) Successful in 4m43s
Deploy Test / Run deploy test suite (pull_request) Successful in 5m31s
K8s Deploy Test / Run deploy test suite on kind/k8s (pull_request) Successful in 8m34s
8430c83e89
dboreham added 1 commit 2024-07-04 21:47:02 +00:00
Remove old comment
All checks were successful
Lint Checks / Run linter (pull_request) Successful in 32s
Webapp Test / Run webapp test suite (pull_request) Successful in 4m39s
Deploy Test / Run deploy test suite (pull_request) Successful in 5m16s
Smoke Test / Run basic test suite (pull_request) Successful in 4m31s
K8s Deploy Test / Run deploy test suite on kind/k8s (pull_request) Successful in 8m6s
0938137df0
dboreham added 1 commit 2024-07-04 21:57:33 +00:00
Observe --force-rebuild
All checks were successful
Lint Checks / Run linter (pull_request) Successful in 37s
Webapp Test / Run webapp test suite (pull_request) Successful in 4m24s
Smoke Test / Run basic test suite (pull_request) Successful in 4m21s
Deploy Test / Run deploy test suite (pull_request) Successful in 5m13s
K8s Deploy Test / Run deploy test suite on kind/k8s (pull_request) Successful in 7m39s
3750f53b91
dboreham merged commit c3a1402042 into main 2024-07-04 22:38:08 +00:00
dboreham deleted branch dboreham/fix-webapp-container-id 2024-07-04 22:38:13 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cerc-io/stack-orchestrator#865
No description provided.