Skip to content

Add AGENTS.md documentation on MiqQueue/Kafka - #23997

Merged
Fryguy merged 1 commit into
ManageIQ:masterfrom
agrare:agents_md_miq_queue_put_unless_exists
Oct 2, 2026
Merged

Fryguy merged 1 commit into
ManageIQ:masterfrom
agrare:agents_md_miq_queue_put_unless_exists

Conversation

@agrare

@agrare agrare commented Oct 2, 2026

Copy link
Copy Markdown
Member

There was a line in the product.md about the miq_queue table but nothing around MiqQueue and its uses, or Kafka and where that is used.

Also specifically there was nothing indicating that e.g. new callers to put_unless_exist should be avoided.

@agrare
agrare requested a review from Fryguy as a code owner October 2, 2026 13:20
There was a line in the product.md about the miq_queue table but nothing
around MiqQueue and its uses, or Kafka and where that is used.

Also specifically there was nothing indicating that e.g.
new callers to `put_unless_exist` should be avoided.
@agrare
agrare force-pushed the agents_md_miq_queue_put_unless_exists branch from 0554af1 to 3cd0f6d Compare October 2, 2026 13:20
@agrare

agrare commented Oct 2, 2026 •

Copy link
Copy Markdown
Member Author

@Fryguy I noticed a PR that added a new caller of put_unless_exists and thought we should document that it should be avoided.

We could probably go much more in depth into MiqQueue here if we want to, other options like deliver_on / server_guid.

@miq-bot

miq-bot commented Oct 2, 2026

Copy link
Copy Markdown
Member

Checked commit agrare@3cd0f6d with ruby 3.3.12, rubocop 1.91.0, haml-lint 0.78.0, and yamllint 1.37.1
0 files checked, 0 offenses detected
Everything looks fine. 🏆

@Fryguy
Fryguy merged commit c1b1ea8 into ManageIQ:master Oct 2, 2026
6 checks passed
@agrare
agrare deleted the agents_md_miq_queue_put_unless_exists branch October 2, 2026 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants