| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 08:23 |
|---|---|---|
| zigo | There's an issue with Magnum and the latest oslo.db: | 08:41 |
| zigo | ImportError: cannot import name 'String' from 'oslo_db.sqlalchemy.types' (/usr/lib/python3/dist-packages/oslo_db/sqlalchemy/types.py) | 08:41 |
| zigo | This worked with oslo.db 12.3.1, but fails in 14.1.0, therefore, I cannot run unit tests (failure during test discovery). | 08:42 |
| zigo | How can this be fixed? Is someone working on this? | 08:42 |
| zigo | Is there a patch for it available ? | 08:42 |
| zigo | 'Support for the MySQL NDB Cluster storage engine has been ' | 08:47 |
| zigo | 'deprecated and will be removed in a future release. Use the ' | 08:47 |
| zigo | 'standard String type from sqlalchemy.types' | 08:47 |
| zigo | This can be read in oslo_db/sqlalchemy/types.py in oslo.db 12.3.1 ... | 08:47 |
| zigo | jakeyip: I've been packaging OpenStack since 2011 in Debian, what I'm doing is actually trying to upload Magnum to Debian Unstable ! :) | 08:57 |
| jakeyip | will a new minor version help? | 08:58 |
| zigo | jakeyip: As long as I have the patch, I can add it to debian/patches, that's ok, thanks ! | 08:59 |
| jakeyip | zigo: so we already backported to Bobcat. I am not good at packaging, will us cutting a minor release ver make it easier for you? | 09:01 |
| zigo | jakeyip: As I just wrote, I don't need a new release, I am just adding the patch to my packaging, it's currently running unit tests (at package build time), and the issue is fixed on my side, so no, I don't need a new minor release, thanks. | 09:02 |
| jakeyip | ok | 09:02 |
| zigo | Though probably it'd be nicer for other distros that may ask the same question to you ... | 09:02 |
| jakeyip | ok also :) | 09:02 |
| zigo | All unit tests ran successfully ! :) | 09:03 |
| zigo | magnum_17.0.0-1_source.changes uploaded ... :P | 09:04 |
| jakeyip | thanks for packaging :) | 09:05 |
| mnasiadka | it has been merged in Bobcat yesterday | 09:06 |
| * zigo is nearly done packaging all of Bobcat ... | 09:08 | |
| jakeyip | mnasiadka: maybe packagers work off tags | 09:09 |
| mnasiadka | jakeyip: then we need a new release ;) | 09:09 |
| jakeyip | https://review.opendev.org/c/openstack/releases/+/897404 | 09:09 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 09:10 |
| jakeyip | mnasiadka: already up :P | 09:10 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 09:21 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 10:34 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 10:48 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 10:56 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 11:38 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 12:47 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 14:30 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!