Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-02-06
14:59:08 tssurya so I will just leave my change as it is then
14:59:28 tssurya and just edit the commit/comment
15:01:58 sean-k-mooney bauzas: we proably should update https://github.com/openstack/nova/blob/master/doc/source/admin/flavors.rst#modify-a-flavor
15:02:32 bauzas I'm out of steam for that but yeah
15:02:46 mriedem bauzas: so add something about that here https://docs.openstack.org/nova/latest/admin/flavors.html#modify-a-flavor
15:03:55 sean-k-mooney bauzas: i can add a note if you like
15:04:08 mriedem you can copy what's in https://specs.openstack.org/openstack/nova-specs/specs/queens/implemented/flavor-description.html if you want
15:04:15 mriedem " Note The only field that can be updated is the description field. Nova has historically intentionally not included an API to update a flavor because that would be confusing for instances already created with that flavor. Needing to change any other aspect of a flavor requires deleting and/or creating a new flavor. "
15:05:29 sean-k-mooney mriedem: ya that makes sense. bauzas want me to do it since your busy or will i leave it to you.
15:05:59 mriedem we can also update the osc command note in https://docs.openstack.org/nova/latest/admin/flavors.html#modify-a-flavor to mention that support was added in openstack CLI in 3.16.0: https://review.openstack.org/#/c/575390/
15:06:17 mriedem or just change the example to use osc
15:06:20 mriedem and drop the nonte
15:06:22 mriedem *note
15:06:30 sean-k-mooney sure
15:06:37 bauzas actually, the spec is a good example
15:06:43 sean-k-mooney ill go do that now
15:06:54 bauzas thanks
15:07:12 sean-k-mooney bauzas: ya you can use the spec as a reference for the customer while i update this
15:07:25 bauzas yeah
15:07:35 bauzas but anyway, I need to provide a hack :(
15:07:40 bauzas and I hate this
15:07:46 sean-k-mooney well you dont
15:07:56 sean-k-mooney we shoud discuss that downstream
15:07:59 bauzas oh yeah, I have to do it
15:08:03 bauzas trust me
15:12:10 stephenfin adrianc: Quick question: how does one identify the "OPN" of a ConnectX-3 NIC?
15:12:25 openstackgerrit Dan Smith proposed openstack/nova master: Fix InstanceMapping to always default queued_for_delete=False https://review.openstack.org/635185
15:12:44 stephenfin I can find the PSID via 'mstflint query' but can't see the OPN http://www.mellanox.com/page/firmware_table_ConnectX3EN
15:12:45 sean-k-mooney stephenfin: OPN?
15:13:08 sean-k-mooney oh its a vendor id
15:13:12 stephenfin sean-k-mooney: The second column in that firmware download list. I assume it's the part ID
15:13:32 sean-k-mooney ya it like intels etag for there firmware
15:13:47 sean-k-mooney at least it looks like it is anyway
15:14:00 stephenfin sean-k-mooney: I think that's only good to distinguish between e.g. the ConnectX-3 and ConnectX-3 Pro, based on http://www.mellanox.com/page/firmware_HCA_FW_identification
15:14:28 stephenfin and the ID I'm getting, 15b3:1003, doesn't give me any deets on https://www.pcilookup.com/?ven=15b3&dev=1003&action=submit
15:15:44 stephenfin adrianc: I had the PSID and just clicked down through the list of OPNs to find what I was looking for. If there's a better way though, I'd be happy to hear about it
15:15:46 stephenfin sean-k-mooney: ^
15:17:02 openstackgerrit Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268
15:17:02 openstackgerrit Balazs Gibizer proposed openstack/nova master: Extend RequestGroup object for mapping https://review.openstack.org/619527
15:17:03 openstackgerrit Balazs Gibizer proposed openstack/nova master: Calculate RequestGroup resource provider mapping https://review.openstack.org/616239
15:17:03 openstackgerrit Balazs Gibizer proposed openstack/nova master: Fill the RequestGroup mapping during schedule https://review.openstack.org/619528
15:17:04 openstackgerrit Balazs Gibizer proposed openstack/nova master: Pass resource provider mapping to neutronv2 api https://review.openstack.org/616240
15:17:04 openstackgerrit Balazs Gibizer proposed openstack/nova master: Recalculate request group - RP mapping during re-schedule https://review.openstack.org/619529
15:17:05 openstackgerrit Balazs Gibizer proposed openstack/nova master: Send RP uuid in the port binding https://review.openstack.org/569459
15:17:05 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317
15:17:06 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove port allocation during detach https://review.openstack.org/622421
15:17:06 openstackgerrit Balazs Gibizer proposed openstack/nova master: Record requester in the InstancePCIRequest https://review.openstack.org/625310
15:17:07 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add pf_interface_name tag to passthrough_whitelist https://review.openstack.org/625311
15:17:07 openstackgerrit Balazs Gibizer proposed openstack/nova master: Ensure that bandwidth and VF are from the same PF https://review.openstack.org/623543
15:17:40 jaypipes stephenfin: ok, will try. (sorry, been out of IRC contact for 6+ days)
15:19:24 stephenfin jaypipes: It's all good. Thanks for even trying
15:30:27 mriedem why oh why doesn't the set -x here https://review.openstack.org/#/c/634962/2/nova/tests/live_migration/hooks/run_tests.sh@12 enable tracing?
15:30:46 mriedem http://logs.openstack.org/62/634962/2/check/legacy-grenade-dsvm-neutron-multinode-live-migration/f8f245f/logs/devstack-gate-post_test_hook.txt.gz#_2019-02-05_22_35_09_439
15:31:12 mriedem oh i bet something downstream unsets it
15:32:53 mriedem aha http://logs.openstack.org/62/634962/2/check/legacy-grenade-dsvm-neutron-multinode-live-migration/f8f245f/logs/devstack-gate-post_test_hook.txt.gz#_2019-02-05_22_35_09_525
15:36:19 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Fix legacy-grenade-dsvm-neutron-multinode-live-migration https://review.openstack.org/634962
15:36:35 bauzas gibi_off: would you mind if I was rebasing https://review.openstack.org/#/c/631559/ on top of the new revision I'm working for https://review.openstack.org/#/c/599208/
15:36:37 bauzas ?
15:37:33 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Allow [pci]alias to take a list https://review.openstack.org/635191
15:37:51 mriedem bauzas: i'm sure he doesn't mind
15:37:57 bauzas ack
15:46:46 openstackgerrit Jack Ding proposed openstack/nova master: Improve libvirt image and snapshot handling https://review.openstack.org/616692
15:55:08 stephenfin sean-k-mooney: https://that.guru/blog/updating-mellanox-connectx-3/
15:55:27 stephenfin adrianc: ^ if you figure that out, I'll update the above
15:59:58 openstackgerrit sean mooney proposed openstack/nova master: update flavor admin docs https://review.openstack.org/635198
16:04:41 sean-k-mooney stephenfin: i assume this should not happen http://paste.openstack.org/show/744632/
16:05:26 stephenfin sean-k-mooney: You've got left over pyc files
16:05:34 openstackgerrit Merged openstack/nova master: Ignore some PendingDeprecationWarnings for os-vif https://review.openstack.org/634928
16:05:35 stephenfin and the interpreter is trying to read them
16:05:43 sean-k-mooney oh even after -r
16:05:46 stephenfin 'find . -type f -name "*.pyc" -delete'
16:05:58 melwitt o/
16:06:35 mriedem sean-k-mooney: stephenfin: we used to have something in tox.ini to delete those on each run to avoid that but someone had an opinion and removed it
16:07:03 sean-k-mooney mriedem: yes i rememebr i was going to check if it was remvoed
16:07:44 stephenfin mriedem: It's still there and that sounds rather passive aggressive. If that's intentional, stop it please. No need for that
16:07:48 stephenfin We just don't do it for docs
16:07:53 sean-k-mooney stephenfin: anyway that seams to have partly fixed it.
16:08:01 sean-k-mooney i now get dot command 'dot' cannot be run (needed for graphviz output), check the graphviz_dot setting
16:08:38 stephenfin sean-k-mooney: Yup, if you fancy sticking a call to '{[testenv]commands}' in the commands for the 'docs' target, it'd be an easy fix to prevent that recurring
16:08:51 stephenfin sean-k-mooney: Guess you need to install dot?
16:08:51 sean-k-mooney ya i can
16:08:54 mriedem huh?
16:09:00 mriedem we don't delete pyc in the tox docs env
16:09:03 mriedem b/c we override commands
16:09:09 mriedem what am i being passive aggressive about?
16:09:30 mriedem i hit this same thing from time to time and have to remember to delete pycs manually which is annoying
16:10:20 sean-k-mooney ill update the docs tox env but i think we need to add graphviz as a docs dep too
16:10:29 mriedem graphviz is a binary
16:10:32 mriedem it's in bindep
16:10:48 sean-k-mooney you are right but why do we now need it for docs
16:10:53 sean-k-mooney we didnt in the past
16:10:59 mriedem we've needed it for a long time
16:11:02 mriedem are you on a new system?
16:11:18 sean-k-mooney im on a system that has never had devstack run
16:11:20 sean-k-mooney so yes
16:11:28 sean-k-mooney ok ill just install it
16:12:09 jackding mriedem: runway for https://review.openstack.org/#/c/620706/ ends tomorrow. I believe I have addressed your comments. Do you have sometime to revisit?
16:12:12 mriedem stephenfin: this is what i was referring to http://lists.openstack.org/pipermail/openstack-dev/2014-September/045873.html
16:12:38 stephenfin mriedem: "we used to have something in tox.ini to delete those on each run to avoid that but someone had an opinion and removed it"
16:12:44 stephenfin mriedem: I figured that was aimed at me, in which case it seemed like I was being singled out for what was almost certainly a change made for good reasons

Earlier   Later