Results 65 comments of Marco Primi

Addressed review feedback. Travis errors are unrelated.

Updated with additional test cases now that https://github.com/nats-io/nats-server/pull/5785 fixed them ``` --- PASS: TestJetStreamClusterConsumeWithStartSequence (2.76s) --- PASS: TestJetStreamClusterConsumeWithStartSequence/Nodes:1,Replicas:1 (0.02s) --- PASS: TestJetStreamClusterConsumeWithStartSequence/Nodes:1,Replicas:1/DurableConsumer (0.00s) --- PASS: TestJetStreamClusterConsumeWithStartSequence/Nodes:1,Replicas:1/DurableConsumerWithBind (0.00s) --- PASS: TestJetStreamClusterConsumeWithStartSequence/Nodes:1,Replicas:1/PreCreatedDurableConsumerWithBind...

@neilalexander Addressed feedback, thank you. While this is open, I added a second commit with a new test (ACK of deleted messages)

Thank you for looking into this. I also have been trying to repro with a variant of `TestSystem_TestProcShutDownWithConfiguredTimeOut`, unsuccessfully so far. Nevertheless our CI pipes (1.27 on Linux via `nix`)...

@F1bonacc1, thank you for investigating this. While it seems likely the problems are related, the issue I originally reported is not exactly what you identified: - I am not using...

Confirming issue is fixed in [v1.40.0](https://github.com/F1bonacc1/process-compose/releases/tag/v1.40.0). Thank you @F1bonacc1

Just bumped into this. Same result: whichever key is first is used, the rest seem to be ignored. Maybe this setup should not be advertised in the frontpage/readme, given that...

Suggested workaround for whoever is reading this: set and use one key at the time. For example i want to refresh 2 submodules that have 2 different deploy keys. The...

> not sure it merits an ADR Fair enough. I won't try to merge this PR. I wanted to collect some feedback/opinions, then I'll close it.

GH Pipeline: https://github.com/nats-io/nats-server/actions/workflows/long-tests.yaml BK Pipeline: https://buildkite.com/synadia/nats-server-long-running-tests BK Analytics: https://buildkite.com/organizations/synadia/analytics/suites/nats-server-long-running-tests PR: https://github.com/nats-io/nats-server/pull/5920