Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-03-27
14:22:19 openstackgerrit sean mooney proposed openstack/nova master: [WIP] cyborg evacuate support https://review.opendev.org/715326
14:25:41 mriedem nightmare_unreal: comments inline
14:25:54 nightmare_unreal thank you :D
14:26:00 mriedem nightmare_unreal: remember to run `tox -e flake8` before pushing up your changes
14:26:07 mriedem err,
14:26:10 mriedem tox -e fast8
14:26:39 nightmare_unreal I'll remember that . So far I was running only tox -e pep8 , tox-e py36
14:27:10 stephenfin or do 'pre-commit install --allow-missing-config'
14:27:19 stephenfin then it'll run automatically whenever you make a commit
14:27:52 nightmare_unreal aah that's nice
14:28:01 nightmare_unreal I will do it
14:34:14 mriedem you don't need py36 if you're doing functional tests
14:35:09 mriedem tox -e functional
14:35:15 mriedem py36 == unit tests
14:35:35 mriedem for nova running all of those tests locally will probably take awhile
14:35:41 mriedem unless you're sean-k-mooney
14:36:21 sean-k-mooney hehe actully that server is kind of unhappy at the momemtn so im back to using a laptop
14:37:07 sean-k-mooney i think the motherboad has issues or maybe the memory. its raising machine check excptions which are not fun
14:38:05 sean-k-mooney but some day ill have 48 thread to make it fast again. some day
14:58:48 openstackgerrit Kevin Zhao proposed openstack/nova master: fix scsi disk unit number of the attaching volume when cdrom bus is scsi https://review.opendev.org/712607
15:16:27 melwitt mriedem: I proposed this yesterday for the troubleshooting doc about the affinity stuff https://review.opendev.org/715092
15:27:44 openstackgerrit Kevin Zhao proposed openstack/nova master: fix scsi disk unit number of the attaching volume when cdrom bus is scsi https://review.opendev.org/712607
15:32:06 openstackgerrit Merged openstack/nova master: Enable hard/soft reboot with accelerators. https://review.opendev.org/697940
15:35:51 mriedem melwitt: soft -1 to put that into a sub-page
15:35:55 mriedem also added dansmith
15:36:39 melwitt mriedem: ack, thanks
15:37:46 mriedem just thought of something else, i'll leave another comment
15:38:50 openstackgerrit Merged openstack/nova master: Enable start/stop of instances with accelerators. https://review.opendev.org/699553
15:41:31 melwitt k
16:40:51 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing ips policies https://review.opendev.org/715477
16:52:29 openstackgerrit Dan Smith proposed openstack/nova master: Separate update and swap volume policies https://review.opendev.org/711194
16:52:29 openstackgerrit Dan Smith proposed openstack/nova master: Allow PUT volume attachments API to modify delete_on_termination https://review.opendev.org/693828
16:52:30 openstackgerrit Dan Smith proposed openstack/nova master: Add functional tests for PATCH volume attachments API https://review.opendev.org/710965
16:52:38 dansmith gibi: gmann: ^
16:54:59 gmann dansmith: thanks i will check. cmt msg title of func tests patch still has 'PATCH'. ^^
16:55:26 dansmith gmann: I didn't update that commit message because I figured we're squashing that after we review the diff, but will -W with that to make it obvious
16:56:03 gmann dansmith: ohk, did not read your comment.
16:56:39 dansmith gmann: I also didn't update the api-ref just yet, so just FYI
16:59:53 gmann got it,
17:11:47 openstackgerrit Sylvain Bauza proposed openstack/nova master: WIP: Support different vGPU types per pGPU https://review.opendev.org/715490
17:11:47 openstackgerrit Sylvain Bauza proposed openstack/nova master: Provide the parent pGPU when creating a new vGPU https://review.opendev.org/715489
17:33:33 gmann dansmith: commented, +1 on adding tag in updatable items
17:37:38 gmann dansmith: policy patch lgtm with 1 comment on passing project_id as target for admin policy.
17:45:59 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Fix os-ips policy to be admin_or_owner https://review.opendev.org/715496
17:46:32 hansmoleman so did the PATCH thing get dropped because https://review.opendev.org/#/c/693828/ is still talking about PATCH
17:50:22 dansmith hansmoleman: I just commented, but as we discussed here and on the last patch, I haven't updated all the meta and docs yet, nor squashed the last patch into the first one as planned because I'm trying to allow for review of just the PATCH->PUT code changes before I go moving everything around
17:52:39 hansmoleman ok so it's diverged quite a bit from the spec, i suppose in an effort to still try to get something in for ussuri?
17:53:01 hansmoleman because i was originally advocating doing this with PUT in the spec but not with a swap volume operation, but then that all changed at some point to PATCH
17:53:11 hansmoleman and now the PATCH thing is getting dropped
17:53:31 dansmith hansmoleman: we discussed that in IRC yesterday
17:53:45 dansmith gibi, gmann, myself and brin
17:58:04 sean-k-mooney dansmith: just an fyi im finishing up for the day shortly. ill continue working on https://review.opendev.org/#/c/715326/ on monday but i think getting rebuild and evacuation working with cyborg should not be too hard
17:58:18 dansmith sean-k-mooney: sounds good, enjoy the weekend
17:58:29 sean-k-mooney you too o/
18:04:51 openstackgerrit Stephen Finucane proposed openstack/nova stable/queens: Functional test for UnexpectedDeletingTaskStateError https://review.opendev.org/715405
18:04:52 openstackgerrit Stephen Finucane proposed openstack/nova stable/queens: Unplug VIFs as part of cleanup of networks https://review.opendev.org/715406
18:07:12 stephenfin artom: if you've time to kill before the end of the day, the series ending here continues work on cleaning up a lot of crap from hardware.py https://review.opendev.org/#/c/714703/
18:07:40 artom stephenfin, tbh probably not today, because kids
18:07:53 stephenfin fair
18:15:47 artom stephenfin, but I'll keep that on my review list
18:16:22 artom stephenfin, and if you're still around, gentle reminder that https://review.opendev.org/#/c/714228/ and the patches on top are still a thing
18:16:59 artom And if you're not, happy 🍺'ing
19:52:58 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing ips policies https://review.opendev.org/715477
20:01:13 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-ips https://review.opendev.org/715529
20:08:23 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing ips policies https://review.opendev.org/715477
20:08:36 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-ips https://review.opendev.org/715529
20:32:47 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-ips policies https://review.opendev.org/715545
23:16:46 openstackgerrit Merged openstack/nova stable/stein: libvirt: Provide the backing file format when creating qcow2 disks https://review.opendev.org/710790
#openstack-nova - 2020-03-28
00:33:12 gmann :)
05:16:27 openstackgerrit Sundar Nadathur proposed openstack/nova master: Bump compute rpcapi version and reduce Cyborg calls. https://review.opendev.org/704227
05:16:27 openstackgerrit Sundar Nadathur proposed openstack/nova master: Enable and use COMPUTE_ACCELERATORS trait. https://review.opendev.org/699554
05:16:28 openstackgerrit Sundar Nadathur proposed openstack/nova master: Add cyborg tempest job. https://review.opendev.org/670999
05:16:28 openstackgerrit Sundar Nadathur proposed openstack/nova master: Block unsupported instance operations with accelerators. https://review.opendev.org/674726
05:25:21 openstackgerrit Kevin Zhao proposed openstack/nova master: fix ut error on arm64 https://review.opendev.org/713163
06:22:34 sapd1 Hi everyone! Can I test this patch https://review.opendev.org/#/q/topic:bp/virt-bfv-instance-rescue+(status:open+OR+status:merged) right now?
13:48:42 openstackgerrit Takashi Kajinami proposed openstack/nova master: WIP: Add new option to restrict migration within az https://review.opendev.org/715621
19:07:08 openstackgerrit Andreas Jaeger proposed openstack/nova-specs master: Update hacking for Python3 https://review.opendev.org/715650
19:13:08 openstackgerrit Andreas Jaeger proposed openstack/os-vif master: Update hacking for Python3 https://review.opendev.org/715651
20:41:23 openstackgerrit Victor Coutellier proposed openstack/python-novaclient master: Microversion 2.83 - Add more filters for the nova list command https://review.opendev.org/713089
#openstack-nova - 2020-03-29
01:57:03 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Fix os-ips policy to be admin_or_owner https://review.opendev.org/715496
02:02:56 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing ips policies https://review.opendev.org/715477
02:05:59 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing ips policies https://review.opendev.org/715477
02:09:03 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-ips https://review.opendev.org/715529
02:09:18 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-ips policies https://review.opendev.org/715545
02:42:26 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Correct limits policy check_str https://review.opendev.org/715672
03:54:42 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing limits policies https://review.opendev.org/715674
03:57:20 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing limits policies https://review.opendev.org/715674
04:03:55 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Correct limits policy check_str https://review.opendev.org/715672
04:04:17 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing limits policies https://review.opendev.org/715674
05:23:34 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Combine the limits policies in single place https://review.opendev.org/715678
05:25:10 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Combine the limits policies in single place https://review.opendev.org/715678
05:38:06 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing limits policies https://review.opendev.org/715674
05:47:52 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in limits policy https://review.opendev.org/715680
11:47:09 openstackgerrit jayaditya gupta proposed openstack/nova master: Support for --overwrite flag for nova-manage placement heal_allocations command Closes-Bug:1868997 https://review.opendev.org/715395
12:02:11 openstackgerrit jayaditya gupta proposed openstack/nova master: Support for nova-manage placement heal_allocations --cell Closes-bug: #1868531 Change-Id: I98b3280583a6d12461d8aa52e5714d7606b84369 https://review.opendev.org/714459
12:02:13 openstack bug 1868531 in OpenStack Compute (nova) "nova manage placement doesn't support registration per Cell." [Undecided,New] https://launchpad.net/bugs/1868531 - Assigned to jayaditya gupta (jayssj11)
14:24:00 openstackgerrit Brin Zhang proposed openstack/nova master: Add instance actions v284 samples test https://review.opendev.org/706251
14:34:06 openstackgerrit Brin Zhang proposed openstack/python-novaclient master: Microversion 2.84 - action event fault details https://review.opendev.org/714561
21:19:06 openstackgerrit Mohammed Naser proposed openstack/nova master: DNM: test out running with pep8 inline comments https://review.opendev.org/715728
21:19:07 openstackgerrit Mohammed Naser proposed openstack/nova master: DNM: this is a failing change for inline pep8 https://review.opendev.org/715729

Earlier   Later