Skip to content

Conversation

DavidFair
Copy link
Collaborator

Blocked by #71

Adds AQ machine name to the metadata we register with. This was based on a request from the ops team.

@DavidFair DavidFair force-pushed the Add_machine_name_to_metadata branch from 4f2d1b8 to 5e1b095 Compare June 8, 2023 10:04
Add the AQ Machine name to the Aquilon metadata, based on a request from
the ops team
@DavidFair DavidFair force-pushed the Add_machine_name_to_metadata branch from 5e1b095 to abcf6c5 Compare June 8, 2023 10:09
@codecov-commenter
Copy link

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.01 🎉

Comparison is base (34a6e5f) 94.01% compared to head (abcf6c5) 94.02%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #72      +/-   ##
==========================================
+ Coverage   94.01%   94.02%   +0.01%     
==========================================
  Files          15       15              
  Lines         935      938       +3     
==========================================
+ Hits          879      882       +3     
  Misses         56       56              
Flag Coverage Δ
rabbit_consumer 94.02% <100.00%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...abbit-Consumer/rabbit_consumer/message_consumer.py 78.62% <100.00%> (ø)
...tack-Rabbit-Consumer/test/test_message_consumer.py 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@DavidFair DavidFair mentioned this pull request Jun 8, 2023
@meoflynn meoflynn merged commit 5a85dc6 into master Jun 9, 2023
@DavidFair DavidFair deleted the Add_machine_name_to_metadata branch June 27, 2023 12:08
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