Posted	Nick	Remark
#openstack-nova - 2018-02-13
23:25:50	openstackgerrit	Merged openstack/nova master: Ensure resource classes correctly https://review.openstack.org/539738
23:30:57	openstackgerrit	Nicolas Bock proposed openstack/nova stable/pike: Fix SUSE Install Guide: Placement port https://review.openstack.org/544131
23:41:58	openstackgerrit	Eric Fried proposed openstack/nova master: Avoid inventory DELETE API (no conflict detection) https://review.openstack.org/539712
23:41:59	efried	jaypipes: Yet another manual rebase +2 please ^
#openstack-nova - 2018-02-14
00:09:23	openstackgerrit	Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: Snapshot https://review.openstack.org/543023
00:59:32	gmann_	mriedem_afk: gibi stephenfin as we faced the nova jobs irrelevant_files and branch restriction issues and have to fix on project-config side, i am thinking to move pipeline list jobs on project side.
01:00:18	gmann_	mriedem_afk: gibi stephenfin this will give nova a full control on those bits. i started a generic discussion on ML- http://lists.openstack.org/pipermail/openstack-dev/2018-February/127320.html
01:00:58	gmann_	please check and if that looks ok/not ok from nova perspective.
01:01:03	gmann_	melwitt: ^^
01:02:53	gmann_	next issue is irrelevant_files which does not work in case of inheritance or pipeline list (current issue where lot of nova jobs run on doc/api-ref changes), i will push some debug patch and then debug with infra team.
01:09:42	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for traits API https://review.openstack.org/524094
01:11:57	openstackgerrit	Rong Han proposed openstack/nova stable/pike: detach instance volumes when VM creation fails https://review.openstack.org/544143
01:18:10	openstackgerrit	Rong Han proposed openstack/nova stable/queens: detach instance volumes when VM creation fails https://review.openstack.org/544144
01:34:16	tetsuro	efried, dansmith, mriedem, jaypipes: Just finished reading back the dictussion about the update_provider_tree. Thank you for picking it up!
01:57:10	mriedem_afk	takashin: i have a couple of comments in https://review.openstack.org/#/c/540256/
01:57:16	mriedem_afk	if you can address and then i'll +W tomorrow
02:02:24	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add admin guide doc on volume multiattach support https://review.openstack.org/544090
02:16:15	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Amend the migration paging spec for uuid in server migrations response https://review.openstack.org/532904
02:20:16	openstackgerrit	jichenjc proposed openstack/nova master: WIP: Add zvm CI information https://review.openstack.org/533512
02:20:16	openstackgerrit	jichenjc proposed openstack/nova master: WIP: Add zvm admin doc https://review.openstack.org/533125
02:28:35	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Amend volume multi-attach spec https://review.openstack.org/544152
02:31:18	takashin
03:19:21	openstackgerrit	jichenjc proposed openstack/nova master: fix race condition of instance host https://review.openstack.org/494458
03:23:05	openstackgerrit	Zhenyu Zheng proposed openstack/nova-specs master: WIP https://review.openstack.org/544158
04:23:24	openstackgerrit	jichenjc proposed openstack/nova master: WIP: fix validator confusing info https://review.openstack.org/544163
04:41:47	openstackgerrit	Arvind Nadendla proposed openstack/nova-specs master: Support traits in Glance https://review.openstack.org/541507
05:40:13	openstackgerrit	jichenjc proposed openstack/nova master: Avoid live migrate to same host https://review.openstack.org/542689
06:32:38	openstackgerrit	jichenjc proposed openstack/nova master: Fix warning for user_id and project_id https://review.openstack.org/544180
06:54:51	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: Fix missing blueprint URLs https://review.openstack.org/542543
06:55:00	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: Fix missing blueprint URLs https://review.openstack.org/542543
07:00:20	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/540256
07:04:34	tonyb	stephenfin: You can remove your -W from https://review.openstack.org/#/c/538954/1 when you're online :)
07:31:11	openstackgerrit	Merged openstack/nova master: Bindep does not catch missing libpcre3-dev on Ubuntu https://review.openstack.org/542340
07:40:03	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/541561
08:49:20	openstackgerrit	Claudiu Belu proposed openstack/nova master: compute: Cleans up allocations after failed resize https://review.openstack.org/543971
08:56:09	hrw	morning
09:21:50	gibi	gmann_: thanks for the ML post, I put my 2 cents in a reply.
09:27:21	gmann_	gibi: thanks, i am not sure about moving from integrated-job but over time we can check those if creating issues
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.

Earlier   Later