Earlier  
Posted Nick Remark
#openstack-nova - 2017-09-20
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: 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

Earlier   Later