Skip to content

Comments

fix(cmdeploy): Rewrite dovecot install logic, update#862

Draft
j4n wants to merge 1 commit intomainfrom
dovecot-install
Draft

fix(cmdeploy): Rewrite dovecot install logic, update#862
j4n wants to merge 1 commit intomainfrom
dovecot-install

Conversation

@j4n
Copy link
Contributor

@j4n j4n commented Feb 19, 2026

The old code did not install updates when the service was running; check
installed version instead of systemd status. Also, rewrite install logic
to extract dovecot version and hashes as module-level constants.

@missytake
Copy link
Contributor

There was some reason why we didn't want to download and install this on every run, @link2xt do you remember?

@j4n
Copy link
Contributor Author

j4n commented Feb 19, 2026

There was some reason why we didn't want to download and install this on every run, @link2xt do you remember?

While I still do have a hunch I may be barking up the wrong tree here, regarding this specifically, it wouldn't, as long as the chatmail repo and the hashes/versions are not touched it won't do it.

The old code did not install updates when the service was running; check
installed version instead of systemd status. Also, rewrite install logic
to extract dovecot version and hashes as module-level constants.
@j4n j4n temporarily deployed to staging-ipv4.testrun.org February 19, 2026 19:04 — with GitHub Actions Inactive
@j4n j4n temporarily deployed to staging2.testrun.org February 19, 2026 19:04 — with GitHub Actions Inactive
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.

2 participants