| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-28 | |||
| 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: Make security_group_rules use check_deltas() for quota https://review.openstack.org/477700 | |
| 21:57:08 | openstackgerrit | melanie witt proposed openstack/nova master: Add scatter gather utilities for cells https://review.openstack.org/475957 | |
| 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 | mriedem | https://bugs.launchpad.net/nova/+bug/1701105 | |
| 22:29:24 | openstack | Launchpad bug 1701105 in OpenStack Compute (nova) "Customizing the OpenStack Compute (nova) Scheduler in Operations Guide" [Medium,Confirmed] | |
| 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: Make security_group_rules use check_deltas() for quota https://review.openstack.org/477700 | |
| 00:10:17 | openstackgerrit | melanie witt proposed openstack/nova master: Add scatter gather utilities for cells https://review.openstack.org/475957 | |
| 00:10:19 | openstackgerrit | melanie witt proposed openstack/nova master: Remove 'reserved' count from used limits https://review.openstack.org/446242 | |
| 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: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 | 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: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: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? | |