| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-14 | |||
| 09:27:25 | gmann_ | gibi: and on irrelevant_files issue, i reply on that your ML thread with additional query. i think avoiding the irrelevant_files in base job/template solve the most of the issue. | |
| 09:37:48 | gibi | gmann_: I'm not sure. The integrated-gate list jobs without irrelevant-files and that means those jobs will run on ever patch and we cannot override that later in project-config or in-tree | |
| 09:38:41 | gibi | gmann_: at least that how I understood the problem in https://review.openstack.org/#/c/538908/ | |
| 09:38:50 | gmann_ | gibi: no, we can still define irrelevant-files in-tree. if those job does not have irrelevant-files in base definition | |
| 09:39:59 | gibi | gmann_: then I'm still confused about the problem in https://review.openstack.org/#/c/538908/ :) | |
| 09:40:52 | gibi | gmann_: what I see there is that we list tempest-full job twice, once in the integrated-gate template without irrelevant-files and once in the project-config with irrelevant-files | |
| 09:41:23 | gibi | gmann_: still the tempest-full is triggered on every patches as the above two listings are understood as two different possible triggers for the job | |
| 09:41:54 | gibi | gmann_: and the definition in integrated-gate matches to every patch (as no irrelevant-files define there) | |
| 09:50:42 | gmann_ | gibi: yea that seems the behavior. | |
| 09:51:49 | gmann_ | gibi: but that is opposite for 'branch' case, which get overridden from template to pipeline list definition - https://review.openstack.org/#/c/542484/2/zuul.d/projects.yaml | |
| 09:54:10 | gmann_ | gibi: i got your point on https://review.openstack.org/#/c/538908/ | |
| 09:56:15 | gmann_ | gibi: after your fix, i ll move those complete bits to nova-tree and there we control and do not need to use template if they create issues. | |
| 10:03:56 | gibi | gmann_: ohh, so branch behaves differently, I did not know that | |
| 10:05:13 | stephenfin | tonyb: Done :) | |
| 10:08:15 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: libvirt: Block swap volume attempts with encrypted volumes https://review.openstack.org/543569 | |
| 10:08:21 | gmann_ | gibi: but issue is same i think, zuul found it in 2 place and run on stable branch also as it did not find any branch restriction on project pipeline list | |
| 10:10:45 | gibi | gmann_: I see. Then we cannot simply disallow defining branch and irrelevant-files for jobs in project-templates | |
| 10:11:22 | gibi | gmann_: we either have a job in the templates that has irrelevant-files and branch definition that is good for every porject using that template, or we cannot have jobs in the template | |
| 10:11:54 | openstackgerrit | Merged openstack/osc-placement stable/queens: Update UPPER_CONSTRAINTS_FILE for stable/queens https://review.openstack.org/537697 | |
| 10:12:38 | gmann_ | gibi: +1. totally agree. as we cannot have generic irrelevant-files for each project i think we remove the integrated-job as you mentioned in ML | |
| 10:17:05 | gibi | gmann_: cool. As I mentioned I'm happy to help with the actual work as well if needed | |
| 10:17:26 | gmann_ | gibi: replied. | |
| 10:17:43 | gmann_ | gibi: thanks that will be helpful. | |
| 10:18:09 | gmann_ | gibi: ll coordinate with you once get final confirmation from on ML | |
| 10:20:36 | gibi | gmann_: thanks | |
| 10:25:03 | bauzas | good morning Nova | |
| 10:28:46 | cdent | lately it feels like morning (pre-coffee) all the time | |
| 10:33:18 | bauzas | I feel always undercaffeinated | |
| 10:34:54 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Block swapping to an encrypted volume when using QEMU to decrypt https://review.openstack.org/544238 | |
| 10:36:29 | gibi | I have some coffee beans covered in chocolate nearby all the time, that helps with my 'balanced' sugar and caffein intake | |
| 10:37:33 | openstackgerrit | Hironori Shiina proposed openstack/nova-specs master: Ironic: Instance switchover https://review.openstack.org/449155 | |
| 10:37:54 | cdent | gibi is wise | |
| 10:39:14 | bauzas | we have something called "Green chaud" here | |
| 10:39:32 | bauzas | https://www.chartreuse.fr/en/recettes/green-chaud/ | |
| 10:39:33 | gmann_ | gibi: bauzas cdent enjoy your morning coffee, it's dinner time here :). | |
| 10:40:14 | gibi | gmann_: actually I'm also closer to lunch than morning coffee. Enjoy your dinner | |
| 10:42:09 | gibi | bauzas: that drink looks to me that somethin that helps to got to sleep at the end of a long day | |
| 10:42:28 | bauzas | gibi: rather at the beginning of a skiing day | |
| 10:42:47 | gibi | bauzas: interesting | |
| 10:43:10 | cdent | a) looks tasty, b) agree it looks sleep, not ski, inducing | |
| 10:55:12 | ameeda | Hi, just a question about gerrit review, I have 2 nova reviews waiting for reviewers (one of them got +2c, waiting for +2w) and another one doesn't have activities from reviewers from awhile , do you follow scheduler to check them or I have to ask you to check them ? | |
| 10:58:33 | cdent | bauzas, gibi, stephenfin ^ | |
| 11:04:34 | gibi | ameeda: it depends. For example priority feature work and high and critical bug fixes tend to get attention automatically the rest of the patches might need some encuragement | |
| 11:05:56 | gibi | ameeda: also I suggest to try to communicate why something is important / needs attention instead of just pinging people with reviews. It helps us reviewers to prioritize properly | |
| 11:07:08 | gibi | ameeda: when I have time I sometimes look at random reviews on gerrit having a clean CI run and some code review | |
| 11:07:36 | gibi | ameeda: but most of the time I have a list of things to make progress with | |
| 11:19:41 | openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: RP list: member_of and resources parameters (v1.3, v1.4) https://review.openstack.org/511183 | |
| 11:19:42 | openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: RP delete inventories (v1.5) https://review.openstack.org/514642 | |
| 11:19:44 | openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: CLI for traits (v1.6) https://review.openstack.org/514643 | |
| 11:19:45 | openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644 | |
| 11:19:47 | openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646 | |
| 11:19:48 | openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647 | |
| 11:19:49 | openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: PUT dict format to /allocations (v1.11, v1.12) https://review.openstack.org/542819 | |
| 11:26:16 | bauzas | cdent: https://bugs.launchpad.net/nova/+bug/1749410 | |
| 11:26:17 | openstack | Launchpad bug 1749410 in OpenStack Compute (nova) "placement api-ref unclear if capacity is meant to be total or current" [Wishlist,Confirmed] | |
| 11:26:30 | openstackgerrit | Claudiu Belu proposed openstack/nova master: compute: Cleans up allocations after failed resize https://review.openstack.org/543971 | |
| 11:28:16 | cdent | bauzas, hmmm, yeah, not entirely clear what the best way to clarify that is. It might be useful for there to be separate glossary of placement terms, so that the tables in the api-ref don't get too clogged up | |
| 11:28:33 | bauzas | cdent: anyway, just FYI | |
| 11:32:13 | mdbooth | Hmm, instance snapshot is root disk only. | |
| 11:37:32 | openstack | Launchpad bug 1749404 in OpenStack Compute (nova) "nova-compute resource tracker ignores 'reserved' while reporting 'max_unit'" [Undecided,New] | |
| 11:37:32 | bauzas | cdent: yet again some good point https://bugs.launchpad.net/nova/+bug/1749404 | |
| 11:40:39 | cdent | bauzas: yup, that is a good point. I suppose an argument could be made that "it depends on allocation ratio" but I think the intent is as stated in the bug. | |
| 11:40:51 | bauzas | cdent: I'll mark it as Wontfix | |
| 11:41:00 | bauzas | it's a semantic point of view | |
| 11:41:05 | cdent | no, I think it should be fixed | |
| 11:41:28 | bauzas | cdent: if so, reopen the bug with your comment please | |
| 11:41:33 | bauzas | I'm fine with reopening it | |
| 11:41:36 | cdent | if allocation_ratio is one and reserved is > 0 then it is wrong | |
| 11:41:37 | cdent | will do | |
| 11:52:13 | openstackgerrit | Chen Hanxiao proposed openstack/nova master: libvirt: remove TODO on validation of scsi model https://review.openstack.org/525055 | |
| 12:00:25 | openstackgerrit | Chen Hanxiao proposed openstack/nova master: libvirt: add Linux distribution guest only description for inject_xxx options https://review.openstack.org/528441 | |
| 12:18:18 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/544277 | |
| 12:30:02 | openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/544282 | |
| 13:19:20 | openstackgerrit | Lajos Katona proposed openstack/osc-placement master: Random names for functional tests https://review.openstack.org/542745 | |
| 13:58:24 | mriedem | happy valentines day everyone <3 | |
| 13:59:48 | gibi | mriedem: same to you :) | |
| 14:02:55 | openstackgerrit | Bence Romsics proposed openstack/nova master: Clarify 'capacity' in placement api-ref https://review.openstack.org/544347 | |
| 14:03:59 | gibi | mriedem: I left two questions in https://review.openstack.org/#/c/543971/3/nova/compute/manager.py@4457 | |
| 14:09:39 | mriedem | replied | |
| 14:12:18 | mriedem | ildikov: i realized yesterday that we never handled this https://review.openstack.org/#/c/544152/1/specs/queens/implemented/multi-attach-volume.rst@136 | |
| 14:12:44 | mriedem | ildikov: but then realized, the only volume types that rely on that, don't support multiattach, so it's not really a problem we need to worry about right now | |
| 14:12:55 | mriedem | things like NFS | |
| 14:13:36 | ildikov | mriedem: good points, thanks for looking into it | |
| 14:14:16 | ildikov | agreed on let's worry about it later | |
| 14:14:38 | ildikov | do you think it should be a PTG micro topic or we're good for now? | |
| 14:15:00 | mriedem | nah | |
| 14:15:09 | mriedem | unless someone really cares about making NFS support multiattach | |
| 14:15:10 | mriedem | which i doubt | |
| 14:15:32 | mriedem | well, NFS + multiattach + snapshot | |
| 14:16:00 | ildikov | yeah, let's not encourage people if don't have to :) | |
| 14:17:03 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Test websocketproxy with TLS in the nova-next job https://review.openstack.org/513160 | |
| 14:22:16 | ildikov | mriedem: I also didn't talk about shared_targets in the docs for now, wonder whether I should or not? | |
| 14:23:07 | mriedem | probably not necessary; nova uses them for locking if the microversion is available | |
| 14:23:14 | mriedem | but it's not required | |
| 14:28:37 | ildikov | ok, fine, I'll leave it then | |
| 14:29:05 | ildikov | it's something more interesting for driver maintainers anyway so does not seem to be admin guide content, but wanted to check | |
| 14:55:30 | openstackgerrit | Merged openstack/nova-specs master: Fix missing blueprint URLs https://review.openstack.org/542543 | |
| 14:56:22 | mriedem | dansmith: i won't be around for the cells v2 meeting today (if there was going to be one) | |
| 14:58:56 | openstackgerrit | Merged openstack/nova-specs master: List/show all server migration types https://review.openstack.org/540256 | |
| 14:59:34 | dansmith | mriedem: ack | |