)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"41c3600d171ae004bf90aa8d89e4a8fc74f09046","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bee49383_e229eb07","updated":"2025-06-03 19:31:03.000000000","message":"+1 for starting this\nill wait for it to render in the job and loop back\nminor nit: please fix the whitespace issues after the jobs report.","commit_id":"a5d4c263e2bff8118098923a15bc2da55480fdfb"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"9d67f59decc79ac2c939f1f9a0ee85e48d24a7d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cefd37b3_19f72934","updated":"2025-06-04 10:21:31.000000000","message":"Thanks for this! it\u0027s rendering properly.\n\nhttps://cb417745cbf2b7f2d721-49bec08735736aac984064f4e640f4b1.ssl.cf1.rackcdn.com/openstack/9bbd958f01ab43fe94b91f82f9eb53bd/docs/integrations/index.html#integration-status-matrix\n\nWould it be good to provide any categorization about the scope of the integration? i.e. if it provides a model collector, actions for the service, etc...","commit_id":"a5d4c263e2bff8118098923a15bc2da55480fdfb"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"3aba24f197d2756827f31b3dfb61aa91682ec44e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c50ab744_e7532d9d","in_reply_to":"8ffad7f1_70c01587","updated":"2025-06-05 16:47:32.000000000","message":"ack, I updated the doc including more details about each service. Thanks for the suggestion.","commit_id":"a5d4c263e2bff8118098923a15bc2da55480fdfb"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"76b22cd680218a4e636d3a70994375679043427b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8ffad7f1_70c01587","in_reply_to":"c4073c1d_9455863d","updated":"2025-06-04 13:55:34.000000000","message":"I agree we should try to keep it simple and not require to maintain on each new call or change but some high level description that make visible the high level different levels of usage between nova (which has its own model collector, specific actions, etc...) and glance or neutron (minimal integration). I think that\u0027d be enough.","commit_id":"a5d4c263e2bff8118098923a15bc2da55480fdfb"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"786fa788bd30adf5deb24d7b200222ce5728bd36","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1318ca63_bd98be4e","in_reply_to":"c50ab744_e7532d9d","updated":"2025-06-05 17:03:01.000000000","message":"Thanks for that, I think it\u0027s what is needed.","commit_id":"a5d4c263e2bff8118098923a15bc2da55480fdfb"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"2d242366aa2df99f419c09307cc3e6f2a1637e62","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c4073c1d_9455863d","in_reply_to":"cefd37b3_19f72934","updated":"2025-06-04 12:04:28.000000000","message":"I think that we could add a Scope column yes. Not sure if it will be harder to maintain, e.g.: if a strategy adds a cinder api call, we would need to update it here. If we remove a action which uses a client, we would need to update too.\nNot sure, but we could try.","commit_id":"a5d4c263e2bff8118098923a15bc2da55480fdfb"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"4395c7e86421c871c1686146d6ff9342bd1abb04","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"605b8baf_9d153441","updated":"2025-06-27 06:34:08.000000000","message":"lgtm!","commit_id":"b788a67c52361a4a41040dee8430fa03f3ac564a"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"8d4c63fea1a72e84fba5324a6cee651af837c5db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"86f64762_cbe805ab","updated":"2025-06-06 14:50:31.000000000","message":"looking through the documentation, there are a few more services that watcher could interact but are not listed here: prometheus, monasca, grafana, gnocchi, ceilometer https://docs.openstack.org/watcher/latest/configuration/configuring.html#configure-the-watcher-service. Should we only document openstack services in this table or all of them? E.g only ceilometer and monasca are openstack services that would be missing","commit_id":"b788a67c52361a4a41040dee8430fa03f3ac564a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"655d6ca5d2154fc0e711434fbcaa721557291b16","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"be2d41bb_55e7dcbf","updated":"2025-06-27 15:57:54.000000000","message":"ok. there is quite a lot of repitation in some of the service descriptions but its fine the doc is pretty short.\n\nlooking at the Integration Status i agree with all or most of the current status\nim happy to merge them as is but we will likely revise this again before the end of the cycle","commit_id":"b788a67c52361a4a41040dee8430fa03f3ac564a"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"6a1b258cdd5729ff622cb17bfb6621bf743653bf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c1ef7fbc_f4b2f0d6","updated":"2025-06-26 12:16:40.000000000","message":"thanks Doug, lgtm!","commit_id":"b788a67c52361a4a41040dee8430fa03f3ac564a"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"bca2983acbc21985811e73ac92ee8950e39bf9cc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d3245a12_6d8ea8aa","in_reply_to":"86f64762_cbe805ab","updated":"2025-06-06 14:56:36.000000000","message":"It is a good point. \nI think that we could split and document those in the Datasources subsection, and maybe move Datasources under Integrations section. We could create a second support table for Datasources too.","commit_id":"b788a67c52361a4a41040dee8430fa03f3ac564a"}],"doc/source/integrations/index.rst":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"0775093079c309e7f105a548ed7e21ab66d00d88","unresolved":true,"context_lines":[{"line_number":49,"context_line":"       * - :ref:`Placement \u003cplacement_integration\u003e`"},{"line_number":50,"context_line":"         - Supported"},{"line_number":51,"context_line":"         - Minimal"},{"line_number":52,"context_line":"         - Unit and Integration"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":".. note::"},{"line_number":55,"context_line":"   Minimal documentation covers only basic configuration and, if available,"}],"source_content_type":"text/x-rst","patch_set":3,"id":"a47505dd_f21310b4","line":52,"updated":"2025-06-06 14:52:46.000000000","message":"In https://review.opendev.org/c/openstack/watcher/+/951440 we document testing level slightly differently, with an `Integration` level implying that `Unit` tests also exist, both approaches seem reasonable to me, but I think we should converge both patch to use the same","commit_id":"b788a67c52361a4a41040dee8430fa03f3ac564a"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"bded6a78e68d02978c1b9320c20754124f71c05b","unresolved":true,"context_lines":[{"line_number":49,"context_line":"       * - :ref:`Placement \u003cplacement_integration\u003e`"},{"line_number":50,"context_line":"         - Supported"},{"line_number":51,"context_line":"         - Minimal"},{"line_number":52,"context_line":"         - Unit and Integration"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":".. note::"},{"line_number":55,"context_line":"   Minimal documentation covers only basic configuration and, if available,"}],"source_content_type":"text/x-rst","patch_set":3,"id":"ec075583_6e48cf6e","line":52,"in_reply_to":"a47505dd_f21310b4","updated":"2025-06-06 15:06:05.000000000","message":"I think that both are different. There are modules that have Unit tests implemented and tox is running all in CI. By Integration, I am checking if we have tempest tests running on CI, which actually validate the integration of this service with Watcher. One is not related with the other in this case","commit_id":"b788a67c52361a4a41040dee8430fa03f3ac564a"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"6a1b258cdd5729ff622cb17bfb6621bf743653bf","unresolved":false,"context_lines":[{"line_number":49,"context_line":"       * - :ref:`Placement \u003cplacement_integration\u003e`"},{"line_number":50,"context_line":"         - Supported"},{"line_number":51,"context_line":"         - Minimal"},{"line_number":52,"context_line":"         - Unit and Integration"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":".. note::"},{"line_number":55,"context_line":"   Minimal documentation covers only basic configuration and, if available,"}],"source_content_type":"text/x-rst","patch_set":3,"id":"bf107f3f_45a9cdc3","line":52,"in_reply_to":"ec075583_6e48cf6e","updated":"2025-06-26 12:16:40.000000000","message":"Acknowledged","commit_id":"b788a67c52361a4a41040dee8430fa03f3ac564a"}]}