)]}'
{".zuul.yaml":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"76f9dd8cee8067c03ba40c5b91224565817441db","unresolved":true,"context_lines":[{"line_number":10,"context_line":"            voting: false"},{"line_number":11,"context_line":"        - monasca-tempest-python3-cassandra"},{"line_number":12,"context_line":"        - legacy-monasca-common-maven-build"},{"line_number":13,"context_line":"        - build-monasca-common-docker-base-image"},{"line_number":14,"context_line":"    gate:"},{"line_number":15,"context_line":"      jobs:"},{"line_number":16,"context_line":"        - monasca-tempest-python3-influxdb:"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"8dab97d6_0904f725","line":13,"updated":"2024-02-06 14:45:15.000000000","message":"should this be run on every patch send to gerrit?","commit_id":"86f83bbfb30dd7ef37e8c7266ac5ded5d12a3f0c"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"a0e6af75ffc14ba82357415df9312f34c3f1cb51","unresolved":false,"context_lines":[{"line_number":10,"context_line":"            voting: false"},{"line_number":11,"context_line":"        - monasca-tempest-python3-cassandra"},{"line_number":12,"context_line":"        - legacy-monasca-common-maven-build"},{"line_number":13,"context_line":"        - build-monasca-common-docker-base-image"},{"line_number":14,"context_line":"    gate:"},{"line_number":15,"context_line":"      jobs:"},{"line_number":16,"context_line":"        - monasca-tempest-python3-influxdb:"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"c86a3fb5_9275851e","line":13,"in_reply_to":"56485134_ba2a2ce5","updated":"2024-02-07 08:18:14.000000000","message":"Ok","commit_id":"86f83bbfb30dd7ef37e8c7266ac5ded5d12a3f0c"},{"author":{"_account_id":35988,"name":"Hasan Acar","display_name":"Hasan Acar","email":"hasan.acar@tubitak.gov.tr","username":"hasanacar"},"change_message_id":"5c67c92006138f473f235e837f35a6cda35ccc86","unresolved":true,"context_lines":[{"line_number":10,"context_line":"            voting: false"},{"line_number":11,"context_line":"        - monasca-tempest-python3-cassandra"},{"line_number":12,"context_line":"        - legacy-monasca-common-maven-build"},{"line_number":13,"context_line":"        - build-monasca-common-docker-base-image"},{"line_number":14,"context_line":"    gate:"},{"line_number":15,"context_line":"      jobs:"},{"line_number":16,"context_line":"        - monasca-tempest-python3-influxdb:"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"56485134_ba2a2ce5","line":13,"in_reply_to":"8dab97d6_0904f725","updated":"2024-02-06 19:05:59.000000000","message":"It should be checked the docker build before merging. Because after the merging, the docker image is aimed to be published.","commit_id":"86f83bbfb30dd7ef37e8c7266ac5ded5d12a3f0c"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"76f9dd8cee8067c03ba40c5b91224565817441db","unresolved":true,"context_lines":[{"line_number":22,"context_line":"        - legacy-monasca-common-localrepo-upload"},{"line_number":23,"context_line":"    periodic:"},{"line_number":24,"context_line":"      jobs:"},{"line_number":25,"context_line":"        - publish-monasca-common-docker-base-image"},{"line_number":26,"context_line":"    release:"},{"line_number":27,"context_line":"      jobs:"},{"line_number":28,"context_line":"        - publish-monasca-common-docker-base-image"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"1ea9b5c4_aec55cbe","line":25,"updated":"2024-02-06 14:45:15.000000000","message":"do You need to run it daily?","commit_id":"86f83bbfb30dd7ef37e8c7266ac5ded5d12a3f0c"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"98038907eb0f93d916bb78abe719d32482294772","unresolved":true,"context_lines":[{"line_number":22,"context_line":"        - legacy-monasca-common-localrepo-upload"},{"line_number":23,"context_line":"    periodic:"},{"line_number":24,"context_line":"      jobs:"},{"line_number":25,"context_line":"        - publish-monasca-common-docker-base-image"},{"line_number":26,"context_line":"    release:"},{"line_number":27,"context_line":"      jobs:"},{"line_number":28,"context_line":"        - publish-monasca-common-docker-base-image"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"5058f9b7_7a477854","line":25,"in_reply_to":"1ea9b5c4_aec55cbe","updated":"2024-02-06 17:09:09.000000000","message":"The only reason to run this daily would be if you expect infrequent udpates to this repo which will update the docker image in the post queue. That does seem like it might be a bit much considering it would be easy to force an image update by merging a change.\n\nMaybe also consider gating on image builds since you expect them to run post merge.","commit_id":"86f83bbfb30dd7ef37e8c7266ac5ded5d12a3f0c"},{"author":{"_account_id":35988,"name":"Hasan Acar","display_name":"Hasan Acar","email":"hasan.acar@tubitak.gov.tr","username":"hasanacar"},"change_message_id":"5c67c92006138f473f235e837f35a6cda35ccc86","unresolved":true,"context_lines":[{"line_number":22,"context_line":"        - legacy-monasca-common-localrepo-upload"},{"line_number":23,"context_line":"    periodic:"},{"line_number":24,"context_line":"      jobs:"},{"line_number":25,"context_line":"        - publish-monasca-common-docker-base-image"},{"line_number":26,"context_line":"    release:"},{"line_number":27,"context_line":"      jobs:"},{"line_number":28,"context_line":"        - publish-monasca-common-docker-base-image"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"a6582dca_c6174729","line":25,"in_reply_to":"5058f9b7_7a477854","updated":"2024-02-06 19:05:59.000000000","message":"We added the periodic job because it was also used in previous releases. But it may not be necessary. We can remove it.","commit_id":"86f83bbfb30dd7ef37e8c7266ac5ded5d12a3f0c"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"a0e6af75ffc14ba82357415df9312f34c3f1cb51","unresolved":false,"context_lines":[{"line_number":22,"context_line":"        - legacy-monasca-common-localrepo-upload"},{"line_number":23,"context_line":"    periodic:"},{"line_number":24,"context_line":"      jobs:"},{"line_number":25,"context_line":"        - publish-monasca-common-docker-base-image"},{"line_number":26,"context_line":"    release:"},{"line_number":27,"context_line":"      jobs:"},{"line_number":28,"context_line":"        - publish-monasca-common-docker-base-image"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"1e23735f_68191924","line":25,"in_reply_to":"a6582dca_c6174729","updated":"2024-02-07 08:18:14.000000000","message":"Ok","commit_id":"86f83bbfb30dd7ef37e8c7266ac5ded5d12a3f0c"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":35988,"name":"Hasan Acar","display_name":"Hasan Acar","email":"hasan.acar@tubitak.gov.tr","username":"hasanacar"},"change_message_id":"d2b0f828d579b6ce3312a8bc3bdbfab3c56cc06c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f3c91652_d61f6dd1","updated":"2024-02-02 05:52:43.000000000","message":"recheck","commit_id":"2f34308ba8bc58d54a151a64f25aec00f413c72b"},{"author":{"_account_id":20865,"name":"Gökhan","email":"gokhan.isik@tubitak.gov.tr","username":"gokhan.isik"},"change_message_id":"3ece540d02925be949f2c48546a72a06a4a2251f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"841a1808_b21f5f61","updated":"2024-02-05 13:30:48.000000000","message":"Hi slaweq, We need to merge this patch for publish new docker image","commit_id":"a0d1b66f6a20ca5ebefb6312186205ff57146b94"},{"author":{"_account_id":20865,"name":"Gökhan","email":"gokhan.isik@tubitak.gov.tr","username":"gokhan.isik"},"change_message_id":"a6e35b37e2d83176cb8de65e3b8e4b5cdd0cfb72","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"7fbf6ed7_bb19941c","updated":"2024-02-08 05:48:45.000000000","message":"Hi @Ghanshyam @frickler and Elod, we need to merge this patch for fixing monasca-api gates. Can you help us for merging this patch.","commit_id":"d9d071b87c37f7406230d68c4bc4a757fc0832ea"},{"author":{"_account_id":13114,"name":"Hoai-Thu Vuong","email":"thuvh87@gmail.com","username":"thuvh"},"change_message_id":"6fdd659e0ba3a0804291020d14488025d3dcf163","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"2294ce07_1730a8ae","updated":"2024-02-07 06:27:46.000000000","message":"LGTM","commit_id":"d9d071b87c37f7406230d68c4bc4a757fc0832ea"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"d477368852132f606c67a54796eedb60ff62feea","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":16,"id":"d26497b9_1eb7120b","updated":"2024-02-08 10:27:13.000000000","message":"To me it sounds weird that you have been PTL for this project for the last cycle, but never got added as core reviewer. I think the TC should discuss this topic.\n\nThat said, looking at the patch itself, I\u0027m worried by the number of jobs still failing, but maybe this is feasible as a first step regardless. You should however urgently look into making unit tests pass for python3.10 (and also python3.11 which still seems to be missing completly), which are the current required versions for this cycle.","commit_id":"d9d071b87c37f7406230d68c4bc4a757fc0832ea"},{"author":{"_account_id":35988,"name":"Hasan Acar","display_name":"Hasan Acar","email":"hasan.acar@tubitak.gov.tr","username":"hasanacar"},"change_message_id":"92570105e61c4ec117fe66b0d058ee0bbdd347dd","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":16,"id":"f81a2025_5dfc3d1a","in_reply_to":"d26497b9_1eb7120b","updated":"2024-02-08 20:48:18.000000000","message":"As you said, after your message, we successfully passed the Python 3.10 unit tests. Due to our internal work, we were not able to contribute in the previous cycle. But  now, as a team, we are focusing on this project. Additionally, Hoai-Thu Vuong wants to contribute to the Monasca project.","commit_id":"d9d071b87c37f7406230d68c4bc4a757fc0832ea"},{"author":{"_account_id":35988,"name":"Hasan Acar","display_name":"Hasan Acar","email":"hasan.acar@tubitak.gov.tr","username":"hasanacar"},"change_message_id":"6bc2cc7e136a2027088f6aace94a1724b9b84653","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":16,"id":"17367dd3_ae05b36b","in_reply_to":"f81a2025_5dfc3d1a","updated":"2024-02-08 20:55:29.000000000","message":"Most of the monasca-api jobs have been fixed, including non-voting jobs that failed here. It needs to be merged, but first monasca-common should be merged. You can review the changes: \n\nhttps://zuul.opendev.org/t/openstack/build/7ef38171b1844fb197bcd46f89b0a030","commit_id":"d9d071b87c37f7406230d68c4bc4a757fc0832ea"},{"author":{"_account_id":20865,"name":"Gökhan","email":"gokhan.isik@tubitak.gov.tr","username":"gokhan.isik"},"change_message_id":"0c20fac9f08d7bddf92d6cd5b299fe823dc3f557","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"58a79f97_14a278bb","updated":"2024-02-09 07:03:15.000000000","message":"frickler, Hasan was PTL but he was not added as a core reviewer. we resolved all of the unit tests. Non voting jobs depends monasca-api. But monasca api docker image job is also depends monasca-common. There is circular dependency 😊 Firstly we need to merge monasca common patch.","commit_id":"fe601fa51b0e5887f9557dab0765b4e415b418eb"}]}
