Skip to content

Commit

Permalink
fix #63589
Browse files Browse the repository at this point in the history
  • Loading branch information
Thomas Phipps committed Apr 27, 2023
1 parent e25da80 commit 19fb8c6
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 0 deletions.
1 change: 1 addition & 0 deletions changelog/63589.fixed.md
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
add documentation note about reactor state ids.
8 changes: 8 additions & 0 deletions doc/topics/reactor/index.rst
Original file line number Diff line number Diff line change
Expand Up @@ -212,6 +212,10 @@ in :ref:`local reactions <reactor-local>`, but as noted above this is not very
user-friendly. Therefore, the new config schema is recommended if the master
is running a supported release.

.. note::
state ids of reactors for runners and wheel should all be unique. they can
overwrite each other when added to the async queue. causing lost reactions

The below two examples are equivalent:

+-------------------------------------------------+-------------------------------------------------+
Expand Down Expand Up @@ -248,6 +252,10 @@ Like :ref:`runner reactions <reactor-runner>`, the old config schema called for
wheel reactions to have arguments passed directly under the name of the
:ref:`wheel function <all-salt.wheel>` (or in ``arg`` or ``kwarg`` parameters).

.. note::
state ids of reactors for runners and wheel should all be unique. they can
overwrite each other when added to the async queue. causing lost reactions

The below two examples are equivalent:

+-----------------------------------+---------------------------------+
Expand Down

0 comments on commit 19fb8c6

Please sign in to comment.