Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-06-28
21:57:08 openstackgerrit melanie witt proposed openstack/nova master: Make security_group_rules use check_deltas() for quota https://review.openstack.org/477700
21:57:09 openstackgerrit melanie witt proposed openstack/nova master: Remove 'reserved' count from used limits https://review.openstack.org/446242
21:57:10 openstackgerrit melanie witt proposed openstack/nova master: Make key_pairs use check_deltas() for quota https://review.openstack.org/477699
21:57:12 openstackgerrit melanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243
21:57:15 openstackgerrit melanie witt proposed openstack/nova master: Count instances to check quota https://review.openstack.org/416521
21:57:47 openstackgerrit melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945
21:57:50 openstackgerrit melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946
22:03:35 openstackgerrit Ed Leafe proposed openstack/nova master: Add traits to the ResourceProviders filters https://review.openstack.org/474602
22:09:38 mriedem o/
22:10:44 melwitt o/ how's it going
22:10:55 mriedem so far so good
22:11:06 melwitt sweet
22:11:27 mriedem went to a huuuge electronics factory direct sales district last night
22:11:36 mriedem like 6 floors of wall to wall stuff
22:12:22 mriedem perfecting my technique for eating noodle soup with chopsticsk
22:12:26 mriedem *chopsticks
22:12:43 melwitt the electronics place description reminds me of yodobashi
22:13:07 melwitt nice
22:13:30 mriedem i can't get through to gmail though so no mailing list stuff all week
22:13:36 mriedem so i assume all is quiet
22:14:04 melwitt hah, quiet
22:19:54 mriedem is there any major nova news?
22:20:43 melwitt no but there's "confusion around 'hosted projects'" thread going on. which looks like an extension of the "big tent" thread fun
22:22:14 mriedem yeah, throw constellations in there too
22:22:59 melwitt so that's what you're missing out on. big time stuff
22:27:07 mriedem dang
22:27:16 mriedem well, it's come up here too at least
22:27:34 mriedem "no core?!" was the quote from monday i believe
22:28:33 melwitt like no core projects?
22:28:45 mriedem yeah
22:28:50 mriedem also, found this which is fun https://docs.openstack.org/ops-guide/ops-customize-compute.html
22:29:24 openstack Launchpad bug 1701105 in OpenStack Compute (nova) "Customizing the OpenStack Compute (nova) Scheduler in Operations Guide" [Medium,Confirmed]
22:29:24 mriedem https://bugs.launchpad.net/nova/+bug/1701105
22:30:01 mriedem sfinucan: ^ might want to update that guide to warn that nova does'nt support plugging scheduler drivers anymore
22:30:05 melwitt ah, yeah
22:47:07 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285
22:56:35 mriedem guh, so i knew this, but don't remember why we don't do it - we record instance action start in the compute api, but don't ever record when an instance action finishes
22:57:32 mriedem maybe that gets handled implicitly by the action event finish?
23:01:51 mriedem mgagne: something else we need to handle in this change https://review.openstack.org/#/c/454322/
23:02:08 mriedem we have to record an instance action for this external event so the client (tempest) can poll for when nova is done doing it's part
23:54:57 openstackgerrit Merged openstack/nova master: placement: refactor driver select_destinations() https://review.openstack.org/478291
#openstack-nova - 2017-06-29
00:00:58 openstackgerrit melanie witt proposed openstack/nova master: Handle version for PUT and POST in PlacementFixture https://review.openstack.org/478683
00:10:17 openstackgerrit melanie witt proposed openstack/nova master: Add scatter gather utilities for cells https://review.openstack.org/475957
00:10:17 openstackgerrit melanie witt proposed openstack/nova master: Make security_group_rules use check_deltas() for quota https://review.openstack.org/477700
00:10:19 openstackgerrit melanie witt proposed openstack/nova master: Make key_pairs use check_deltas() for quota https://review.openstack.org/477699
00:10:19 openstackgerrit melanie witt proposed openstack/nova master: Remove 'reserved' count from used limits https://review.openstack.org/446242
00:10:21 openstackgerrit melanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243
00:10:25 openstackgerrit melanie witt proposed openstack/nova master: Count instances to check quota https://review.openstack.org/416521
00:10:52 openstackgerrit melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945
00:10:55 openstackgerrit melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946
01:30:21 gaoyuan anyone doing bp/remove-mox-pike online?
01:32:01 gaoyuan I have a little question
01:50:00 gmann gaoyuan, yes, what is question?\
02:12:09 gaoyuan gmann: please see this review https://review.openstack.org/#/c/477812/2
02:12:37 gaoyuan Ken'ichi Ohmichi says he prefer another patch using mock
02:13:06 gaoyuan what in the bp description of remove-mox-pike, it states Please note: self.stub_out does not make use of mox, it is there to replace to use of "self.stub.Set" in an easy to review way. Moreover, and proposals to remove the use of stub_out should not be part of this spec.
02:13:23 gmann gaoyuan, checking
02:13:36 gmann gaoyuan, left comment on https://review.openstack.org/#/c/477904/1
02:13:55 gmann gaoyuan, on preferable way of mocking things.
02:14:34 gaoyuan Does this mean we should use self.stub_out to replace self.stub.Set?
02:14:41 gaoyuan or I misunderstood
02:15:07 gaoyuan gmann, thanks for the updating.
02:16:32 gmann gaoyuan, yes, as of now we are replacing the self.stub.Set with self.stub_out
02:17:10 gmann gaoyuan, because if there are lot of changes in review changing the self.stub.Set to mock (i saw in some of the review) then it is very hard to review
02:17:57 gmann gaoyuan, in that case you pointed it is just 1 line and thats why may be kenichi prefer the mock instead of stub_out
02:17:57 gaoyuan gmann, but it's got a -1, and I already abandoned it, https://review.openstack.org/#/c/477384/, this one got +2 already
02:18:29 gaoyuan OK, I understand
02:18:59 gmann gaoyuan, i saw, seems like other patch was initially proposed. but as i mentioned as it is just 1 line so it is +2
02:19:20 gmann gaoyuan, but if its too many changes then stub_out is preferable as of now
02:19:54 gaoyuan gmann, finally, if you still have time, https://review.openstack.org/#/c/451614/, please have a look at this one
02:20:44 gmann gaoyuan, you should comment on that patch instead of new patch (just matching the timing of your review on another patch and your patch :))
02:21:10 gmann gaoyuan, but main idea is same as you understood. self.stub.Set -> self.stub_out
02:21:51 gmann gaoyuan, sure, i will check but may be in evening or tomorrow for sure. added in queue
02:21:56 gaoyuan gmman, in test_vmops.py, I used a function mock_out_object which is created by myself in nova/test.py
02:22:30 gaoyuan OK, good day then
02:22:56 gaoyuan gmann, I hope I am not too much disturbing.
02:22:57 gmann gaoyuan, but stub_out does not work there ?
02:23:29 gmann gaoyuan, not at all, we really appreciate your help on fixing mox things and rally nice work.
02:23:59 gaoyuan no
02:24:03 gaoyuan I have commetns
02:24:11 gaoyuan unless I am wrong
02:24:16 gmann gaoyuan, ok, let me check the changes in detail ad
02:24:23 gmann and will get back to you
02:24:47 gaoyuan gmann, 2ks
02:24:49 gaoyuan 3ks
05:01:16 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix a missing classifier https://review.openstack.org/478733
05:27:05 openstackgerrit Alex Xu proposed openstack/nova master: placement: add traits support to get resource_providers with shared https://review.openstack.org/478464
05:38:10 mkrai_ Hi I have question regarding nova serial console. How does it work?
05:38:59 mkrai_ In Ironic, a new shellinabox process is started which serves the serial console via ipmitool. How is it done in Nova?
05:39:09 mkrai_ Link to code or any doc will be helpful.
05:39:16 mkrai_ Thanks in advance.
06:08:33 openstackgerrit Gao Yuan proposed openstack/nova master: Remove mox from nova.tests.unit.virt.test_block_device https://review.openstack.org/478749
06:11:37 openstackgerrit Gao Yuan proposed openstack/nova master: Remove mox from nova.tests.unit.virt.xenapi.test_vm_utils.py https://review.openstack.org/478749
06:56:46 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi: Exception Error logs shown in Citrix XenServer CI https://review.openstack.org/478437
07:03:07 openstackgerrit Radoslav Gerganov proposed openstack/nova master: VMware: Live migration of instances https://review.openstack.org/270116
07:03:07 openstackgerrit Radoslav Gerganov proposed openstack/nova master: VMware: Factor out relocate_vm() https://review.openstack.org/270115
07:19:06 gmann mordred, can you check this - https://review.openstack.org/#/c/459172/ (could not find you on #openstack)
07:30:16 openstackgerrit Béla Vancsics proposed openstack/nova master: [WIP] Transform live_migration_post_dest notification https://review.openstack.org/469784
07:35:33 openstackgerrit Chason Chan proposed openstack/nova master: import the admin guide content from openstack-manuals https://review.openstack.org/477497
07:52:04 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285

Earlier   Later