| opendevreview | Merged openstack/glance master: Remove unused run_sql_cmd https://review.opendev.org/c/openstack/glance/+/915791 | 00:48 |
|---|---|---|
| opendevreview | OpenStack Proposal Bot proposed openstack/glance master: Imported Translations from Zanata https://review.opendev.org/c/openstack/glance/+/915799 | 05:12 |
| opendevreview | Pranali Deore proposed openstack/glance master: Change DB migration constant to 2024_2 https://review.opendev.org/c/openstack/glance/+/916162 | 05:49 |
| tkajinam | abhishekk, hi. do you have a min to review/merge this https://review.opendev.org/c/openstack/glance-tempest-plugin/+/906238 ? | 06:57 |
| tkajinam | to reduce zuul config warnings | 06:57 |
| abhishekk | tkajinam, I will do it in short time | 06:57 |
| tkajinam | abhishekk, thx ! | 06:58 |
| abhishekk | tkajinam, done, thank you! | 07:31 |
| opendevreview | Merged openstack/glance-tempest-plugin master: Replace deprecated perl-style regex https://review.opendev.org/c/openstack/glance-tempest-plugin/+/906238 | 08:17 |
| opendevreview | Pranali Deore proposed openstack/glance-specs master: [Spec Lite] Deprecate metadata_encryption_key https://review.opendev.org/c/openstack/glance-specs/+/916178 | 09:39 |
| *** jph5 is now known as jph | 10:15 | |
| opendevreview | Pranali Deore proposed openstack/glance-specs master: Dalmatian project priorities https://review.opendev.org/c/openstack/glance-specs/+/916279 | 11:50 |
| *** jph6 is now known as jph | 12:45 | |
| pdeore | whoami-rajat, Is it possible for you to join today's glance meeting at 1400 UTC? we have some questions about how nfs store is handled in cider since we are going to introduce it as a substore for FS in glance as well | 13:22 |
| pdeore | abhishekk, rosmaita, dansmith, croelandt, mrjoshi glance weekly meeting in 10 minutes at #openstack-meeting | 13:50 |
| mrjoshi | thanks! | 13:51 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: DNM: Run py312 unit tests with modified taskflow https://review.opendev.org/c/openstack/glance/+/916301 | 13:56 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: DNM: Run py312 unit tests with modified taskflow https://review.opendev.org/c/openstack/glance/+/916301 | 14:03 |
| abhishekk | dansmith, those tests were skipped for functional-py310 upstream | 14:41 |
| abhishekk | 2024-04-18 06:04:55.353138 | ubuntu-jammy | {4} glance.tests.functional.test_glance_manage.TestGlanceManage.test_sync ... SKIPPED: test requires exe: sqlite3 | 14:41 |
| dansmith | abhishekk: mmhmm :) | 14:42 |
| abhishekk | and both are using oslo.db 15.0.0 | 14:42 |
| *** pdeore is now known as pdeore|afk | 14:43 | |
| abhishekk | depends_on_exe decorator | 14:47 |
| abhishekk | dansmith, https://paste.opendev.org/show/bPJ4HpGSRg5CxMgfVA3r/ these changes helps functional py311 test to pass | 16:32 |
| dansmith | ack, cool | 16:33 |
| abhishekk | could you please confirm it works for you as well | 16:34 |
| dansmith | hmm, nope: | 16:37 |
| dansmith | https://paste.opendev.org/show/b0skT3pQcwVi3IcfVPku/ | 16:37 |
| abhishekk | 1 test is still failing | 16:38 |
| abhishekk | test_check | 16:38 |
| dansmith | all 6 still fail | 16:38 |
| dansmith | - Failed: 6 | 16:38 |
| abhishekk | for me Ran 6 tests in 12.598s | 16:38 |
| abhishekk | FAILED (failures=1) | 16:38 |
| abhishekk | and that is because same change might be needed in check_db | 16:39 |
| abhishekk | yes, same change needed in _check_db function as well | 16:40 |
| abhishekk | https://paste.opendev.org/show/bkV5YoqYIGRdHevR6iOh/ | 16:42 |
| dansmith | okay sorry, I'm not sure what you' | 16:43 |
| dansmith | are saying, but I'm in another meeting right now and can' really focus | 16:43 |
| abhishekk | aah, sorry | 16:44 |
| tkajinam | ugh. I thought I fixed it but https://review.opendev.org/c/openstack/glance/+/906976 was incomplete it seems | 16:44 |
| abhishekk | let me push wip patch | 16:44 |
| abhishekk | yeah, it additionally needs changes in above file | 16:44 |
| tkajinam | we merged https://review.opendev.org/c/openstack/oslo.db/+/798141 which removed deprecated options so that needs to be fixed... | 16:44 |
| abhishekk | tkajinam, can you push the patch? | 16:44 |
| tkajinam | abhishekk, sure. give me a sec | 16:45 |
| abhishekk | no worries, thank you!! | 16:45 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Replace remaining usage of [DEFAULT] sql_connection https://review.opendev.org/c/openstack/glance/+/916321 | 16:50 |
| abhishekk | thank you! | 16:50 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Replace remaining usage of [DEFAULT] sql_connection https://review.opendev.org/c/openstack/glance/+/916321 | 16:51 |
| tkajinam | fixed a typo in the commit message. I also tested this locally. | 16:52 |
| tkajinam | this needs to be backported to stable/2024.1 because oslo.db 15.0.0 is in u-c for 2024.1 | 16:53 |
| abhishekk | yes | 16:56 |
| opendevreview | Takashi Kajinami proposed openstack/glance-tempest-plugin master: Update stable branch testing https://review.opendev.org/c/openstack/glance-tempest-plugin/+/916332 | 19:20 |
| *** jph1 is now known as jph | 23:37 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!