)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":37855,"name":"Amir Hossein Ahmadi","display_name":"Amir Hossein","email":"sahahmadi96@gmail.com","username":"amir58118"},"change_message_id":"9f4a109de03ac6bbe442930778d7dd1ce2b34d86","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"97a010b9_f8f62ab4","updated":"2025-03-28 08:31:52.000000000","message":"Hello everyone,\n\nI\u0027m pleased to submit a patch that restores support for deploying Swift by reintroducing the Swift code and modernizing its role structure.\n\nI want to highlight that the bulk of the work was accomplished by other community contributors, and I have gathered and integrated their efforts. I made a few modifications based on my years of experience using OpenStack, which I believe improve the overall robustness and usability of Swift support in Kolla-Ansible.\n\nMoving forward, I intend to contribute more actively to the development of Kolla-Ansible. I appreciate your time in reviewing this patch and look forward to your feedback.\n\nBest regards,","commit_id":"a11d82fcfa93019f201c0d9b337e08121b391281"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"d1d6e9078d12e67a141fe858c818b4f74a8e3948","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f526f223_b51a1b9b","updated":"2025-03-28 08:53:53.000000000","message":"Zuul config missing - there need to be swift based CI jobs, you missed that.","commit_id":"a11d82fcfa93019f201c0d9b337e08121b391281"},{"author":{"_account_id":37855,"name":"Amir Hossein Ahmadi","display_name":"Amir Hossein","email":"sahahmadi96@gmail.com","username":"amir58118"},"change_message_id":"d3ab9012aa0fefba6d1e58127cc79806a985d091","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ce383f30_f3133f99","in_reply_to":"205bf0f9_d77e4107","updated":"2025-03-28 09:27:56.000000000","message":"@mnasiadka@gmail.com\n\nThe issue has been resolved, and the updated patch has been submitted. Thanks for your feedback!","commit_id":"a11d82fcfa93019f201c0d9b337e08121b391281"},{"author":{"_account_id":37855,"name":"Amir Hossein Ahmadi","display_name":"Amir Hossein","email":"sahahmadi96@gmail.com","username":"amir58118"},"change_message_id":"055d5b5ff40d37f8cb931b0d695091f1c74d9218","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"56bb084f_6c0c5cb9","in_reply_to":"97a010b9_f8f62ab4","updated":"2025-03-28 08:41:17.000000000","message":"I would like to acknowledge that this patch is the result of a collaborative team effort. Special thanks to:\n\n    Mahrad Hanaforoosh: mahrad.hanaforoosh@gmail.com\n\n    Mostafa Salari: msgm68@gmail.com\n\n    Sara Khosravi: sarah.kh09@gmail.com","commit_id":"a11d82fcfa93019f201c0d9b337e08121b391281"},{"author":{"_account_id":37855,"name":"Amir Hossein Ahmadi","display_name":"Amir Hossein","email":"sahahmadi96@gmail.com","username":"amir58118"},"change_message_id":"6fa1dcbce5a81edcc92468f8d762cb28a8c8711e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"205bf0f9_d77e4107","in_reply_to":"f526f223_b51a1b9b","updated":"2025-03-28 09:08:15.000000000","message":"Thanks for the feedback. Could you please clarify which specific Swift CI jobs you have in mind? Any pointers or examples you could share would be really helpful in guiding me on how to integrate them into the Zuul configuration. Thank you!","commit_id":"a11d82fcfa93019f201c0d9b337e08121b391281"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"1f6530dcb10dbb827af309b7dd3d3b147303650f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fe7c753a_412793c6","updated":"2025-03-28 12:37:54.000000000","message":"I\u0027m sorry if this sounds harsh, but things don\u0027t work this way. You come along as a new contributor with no previous activity in gerrit, don\u0027t even know how to properly format a commit message and then expect us to spend time helping and reviewing this. Previous experience shows that there is a high probability that once this change is merged, you will disappear again and we as a project will in the same state as before. In my opinion you would first have to regularly contribute to the general project for say at least half a year before we could have some confidence that something sustainable will come out of this change.","commit_id":"49d48bd12ba9bce1c3ec3b07af2c62e24526aadc"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"f7bcdd0f4bf2fe94b49a831ef2cb521d2c9ea8f0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5c662ef7_d396b2ad","updated":"2025-03-28 16:45:03.000000000","message":"Let me contribute my two cents and maybe, in the end, explain it a bit more concretely using myself as an example.\n\nThe main point here is that we, as the Kolla team, sent a community email announcing the deprecation and subsequent removal. What happened? Basically nothing — the response was minimal, bordering on zero — so up until now, we decided to actually remove Swift.\n\nAs for me personally, I would have liked Swift to remain part of the ecosystem in Kolla, and by extension in Kolla-Ansible. The problem is that Swift should have proper tests, it should follow the coding style of the other projects, and honestly — every major change was running into issues with Swift, where often it wasn’t even clear if the implementation in Swift was correct (since Swift is a bit different from other services)... because almost no one uses it in Kolla, tests were failing, and so on.\n\nThis also played a role in our company when we were deciding whether to deploy Swift using Kolla — in the end, we wrote our own complete Swift deployment and still use it today. If I tried to push through all the sensible changes into Kolla, I wouldn\u0027t live to see it happen — simply because there are no people who would review it. From my side, I’d rather see a gradual deployment concept, step by step, with all the tests and so on. Not a single commit that’s supposed to do everything and basically throw a wrench in the works again and take us back in time.\n\nPS: After 30 seonds I just found this patch for example completly don\u0027t follow the pattern how services are now restarted in kolla-ansible (configure multiple times, restart once), which is actually showing you don\u0027t follow kolla-ansible development, don\u0027t rebase your patch, not maintaining.","commit_id":"49d48bd12ba9bce1c3ec3b07af2c62e24526aadc"},{"author":{"_account_id":37855,"name":"Amir Hossein Ahmadi","display_name":"Amir Hossein","email":"sahahmadi96@gmail.com","username":"amir58118"},"change_message_id":"baa975f176c53a7f01877801683e06ba48eb42bf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"867b3d24_2cd4aead","in_reply_to":"23872702_ae803dc0","updated":"2025-03-29 10:03:10.000000000","message":"Acknowledged","commit_id":"49d48bd12ba9bce1c3ec3b07af2c62e24526aadc"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"4553d304639dad2022c1054dd5d08c1b691f5f56","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ae35caab_7d0f1530","in_reply_to":"27544b4e_56bcd21b","updated":"2025-03-28 16:06:21.000000000","message":"I never questioned your expertise, I would be happy to see you demonstrate your commitment to contributing by actually doing so and then when the time comes we can maybe revisit this patch. If you need further guidance on how to make useful contributions, best join us in #openstack-kolla on OFTC.","commit_id":"49d48bd12ba9bce1c3ec3b07af2c62e24526aadc"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"483260d65ddef5d29a1ffb42709d107aa91d3577","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c022a408_8ca9561f","in_reply_to":"5c662ef7_d396b2ad","updated":"2025-03-28 19:29:47.000000000","message":"I agree with Michal, in addition to that I have one major issue with Swift in Kolla/Kolla-Ansible.\njust like Ceph (which we stopped to be a deployer for), Swift also needs to be performant, scalable and so on. I can’t see it being such with us using the same keepalived/haproxy as for the OpenStack API (amingst other flaws of the current implementation) - and I would prefer us not trying to be in the storage deployer game anymore.\nI’m happy to support external Swift - but bringing back Swift in K-A needs extra care, which we as a small team can’t do.\nif that contribution would be discussed on the meeting/PTG and would come from an invested contributor/company - that would be a different situation - but still I would not be convinced that Swift needs to be managed by Ansible - because I feel it can’t do anything beyond a dev/small production deployment that is not performant enough for modern use cases.","commit_id":"49d48bd12ba9bce1c3ec3b07af2c62e24526aadc"},{"author":{"_account_id":37509,"name":"Mostafa Salari","display_name":"Mosal","email":"msgm68@gmail.com","username":"msgm68"},"change_message_id":"d0f288ab710cc03487ff3a313e1bb2567f8a8567","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e1a89001_ef1818c1","in_reply_to":"867b3d24_2cd4aead","updated":"2025-04-13 12:27:55.000000000","message":"@mnasiadka@gmail.com\nYou said \"I would not be convinced that Swift needs to be managed by Ansible - because I feel it can’t do anything beyond a dev/small production deployment that is not performant enough for modern use cases.\"  I didnt understand what you mean by \"it\"? please tell more clearly. Do you think using Ansible is not efficient enough for managing Swift deployment?","commit_id":"49d48bd12ba9bce1c3ec3b07af2c62e24526aadc"},{"author":{"_account_id":37855,"name":"Amir Hossein Ahmadi","display_name":"Amir Hossein","email":"sahahmadi96@gmail.com","username":"amir58118"},"change_message_id":"cccdbe9ce352655e08f85b9636acb5f99eafeb9e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1844a05a_9e2deeb8","in_reply_to":"ae35caab_7d0f1530","updated":"2025-03-29 08:11:32.000000000","message":"I appreciate your guidance. I am already a member of the #openstack-kolla channel on OFTC, and I look forward to further engaging with the community and collaborating more effectively.","commit_id":"49d48bd12ba9bce1c3ec3b07af2c62e24526aadc"},{"author":{"_account_id":37855,"name":"Amir Hossein Ahmadi","display_name":"Amir Hossein","email":"sahahmadi96@gmail.com","username":"amir58118"},"change_message_id":"128ff98c05b21af7c58e7178bc84e25180b21d81","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"23872702_ae803dc0","in_reply_to":"c022a408_8ca9561f","updated":"2025-03-29 08:44:46.000000000","message":"@michal.arbet@ultimum.io\n@mnasiadka@gmail.com\n\nDear Michal Arbet and Michal Nasiadka,\n\nThank you very much for your detailed feedback and for sharing your insights. We truly appreciate your constructive comments.\n\nI would like to clarify that the reconfiguration issue you observed was due to an oversight in saving the file in the IDE. In Patch 3, I have double-checked and ensured that the Swift role is fully aligned with the new Kolla-Ansible standards, including proper reconfiguration handling.\n\nPlease rest assured that our commitment to this project goes well beyond this patch. Our team and company are eager to engage more deeply with the Kolla community and take on the development of Swift in Kolla as dedicated contributors. We are committed to working closely with you and the team to ensure that any future contributions meet the project\u0027s high standards.\n\nThank you again for your valuable guidance.\n\nBest regards,\nAmir Hossein (sahahmadi96@gmail.com)","commit_id":"49d48bd12ba9bce1c3ec3b07af2c62e24526aadc"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"b46bdac472fe29d6f32f6fd9aeaffe60b3f12fdc","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2b8e583d_c1200fa3","in_reply_to":"e1a89001_ef1818c1","updated":"2025-04-15 03:31:45.000000000","message":"I mean that Kolla-Ansible based deployment utilising the API services haproxy for pushing big volumes of Swift data is not something I can willingly support and it would be better if there would be some external tool for this and Kolla-Andible would only have support for external Swift (just like we only support external Ceph today).","commit_id":"49d48bd12ba9bce1c3ec3b07af2c62e24526aadc"},{"author":{"_account_id":37855,"name":"Amir Hossein Ahmadi","display_name":"Amir Hossein","email":"sahahmadi96@gmail.com","username":"amir58118"},"change_message_id":"37ca5c3dbb14860f41057c7d132b1b862be06d6a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"27544b4e_56bcd21b","in_reply_to":"fe7c753a_412793c6","updated":"2025-03-28 13:58:10.000000000","message":"@frickler@offenerstapel.de\n\nDear Jens Harbott,\n\nThank you for your response. I understand your concerns regarding sustainability and long-term contribution to the project.\n\nI’d like to clarify that I have been working as a cloud product developer with a focus on Swift for nearly five years at Burna (https://burna.ir/).\n\nAdditionally, our team has developed and commercially deployed a Swift-based product, which we offer to our customers. A free version of this product is available at:\nhttps://zdrive.ir/\n\nIf needed, we would be happy to demonstrate our expertise in OpenStack and Kolla-Ansible through an online meeting.\n\nMoreover, we are committed to contributing continuously to OpenStack projects moving forward.\n\nLooking forward to your feedback.","commit_id":"49d48bd12ba9bce1c3ec3b07af2c62e24526aadc"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"4db37b72376e1625fe769d99244de1a2dd39930a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e190a472_724db745","updated":"2025-05-26 14:55:01.000000000","message":"where are tests for swift ? I can\u0027t see any ..","commit_id":"7da13bdcc9aeeaaee04a0bd14b8a767a32a85ce2"},{"author":{"_account_id":37855,"name":"Amir Hossein Ahmadi","display_name":"Amir Hossein","email":"sahahmadi96@gmail.com","username":"amir58118"},"change_message_id":"685a9e1d7f257fe0fce10b1ce0541873344797bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ea3d28f0_5b319b0c","in_reply_to":"e190a472_724db745","updated":"2025-05-31 07:09:41.000000000","message":"The tests and Zuul jobs are included in patchset 3.\n\nHowever, we\u0027re currently preparing a new patch for Kolla Ansible, which might be ready later next week.","commit_id":"7da13bdcc9aeeaaee04a0bd14b8a767a32a85ce2"}]}
