ReviewOS

also looking at this

stacks/ts-cloud

fix(tls): wait for DNS before ACME issuance

#166
Merged chrisbbreuer wants to merge agent/wait-for-rpx-dns into main
4 files +42 -4
packages/ts-cloud/test/drivers/compute-deploy.test.tsmodified+4-1
Changes to packages/ts-cloud/test/drivers/compute-deploy.test.ts
@@ -547,7 +547,10 @@ describe('renewRpxCertificates', () => {
547547 expect(ok).toBe(true)
548548 const call = (driver.runRemoteDeploy as ReturnType<typeof mock>).mock.calls[0][0]
549549 expect(call.targets[0].id).toBe('i-abc123')
550 expect(call.commands).toEqual(['systemctl start rpx-cert-renew-my-app.service'])
550 expect(call.commands).toEqual([
551 'systemctl start rpx-cert-renew-my-app.service',
552 "test -s '/etc/rpx/certs/my-app.example.com.crt'",
553 ])
551554 expect(call.tags.Role).toBe('app')
552555 })
553556