| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-04 | |||
| 10:59:45 | asettle | Also - just left minor nits/comments on your Contrib patch. If it's easier, I'm happy to pull down and do the updates. | |
| 10:59:56 | sdague | http://www.sphinx-doc.org/en/stable/config.html#confval-suppress_warnings | |
| 11:00:07 | asettle | Looking | |
| 11:00:32 | sdague | asettle: sure, feel free to just jump in and make those changes | |
| 11:00:51 | asettle | Coolio - gotta run for luncheon soon. Can do upon return :) | |
| 11:01:04 | openstackgerrit | Sean Dague proposed openstack/nova master: doc: Import administration guide https://review.openstack.org/477497 | |
| 11:01:10 | sdague | ok, I think that will build now | |
| 11:02:06 | asettle | Okay cool :) | |
| 11:02:09 | asettle | brb | |
| 11:02:33 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add Port Profile info to VIF objects Linux Bridge plugin https://review.openstack.org/490829 | |
| 11:59:48 | openstackgerrit | Merged openstack/nova master: doc: Import installation guide https://review.openstack.org/477488 | |
| 12:03:40 | openstackgerrit | Merged openstack/nova master: doc: Import administration guide https://review.openstack.org/477497 | |
| 12:08:51 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091 | |
| 12:41:10 | openstackgerrit | Alexandra Settle proposed openstack/nova master: Update install guide to clearly define between package installs https://review.openstack.org/490844 | |
| 12:45:58 | openstackgerrit | Alexandra Settle proposed openstack/nova master: Update install guide to clearly define between package installs https://review.openstack.org/490844 | |
| 12:46:02 | stephenfin | sdague: We'll leave stripping out the euca stuff til later. I put a TODO in there | |
| 12:46:53 | stephenfin | sdague: might be good to assign willing SMEs to each doc for a thorough review at the PTG. There's a lot of outdated stuff in there that needs to be updated/removed/"synergized" | |
| 12:47:04 | stephenfin | possible PTG session, actually? | |
| 12:48:11 | stephenfin | sdague: RE: resolving the ToC issues. This is what Django do | |
| 12:48:24 | stephenfin | https://docs.djangoproject.com/en/1.8/contents/ | |
| 12:54:58 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/490859 | |
| 12:56:26 | leakypipes | gibi_chaosmonkey: cool, will check it out shortly. | |
| 13:01:35 | bauzas | gibi_chaosmonkey: do you have more explanations for https://bugs.launchpad.net/nova/+bug/1708637 ? | |
| 13:01:36 | openstack | Launchpad bug 1708637 in OpenStack Compute (nova) "nova does not properly claim resources when server resized to a too big flavor" [High,New] | |
| 13:02:12 | bauzas | gibi_chaosmonkey: tbc, CoreFilter shall not be used in Pike unless you uses CachingScheduler | |
| 13:02:13 | tikitavi_ | leakypipes: Hi. We are ec2-api team and now we have problem in gating because of bug in nova #1708210. It is fixed in https://review.openstack.org/#/c/490340/ so we are looking forward when it will be merged. | |
| 13:34:16 | dansmith | leakypipes: did you see where the patch is? one unit and one functional failure that I didn't resolve | |
| 13:34:31 | dansmith | the unit test is so mocked up the butt I can't really even tell if it's useful | |
| 13:34:46 | dansmith | and the functional one was a gabbit thing and I dunno why it would have changed or what to do about it | |
| 13:35:08 | dansmith | leakypipes: and did you see my comments about the fixes for the ocata-to-pike one? | |
| 13:35:54 | dansmith | hmm, I guess the gabbit one didn't fail in jenkins, that's weird | |
| 13:36:06 | stephenfin | leakypipes: Two easy reviews for you for later https://review.openstack.org/#/c/486536/ https://review.openstack.org/#/c/480092/ | |
| 13:36:18 | stephenfin | (gets os-vif off of https://doughellmann.com/doc-migration/) | |
| 13:45:10 | leakypipes | dansmith: no, sorry superdan, just getting back to things this morning. | |
| 13:45:39 | openstackgerrit | Merged openstack/os-vif master: Update the documentation link for doc migration https://review.openstack.org/486536 | |
| 13:47:25 | dansmith | heh, always with the friday guilt | |
| 13:47:33 | leakypipes | :) | |
| 13:48:31 | superdan | leakypipes: so the remaining unit test fails because the fake resource tracker has no compute nodes, but tries to pull self.compute_nodes[cn].uuid out and fails with keyerror | |
| 13:48:46 | leakypipes | superdan: k. | |
| 13:48:49 | superdan | but the test seriously has like 30 mocks and so it's really hard to tell if it's even useful | |
| 13:48:58 | leakypipes | superdan: I know, it's rough | |
| 13:49:08 | leakypipes | superdan: am I good to take the baton and pull code? | |
| 13:49:10 | superdan | when I tried adding a compute node to it, it failed much earlier, which I found amusing | |
| 13:49:13 | superdan | leakypipes: yes | |
| 13:49:36 | superdan | leakypipes: look at my two latest comments on the review though and make sure my fixes for your ocata to pike test are legit | |
| 13:51:07 | mriedem | o/ | |
| 13:53:04 | mriedem | stephenfin: the admin guide import was missing several things i pointed out https://review.openstack.org/#/c/477497/ | |
| 13:53:29 | mriedem | like https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/admin-guide/source/compute-flavors.rst | |
| 13:53:32 | stephenfin | mriedem: s/compute-// | |
| 13:53:39 | stephenfin | He just dropped all the prefixes | |
| 13:53:47 | stephenfin | They're there alright :) | |
| 13:54:23 | mriedem | all of the cli-nova-* stuff from https://github.com/openstack/openstack-manuals/tree/stable/ocata/doc/admin-guide/source isn't in there | |
| 13:54:30 | mriedem | like evacuate and specify-host | |
| 13:54:50 | ftersin | mriedem: hi. who could merge https://review.openstack.org/#/c/490340/ ? it fixes a regression bug which blocks CI for ec2api | |
| 13:55:02 | stephenfin | mriedem: Yeah, I'm not sure where those are going yet, tbh | |
| 13:55:07 | leakypipes | superdan: yup, will do. just wrapping up meetings. | |
| 13:55:47 | stephenfin | They're going to need some work as they're pretty out of date and don't tie into the 'oslo_config.sphinxext'. I'll take a shot at it shortly | |
| 13:56:22 | mriedem | stephenfin: there is also https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/admin-guide/source/cli-set-compute-quotas.rst | |
| 13:56:27 | mriedem | so the naming pattern isn't consistent | |
| 13:56:40 | mriedem | https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/admin-guide/source/cli-os-migrate-cfg-ssh.rst | |
| 13:56:45 | stephenfin | mriedem: Good point. Looks like I'll need to go through them all individually :( | |
| 13:57:01 | mriedem | stephenfin: i'll just open a bug to track it | |
| 13:57:07 | stephenfin | mriedem: In any case, sdague and I figured it was better to just get _something_ in and build upon it | |
| 13:57:14 | stephenfin | Do - you can assign it to me | |
| 13:59:15 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/488086 | |
| 14:00:12 | stephenfin | mriedem: Can we still merge those req updates if they make sense ^ | |
| 14:00:15 | stephenfin | *? | |
| 14:00:39 | mriedem | we can't release them for pike at this point | |
| 14:00:43 | mriedem | or in stable/pike | |
| 14:00:54 | mriedem | i'd just hold off on those until queens | |
| 14:00:58 | mriedem | for the libraries | |
| 14:01:39 | openstack | Launchpad bug 1708666 in OpenStack Compute (nova) "docs: admin guide import is missing several compute-specific docs" [High,Confirmed] - Assigned to Stephen Finucane (stephenfinucane) | |
| 14:01:39 | mriedem | stephenfin: https://bugs.launchpad.net/nova/+bug/1708666 | |
| 14:04:07 | stephenfin | mriedem: (y) (for both) | |
| 14:15:22 | mriedem | ftersin: the test needs work if you want to touch it up | |
| 14:15:30 | mriedem | takashin is probably no longer working | |
| 14:20:06 | ftersin | mriedem: thx. we'll ping takashi tomorrow | |
| 14:20:53 | ftersin | uhhm... tomorrow, in monday :/ | |
| 14:21:08 | mriedem | heh, ok, or i'll just update it | |
| 14:21:12 | mriedem | it's a 1 line assertion in the test | |
| 14:23:38 | ftersin | our developer's gone home, i'm busy with other things and do not have OS sources with git, etc right now | |
| 14:23:50 | mriedem | that's fine | |
| 14:26:59 | openstackgerrit | Merged openstack/nova master: Update install guide to clearly define between package installs https://review.openstack.org/490844 | |
| 14:30:28 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs to instance_extra https://review.openstack.org/457711 | |
| 14:45:16 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix getting instance bdms in multiple cells https://review.openstack.org/490340 | |
| 14:47:13 | mriedem | superdan: you should probably check this out - multi-cell fun https://review.openstack.org/#/c/490340/ | |
| 14:48:27 | superdan | cool | |
| 14:51:37 | openstackgerrit | Sean Dague proposed openstack/nova master: Add Contributor Guide section page https://review.openstack.org/490644 | |
| 14:51:42 | sdague | sfinucan: ^^^ | |
| 14:51:58 | stephenfin | sdague: looking | |
| 14:52:08 | sdague | sfinucan: also, just for you, I changed my word wrap for rst mode to 79 columns :) | |
| 14:52:16 | stephenfin | Yay! :D | |
| 14:58:10 | stephenfin | sdague: Yeah, I personally prefer that bullet point style. It's used in both the OVS and Django docs, and gives way more context | |
| 15:00:13 | superdan | mriedem: not sure how that bdms.update() is doing the right thing | |
| 15:03:26 | mriedem | superdan: if no bdms are found in a cell for the list of instance uuids it should just return {} | |
| 15:03:35 | leakypipes | superdan: k, just finished up with meetings. heading to code now. | |
| 15:03:40 | leakypipes | sorry for delay | |
| 15:04:09 | mriedem | leakypipes: before heading back into the RT fun, mind looking at this regression fix? https://review.openstack.org/#/c/489763/ | |
| 15:04:37 | leakypipes | mriedem: yup, will do. | |
| 15:04:41 | superdan | mriedem: the object method returns a List object though, not a dict | |