| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-20 | |||
| 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? | |
| 13:58:22 | sahid | stephenfin: the admin is going to configure the flavor, it seems reasonable to me to add a note saying what i do have mentioned on the review | |
| 13:58:42 | mdbooth | Unfortunately, it also means the logs are not correctly sorted :/ | |
| 13:58:58 | openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: [WIP] CLI for aggregates https://review.openstack.org/505643 | |
| 13:58:58 | stephenfin | sahid: Right, but vcpu_pin_set is not a flavor property. I've mentioned pinned CPUs because it is | |
| 13:59:03 | mdbooth | This is not conducive to merge sorting | |
| 13:59:37 | stephenfin | You also need to configure things like isolate the CPUs and use a properly configured guest, but I don't mention those there because they're nothing to do with configuring flavor properties | |
| 13:59:55 | sahid | isolate the CPUs? | |
| 14:00:13 | stephenfin | sahid: From the host? | |
| 14:00:29 | stephenfin | Whatever it is that replace the isolcpus boot parameter | |
| 14:00:31 | sahid | that is not related to Nova, i'm talking about an option which is related to Nova, that is why i think a small note is important | |
| 14:01:25 | stephenfin | and vcpu_pin_set is not related to flavors. This is an flavor (extra_spec) overview doc | |
| 14:01:35 | sahid | if you don't mention that an admin could just think that after to have configured the host, enabling cpu_realtime=yes in flavor is enough | |
| 14:02:41 | stephenfin | They could also boot an standard Linux guest kernel or forget to configure isolcpus | |
| 14:03:00 | stephenfin | The point is that this is just an overview of the flavor extra specs available and the interactions between them | |
| 14:03:18 | stephenfin | We should add a real-time doc but it should be separate, like this: | |
| 14:03:55 | stephenfin | https://docs.openstack.org/nova/latest/admin/cpu-topologies.html | |
| 14:04:18 | sahid | stephenfin: ok, i just gave to you my point, i would have added that note but feel free to not mention it | |
| 14:05:07 | stephenfin | sahid: Yup, and I appreciate it :) I'm countering that I don't think it's necessary here, and would make more sense in the upcoming larger doc | |
| 14:05:16 | stephenfin | ...where I'll definitely mention it | |
| 14:06:02 | mriedem | mdbooth: it's likely a problem in the customers log config | |
| 14:06:13 | mriedem | mdbooth: i saw something like that with our new super conductor logs in devstack, the fix for that was in devstack https://review.openstack.org/#/c/497944/1/lib/nova | |
| 14:06:22 | mdbooth | mriedem: Looking. | |
| 14:06:22 | mriedem | but you should checkout what devstack does for log config | |