| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-20 | |||
| 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: Live migration of instances https://review.openstack.org/270116 | |
| 10:43:57 | openstackgerrit | Radoslav Gerganov proposed openstack/nova master: VMware: Factor out relocate_vm() https://review.openstack.org/270115 | |
| 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 | migi | jhesketh: so proper link should be: https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt?h=liberty-eol | |
| 11:18:50 | jhesketh | migi: the tag is in the correct place though right? it's just that the code needs the branch | |
| 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 | efried | Yeah, getting it, stand by. | |
| 12:49:50 | sdague | I guess it didn't hit my radar yet | |
| 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 | |
| 12:57:45 | sdague | this is the ocata ppa we're working with | |
| 12:57:52 | sdague | it's ubuntu supported | |
| 12:58:51 | kashyap | Ah, okay. For a moment, I assumed PPA in this context meant - "random unsupported packages done by a contributor, which is used only for personal testing" | |
| 13:02:14 | stephenfin | sahid: RE: https://review.openstack.org/#/c/501132/, could you add a summary of the comments with sean-k-mooney to the commit message? After that, it's an easy +2 | |
| 13:05:45 | alex_xu | nova api meeting is running at #openstack-meeting-4 | |
| 13:14:15 | openstackgerrit | Merged openstack/nova-specs master: Spec to remove the hide server address config options https://review.openstack.org/502516 | |
| 13:28:23 | openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for placement conf & requests https://review.openstack.org/492247 | |
| 13:46:48 | openstackgerrit | Merged openstack/nova master: Fix a typo https://review.openstack.org/505062 | |
| 13:49:34 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Split flavors docs into admin and user guides https://review.openstack.org/501342 | |
| 13:49:35 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add documentation for cpu_realtime, cpu_realtime_mask https://review.openstack.org/502056 | |
| 13:49:35 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add documentation for emulator_thread_policy https://review.openstack.org/501721 | |
| 13:50:10 | stephenfin | sdague: Could you take a look at the first of those? gibi has reviewed it a few times but it keeps getting into merge conflicts :( | |
| 13:52:49 | sdague | +2 | |
| 13:52:55 | openstackgerrit | sahid proposed openstack/nova master: libvirt: bandwidth param should be set in guest migrate https://review.openstack.org/497455 | |
| 13:52:56 | sdague | that all looks very reasonable | |
| 13:52:56 | openstackgerrit | sahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/497457 | |
| 13:52:56 | openstackgerrit | sahid proposed openstack/nova master: libvirt: add method to configure migration speed https://review.openstack.org/497456 | |
| 13:53:19 | stephenfin | sdague: Thank you, sir | |
| 13:53:20 | gibi | stephenfin, sdague: I'm also checking that rebase with an intent to approve it | |
| 13:53:34 | stephenfin | gibi: and you too :) | |
| 13:55:33 | stephenfin | sahid: Could you take a look at https://review.openstack.org/#/c/502056/ again? | |
| 13:56:05 | stephenfin | I get that we need to do more configuration that what's there, but there's a lot of stuff to do. I'd prefer to add an admin guide doc for that in the future | |
| 13:56:16 | stephenfin | ...which I should do sometime in the next few weeks | |
| 13:57:04 | mdbooth | 2017-09-14 15:54:39.689 120626 ERROR nova.s2017-09-14 15:54:39.690 120562 ERROR nova.servicegroup.drivers.db [-] Unexpected error while reporting service status | |
| 13:57:16 | gibi | stephenfin: +2+W | |
| 13:57:20 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646 | |
| 13:57:23 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708 | |
| 13:57:25 | stephenfin | gibi: Yay. Thanks :) | |
| 13:57:35 | mdbooth | With the continuation of the first log later in the file without its timestamp | |
| 13:57:43 | mdbooth | Are we not locking in the logger? | |