| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-09-18 | |||
| 09:41:30 | naichuans_ | bauzas: Got it, thank you very much~ And for so much information :) | |
| 09:42:58 | bauzas | naichuans_: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L566 | |
| 09:43:12 | naichuans_ | bauzas: will check it | |
| 09:49:50 | naichuans_ | bauzas: According to your spec, multiple vgpu support would not be implemented in stein, right? | |
| 09:50:00 | naichuans_ | https://review.openstack.org/#/c/602474/2/specs/stein/approved/vgpu-stein.rst | |
| 09:52:11 | kashyap | tox -epep8 -- -HEAD | |
| 09:52:21 | kashyap | Oeps | |
| 09:54:10 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_PARALLELS_SET_ADMIN_PASSWD https://review.openstack.org/562313 | |
| 09:59:27 | pvc | hi bauzas | |
| 09:59:29 | pvc | it is okay now :) | |
| 09:59:42 | pvc | i have 13 supported types for my one p100 | |
| 10:04:37 | pvc | do i need to do something more bauzas? | |
| 10:21:53 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Spec: Support filtering by forbidden aggregate https://review.openstack.org/603352 | |
| 10:32:07 | openstackgerrit | Vlad Gusev proposed openstack/nova stable/queens: Remove mox in libvirt/test_driver.py (3) https://review.openstack.org/603353 | |
| 10:38:29 | stephenfin | gibi: Any chance I could get a copy of the commands you were using in the bandwidth-aware scheduling demo last week? | |
| 10:45:30 | openstackgerrit | Vlad Gusev proposed openstack/nova stable/queens: libvirt: Reduce calls to qemu-img during update_available_resource https://review.openstack.org/603358 | |
| 10:58:11 | Luzi | Hello, I'm new to nova. How do you propose patches or ideas? Do you use Launchpad or Storyboard? | |
| 10:59:16 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Spec: Support filtering by forbidden aggregate https://review.openstack.org/603352 | |
| 11:05:53 | s10 | Luzi: Hello. We should use gerrit for patches proposal, see https://docs.openstack.org/infra/manual/developers.html . AFAIK, bugs and RFE are proposed in Nova by Launchpad. | |
| 11:07:01 | Luzi | yeah, i work in a team, which implements security enhancements and we want to contribute to openstack | |
| 11:07:32 | Luzi | for now we want to propose a signature check after an image is cached on a compute node | |
| 11:11:57 | gibi | stephenfin: hi | |
| 11:12:18 | gibi | stephenfin: let me talk to rubasov if he have an easy to digest version of the commands | |
| 11:13:08 | gibi | stephenfin: hi did promise to write up the whole thing as a blogpost but I don't know the timing of that | |
| 11:13:34 | s10 | Luzi: So basically you want to implement signature check that is described in https://docs.openstack.org/glance/latest/user/signature.html ? | |
| 11:14:33 | Luzi | s10, well nova already checks the signature when downloading an image from glance | |
| 11:14:56 | Luzi | but not when it was cached on the compute node | |
| 11:18:57 | s10 | Luzi: I think we should wait for nova cores to wake up (15:00-16:00 UTC+0) for their thoughts. I believe good starting point is to open a bug in https://bugs.launchpad.net/nova describing this issue | |
| 11:19:20 | Luzi | okay, thank you s10 :) | |
| 11:44:23 | kashyap | 'Citrix XenServer CI' is a 'voting' job, is it? | |
| 11:55:29 | sean-k-mooney | kashyap: i think so | |
| 11:56:01 | gibi | stephenfin: does it work for you if there will be a blogpost still this week? | |
| 11:56:09 | kashyap | sean-k-mooney: Trying to figure out if this change is failing because of it: https://review.openstack.org/#/c/602592/ ("Use 'virt' as the default machine type for ARMv7") | |
| 11:57:19 | sean-k-mooney | kashyap: well you are chaing the libvirt driver the cirtx xenserver ci should be using the native xen driver | |
| 11:57:21 | kashyap | sean-k-mooney: Also that 'neutron-grenade' failure is irrelevant | |
| 11:57:27 | kashyap | sean-k-mooney: Exactly | |
| 11:57:52 | sean-k-mooney | kashyap: the neutron-grenade job is voting hence the zuul -1 | |
| 11:58:06 | kashyap | sean-k-mooney: That Grenade thing was failing due to the presence of untracked working tree files: http://logs.openstack.org/92/602592/3/check/neutron-grenade/bfeb3e6/logs/devstack-gate-setup-workspace-old.txt | |
| 11:58:16 | kashyap | Yesterday mriedem was saying he saw it, but no clue why we're hitting it | |
| 11:58:28 | kashyap | Who can I bother about "Grenade issues"? | |
| 11:58:46 | sean-k-mooney | kashyap: infra but i would do a recheck first | |
| 11:58:49 | kashyap | A 'recheck' today didn't fix it; and weirdly enough, it didn't result in fresh logs. | |
| 11:59:04 | kashyap | (As in, it's still posted yesterday's 'neutron-grenade' logs) | |
| 11:59:08 | sean-k-mooney | has it actully run again? | |
| 11:59:29 | johnthetubaguy | Luzi: curious about the image signature issue, Is this because you are worried about on disk changes happening after the image gets in the cache? | |
| 11:59:32 | sean-k-mooney | kashyap: zuul has not commented back since the recheck | |
| 11:59:49 | kashyap | sean-k-mooney: I ran at 11:33 AM ("Gerrit Time"). And it's showing '-1' isn't it? | |
| 12:00:08 | sean-k-mooney | that is from before the recheck | |
| 12:00:27 | sean-k-mooney | kashyap: your job is still running | |
| 12:00:47 | Luzi | johnthetubaguy, yes the image could be replaced, that's because a recheck would be fine | |
| 12:00:49 | kashyap | sean-k-mooney: Hmm, in that case, I thought it'd "reset" the -1. | |
| 12:01:01 | kashyap | sean-k-mooney: You showed me a live console in the past, can you point me to that again? | |
| 12:01:01 | johnthetubaguy | Luzi: if you didn't bump across it yet, this was the original design spec: http://specs.openstack.org/openstack/nova-specs/specs/rocky/implemented/nova-validate-certificates.html | |
| 12:01:19 | johnthetubaguy | Luzi: well it references the original bit at the bottom of there | |
| 12:01:27 | sean-k-mooney | kashyap: that is the last job that is running http://zuul.openstack.org/stream.html?uuid=e576a55aa1d1464fbbecbdea0cd574d8&logfile=console.log | |
| 12:02:04 | sean-k-mooney | it just finished devstack and is tarting the tempest and will be starting the tempest run soon | |
| 12:02:27 | kashyap | sean-k-mooney: Most excellent. I _guessed_ so, but thought it shouldn't take 3+ hours. Apparently it does :-) | |
| 12:02:50 | sean-k-mooney | kashyap: if you go to http://zuul.openstack.org/ and filter by 602592 you will see the rest | |
| 12:03:17 | kashyap | sean-k-mooney: You're clairvoyant. That was my next question :-) To remind how I can figure it out next time | |
| 12:03:20 | kashyap | Thanks! | |
| 12:07:58 | johnthetubaguy | Luzi: this is an email thread that might be worth reading through, around when we used to do image verification: http://lists.openstack.org/pipermail/openstack-operators/2016-May/010554.html | |
| 12:09:11 | johnthetubaguy | Luzi: largely some basic checks we removed because once you have access to the image cache, its basically game over for a heap of reasons. Could well be something wrong in that analysis, but its worth a read of the thread for context | |
| 12:09:40 | Luzi | johnthetubaguy, thank you, i will read through | |
| 12:24:38 | openstackgerrit | Merged openstack/os-traits master: import zuul job settings from project-config https://review.openstack.org/601377 | |
| 12:33:18 | openstackgerrit | Merged openstack/os-traits master: switch documentation job to new PTI https://review.openstack.org/601378 | |
| 12:33:20 | openstackgerrit | Merged openstack/os-traits master: add python 3.6 unit test job https://review.openstack.org/601379 | |
| 12:44:06 | bauzas | naichuans_: nope, just the fact that we would only support multiple types once reshape is fully done | |
| 12:44:21 | bauzas | pvc: nope, you're all good, enjoy | |
| 12:47:19 | pvc | thank you so much bauzas :) | |
| 12:47:57 | stephenfin | gibi: Blogpost this week is a-ok. A dump of the commands, assuming there wasn't anything confidential in them, is also a-ok :) | |
| 12:48:23 | gibi | stephenfin: then everything is set :) rubasov will make a post and distribute it on the ML | |
| 12:48:29 | naichuans_ | bauzas: got it, thanks | |
| 12:48:33 | stephenfin | gibi: Sweet! | |
| 12:48:48 | stephenfin | gibi: Totally unrelated, fancy grabbing this patch at some point? Will unblock it's successor https://review.openstack.org/#/c/565472/ | |
| 12:48:52 | stephenfin | *its | |
| 12:48:57 | gibi | stephenfin: it was so unconfidental that you saw the demo earlier than anybody in the company ;) | |
| 12:49:11 | stephenfin | :D Touché | |
| 12:52:31 | gibi | stephenfin: ohh I +2d the successor earlier :) | |
| 12:53:48 | gibi | stephenfin: done | |
| 13:02:39 | openstackgerrit | Christian Berendt proposed openstack/nova master: Add missing backticks in nova-manage docs https://review.openstack.org/603370 | |
| 13:03:53 | bauzas | stephenfin: looks like someone saw your presentation ^ | |
| 13:05:22 | openstackgerrit | Michele Baldessari proposed openstack/nova master: Ignore ArgsAlreadyParsedError exceptions when registering logging options https://review.openstack.org/603372 | |
| 13:05:32 | efried | ō/ | |
| 13:07:21 | mriedem | brinzhang: hi | |
| 13:07:37 | brinzhang | mriedem: Hi | |
| 13:07:56 | mriedem | you had some questions? | |
| 13:08:25 | bauzas | mriedem: FWIW, working on your change locally | |
| 13:08:30 | brinzhang | yes. | |
| 13:08:37 | brinzhang | 1. about this https://review.openstack.org/#/c/599276/23/api-guide/source/polling_changes-since_parameter.rst | |
| 13:09:19 | mriedem | yeah jay is right, git mv would be better to preserve the history | |
| 13:10:55 | brinzhang | But I think "polling_changes-since_and_changes-before_parameters.rst" already contains the contents of "polling_changes-since_parameter.rst", so I think this should be replaced by current doc. As mriedem said, I need to do a redirectmatch for old links. | |
| 13:11:21 | brinzhang | ignore the last sentence. ^^ | |
| 13:12:39 | brinzhang | I think this do a rediecetmatch for old links is better | |
| 13:13:16 | mriedem | why not just rename the file as jaypipes suggested to preserve the history? | |
| 13:13:20 | mriedem | git history i mean | |
| 13:13:32 | mriedem | you have to add a redirect link either way | |
| 13:14:03 | mriedem | then it's also easier, as a reviewer, to see what you changed about the document when adding the changes-before conent | |
| 13:14:05 | mriedem | *content | |
| 13:15:07 | brinzhang | I don't think I understand the meaning of jaypipes :-( | |
| 13:16:10 | mriedem | just git mv the file to rename it | |