summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #1283 from rabbitmq/rabbitmq-server-1280Michael Klishin2017-07-0812-88/+210
|\ | | | | Handle concurrent vhost creation and deletion better
| * Wording, correct a type specMichael Klishin2017-07-072-5/+5
| |
| * WordingMichael Klishin2017-07-062-5/+4
| |
| * Rename functionMichael Klishin2017-07-061-3/+3
| |
| * WordingMichael Klishin2017-07-061-2/+2
| |
| * It can be any error, log it as suchMichael Klishin2017-07-061-1/+1
| |
| * WordingMichael Klishin2017-07-061-3/+0
| |
| * Return an error hereMichael Klishin2017-07-061-1/+1
| |
| * Emit vhost_deleted first, like beforeMichael Klishin2017-07-051-2/+2
| | | | | | | | | | It wasn't an intended change and it breaks per-vhost connection limit test suite.
| * Merge branch 'master' into rabbitmq-server-1280Michael Klishin2017-07-051-1/+1
| |\
| * | if rabbit_amqqueue_sup_sup:start_for_vhost/1 fails, log the reason and throwMichael Klishin2017-07-051-5/+4
| | |
| * | rabbit_vhost_sup_watcher: log a warningMichael Klishin2017-07-051-3/+3
| | | | | | | | | | | | | | | | | | | | | The watcher can discover that a vhost does not exits but the process tree is still there in normal operation (concurrent vhost setup and deletion), so a warning makes more sense here.
| * | Make this message up-to-dateMichael Klishin2017-07-051-1/+1
| | |
| * | rabbit_vhost:delete/2: reduce log noise when vhost is already deletedMichael Klishin2017-07-051-1/+4
| | | | | | | | | | | | | | | | | | Some of these errors are a consequence of the current implementation where queues, exchanges, etc are deleted in multiple transactions instead of one.
| * | Make rabbit_vhost:add/2 return an error when it failsMichael Klishin2017-07-051-6/+16
| | | | | | | | | | | | This propagates the error the caller, e.g. the CLI command or HTTP API handler.
| * | Log before throwing hereMichael Klishin2017-07-051-1/+4
| | |
| * | Virtual host watcher: stop vhost's top level supervisor instead of exitingMichael Klishin2017-07-051-1/+8
| | | | | | | | | | | | | | | Per suggestion from @hairyhum. This should be a safer option when rabbit.vhost_restart_strategy is set to stop_node.
| * | Merge branch 'master' into rabbitmq-server-1280Michael Klishin2017-07-051-143/+7
| |\ \
| * | | A typoMichael Klishin2017-07-041-1/+1
| | | |
| * | | Terminate when message stores fail to startMichael Klishin2017-07-041-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes rabbit.vhost_restart_strategy work as expected: higher up supervisors should decide whether the node should stop or keep going. Per discussion with @hairyhum.
| * | | Drop the "ignore" clauseMichael Klishin2017-07-041-3/+1
| | | | | | | | | | | | | | | | It was never a part of a GA release.
| * | | Merge branch 'master' into rabbitmq-server-1280Michael Klishin2017-07-0411-179/+217
| |\ \ \
| * | | | Correct a typoMichael Klishin2017-07-041-1/+1
| | | | |
| * | | | Handle concurrent vhost creation and deletion betterMichael Klishin2017-07-0111-79/+183
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the case of a vhost creation and immediate deletion those operations can end up executing concurrently. This changes a number of things to be more resilient: * Default vhost restart strategy changes to "continue" (previosly "ignore"). It's a tough default to pick but we believe the damage of a single vhost terminating the entire node that may have a lot of users is greater than the need to set up an alternative strategy for environments where only one vhost is used. Note that the event of message store or vhost supervision tree termination should be very rare. Per discussion with @kjnilsson. * For vhost [message store] recovery, we now log "no_such_vhost" errors in a sensible way instead of a bunch of process terminations. * Max restart intensity settings for rabbit_vhost_sup_wrapper are changed to accommodate integration test suites that rapidly set up temporary vhost and delete them. In most failure message store scenarios the new value is still low enough to lead to vhost supervision tree termination when recovery is impossible. There are also drive by doc changes. Fixes #1280.
* | | | | Merge pull request #1288 from rabbitmq/fix-parseInt-usageMichael Klishin2017-07-081-2/+2
|\ \ \ \ \ | |_|_|_|/ |/| | | | Fix verbiage to read a little better
| * | | | Fix verbiage to read a little betterLuke Bakken2017-07-071-2/+2
|/ / / /
* | | | sync_detection_SUITE: Wait for sync status for 30 secondsJean-Sébastien Pédron2017-07-051-1/+1
| |_|/ |/| | | | | | | | | | | ... instead of the previous 10 seconds. This helps eager_sync_SUITE which times out quite often with Erlang 20.
* | | rabbit_nodes: move a few more functions to rabbit_nodes_commonMichael Klishin2017-07-041-143/+7
| |/ |/| | | | | they are required by or can be useful to CLI tools.
* | term_to_binary_compat:queue_name_to_binary/1 was supersededMichael Klishin2017-07-041-1/+1
| |
* | Merge branch 'stable'Daniil Fedotov2017-07-040-0/+0
|\ \
| * | Do not require a node to be running to execute decode, list_ciphers and ↵Daniil Fedotov2017-07-041-1/+2
| | | | | | | | | | | | list_hashes commands
| * | Mention ctl decode arguments on manpage.Daniil Fedotov2017-07-041-1/+19
| | |
* | | Update man entries for encode and decode commands correspond to commands API.Daniil Fedotov2017-07-041-23/+10
| | |
* | | Merge pull request #1250 from rabbitmq/rabbitmq-server-1246-masterMichael Klishin2017-07-043-47/+67
|\ \ \ | | | | | | | | Change directory name generation function for queue indexes and vhosts
| * \ \ Merge branch 'master' into rabbitmq-server-1246-masterMichael Klishin2017-07-038-114/+145
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #1281 from rabbitmq/rabbitmq-common-208-masterMichael Klishin2017-07-032-32/+38
|\ \ \ \ | | | | | | | | | | Use delegate:invoke and delegate:invoke_no_result
| * | | | Use delegate:invoke and delegate:invoke_no_resultDiana Corbacho2017-06-302-32/+38
| | | | | | | | | | | | | | | | | | | | | | | | | Further simplifications to rabbitmq-server#208 which where too large to apply in 3.6.11. Now invoke and invoke_no_result are called directly
* | | | | rabbit_looking_glass: Tell xref to ignore call to lg:trace/4Jean-Sébastien Pédron2017-07-031-0/+2
|/ / / / | | | | | | | | | | | | [#140125673]
* | | | Merge branch 'stable'Jean-Sébastien Pédron2017-06-291-26/+10
|\ \ \ \ | | |/ / | |/| |
| * | | unit_SUITE: Adapt rabbitmqctl_encode testcase to the new rabbit_control_pbe APIrabbitmq_v3_6_11_milestone3Jean-Sébastien Pédron2017-06-291-26/+10
| | | |
* | | | rabbit_runtime_parameters: Pass `ActingUser` to clear_component()Jean-Sébastien Pédron2017-06-292-4/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In rabbit_exchange_parameters, use ?INTERNAL_USER when calling clear_component(). This fixes the call to clear() which expects this `ActingUser`. This went unnoticed because the function was called as M:F() instead of F() even though the function is local. Reported by xref. [#140125673]
* | | | rabbit_exchange_parameters, rabbit_runtime_parameters: Remove unused importsJean-Sébastien Pédron2017-06-292-3/+1
| | | |
* | | | docs/rabbitmqctl.8: Add list_hashes, list_ciphers and decode commandsJean-Sébastien Pédron2017-06-291-0/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This changes are ported from the following commits to rabbitmqctl.1.xml in `stable`: - commit 020fa730ce65e2e19ba6bb483bfc5c458580c533 - commit 6b65e41330a40060919b0cc19b895a57348493a5
* | | | Merge branch 'stable'Jean-Sébastien Pédron2017-06-292-50/+63
|\ \ \ \ | |/ / / | | | / | |_|/ |/| |
| * | docs/rabbitmqctl.1.xml: Fix quoting of {encrypted, <<"...">>}Jean-Sébastien Pédron2017-06-291-3/+3
| | | | | | | | | | | | There was three single quotes, so one extra.
| * | rabbit_runtime_parameters: Remove module name for local callsJean-Sébastien Pédron2017-06-281-4/+4
| | |
| * | Merge pull request #1276 from rabbitmq/rabbitmq-server-ctl-decodeMichael Klishin2017-06-283-52/+146
| |\ \ | | | | | | | | Add CTL commands to decode encoded value and list ciphers and hashes
| | * | CLI: ditto for decodeMichael Klishin2017-06-281-0/+25
| | | |
| | * | CLI: add list_hashes and list_ciphers in rabbitmqctl(1)Michael Klishin2017-06-281-3/+33
| | | |
| | * | Merge branch 'stable' into rabbitmq-server-ctl-decodeMichael Klishin2017-06-2821-1210/+6488
| | |\ \ | | |/ / | |/| |
View Lorry Controller Status