summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Use erlang:system_info(creation) instead of GUIDlh-kura/ad-on-stopMichal Kuratczyk2021-10-291-4/+7
| | | | | | | | | | GUIDs are generated and announced when rabbit app starts. In some cases, that takes minutes (many queues/bindings). During that time, we can get false positives and declare pause_minority because the Erlang VM responds but the GUID has not been updated. By using and checking erlang:system_info(creation) we get a unique value immediately after Erlang VM restart so we can tell whether the node was restarted.
* WIPLoïc Hoguin2021-10-264-9/+74
|
* Merge pull request #3599 from rabbitmq/mgmt-ui-prometheus-port-fixMichael Klishin2021-10-213-54/+53
|\ | | | | Prometheus plugin: fix TLS port handling
| * Fix ssl port handling in prometheus pluginmgmt-ui-prometheus-port-fixAlexey Lebedeff2021-10-203-54/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All ssl options were stored in the same proplist, and the code was then trying to determine whether an option actually belongs to ranch ssl options or not. Some keys landed in the wrong place, like it did happen in #2975 - different ports were mentioned in listener config (default at top-level, and non-default in `ssl_opts`). Then `ranch` and `rabbitmq_web_dispatch` were treating this differently. This change just moves all ranch ssl opts into proper place using schema, removing any need for guessing in code. The only downside is that advanced config compatibility is broken.
* | Merge pull request #3576 from rabbitmq/coordinator-error-handlingMichael Klishin2021-10-212-0/+7
|\ \ | | | | | | Handling of `stream_not_found` return value when init a stream client
| * | Handling of `stream_not_found` return value when init a stream clientcoordinator-error-handlingdcorbacho2021-10-132-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | This error probably shouldn't happen if the system is correct, however `stream_not_found` is a valid return value for the functions called - it comes from `register_listener`. Not handling it causes a case_clause (and long stacktrace) that is sent to the client as an internal error, instead of a known protocol error. Seen while debugging another issue.
* | | Merge pull request #3598 from rabbitmq/mgmt-ui-persistent-messages-fixMichael Klishin2021-10-211-0/+3
|\ \ \ | | | | | | | | Fix persistent message count display in mgmt UI
| * | | Fix persistent message count display in mgmt UImgmt-ui-persistent-messages-fixAlexey Lebedeff2021-10-201-0/+3
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #3396 As can be seen in https://github.com/rabbitmq/rabbitmq-server/blob/9cf18e83f279408e20430b55428a2b19156c90d7/deps/rabbitmq_management/priv/www/js/tmpl/queue.ejs#L171 there should be 3 colums for a classic queue. While it's possible to share the same field with the quorum queues case, I did it this way so that conditions for the labels and for the values will have the same shape.
* | | Disable stream test in mixed versions modeArnaud Cogoluègnes2021-10-202-5/+8
| | | | | | | | | | | | | | | Not worth testing in this mode, as the stream plugin does not start without the stream_queue feature flag.
* | | Enable stream_queue flag with env variable in test suiteArnaud Cogoluègnes2021-10-202-3/+4
| | | | | | | | | | | | | | | For mixed version mode, where feature flags are not explictely enabled.
* | | Enable stream_queue flag in test suiteArnaud Cogoluègnes2021-10-201-1/+3
| | | | | | | | | | | | | | | For mixed version mode, where feature flags are not explictely enabled.
* | | Format stream pluginArnaud Cogoluègnes2021-10-201-10/+12
|/ /
* | Merge branch 'RentTheRunway-falconertc/fix_duplicate_binding_exchange'Michael Klishin2021-10-192-5/+99
|\ \
| * | Improve a commentMichael Klishin2021-10-191-2/+1
| | |
| * | Finalize 3.9.8 release notesLoïc Hoguin2021-10-191-0/+41
| | |
| * | Fix binding deletion in duplicate-binding scenariosFalcon Taylor-Carter2021-10-181-3/+3
| | |
| * | Add tests for duplicate binding scenariosFalcon Taylor-Carter2021-10-181-1/+96
| | |
* | | Merge pull request #3591 from rabbitmq/3.9.8-release-notesMichael Klishin2021-10-190-0/+0
|\ \ \ | | | | | | | | Finalize 3.9.8 release notes
| * | | Finalize 3.9.8 release notesMichael Klishin2021-10-191-0/+41
| |/ /
* | | Finalize 3.9.8 release notesLoïc Hoguin2021-10-191-0/+41
|/ /
* | Merge pull request #3586 from rabbitmq/declare-stream-with-node-down-fixMichael Klishin2021-10-182-2/+18
|\ \ | | | | | | Declare stream with node down fix
| * | Take all clustered nodes into account when declaring stream.Karl Nilsson2021-10-182-2/+18
| | | | | | | | | | | | | | | | | | Deriving a max-cluster-size only from running nodes would create situations where in a three-node with only two nodes running cluster it would select an non-running node as follower.
* | | Update 3.9.8 release notesMichael Klishin2021-10-181-9/+5
| | |
* | | More accurate wording in the release notesMichal Kuratczyk2021-10-181-3/+3
| | |
* | | Mention osiris hostname fix in the release notesMichal Kuratczyk2021-10-181-0/+5
| | |
* | | Merge pull request #3584 from ↵Michael Klishin2021-10-168-12/+12
|\ \ \ | | | | | | | | | | | | | | | | rabbitmq/dependabot/github_actions/actions/checkout-2.3.5 Bump actions/checkout from 2.3.4 to 2.3.5
| * | | Bump actions/checkout from 2.3.4 to 2.3.5dependabot/github_actions/actions/checkout-2.3.5dependabot[bot]2021-10-158-12/+12
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [actions/checkout](https://github.com/actions/checkout) from 2.3.4 to 2.3.5. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](https://github.com/actions/checkout/compare/v2.3.4...v2.3.5) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
* | | Merge pull request #3574 from rabbitmq/fix-stream-coordinator-logMichael Klishin2021-10-151-2/+2
|\ \ \ | | | | | | | | Fix stream coordinator log message
| * | | Fix stream coordinator log messagefix-stream-coordinator-logdcorbacho2021-10-141-2/+2
| |/ /
* | | Merge pull request #3582 from luos/channel-q-pid-leak-fix-v39xMichael Klishin2021-10-152-4/+6
| | | | | | | | | | | | | | | | | | Fix an Erlang monitor leak when queue churn is high (cherry picked from commit a78b76afcda60820cf78f16a509b50b8ec32789b)
* | | Merge pull request #3579 from tvhong-amazon/tvhong/log_errorMichael Klishin2021-10-152-11/+20
|\ \ \ | | | | | | | | Log response body of failed AWS requests
| * | | Log response body of failed AWS requestsVy Hong2021-10-082-11/+20
| | | |
* | | | Merge pull request #3562 from rabbitmq/disable_stream_plugin_3557Michael Klishin2021-10-149-16/+66
|\ \ \ \ | | | | | | | | | | Disable the stream_plugin if stream feature flag is not enabled
| * | | | WordingMichael Klishin2021-10-141-2/+2
| | | | |
| * | | | Formatting, wordingMichael Klishin2021-10-141-12/+12
| | | | |
| * | | | change messagedisable_stream_plugin_3557Gabriele Santomaggio2021-10-132-2/+2
| | | | |
| * | | | Disable the stream_plugin and stream management if the feature flagGabriele Santomaggio2021-10-139-14/+52
| | | | | | | | | | | | | | | | | | | | | | | | | is not enabled. fixes https://github.com/rabbitmq/rabbitmq-server/issues/3557
| * | | | Disable the stream_plugin if the feature flagGabriele Santomaggio2021-10-121-5/+17
| | | | | | | | | | | | | | | | | | | | | | | | | is not enabled. fixes https://github.com/rabbitmq/rabbitmq-server/issues/3557
* | | | | Attach add/delete super stream command to streams scope onlyArnaud Cogoluègnes2021-10-142-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Not to ctl. References #3503
* | | | | Merge pull request #3503 from rabbitmq/super-stream-cliArnaud Cogoluègnes2021-10-1410-125/+1283
|\ \ \ \ \ | | | | | | | | | | | | Add functions to create/delete super stream in manager
| * | | | | Handle stream arguments in add_super_stream commandsuper-stream-cliArnaud Cogoluègnes2021-10-116-11/+295
| | | | | | | | | | | | | | | | | | | | | | | | max-age, leader-locator, etc.
| * | | | | Embed RPC call function in testArnaud Cogoluègnes2021-10-111-2/+6
| | | | | |
| * | | | | Fix group spec in testArnaud Cogoluègnes2021-10-111-1/+1
| | | | | |
| * | | | | Add delete_super_stream CLI commandArnaud Cogoluègnes2021-10-113-8/+170
| | | | | |
| * | | | | Add add_super_stream CLI commandArnaud Cogoluègnes2021-10-113-8/+212
| | | | | |
| * | | | | Add functions to create/delete super stream in managerArnaud Cogoluègnes2021-10-115-121/+625
| | |/ / / | |/| | |
* | | | | Merge pull request #3544 from rabbitmq/bump-otp-for-ociMichael Klishin2021-10-131-2/+2
|\ \ \ \ \ | | | | | | | | | | | | Adopt otp 24.1.2 for OCI workflow
| * | | | | Adopt otp 24.1.2 for OCI workflowGitHub2021-10-061-2/+2
| | | | | |
* | | | | | Merge pull request #3570 from rabbitmq/lh-fix-memory_breakdownMichael Klishin2021-10-133-1/+69
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | Fix memory_breakdown for connection procs
| * | | | | Fix memory_breakdown for connection procsLoïc Hoguin2021-10-133-1/+69
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | Instead of trying to read the ranch_server ETS table directly, use an internal Ranch function that does the same. While neither are documented, the latter is less likely to change.