| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-26 | |||
| 09:53:50 | bauzas | I'm not that seasoned with docs | |
| 09:53:59 | bauzas | although I patched docs in 2013 :p | |
| 09:54:28 | bauzas | but IIRC, it was far less complicated, either you deployed with Ubuntu or Fedora | |
| 09:55:35 | openstackgerrit | John Garbutt proposed openstack/nova master: Add description to policies in floating_ip files https://review.openstack.org/459791 | |
| 09:55:50 | bauzas | johnthetubaguy: asettle: so I guess if it's that simple, we should just have variable doc depending on which distro they use nope? | |
| 09:56:05 | johnthetubaguy | alex_xu: cool, glad we got that deprecation sorted, makes those docs much simpler ^ | |
| 09:56:10 | asettle | bauzas: the variables are easy. We can manage that. We already do. So. Yep? | |
| 09:56:17 | johnthetubaguy | bauzas: thats what we do | |
| 09:56:41 | bauzas | asettle: could we then just try to have that reviewed by most of the distros ? | |
| 09:56:56 | bauzas | and having endpoints different between them ? | |
| 09:57:11 | bauzas | idk, just seems to me the easiest solution | |
| 09:58:14 | openstackgerrit | John Garbutt proposed openstack/nova master: Add description to policies in migrate_server.py https://review.openstack.org/459711 | |
| 09:58:36 | openstackgerrit | John Garbutt proposed openstack/nova master: Add description to policies in migrate_server.py https://review.openstack.org/459711 | |
| 10:04:02 | cdent | asettle, bauzas, johnthetubaguy did you resolve the above discussion satisfactorily? I tend to agree with johnthetubaguy: make the docs match the packages and leave it at that (for now) | |
| 10:04:16 | asettle | cdent: uh not quite. I think bauzas lost me with his last comments. | |
| 10:11:44 | Dinesh_Bhor | johnthetubaguy, sfinucan: Hi, if you get time then please take a look at it: https://review.openstack.org/#/c/457890/ | |
| 10:12:00 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(2/4) https://review.openstack.org/459569 | |
| 10:15:58 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.soft_delete notifications https://review.openstack.org/460029 | |
| 10:22:57 | sfinucan | Dinesh_Bhor: I'd seen that but I'm not 100% sure if we can do it. Comments now left saying as much | |
| 10:26:59 | Dinesh_Bhor | sfinucan: ok, will wait for others to take a look at it. thanks you so much | |
| 10:41:07 | cdent | johnthetubaguy: if you're willing and able this looks good to go: https://review.openstack.org/#/c/455710/ | |
| 10:41:18 | cdent | or alex_xu | |
| 10:49:31 | cdent | thanks sdague, you're up early | |
| 10:53:20 | sdague | cdent: not that early :) | |
| 10:53:33 | cdent | anything before noon is early to me | |
| 10:53:51 | sdague | cdent: I just went through the devstack nova wsgi patch about what the transition plan needs to be | |
| 10:54:04 | cdent | it's a bit messy... | |
| 10:56:32 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-exists notification https://review.openstack.org/403660 | |
| 10:58:12 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660 | |
| 10:58:15 | cdent | i'll need to digest that sdague, I was thinking it might be possible to have grenade from-ocata to switch to uwsgi hosting the api, but I guess there's stuff that's relying on the URL for the API saying the same? | |
| 10:59:20 | cdent | another way to achieve the same thing would be to run uwsgi, but on a port (when in grenade) | |
| 10:59:26 | cdent | (as I've done for the metadata service) | |
| 11:00:11 | sdague | cdent: so in every transition we made here before we've left the configuration the same, we've not modeled a transition between 2 different service modes | |
| 11:00:56 | sdague | because I think that's really a post upgrade decision. Get onto the new code, then transition from one run mode to the other. | |
| 11:02:10 | cdent | yeah, I've never understood that because that means two events that are effectively upgrades. If I were deploying I'd change both at the same time (assuming some out of band testing beforehand) | |
| 11:07:19 | sdague | cdent: I wouldn't. I would validate the same configuration path worked as before before changing that out. | |
| 11:08:44 | cdent | different strokes. but that's fine. I'll have to stick a lot of stuff back into lib/nova, but that's also fine. | |
| 11:23:11 | jaypipes | *yawn* | |
| 11:23:13 | jaypipes | morning. | |
| 11:24:17 | ralonsoh | jaypipes: good morning. Flash question: https://review.openstack.org/#/c/451777/6/nova/objects/pci_device_pool.py | |
| 11:24:34 | ralonsoh | jaypipes: do we need to bump the version of an object if the parent is updated? | |
| 11:24:51 | jaypipes | ralonsoh: no | |
| 11:24:54 | ralonsoh | thanks | |
| 11:25:06 | jaypipes | ralonsoh: flash answer :) | |
| 11:25:08 | ralonsoh | that's all, enjoy your coffee | |
| 11:25:14 | jaypipes | hehe | |
| 11:41:43 | jaypipes | moshele: mornin. just approved the hw-tc-offload os-vif patch. excellent work on that. | |
| 11:42:23 | moshele | jaypipes: thanks :) | |
| 12:45:34 | alex_xu | nova api meeting is in 15 mins at #openstack-meeting-4 | |
| 12:50:55 | openstackgerrit | sahid proposed openstack/nova master: numa: fix an uninitialized local variable https://review.openstack.org/458848 | |
| 12:52:21 | openstackgerrit | Stephen Finucane proposed openstack/os-vif master: Require OVS 2.8 (presumed) and kernel 4.8 for offload https://review.openstack.org/460093 | |
| 12:52:22 | openstackgerrit | Stephen Finucane proposed openstack/os-vif master: Standardize README https://review.openstack.org/460094 | |
| 12:53:04 | sfinucan | jaypipes. moshele: noddy doc fixes ye should check out ^ | |
| 12:53:14 | sfinucan | sean-k-mooney too ^ | |
| 12:53:30 | johnthetubaguy | sfinucan: about your comments here, I am just quoting from the api-ref, which seemed the correct thing to do: https://review.openstack.org/#/c/459711/3 | |
| 12:55:33 | sfinucan | johnthetubaguy: Yup, I get your reasoning. I'd rather have it "correct" in one place myself but I'm not too worried to hold it up on that | |
| 12:55:47 | sfinucan | the reno/left over vars were what the hold up was for | |
| 12:56:07 | johnthetubaguy | sfinucan: left over vars? I thought that was the other patch | |
| 12:56:25 | sfinucan | johnthetubaguy: I might be mixing up patches. Oops | |
| 12:56:35 | johnthetubaguy | sfinucan: no worries, I keep doing that too | |
| 12:57:37 | johnthetubaguy | sfinucan: my grammar is fairly terrible, so they all read the same to me, I just don't want to flip flop between different wordings | |
| 12:58:03 | sfinucan | johnthetubaguy: Hmm, could I be super-nitty and ask you to fix it? :/ | |
| 12:58:07 | sfinucan | I can do it too | |
| 12:58:17 | johnthetubaguy | sfinucan: I can fix it up, thats fine | |
| 12:58:26 | johnthetubaguy | sfinucan: just wanted to be sure thats the way you want it | |
| 12:58:41 | sfinucan | johnthetubaguy: Ideally, yes :) | |
| 12:58:47 | sfinucan | I tend to write those things in imperative mood, present tense | |
| 12:58:51 | sfinucan | like a command | |
| 12:58:59 | sfinucan | ditto for commit message, fwiw ;) | |
| 13:00:19 | alex_xu | nova api meeting is running | |
| 13:00:59 | johnthetubaguy | sfinucan: yeah, I don't really know the difference really, never got taught any of that, except a small amount in french and german, its kinda frustraighting they culled all that for a few years over here, I should probably go back and read up on all that at some point. | |
| 13:01:55 | kashyap | johnthetubaguy: Git itself uses the "imperative mood" | |
| 13:02:03 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Sensibly wrap policy help messages https://review.openstack.org/460097 | |
| 13:02:20 | kashyap | Merge branch 'blah' | |
| 13:02:22 | sfinucan | johnthetubaguy, sdague: You were asking for this ^ | |
| 13:02:24 | sfinucan | ? | |
| 13:02:30 | kashyap | (Revert "Foo", etc) | |
| 13:02:51 | sfinucan | johnthetubaguy: I don't know where I learnt it, though I'm guessing school | |
| 13:03:10 | kashyap | Speaking of grammar, /me _just_ gave a "level-3" exam of his 4th language, Dutch | |
| 13:03:34 | johnthetubaguy | sfinucan: yeah, its stuff they brought back in just after I left school, I believe | |
| 13:03:34 | kashyap | And _just_ managed to pass (52% in writing; 71.% marks in speaking) | |
| 13:04:02 | sfinucan | johnthetubaguy: We did things like cursive writing (? the one where you join letters etc.) in England, but I don't recall them teaching any of that in Ireland :/ | |
| 13:04:23 | mriedem | alex_xu: api meeting? | |
| 13:04:27 | sfinucan | My siblings' writing is absolutely awful | |
| 13:04:34 | sfinucan | mriedem: Yup, openstack-meeting-4 | |
| 13:04:51 | mriedem | cursive is old news | |
| 13:04:59 | sfinucan | kashyap: Congrats, dude :) That's outrageously impressive | |
| 13:05:10 | johnthetubaguy | sfinucan: oh, funky, they did a slightly strange course at mine I think, missed out the useful bits! | |
| 13:05:20 | johnthetubaguy | sfinucan: I don't remember its name now | |
| 13:05:47 | kashyap | sfinucan: Duh, that 52% in writing was danglingly close! The goddmaned thing trips me up is the "inversion" (which is same in Dutch and German) | |
| 13:05:52 | kashyap | E.g: "If I am sick, I go to the doctor", it becomes: "If I am sick, go I to the doctor" | |
| 13:05:57 | sfinucan | johnthetubaguy: I blame Thatcher | |
| 13:06:04 | sfinucan | Always blame Thatcher | |
| 13:07:11 | kashyap | sfinucan: But yeah, not complaining; tomorrow night (right after work) starts my level-4 -- "maintain the momentum" :P | |
| 13:07:13 | sfinucan | kashyap: It's be "If I sick am", no? | |
| 13:07:13 | sdague | sfinucan: yeh, though is that going to destroy intentional formatting? | |
| 13:07:34 | kashyap | sfinucan: That, too! | |
| 13:07:37 | sfinucan | i.e. Wenn ich krank bin, | |
| 13:07:39 | sfinucan | ... | |
| 13:07:54 | sfinucan | yada yada random guttural words | |