summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Refactor to include retriesLuke Bakken2020-05-122-26/+27
|
* Revert "Remove publisher_confirms & confirm_nack from mirrored_queue tests"Luke Bakken2020-05-121-34/+11
| | | | This reverts commit 838632f3dffd56c9112c0d546027e8ce45ec08fc.
* Revert "Remove rejects_survive test flake"Luke Bakken2020-05-121-0/+2
| | | | This reverts commit eb079377b3064a646cf4f24f88fafcc2ba1a4a46.
* Revert "Remove tests from ct-simple_ha that fail frequently"Luke Bakken2020-05-121-1/+51
| | | | This reverts commit 9aced82a5bdf25aa9819556e8ee1f5134668af18.
* Revert "Remove variable_queue_fold test flake"Luke Bakken2020-05-122-5/+43
| | | | | | This reverts commit 229ac6bfb7429b12bfb9ebe37eb5431d90f7268c. Fix syntax errors
* Revert "Remove calculate_client_local test flake"Luke Bakken2020-05-121-1/+10
| | | | This reverts commit 12e3ad79eef8c21b39502f3e3ff55f63ea479e21.
* Revert "Remove mixed_dead_alive_queues_reject test flake"Luke Bakken2020-05-121-2/+95
| | | | This reverts commit d8771acee402446071b945c76b9dc737e572b992.
* Revert "Remove member_death test flake"Luke Bakken2020-05-121-0/+27
| | | | | | This reverts commit 598c14f76ff593f19faa0a01458273dc5b09197b. Add a broadcast
* Merge pull request #2340 from rabbitmq/mk-missing-license-headersMichael Klishin2020-05-126-1/+97
|\ | | | | Add some missing license headers
| * Add some missing license headersMichael Klishin2020-05-126-1/+97
|/
* GitHub Actions: Regen workflowsJean-Sébastien Pédron2020-05-112-0/+76
|
* GitHub Actions: Regen workflowsJean-Sébastien Pédron2020-05-112-382/+6488
|
* Merge pull request #2339 from rabbitmq/fix-dialyzer-errorsJean-Sébastien Pédron2020-05-1110-33/+19
|\ | | | | Fix errors reported by Dialyzer
| * rabbit_vhost: Vhost limits are never `undefined`Jean-Sébastien Pédron2020-05-111-1/+1
| | | | | | | | | | | | | | The type spec of vhost limits is a list, possibly empty. A vhost is initialized with an empty list. Reported by Dialyzer.
| * rabbit_ssl: Fix return value of `get_highest_protocol_version/0`Jean-Sébastien Pédron2020-05-111-2/+3
| | | | | | | | | | | | | | The result of this function is passed to `ssl:cipher_suites/2` which expects the atom-based SSL/TLS versions. Reported by Dialyzer.
| * rabbit_quorum_queue: Fix type specJean-Sébastien Pédron2020-05-111-1/+2
| | | | | | | | Reported by Dialyzer.
| * rabbit_peer_discovery: Fix type spec of lock/0Jean-Sébastien Pédron2020-05-112-8/+1
| | | | | | | | | | | | | | | | Related to this: in `rabbit_mnesia`, remove dead code from the case clause handling the return value of `rabbit_peer_discovery:lock/0`. Indeed, the function never returns `Data` only. Reported by Dialyzer.
| * rabbit_feature_flags: Fix type specsJean-Sébastien Pédron2020-05-111-1/+6
| | | | | | | | Reported by Dialyzer.
| * rabbit_definitions: Use maps:update_with/4 directlyJean-Sébastien Pédron2020-05-111-14/+1
| | | | | | | | | | | | | | ... instead of using a copy of the function from Erlang 20. This is possible because the requirement was bumped to Erlang 21. Reported by Dialyzer.
| * rabbit_connection_tracking: Disable `race_conditions` check for ↵Jean-Sébastien Pédron2020-05-111-0/+1
| | | | | | | | | | | | | | | | | | | | register_connection/1 There is a race condition reported by Dialyzer because do a read/write outside of a Mnesia transaction. Historically there was a transaction, but it was removed in commit e5df4e6ee105900087744ed40e58b8041738b9cd.
| * rabbit_auth_backend_internal: Removed never used case clauseJean-Sébastien Pédron2020-05-111-4/+0
| | | | | | | | | | | | | | | | | | ... in put_user/3. If the ets call behind `lookup_user/1` fails for another reason than the entry was not found, it would be an exception anyway. Reported by Dialyzer.
| * rabbit_amqqueue: Fix spec of ↵Jean-Sébastien Pédron2020-05-111-2/+4
|/ | | | | | | | list_local_mirrored_classic_without_synchronised_mirrors_for_cli/0 The return value was incorrectly spec'd. Reported by Dialyzer.
* GitHub Actions: Regen workflowsJean-Sébastien Pédron2020-05-072-176/+176
|
* upgrade_preparation_SUITE: Skip test if quorum queues are unsupportedJean-Sébastien Pédron2020-05-071-1/+5
|
* GitHub Actions: Regen workflowsJean-Sébastien Pédron2020-05-072-182/+4604
|
* Restrict build badges to specific branchGerhard Lazu2020-05-041-2/+2
| | | | | | | Otherwise they will show as failing if the latest build failed, even when this build ran on a different branch. Signed-off-by: Gerhard Lazu <gerhard@lazu.co.uk>
* Merge pull request #2334 from rabbitmq/change-quorum-defaultsMichael Klishin2020-05-015-6/+7
|\ | | | | Changed default configuration values related to quorum queues
| * Changed or overridden default configuration values related to quorum queuesVanlightly2020-05-015-6/+7
| | | | | | | | | | | | Specifically: - changed default quorum_commands_soft_limit from 256 to 32 - override Ra wal_max_batch_size to 4096
* | Trigger tests on new-commit-to-dep-release-branch repository eventsGerhard Lazu2020-05-012-2/+10
|/ | | | Signed-off-by: Gerhard Lazu <gerhard@lazu.co.uk>
* Fix README build badgesGerhard Lazu2020-04-301-2/+2
| | | | Signed-off-by: Gerhard Lazu <gerhard@lazu.co.uk>
* Merge pull request #2326 from rabbitmq/improve-product_info-apiGerhard Lazu2020-04-301-42/+79
|\ | | | | Improve handling of product name & version
| * rabbit: Include base product name & version in "Starting" logged messageJean-Sébastien Pédron2020-04-301-3/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the product name & version are not overridden, the common logged message is the following one: 2020-04-24 15:08:51.754 [info] <0.263.0> Starting RabbitMQ 3.8.0+rc.1.278.g5cdf562.dirty on Erlang 21.3.8.13 Copyright (c) 2007-2020 VMware, Inc. or its affiliates. Licensed under the MPL 1.1. Website: https://rabbitmq.com Now, when the product name or version are overridden, the "Starting" line mentions them. There is a second line added with the base product name & version: 2020-04-24 15:08:51.754 [info] <0.263.0> Starting RabbitMQ Enterprise Edition 2020.04 on Erlang 21.3.8.13 Based on RabbitMQ 3.8.0+rc.1.278.g5cdf562.dirty Copyright (c) 2007-2020 VMware, Inc. or its affiliates. Licensed under the MPL 1.1. Website: https://rabbitmq.com
| * rabbit: Report base and overridden product info separatelyJean-Sébastien Pédron2020-04-301-39/+65
|/ | | | | | | | | | | | | | | | | | | | | | | | | Now, `rabbit:product_info/0` will report the base name & version ("RabbitMQ" and its actual version) separately from the final product name & version (if any). If the product name & version were not overridden in the configuration or the environment, they will be missing from the product info return value. It allows caller to distinguish situations where the product name & version were set or not. An example use case is `rabbitmqctl status`: we want to report the product name & version only if they were overridden. If they were not, the "Product name" and "Product version" won't be added to the output. Only "RabbitMQ version" will be displayed. To help with the above, the map returned by `rabbit:product_info/0` also contains a flag, `product_overridden`, set to true if they were. Finally, the RabbitMQ version is often associated with the Erlang/OTP release. So include this into the returned map. This will save an additional RPC call if the remote caller is interested in both.
* Revert "Merge pull request #2325 from ↵Luke Bakken2020-04-301-40/+8
| | | | | | | rabbitmq/revert-rabbitmq-server-2308-dead_queue_rejects" This reverts commit 55d28f5ad3ad3a47791989bd118c6d8ce954eeca, reversing changes made to 0bf8d0841c2467b93ab849f4f53a7bc99ab854e7.
* GitHub Actions: Regen workflowsJean-Sébastien Pédron2020-04-302-178/+1064
|
* Merge pull request #2332 from rabbitmq/lrb-handle-closed-from-port_pleaseMichael Klishin2020-04-301-34/+22
|\ | | | | Handle "closed" return value
| * Handle "closed" return valueLuke Bakken2020-04-291-34/+22
| | | | | | | | | | | | | | | | Turns out `port_please/2` and `port_please/3` can return `closed` - https://github.com/erlang/otp/pull/2616 We need to handle it here. First reported via RabbitMQ public slack channel - https://rabbitmq.slack.com/archives/C1EDN83PA/p1588166195464400
* | Use a 2 minute timeout when renaming nodes in cluster_rename_SUITEPhilip Kuryloski2020-04-291-1/+1
|/ | | | This suite has been flaky, but it only fails with a timeout trap on the whole test. By setting a timeout for invidual rabbitmqctl commands, it should be easier to discern in a flake where things got stalled.
* Express connection count assertion more generallyPhilip Kuryloski2020-04-281-3/+4
| | | | | | So that more information is available in the logs upon test failure https://3.basecamp.com/4467044/buckets/16481075/todos/2619089282
* Merge pull request #2331 from rabbitmq/mk-remove-outdated-commentMichael Klishin2020-04-281-2/+0
|\ | | | | Remove an outdated comment
| * Remove an outdated commentmk-remove-outdated-commentMichael Klishin2020-04-281-2/+0
| |
* | GitHub Actions: Regen workflowsJean-Sébastien Pédron2020-04-282-0/+2
| |
* | Merge pull request #2330 from rabbitmq/qq-delivery-limit-bugMichael Klishin2020-04-282-7/+34
|\ \ | |/ |/| QQ: Fix crash bug when reaching delivery limit
| * QQ: Fix crash bug when reaching delivery limitkjnilsson2020-04-282-7/+34
| | | | | | | | | | | | This only happens when using basic.get but would crash the quorum queue when the delivery limit was reached due to the transient basic.get consumer being removed
* | GitHub Actions: Regen workflowsJean-Sébastien Pédron2020-04-271-2/+2
| |
* | Merge pull request #2325 from ↵Jean-Sébastien Pédron2020-04-271-8/+40
|\ \ | | | | | | | | | | | | rabbitmq/revert-rabbitmq-server-2308-dead_queue_rejects Revert "Remove dead_queue_rejects test flake"
| * | Revert "Remove dead_queue_rejects test flake"Luke Bakken2020-04-231-8/+40
| | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 9772aa8f8256a410bcc13ae5038f949ffe1d5b84. Check RPC return value Try a cast to fix test
* | | GitHub Actions: Regen workflowsJean-Sébastien Pédron2020-04-272-2/+2
| | |
* | | GitHub Actions: Generate workflows using `make github-actions`Jean-Sébastien Pédron2020-04-273-419/+374
| | |
* | | Git: Ignore rabbitmq-deps.mk fileJean-Sébastien Pédron2020-04-271-1/+1
| | |