)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"20d4206109cb8decc6208009826ac2f5b9ad7c41","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"30235c03_964ffebc","updated":"2025-02-19 00:14:25.000000000","message":"actually i have kind of changed my mind i think we need a minimal devstack job (without tempest or just tempest smoke) in the repo that uses this plugin before we start extending the jobs with cross gating jobs","commit_id":"da67cebe306f074591f91804a8c5aa92300a1287"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3eb1d35d299c8d001584282b57eeef58420b51bf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5c9688be_6083ecc0","updated":"2025-02-18 23:49:41.000000000","message":"i guess cross gating like this does make sense \nalthogh we should keep the total number of cross-gating jobs small.\n\nso likely we shoudl have at most one of the telemetry jobs that currently uses prometious form sgcore repo as a represeitive job to ensure they also are not regressed by an changes we make.\n\nsince this is the first release wehre this plugin will exist i think its ok to keep this nobel only fpr now instead of having a jammy (22.04) varint in the repo.\n\nzuul will not merge this until the depend on is merged so +2w","commit_id":"da67cebe306f074591f91804a8c5aa92300a1287"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"cac4adae25bec40be6e6ddb1b4808937468a71ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"21ecde60_1ab9d248","updated":"2025-02-19 21:12:57.000000000","message":"rechecck\n\nwe chatted about some of the design comments i had on the eailer patches.\nwhile im not a fan of some of the choices made by node_exporter specificly the fact that it does not suprpot envionment variabel or config files as confiugration options, we have to live with that for now.\n\nwe can and will add a minimal devstack jobin a follwoup so reapproving this","commit_id":"da67cebe306f074591f91804a8c5aa92300a1287"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"1300575c0270999b5416f64bf2e8a1019e9e9b06","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"761f4dc8_8ee2b9b6","updated":"2025-02-17 19:33:14.000000000","message":"thanks for adding it","commit_id":"da67cebe306f074591f91804a8c5aa92300a1287"}]}