| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-04-12 | |||
| 15:09:29 | owalsh | melwitt: he's assuming rc==0, yes | |
| 15:09:33 | melwitt | I assumed so. I had linked it on the BZ | |
| 15:09:42 | owalsh | dansmith: yea... not how I did it initially but reasons... | |
| 15:10:23 | dansmith | owalsh: that's unfortunate.. giving computes credentials to the api DB just to run this thing the controllers should do.. do those creds persist on the compute? | |
| 15:10:47 | dansmith | melwitt: okay I hadn't read it that way.. so he's just misunderstanding what your patch is doing you mean'? | |
| 15:11:09 | melwitt | I'm concerned that he is, yeah | |
| 15:11:43 | melwitt | tssurya: you're right, I need to add to the docs separately, the docstrings don't go in there automatically http://logs.openstack.org/47/651947/2/check/openstack-tox-docs/d6d7085/html/cli/nova-manage.html | |
| 15:11:48 | dansmith | melwitt: but he's quoting from owalsh who says they'll use a retry loop, which is fine yeah? | |
| 15:12:49 | melwitt | yeah, I guess so. I was hyperfocusing on the reply and not the quote | |
| 15:13:03 | owalsh | dansmith: not sure where the creds are coming from now actually... previously when I moved this to the computes I embedded them in the script and unlink the script as soon as it was run to avoid leaving creds around | |
| 15:13:16 | dansmith | owalsh: okay | |
| 15:13:31 | owalsh | dansmith: but it's been rewritten since, I'll take a look | |
| 15:15:50 | dansmith | owalsh: "talking to the api database" is something we generally expect a compute not to ever do.. I guess in this case you're using it as the admin node since you're just taking action on that one, | |
| 15:16:06 | dansmith | but in ansible lingo, I would "delegate" the discovery task to one of the controller nodes | |
| 15:16:47 | owalsh | dansmith: yea, the reasons for this were that the controller nodes were being excluded from tripleo scale outs to speed things up, so it had to be on a compute... there isn't anything else | |
| 15:17:49 | dansmith | right, because tripleo/heat reasons yeah? If it was pure ansible delegating a single admin command to one controller node after you're done would be easy, but I know that heat probably makes that hard | |
| 15:18:25 | owalsh | might be possible now that tripleo == heat generating ansible playbook, but not sure if it would backport | |
| 15:27:28 | fried_rice | finucannot: a little sphinx help? | |
| 15:27:35 | finucannot | shoot | |
| 15:27:55 | fried_rice | finucannot: I'm in a spec | |
| 15:28:03 | fried_rice | I want to make a link that gets referenced from a couple of places | |
| 15:28:18 | fried_rice | .. `some text`: https://... | |
| 15:28:18 | fried_rice | so it's like | |
| 15:28:28 | fried_rice | but in another place I want to use different text | |
| 15:28:41 | finucannot | :ref:`the text you want <the-anchor-name>` | |
| 15:28:42 | fried_rice | I tried `different text <some text>`_ | |
| 15:28:53 | finucannot | .. _the-anchor-name: https://... | |
| 15:28:57 | finucannot | Oh/ | |
| 15:28:58 | fried_rice | yeah, I tried :ref:`different text <some text>` | |
| 15:29:00 | finucannot | *? | |
| 15:29:01 | fried_rice | it didn't work | |
| 15:29:08 | finucannot | Hmm | |
| 15:29:22 | fried_rice | .. _`some text`: https:// | |
| 15:29:22 | fried_rice | sorry, yes | |
| 15:29:26 | fried_rice | forgot the _ originally | |
| 15:29:33 | fried_rice | The `some text`_ link works fine | |
| 15:29:39 | fried_rice | it's the "different text" I can't suss. | |
| 15:35:03 | finucannot | fried_rice: Ah, yeah, ref is for internal cross-referencing. My bad. | |
| 15:35:23 | finucannot | fried_rice: It's ugly, but I think the only way is to do something like this | |
| 15:35:43 | finucannot | `some text`_ and `some other text`_ | |
| 15:35:55 | finucannot | .. _some text: https://... | |
| 15:36:20 | finucannot | .. _`some other text`: `some text`_ | |
| 15:36:25 | finucannot | which doesn't really help :/ | |
| 15:37:47 | fried_rice | finucannot: It helps a little I guess. Point being to only specify the https://... in one place. | |
| 15:37:57 | finucannot | fried_rice: Wait, no, got it. `some other text <some text_>`_ | |
| 15:38:14 | finucannot | You need the first underscore to covert that to reference, otherwise it's taken literally | |
| 15:38:30 | finucannot | i.e. you'll end up at '/some text' | |
| 15:38:50 | fried_rice | yeah, that was the result I was getting. Trying.... | |
| 15:39:02 | finucannot | Or from earlier, `the text you want <the-anchor-name_>`_ | |
| 15:39:47 | fried_rice | finucannot: \o/ works! finuCAN! | |
| 15:40:06 | finucannot | Side note: 'sphinx-quickstart' in 2.0 has much less options and is now generating non-crufty conf.py files. Hurrah! | |
| 15:40:23 | fried_rice | what's that? | |
| 15:40:46 | finucannot | It's a tool Sphinx provides to bootstrap a new documentation project | |
| 15:41:05 | finucannot | tox has a similar one (tox-quickstart) inspired by Sphinx | |
| 15:41:18 | fried_rice | ah, cool | |
| 15:50:50 | zigo | mriedem: I proposed the same kind of change on python-novaclient too, but it looks like nobody was paying much attention to Python 3 at the time, and my patch got rejected there, when it was accepted in osc. I use exclusively osc anyway, so I didn't insist much ! :) | |
| 15:51:12 | zigo | Probably more out of lazyness, sorry for that. | |
| 15:52:11 | openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Add instance hard delete https://review.openstack.org/570202 | |
| 15:52:12 | openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Enable rebuild for instances in cell0 https://review.openstack.org/570203 | |
| 15:52:12 | openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Add requested_networks to RequestSpec https://review.openstack.org/570201 | |
| 15:53:56 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: Updates to image-metadata-prefiltering spec https://review.openstack.org/652101 | |
| 15:54:48 | fried_rice | sean-k-mooney, dansmith, jaypipes, alex_xu, finucannot: ^ | |
| 15:55:47 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: Updates to image-metadata-prefiltering spec https://review.openstack.org/652101 | |
| 16:06:38 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Bump to hacking 1.1.0 https://review.openstack.org/651553 | |
| 16:06:39 | openstackgerrit | Stephen Finucane proposed openstack/nova master: hacking: Resolve W503 issues https://review.openstack.org/651555 | |
| 16:06:39 | openstackgerrit | Stephen Finucane proposed openstack/nova master: hacking: Resolve E731 issues https://review.openstack.org/651554 | |
| 16:06:40 | openstackgerrit | Stephen Finucane proposed openstack/nova master: hacking: Resolve W605 (invalid escape sequence) https://review.openstack.org/652104 | |
| 16:06:40 | openstackgerrit | Stephen Finucane proposed openstack/nova master: hacking: Resolve E741 (ambiguous variable name) https://review.openstack.org/652103 | |
| 16:07:53 | finucannot | fried_rice: done | |
| 16:08:05 | finucannot | *pub | |
| 16:17:57 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: Updates to image-metadata-prefiltering spec https://review.openstack.org/652101 | |
| 16:34:32 | openstackgerrit | Dustin Cowles proposed openstack/nova master: WIP/PoC: Introduces the openstacksdk to nova https://review.openstack.org/643664 | |
| 16:34:46 | openstackgerrit | melanie witt proposed openstack/nova master: Warn for duplicate host mappings during discover_hosts https://review.openstack.org/651947 | |
| 16:37:09 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: Updates to image-metadata-prefiltering spec https://review.openstack.org/652101 | |
| 16:53:23 | spatel | sean-k-mooney: hey!!! | |
| 16:53:31 | spatel | How are you doing sean!! | |
| 16:55:33 | spatel | Does Google cloud provide SR-IOV function if anyone aware of it? | |
| 17:11:26 | sean-k-mooney | spatel: am im not sure. i have not looked at gce much | |
| 17:12:00 | sean-k-mooney | spatel: i konw aws whas some offerenings that have gpus and fpgas so they might but not sure where google stands in that regard | |
| 17:12:00 | spatel | I don't think they provide SR-IOV because i am not seeing anywhere mentioned | |
| 17:12:03 | artom | It can be googled in 2 seconds, FWIW... https://stackoverflow.com/questions/33114144/does-google-compute-engine-offer-sr-iov-single-root-i-o-virtualization | |
| 17:12:10 | artom | ... and has nothing to do with OpenStack? | |
| 17:12:44 | spatel | Google using DPDK style enhance networking... if i am not wrong | |
| 17:13:14 | sean-k-mooney | artom: spatel has recently (last 6 months) moved form aws to runign an sriov based openstack cloud | |
| 17:13:35 | sean-k-mooney | artom: im guessing spatel you are interested in comparing it to a few other options | |
| 17:16:08 | melwitt | fried_rice: we've been talking about SchedulerReportClient whether it's heavy to construct, if you might have anything to add https://review.openstack.org/#/c/638073/24/nova/quota.py@1154 | |
| 17:19:49 | spatel | sean-k-mooney: artom we are using aws in many place and costing is killing us so looking for other cloud provide like Google or tencent etc.. | |
| 17:20:20 | spatel | We have requirement of high performance networking so exploring options related SR-IOV | |
| 17:21:05 | spatel | artom: i did read that post already but not sure if Google change something recently and someone aware here so i post question here | |
| 17:21:32 | spatel | That post is 4 year old also :( | |
| 17:25:39 | cdent | fried_rice: what you know of the state of https://bugs.launchpad.net/nova/+bug/1724633 . Still alive? | |
| 17:25:40 | openstack | Launchpad bug 1724633 in OpenStack Compute (nova) "AllocationCandidates.get_by_filters hits incorrectly when traits are split across the main RP and aggregates" [Medium,Confirmed] | |
| 18:13:04 | mriedem | who would have know that our actions api is not completely solid https://bugs.launchpad.net/nova/+bug/1824420 | |
| 18:13:06 | openstack | Launchpad bug 1824420 in OpenStack Compute (nova) "Live migration succeeds but instance-action-list still has unexpected Error status" [Undecided,Triaged] | |
| 18:13:07 | mriedem | *known | |
| 18:28:40 | eandersson | mriedem, if you have some time can you take a stab at backporting that patch? If not I can probably do it. | |
| 18:29:01 | eandersson | The allocation leak from resize/cold migration | |
| 18:29:36 | mriedem | oh yeah sec | |
| 18:33:19 | mriedem | eandersson: btw did you figure out where you had migration failures leaking allocations? | |
| 18:34:04 | eandersson | Yea - we are doing a lot of migration testing, and those are causing it. | |