cirruslabs / orchard

Orchestrator for running Tart Virtual Machines on a cluster of Apple Silicon devices
Other
194 stars 16 forks source link

Order of `logs` lines sometimes is messed up #88

Closed grigorye closed 1 year ago

grigorye commented 1 year ago
$ orchard --version
orchard version 0.7.0-60e564d

Steps to reproduce:

orchard create vm v --startup-script 'for i in $(seq 1 100); do echo "$i."; done'
...
orchard logs vm v

Expected output (as long as #87 is not merged):

1.2.3.4.5.6.7.8.9.10.11.12.13.14.15.16.17.18.19.20.21.22.23.24.25.26.27.28.29.30.31.32.33.34.35.36.37.38.39.40.41.42.43.44.45.46.47.48.49.50.51.52.53.54.55.56.57.58.59.60.61.62.63.64.65.66.67.68.69.70.71.72.73.74.75.76.77.78.79.80.81.82.83.84.85.86.87.88.89.90.91.92.93.94.95.96.97.98.99.100.$

Examples of actual output (which differ from case to case):

100.1.2.3.12.13.14.15.16.17.18.19.20.21.4.22.23.24.25.26.27.28.29.30.31.5.32.33.34.35.36.37.38.39.40.41.6.42.43.44.45.46.47.48.49.50.51.7.52.53.54.55.56.57.58.59.60.61.8.62.63.64.65.66.67.68.69.70.71.9.72.73.74.75.76.77.78.79.80.81.10.82.83.84.85.86.87.88.89.90.91.11.92.93.94.95.96.97.98.99.$
1.67.2.68.11.77.78.12.13.79.80.14.15.81.16.82.83.17.84.18.19.85.20.86.69.3.21.87.88.22.89.23.24.90.91.25.92.26.93.27.94.28.29.95.96.30.70.4.31.97.32.98.99.33.34.100.35.36.37.38.39.40.5.71.41.42.43.44.45.46.47.48.49.50.72.6.51.52.53.54.55.56.57.58.59.60.73.7.61.62.63.64.65.66.74.8.75.9.76.10.$

Notes

What's interesting, is that if the number of lines is not that big, it is not reproducible:

$ orchard-local create vm v --startup-script 'for i in $(seq 1 10); do echo "$i."; done'
$ orchard logs vm v
1.2.3.4.5.6.7.8.9.10.$
grigorye commented 1 year ago

Hey folks, if I get it correctly, #87 should fix gluing, but not this (ordering of lines) issue - it's a different one. As far as I recall, it's still reproducible even after fixing gluing, sorry.