| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-28 | |||
| 20:19:10 | openstackgerrit | Merged openstack/nova master: Add a reset for traits DB sync https://review.openstack.org/478564 | |
| 20:38:10 | melwitt | dansmith: I'm wondering if we should skip the cells meeting today too. I don't think mriedem or dtp are around | |
| 20:38:23 | dansmith | melwitt: heh I was just about to suggest that | |
| 20:38:25 | dansmith | mriedem isn't, | |
| 20:38:34 | dansmith | dtp seems to never show up until right before | |
| 20:38:48 | dansmith | but regardless, | |
| 20:38:48 | openstackgerrit | Steve Noyes proposed openstack/nova master: update live migration to use v3 cinder api https://review.openstack.org/463987 | |
| 20:38:50 | dansmith | the status is pretty clear I think so I'm good with that | |
| 20:39:05 | melwitt | cool, agree | |
| 20:40:58 | lyarwood | jaypipes: hey sorry I keep missing your pings re the libvirt and scsi bus values, did you get an answer in the end? | |
| 20:53:49 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285 | |
| 21:02:59 | dtp | no cells today? | |
| 21:09:56 | melwitt | dtp: yeah, we decided to cancel it about 30 minutes ago bc mriedem isn't around (he's in china this week). sorry for the short notice | |
| 21:10:08 | dtp | that's ok | |
| 21:24:57 | dtp | do you guys need anything? | |
| 21:31:28 | melwitt | dtp: nope, not for me anyway. think things have been progressing along well. dansmith, did you have anything? ^ | |
| 21:31:40 | dansmith | nope | |
| 21:31:54 | dansmith | dtp: note that if you were in irc earlier we'd have polled you for topics before canceling :) | |
| 21:32:30 | melwitt | dtp: is there anything you needed? looks like mriedem still WIP on the hypervisors uuid stuff | |
| 21:32:46 | dtp | i'm afraid i was indisposed this morning. i'm sorry about that :) | |
| 21:33:15 | dtp | not really melwitt. just thinking about getting back in the game, if you know what i mean | |
| 21:36:21 | openstackgerrit | Merged openstack/nova master: Simplify a condition https://review.openstack.org/476637 | |
| 21:39:29 | melwitt | dtp: no worries, just wanted to make sure if you had anything you had wanted to talk about at the meeting | |
| 21:53:03 | jaypipes | lyarwood: no, but we've seen this come up with one of our customers now, too. | |
| 21:53:23 | jaypipes | lyarwood: ocata. boot from volume. volume created from image with virtio-scsi image metadata. | |
| 21:57:08 | openstackgerrit | melanie witt proposed openstack/nova master: Add scatter gather utilities for cells https://review.openstack.org/475957 | |
| 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 | |