| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-20 | |||
| 00:23:24 | mriedem | yeah | |
| 00:23:25 | dansmith | mriedem: also, I think the live migration job is fairly fscked | |
| 00:23:33 | mriedem | yup, see me bring that up earlier? | |
| 00:23:42 | dansmith | oh no, sorry | |
| 00:23:46 | mriedem | top fail right now http://status.openstack.org/elastic-recheck/#1718295 | |
| 00:23:56 | dansmith | cool, sorry I didn't see it | |
| 00:24:15 | mriedem | i'll send something to the dev list | |
| 00:26:41 | mriedem | oh shit | |
| 00:26:52 | mriedem | https://review.openstack.org/#/c/501224/ | |
| 00:27:00 | mriedem | i bet $7.75 it's that | |
| 00:27:48 | dansmith | hoo boy | |
| 00:30:19 | mriedem | revert in progress | |
| 00:30:29 | mriedem | although (1) gerrit is hung and (2) no devstack cores around | |
| 00:30:46 | mriedem | https://review.openstack.org/#/c/505446/ | |
| 00:31:02 | mriedem | nice, new gerrit includes the owner and reviewers from the change that you revert | |
| 00:39:28 | mriedem | and with that, i'm going to slurp night time cold medicine tea and pass out | |
| 00:41:11 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: fix nova accepting invalid availability zone name with ':' https://review.openstack.org/490722 | |
| 01:10:47 | openstackgerrit | Merged openstack/nova stable/ocata: libvirt: Fix getting a wrong guest object https://review.openstack.org/498478 | |
| 01:15:42 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646 | |
| 01:15:45 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708 | |
| 01:16:02 | jichen | alex_xu: hi , have time to take a look at https://review.openstack.org/#/c/485121/ ? | |
| 01:19:15 | openstackgerrit | Dan Smith proposed openstack/nova master: Remove legacy fault-loading routines https://review.openstack.org/505456 | |
| 02:08:38 | openstackgerrit | octopuszhang proposed openstack/nova master: remove re-auth logic for ironic client wrapper https://review.openstack.org/502382 | |
| 02:23:59 | openstackgerrit | jichenjc proposed openstack/nova master: Fix warning on {'cell_id': 1} is an invalid UUID https://review.openstack.org/505471 | |
| 02:37:25 | openstackgerrit | jichenjc proposed openstack/nova master: WIP: conf: remove deprecated 'project_id_regex' opt https://review.openstack.org/501057 | |
| 02:41:01 | openstackgerrit | Dan Smith proposed openstack/nova master: Remove legacy fault-loading routines https://review.openstack.org/505456 | |
| 03:22:04 | jichen | alex_xu: shuai ge, have time to take a look at https://review.openstack.org/#/c/485121/ ? | |
| 03:36:03 | alex_xu | jichen: yea, I will check that after the meeting... | |
| 03:36:27 | jichen | alex_xu: ok, perfect, thanks | |
| 03:36:36 | jichen | by the way, which meeting? nova or nova-api? | |
| 03:37:56 | alex_xu | jichen: intel internal meeting... | |
| 03:38:21 | jichen | alex_xu: ok, thanks a lot | |
| 03:38:31 | alex_xu | jichen: np | |
| 03:48:50 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: fix nova accepting invalid availability zone name with ':' https://review.openstack.org/490722 | |
| 04:58:30 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif stable/newton: Updated from global requirements https://review.openstack.org/373293 | |
| 06:50:50 | openstackgerrit | Alex Xu proposed openstack/nova-specs master: Add trait support in the allocation candidates API https://review.openstack.org/497713 | |
| 06:59:18 | pratapagoutham | #openstack-tempest | |
| 08:28:50 | openstackgerrit | Elod Illes proposed openstack/nova master: Add instance.interface_attach notification https://review.openstack.org/503089 | |
| 08:45:43 | openstackgerrit | Merged openstack/nova master: Enable custom certificates for keystone communication https://review.openstack.org/485121 | |
| 08:54:22 | openstackgerrit | jichenjc proposed openstack/nova stable/pike: Enable custom certificates for keystone communication https://review.openstack.org/505546 | |
| 09:04:25 | openstackgerrit | jichenjc proposed openstack/nova master: Remove 400 as expected error https://review.openstack.org/505485 | |
| 09:17:49 | openstackgerrit | jichenjc proposed openstack/nova master: Remove 400 as expected error https://review.openstack.org/505485 | |
| 09:18:07 | abhishekk | sean-k-mooney: hi you around? | |
| 10:43:57 | openstackgerrit | Radoslav Gerganov proposed openstack/nova master: VMware: Factor out relocate_vm() https://review.openstack.org/270115 | |
| 10:43:57 | openstackgerrit | Radoslav Gerganov proposed openstack/nova master: VMware: Live migration of instances https://review.openstack.org/270116 | |
| 11:01:15 | openstackgerrit | garyk proposed openstack/nova master: VMware: ensure that live migration attaches to correct interface https://review.openstack.org/478797 | |
| 11:14:46 | migi | hi, I am facing issue with the rename of branch stable/liberty to liberty-eol tag. The liberty-eol contains tox.ini with wrong pointer to the upper-constraints.txt which I want to fix, but I do not know how to sent this for review cause liberty-eol it's tag and not a branch so can't use gerrit | |
| 11:14:59 | migi | any hint will be appriciated | |
| 11:15:46 | migi | jhesketh: ^^, since you created tags | |
| 11:16:45 | jhesketh | migi: firstly is the tag in the correct position? ie was the branch broken before the EOL? | |
| 11:18:09 | migi | jhesketh: see here: https://github.com/openstack/nova/blob/liberty-eol/tox.ini#L12 | |
| 11:18:20 | migi | jhesketh: before it was ok | |
| 11:18:40 | jhesketh | migi: ah, I see the issue | |
| 11:18:50 | jhesketh | migi: the tag is in the correct place though right? it's just that the code needs the branch | |
| 11:18:50 | migi | jhesketh: so proper link should be: https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt?h=liberty-eol | |
| 11:19:03 | migi | jhesketh: tag is ok | |
| 11:19:11 | jhesketh | okay, just checking :-) | |
| 11:19:38 | robcresswell | Could anyone from Nova shed any light on this change https://review.openstack.org/#/c/501564/ ? It sounds like an old microversion change, but I'm unsure what the correct path here is. Does novaclient default to 2.1 if no version is supplied? | |
| 11:19:43 | migi | jhesketh: same will happen with kilo/mitaka | |
| 11:20:07 | jhesketh | right | |
| 11:20:13 | jhesketh | so the point of EOL'ing is to not make any more changes to a release. And tags typically shouldn't be moved | |
| 11:20:30 | jhesketh | so while there are a few technical ways of fixing it, I think this is more a discussion for the nova and stable team to have | |
| 11:20:43 | migi | jhesketh: actually kilo is OK as it has no such pointer, just mitaka | |
| 11:21:31 | migi | jhesketh: meaning who exactly? | |
| 11:22:08 | jhesketh | migi: my suggestion would be to raise it on the dev mailing list either in the EOL thread or as a new one | |
| 11:22:27 | migi | jhesketh: yep, will do, thanks | |
| 11:22:48 | jhesketh | there are pros and cons to these kind of changes and there is an argument to be had whether it's something to allow from the point of view of discontinued development | |
| 11:23:26 | jhesketh | my personal thought is that if you needed to continue working on an EOL release that you could create a fork of the project and host the branch yourself somewhere | |
| 11:23:42 | rgerganov | gibi, stephenfin: could you please take a look at https://review.openstack.org/#/c/450636/ ? This patch finishes the implementation of console log for VMware. Thanks! | |
| 11:23:55 | jhesketh | but clearly that has huge implications and is a massive amount of work given the number of repos etc | |
| 11:46:48 | openstackgerrit | Elod Illes proposed openstack/nova master: Add instance.interface_attach notification https://review.openstack.org/503089 | |
| 12:07:41 | openstackgerrit | Bob Ball proposed openstack/nova master: XenAPI: Unit tests must mock os_xenapi calls https://review.openstack.org/499573 | |
| 12:13:48 | efried | alex_xu Would you mind if I posted an edit on https://review.openstack.org/#/c/497713/ just for spelling/grammar? | |
| 12:27:00 | openstackgerrit | Moshe Levi proposed openstack/nova master: don't overwrite binding-profile https://review.openstack.org/505613 | |
| 12:43:23 | sdague | something seems to have gone pretty sideways on livemigration jobs http://logs.openstack.org/55/502155/5/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/52b53e8/logs/screen-n-api.txt.gz#_Sep_20_08_32_57_608522 | |
| 12:48:33 | efried | sdague - mriedem submitted a fix in devstack last night. | |
| 12:49:40 | sdague | efried: url? | |
| 12:49:50 | sdague | I guess it didn't hit my radar yet | |
| 12:49:50 | efried | Yeah, getting it, stand by. | |
| 12:50:17 | efried | sdague https://bugs.launchpad.net/nova/+bug/1718295 | |
| 12:50:18 | openstack | Launchpad bug 1718295 in devstack "Unexpected exception in API method: MigrationError_Remote: Migration error: Disk info file is invalid: qemu-img failed to execute - Failed to get shared "write" lock\nIs another process using the image?" [High,Fix released] - Assigned to Matt Riedemann (mriedem) | |
| 12:50:52 | sdague | oh, the pike ppa went sideways? | |
| 12:51:17 | efried | a>ppa<rently | |
| 12:51:20 | sdague | good to know | |
| 12:51:25 | sdague | "P | |
| 12:51:27 | sdague | "P | |
| 12:51:39 | sdague | :P (damn kept missing keys) | |
| 12:51:48 | efried | I knew what you meant. | |
| 12:52:38 | efried | sdague Looks like you juuuust missed the fix with that patch :) | |
| 12:52:55 | efried | cdent gotcher back | |
| 12:53:49 | cdent | efried: eh? wat? zzzzzzzzz | |
| 12:54:16 | efried | cdent Your timely recheck on https://review.openstack.org/#/c/502155/ a couple hours ago. | |
| 12:54:39 | sdague | efried: apparently, well, at least it's fixed now | |
| 12:54:58 | sdague | I wish the ppa game wasn't such a coin flip | |
| 12:55:41 | efried | sdague Reverted, anyway. It's possible an actual fix is in the works, based on jamespage comments in the bug. | |
| 12:56:35 | kashyap | Can someon remind, in what context are we using Ubuntu PPA (Personal Package Archives) | |
| 12:57:02 | sdague | kashyap: to get newer libvirt on xenial | |
| 12:57:03 | cdent | efried: thanks, I’m still relocating my landmarks | |
| 12:57:31 | kashyap | sdague: Is PPA considered 'official' - as in upstream Ubuntu maintainers maintain it? | |
| 12:57:37 | sdague | kashyap: yes | |