| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-25 | |||
| 10:27:18 | openstack | Launchpad bug 1745340 in OpenStack Compute (nova) "Nova assumes that USB Host is present" [Undecided,New] | |
| 10:28:52 | stephenfin | More complete? | |
| 10:29:43 | hrw | stephenfin: more useful to anyone who would look at issue | |
| 10:29:53 | hrw | if anyone at all ;D | |
| 10:30:07 | stephenfin | I think you've pretty much got it | |
| 10:30:39 | stephenfin | I assume the solution would be to check if we have a USB hub and either (a) add it if not or (b) refuse to start with a USB tablet? | |
| 10:31:04 | stephenfin | *refuse to start with a USB tablet if the platform doesn't support the USB hub | |
| 10:31:25 | stephenfin | hrw: I can point you to the code that needs modification, if you fancy the challenge? ;) | |
| 10:31:44 | hrw | stephenfin: please do. in a bug comment | |
| 10:32:08 | hrw | stephenfin: it may end on my or kevinz's plate or maybe we assign it to someone else at Linaro | |
| 10:32:36 | hrw | stephenfin: imho the proper way would be to add usb-host device if not present | |
| 10:33:02 | hrw | as the goal is to have graphical console in Rocky (as it is too late for Queens) | |
| 10:34:21 | hrw | during ptg I have to track nova sticker for my laptop ;D to put it next to kolla one | |
| 10:36:07 | stephenfin | hrw: https://bugs.launchpad.net/nova/+bug/1745340/comments/3 | |
| 10:36:08 | openstack | Launchpad bug 1745340 in OpenStack Compute (nova) "Nova assumes that USB Host is present" [Undecided,New] | |
| 10:37:26 | hrw | stephenfin: exactly where I was looking at | |
| 10:37:40 | hrw | stephenfin: probably some code in nova.virt.libvirt.* too | |
| 10:37:52 | hrw | to 'plug' host | |
| 10:38:14 | hrw | will later look into | |
| 10:38:56 | Spazmotic | Come on tempest... you running slow but i have faith in you | |
| 11:46:01 | stephenfin | sean-k-mooney: Think I've reviewed all of bp/enable-sriov-nic-features now :) | |
| 11:46:13 | stephenfin | Lemme know if you have questions | |
| 11:55:56 | Spazmotic | hey huanxie you still around | |
| 11:56:51 | Spazmotic | or jianghuaw can you explain to me what XS expects of these VDI MAPS on a live migration? | |
| 12:06:12 | jianghuaw | Spazmotic, I've talked with Huan. You can chat with her on tomorrow. | |
| 12:06:29 | Spazmotic | Sounds good, thanks, i'll stop by tomorrow :) | |
| 12:06:48 | lyarwood | mriedem: iirc live snapshot was only for local images and not volumes right? Even then we skip it for rbd? | |
| 12:12:55 | Spazmotic | IBM PowerKVM CI failing constantly | |
| 12:17:10 | Spazmotic | Looks like past 4 or 5 hours all failures on its tempest-dsvm-full-xenialchecks | |
| 12:17:37 | sean-k-mooney | stephenfin: just in so ill take a look at them now | |
| 12:27:02 | owalsh | stephenfin: is vencrypt_ca_certs required when vencrypt is enabled? doesn't looks like it default to the system ca certs so I'm guessing yea | |
| 12:38:49 | openstackgerrit | Brianna Poulos proposed openstack/nova master: Reduce complexity of _from_db_object https://review.openstack.org/537728 | |
| 12:39:02 | openstackgerrit | Gaudenz Steinlin proposed openstack/nova master: Ignore some updates from virt driver https://review.openstack.org/523006 | |
| 13:20:48 | ameeda | jaypipes: Sir are you around ? | |
| 13:21:51 | openstackgerrit | Brianna Poulos proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408 | |
| 13:21:51 | openstackgerrit | Brianna Poulos proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949 | |
| 13:21:52 | openstackgerrit | Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | |
| 13:21:52 | openstackgerrit | Brianna Poulos proposed openstack/nova master: Add trusted_certs to instance_extra https://review.openstack.org/537897 | |
| 13:41:11 | stephenfin | owalsh: I think so, yup. I should probably encode that | |
| 13:51:39 | Spazmotic | IBM CI what is wrong with you :( http://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/index.html | |
| 13:52:06 | mmedvede | Spazmotic: it should be fixed now, at least the original problem that was causing earlier failures | |
| 13:52:16 | Spazmotic | Ahhh thanks mmedvede | |
| 13:52:19 | Spazmotic | I'll give it a new recheck hehe | |
| 13:53:13 | mriedem | Roamer`: i won't be +Wing your change because we require 2 +2s | |
| 13:58:26 | mriedem | gibi_: we should probably consider bumping the nova functional job timeouts as a change on top of your patch to move them into the nova tree | |
| 14:00:00 | hrw | stephenfin: turns out that one change to libvirt makes life easier. | |
| 14:00:20 | hrw | stephenfin: will wait for libvirt devs to respond before testing it further | |
| 14:00:29 | mriedem | nova meeting starting in #openstack-meeting | |
| 14:00:46 | gibi_ | mriedem: I can do that | |
| 14:01:34 | gibi_ | mriedem: is it just the functional or also the functional-py35 job that times out frequently? | |
| 14:01:40 | mriedem | both | |
| 14:01:45 | gibi_ | mriedem: OK | |
| 14:04:06 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Don't filter out sibling sets with one core https://review.openstack.org/537361 | |
| 14:04:06 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Ensure emulator threads are always calculated https://review.openstack.org/537363 | |
| 14:04:07 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Always pass 'NUMACell.siblings' to _pack_instance_onto_cores' https://review.openstack.org/537364 | |
| 14:13:37 | stephenfin | mriedem: Could you take a look at this after the meeting? https://review.openstack.org/#/c/527182/ It's blocking https://review.openstack.org/#/c/526329/ | |
| 14:14:04 | stephenfin | Maybe gibi_ too, if you have the time ^ | |
| 14:15:40 | mriedem | sure | |
| 14:15:54 | mriedem | stephenfin: trade you https://review.openstack.org/#/c/140733/ | |
| 14:16:22 | stephenfin | Sounds good | |
| 14:16:46 | efried | holy crap, that one's >3 years old | |
| 14:19:26 | mriedem | b/c the cinder driver for it was removed awhile back | |
| 14:20:06 | Roamer` | stephenfin, efried, we (StorPool) and, actually, also Canonical would be very, very happy for if this could go in | |
| 14:20:09 | Roamer` | sorry... | |
| 14:20:18 | Roamer` | I honestly hadn't realized that it needed a second +2 vote, my bad, my very bad :( | |
| 14:21:03 | Roamer` | and yeah, there's history... we tried submitting a driver too late for Juno, then it went into Kilo, then we messed up and it was removed, and now we're trying to get it back | |
| 14:21:07 | Roamer` | aaaaand I'll shut up now | |
| 14:22:45 | kashyap | mriedem: Heya, so...I wrote this: https://review.openstack.org/#/c/523958/18. I agree with you on most / all topics. Besides this. Wonder what you think of my POV there. | |
| 14:26:19 | openstackgerrit | sahid proposed openstack/nova master: libvirt: disconnect volume from host during detach https://review.openstack.org/515008 | |
| 14:26:43 | gibi_ | stephenfin: lgtm! +2 | |
| 14:27:18 | efried | gibi_ But don't +W it, cause the gate's broken. Or whatever. | |
| 14:28:09 | gibi_ | efried: I was the first +2 | |
| 14:35:21 | gibi | stephenfin: could you look at these? https://review.openstack.org/#/q/topic:bp/versioned-notification-transformation-queens+status:open+label:Code-Review%253D%252B2 to easy notification transformation before FF :) | |
| 14:35:36 | mriedem | gibi: stephenfin: i'll look at https://review.openstack.org/#/c/465081/ again, | |
| 14:35:39 | mriedem | since i was -1 on it | |
| 14:36:02 | gibi | mriedem: thanks | |
| 14:36:22 | stephenfin | gibi: Sure, it's been on my list | |
| 14:36:28 | takashin | mriedem: Thank you. | |
| 14:36:31 | stephenfin | Along with some of gmann's API stuff | |
| 14:36:53 | gibi | stephenfin: thanks | |
| 14:39:19 | ameeda | hello, how to run pep8 for one class ? | |
| 14:39:46 | ameeda | tox -e pep8 -- -n nova/tests/unit/compute/test_compute_api.py , doesn't work | |
| 14:41:07 | maciejjozefczyk | ameeda: tox -e pep8 nova/virt/libvirt/driver.py ? | |
| 14:41:19 | mriedem | kashyap: replied | |
| 14:41:19 | stephenfin | ^ that | |
| 14:42:18 | ameeda | maciejjozefczyk: Thanks, let me try | |
| 14:42:27 | SpazZz | Why is it still 40 degrees in south texas | |
| 14:42:28 | ameeda | maciejjozefczyk: Thanks ^_^ | |
| 14:42:29 | efried | ameeda You can run flake8 directly from the pep8 venv. source .tox/pep8/bin/activate; flake8 path/to/file.py path/to/another.py etc.py | |
| 14:42:31 | SpazZz | One more day of this and i'll die | |
| 14:42:33 | SpazZz | thems the rules | |
| 14:42:38 | efried | SpazZz ++ | |
| 14:42:41 | ameeda | thanks all | |
| 14:42:49 | maciejjozefczyk | ameeda: :) | |
| 14:42:59 | sean-k-mooney | ameeda: if you have commited the chagne you can use tox -e fast8 to just run tox on the files you changed in the last commit | |
| 14:43:08 | maciejjozefczyk | sean-k-mooney: good to know | |
| 14:43:57 | SpazZz | If any cores got some cycles today for this would be much appreciated. +1'd by two citrix folks, need mah +2 love now :) https://review.openstack.org/#/c/533168/ | |
| 14:43:57 | efried | And to nerd out just a little bit more, to check the last e.g. 6 commits in a series, I do this from my pep8 venv: git diff --name-only HEAD~6 | grep '\.py$' | xargs flake8 | |
| 14:44:01 | mdbooth | sean-k-mooney: If that's the same as 'tox -e pep8 -- -HEAD' it will also include uncommitted changes in the working directory. | |
| 14:44:19 | SpazZz | Ignore the IBM.. it'll go away eventually :P | |