Commit d979972
committed
CLDSRV-992: Wait for the file data daemon before the S3 API
`yarn start` launches the S3 API and the file daemons as separate parallel
processes (npm-run-all --parallel start_dmd start_s3server), so they race. On
run 33734792029 the API bound :8000 at t+10s and served writes from t+15s while
the dataserver only bound :9991 at t+84s, and the ten HTTP 500s in between are
all ECONNREFUSED 0.0.0.0:9991. The same subdir pre-creation step took 2.7s on a
healthy runner, which is why this is intermittent.
Context on why cloudserver does not catch this itself. metadata.setup() opens
the metadata client and throws if 9990 is unreachable, so the API cannot bind
without working metadata, which matches the run: CreateBucket and every metadata
read succeeded and only the data path failed. Nothing guards the data daemon,
because clientCheck returns a hardcoded { code: 200, message: 'OK' } for clients
that implement no probe and DataFileInterface implements none. So the deep
healthcheck only really checks metadata, and only for the mongo and bucketd
clients. Adding the file backend to the deep check would let the product gate
itself and make this change unnecessary, but that belongs in Arsenal.
Gate on 9991 in the four jobs that use the file data backend: file-ft-tests,
kmip-ft-tests, kmip-cluster-ft-tests and sse-kms-migration-tests, the last of
which starts a fresh cloudserver container in a second phase and needs the gate
there too. 120s because 84s was observed; a healthy run pays nothing since the
wait returns as soon as the port answers.
Clears CLDSRV-992 row F1 (ten of its twelve failures) and is the leading
candidate for rows F2 and F3, whose jobs share this boot path.
Issue: CLDSRV-9921 parent c79db25 commit d979972
1 file changed
Lines changed: 10 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
532 | 532 | | |
533 | 533 | | |
534 | 534 | | |
| 535 | + | |
| 536 | + | |
535 | 537 | | |
536 | 538 | | |
537 | 539 | | |
| |||
831 | 833 | | |
832 | 834 | | |
833 | 835 | | |
| 836 | + | |
| 837 | + | |
834 | 838 | | |
835 | 839 | | |
836 | 840 | | |
| |||
909 | 913 | | |
910 | 914 | | |
911 | 915 | | |
| 916 | + | |
| 917 | + | |
912 | 918 | | |
913 | 919 | | |
914 | 920 | | |
| |||
1009 | 1015 | | |
1010 | 1016 | | |
1011 | 1017 | | |
| 1018 | + | |
| 1019 | + | |
1012 | 1020 | | |
1013 | 1021 | | |
1014 | 1022 | | |
| |||
1053 | 1061 | | |
1054 | 1062 | | |
1055 | 1063 | | |
| 1064 | + | |
| 1065 | + | |
1056 | 1066 | | |
1057 | 1067 | | |
1058 | 1068 | | |
| |||
0 commit comments