Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update message schema to include __str__, agent_name, groups, and app_name #4950

Closed
ryanlerch opened this issue Jan 11, 2023 · 0 comments · Fixed by #4951
Closed

Update message schema to include __str__, agent_name, groups, and app_name #4950

ryanlerch opened this issue Jan 11, 2023 · 0 comments · Fixed by #4951
Assignees

Comments

@ryanlerch
Copy link
Collaborator

ryanlerch commented Jan 11, 2023

Currently the bodhi messaging schema is missing a handful of properties for proper compatibility with upcoming fmn work.

  • str -- two messages currently include the str property (which is used to Return a human-readable representation of a message, but longer than the summary i.e. an email body). we need to implement str in the majority of messages:
  • agent_name -- we currently already have agent -- just need to expose this as agent_name for full compatability
  • app_name -- pretty self explainatory

finally, we also have

  • groups -- which i dont think we need to implement in bodhi, as it isnt really aware of FAS groups
@ryanlerch ryanlerch self-assigned this Jan 11, 2023
@ryanlerch ryanlerch added this to Backlog in Fedora Messaging Schemas via automation Jan 11, 2023
@ryanlerch ryanlerch moved this from Backlog to In Progress in Fedora Messaging Schemas Jan 11, 2023
@mergify mergify bot closed this as completed in #4951 Jan 21, 2023
Fedora Messaging Schemas automation moved this from In Progress to Done Jan 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging a pull request may close this issue.

1 participant