| *** pmannidi has quit IRC | 00:11 | |
| *** r-daneel has quit IRC | 00:17 | |
| *** tuanluong has joined #openstack-ironic | 00:18 | |
| *** bfournie has joined #openstack-ironic | 00:20 | |
| *** bfournie has quit IRC | 00:22 | |
| *** bfournie has joined #openstack-ironic | 00:25 | |
| *** mtanino has quit IRC | 00:25 | |
| *** pmannidi has joined #openstack-ironic | 00:31 | |
| *** rloo has quit IRC | 00:34 | |
| jtanner | what's the right value to set on this for a virtualbmc+libvirt vm? | 00:41 |
|---|---|---|
| jtanner | | console | False | Missing 'ipmi_terminal_port' parameter in node's driver_info. | 00:41 |
| *** shuyingya has joined #openstack-ironic | 00:45 | |
| *** phuongnh has joined #openstack-ironic | 01:04 | |
| *** priteau has joined #openstack-ironic | 01:08 | |
| *** priteau has quit IRC | 01:12 | |
| *** gcb has quit IRC | 01:17 | |
| *** gcb has joined #openstack-ironic | 01:17 | |
| *** zhurong has joined #openstack-ironic | 01:28 | |
| *** SerenaFeng has joined #openstack-ironic | 01:37 | |
| *** crushil has joined #openstack-ironic | 01:40 | |
| *** chlong has quit IRC | 01:50 | |
| *** jiangwei has joined #openstack-ironic | 02:02 | |
| *** jiangwei is now known as FrankJ | 02:04 | |
| FrankJ | hello everyone | 02:04 |
| FrankJ | I met a issue with access mysql db. if I use "ironic node-list", the output is fine. However ,when I use "ironic driver-list", the output is empty" | 02:06 |
| jtanner | do you have nodes? | 02:10 |
| FrankJ | yes | 02:12 |
| FrankJ | I find some problems in mysqldb | 02:12 |
| FrankJ | so I enter mysqldb,and access conductor table, and find that the "update_at" argument has not been update for server days. so when use ''ironic driver-list" command, all records are been filtered | 02:12 |
| jtanner | sorry, no idea then ... i'm new | 02:13 |
| FrankJ | the states of conductors are updated by function<_conductor_service_record_keepalive> | 02:13 |
| FrankJ | I doubt that this green thread has been killed by some operations which I haven't found | 02:15 |
| FrankJ | does anyone met this situation? thanks and thanks jtanner:) | 02:16 |
| *** litao__ has joined #openstack-ironic | 02:16 | |
| *** SerenaFeng has quit IRC | 02:24 | |
| *** r-daneel has joined #openstack-ironic | 02:26 | |
| *** crushil has quit IRC | 02:26 | |
| *** zhurong has quit IRC | 02:30 | |
| *** crushil has joined #openstack-ironic | 02:31 | |
| *** r-daneel has quit IRC | 02:33 | |
| *** mat128 has quit IRC | 02:36 | |
| *** mat128 has joined #openstack-ironic | 02:39 | |
| *** mat128 has quit IRC | 02:47 | |
| *** mat128 has joined #openstack-ironic | 02:48 | |
| *** cdearborn_ has quit IRC | 02:50 | |
| *** shuyingya has quit IRC | 02:50 | |
| *** shuyingya has joined #openstack-ironic | 02:51 | |
| *** links has joined #openstack-ironic | 02:52 | |
| *** mat128 has quit IRC | 02:52 | |
| *** shuyingy_ has joined #openstack-ironic | 02:53 | |
| *** shuyingya has quit IRC | 02:57 | |
| *** vikrant has joined #openstack-ironic | 03:02 | |
| *** shuyingy_ has quit IRC | 03:08 | |
| *** shuyingya has joined #openstack-ironic | 03:08 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 03:09 | |
| openstackgerrit | Anup Navare proposed openstack/ironic-ui master: Display enabled interfaces for underlying driver https://review.openstack.org/467804 | 03:10 |
| *** zackf has joined #openstack-ironic | 03:20 | |
| *** zackf has quit IRC | 03:24 | |
| *** FrankJ has quit IRC | 03:25 | |
| *** jiangwei has joined #openstack-ironic | 03:28 | |
| *** jiangwei has quit IRC | 03:28 | |
| openstackgerrit | Anup Navare proposed openstack/ironic-ui master: Display enabled interfaces for underlying driver https://review.openstack.org/467804 | 03:28 |
| *** FrankJ has joined #openstack-ironic | 03:32 | |
| *** crushil has quit IRC | 03:38 | |
| *** crushil has joined #openstack-ironic | 03:40 | |
| *** crushil has quit IRC | 03:40 | |
| *** zhurong has joined #openstack-ironic | 04:00 | |
| *** shuyingy_ has joined #openstack-ironic | 04:11 | |
| *** pmannidi has quit IRC | 04:12 | |
| *** shuyingya has quit IRC | 04:12 | |
| *** FrankJ has quit IRC | 04:13 | |
| *** Nisha_Agarwal has quit IRC | 04:20 | |
| *** FrankJ has joined #openstack-ironic | 04:26 | |
| *** pmannidi has joined #openstack-ironic | 04:29 | |
| *** hoonetorg has quit IRC | 04:29 | |
| *** zhurong has quit IRC | 04:30 | |
| *** zhurong has joined #openstack-ironic | 04:40 | |
| *** shuyingy_ has quit IRC | 04:42 | |
| *** shuyingya has joined #openstack-ironic | 04:46 | |
| *** hoonetorg has joined #openstack-ironic | 04:46 | |
| *** pmannidi has quit IRC | 04:54 | |
| *** pcaruana has joined #openstack-ironic | 04:56 | |
| *** shuyingy_ has joined #openstack-ironic | 05:00 | |
| *** shuyingya has quit IRC | 05:00 | |
| *** adisky__ has joined #openstack-ironic | 05:05 | |
| *** priteau has joined #openstack-ironic | 05:08 | |
| *** dims has quit IRC | 05:09 | |
| *** pmannidi has joined #openstack-ironic | 05:12 | |
| *** priteau has quit IRC | 05:12 | |
| *** pcaruana has quit IRC | 05:14 | |
| *** FrankJ has quit IRC | 05:18 | |
| *** mvpnitesh has joined #openstack-ironic | 05:36 | |
| *** l_jacob has quit IRC | 05:37 | |
| *** dims has joined #openstack-ironic | 05:41 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 05:43 | |
| openstackgerrit | Debayan Ray proposed openstack/sushy master: Add manager resource https://review.openstack.org/441236 | 05:49 |
| *** mjura has joined #openstack-ironic | 05:54 | |
| *** mjura has quit IRC | 05:54 | |
| *** mjura has joined #openstack-ironic | 05:54 | |
| *** hshiina has joined #openstack-ironic | 06:05 | |
| openstackgerrit | Merged openstack/bifrost master: Ensure ~/.config exists https://review.openstack.org/471328 | 06:07 |
| *** jaosorior_away is now known as jaosorior | 06:11 | |
| openstackgerrit | M V P Nitesh proposed openstack/ironic master: Allow setting {provisioning,cleaning}_network in node driver_info https://review.openstack.org/460564 | 06:13 |
| *** udesale has joined #openstack-ironic | 06:19 | |
| *** rcernin has joined #openstack-ironic | 06:21 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Support hpsum based `update_firmware` for iLO drivers https://review.openstack.org/422572 | 06:24 |
| *** armax has quit IRC | 06:36 | |
| *** hjensas has quit IRC | 06:42 | |
| *** amarao has quit IRC | 06:46 | |
| *** athomas has joined #openstack-ironic | 06:53 | |
| openstackgerrit | Merged openstack/proliantutils master: Redfish: Add system power operations https://review.openstack.org/470353 | 06:57 |
| *** afazekas has quit IRC | 07:01 | |
| *** afazekas has joined #openstack-ironic | 07:01 | |
| *** tuanluong_ has joined #openstack-ironic | 07:01 | |
| *** pcaruana has joined #openstack-ironic | 07:03 | |
| *** tuanluong has quit IRC | 07:04 | |
| *** FrankJ has joined #openstack-ironic | 07:05 | |
| *** FrankJ has quit IRC | 07:05 | |
| *** rbartal has joined #openstack-ironic | 07:09 | |
| *** Nisha_Agarwal has quit IRC | 07:09 | |
| openstackgerrit | Akhil jain proposed openstack/ironic master: Replace get_transport with get_rpc_transport https://review.openstack.org/471603 | 07:10 |
| *** tesseract has joined #openstack-ironic | 07:11 | |
| *** jaganathan has joined #openstack-ironic | 07:16 | |
| openstackgerrit | Akhil jain proposed openstack/ironic master: Replace get_transport with get_rpc_transport https://review.openstack.org/471603 | 07:18 |
| openstackgerrit | Debayan Ray proposed openstack/sushy master: Add manager resource https://review.openstack.org/441236 | 07:27 |
| *** ccamacho has quit IRC | 07:30 | |
| *** fragatina has joined #openstack-ironic | 07:32 | |
| *** ccamacho has joined #openstack-ironic | 07:33 | |
| *** hjensas has joined #openstack-ironic | 07:33 | |
| *** foutatoro has joined #openstack-ironic | 07:39 | |
| pas-ha | morning ironic :) | 07:40 |
| *** rcernin has quit IRC | 07:52 | |
| *** priteau has joined #openstack-ironic | 07:52 | |
| *** rcernin has joined #openstack-ironic | 07:52 | |
| *** tuanluong has joined #openstack-ironic | 07:53 | |
| openstackgerrit | Vu Cong Tuan proposed openstack/sushy master: Change assertTrue(isinstance()) by optimal assert https://review.openstack.org/471617 | 07:55 |
| *** tuanluong_ has quit IRC | 07:55 | |
| *** hw_wutianwei has joined #openstack-ironic | 07:56 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| dtantsur | morning ironic | 08:00 |
| *** fragatina has quit IRC | 08:00 | |
| *** fragatina has joined #openstack-ironic | 08:01 | |
| *** pmannidi has quit IRC | 08:02 | |
| *** mvpnitesh has quit IRC | 08:06 | |
| *** fragatina has quit IRC | 08:06 | |
| *** lucas-afk is now known as lucasagomes | 08:16 | |
| lucasagomes | dtantsur, pas-ha morning! | 08:16 |
| lucasagomes | morning all | 08:16 |
| dtantsur | o/ | 08:18 |
| dtantsur | how's networking life, lucasagomes? :) | 08:18 |
| *** mvpnitesh has joined #openstack-ironic | 08:18 | |
| *** mvpnitesh has quit IRC | 08:25 | |
| lucasagomes | dtantsur, doing well! The good thing is that ovn is kinda new so there are plenty of stuff to do yet | 08:25 |
| *** mgoddard_ has joined #openstack-ironic | 08:26 | |
| dtantsur | cool, yeah. that's what I'd expect | 08:28 |
| *** foutatoro has quit IRC | 08:36 | |
| *** derekh has joined #openstack-ironic | 08:37 | |
| *** mvpnitesh has joined #openstack-ironic | 08:40 | |
| *** amarao has joined #openstack-ironic | 08:44 | |
| *** zhurong has quit IRC | 08:47 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 08:56 | |
| *** shuyingy_ has quit IRC | 08:57 | |
| *** shuyingya has joined #openstack-ironic | 08:58 | |
| mkrai | Hi, A task in Ironic is a thread forked out of conductor service to do some operations. Right? | 09:00 |
| mkrai | Or is it a separate process? | 09:01 |
| dtantsur | mkrai: hi. not necessary, but in many cases this is right. it's a green thread (we're using eventlet) | 09:01 |
| *** zhurong has joined #openstack-ironic | 09:02 | |
| mkrai | dtantsur: Thank you for your response. But what is it in case of console? | 09:02 |
| dtantsur | note that a "task" object in the source code is just a wrapper around node, its ports, and a few other related objects | 09:03 |
| dtantsur | then we often attach a thread to such tasks object to do something on nodes | 09:03 |
| openstackgerrit | Aparna proposed openstack/ironic master: Add API and conductor methods for [un]rescue https://review.openstack.org/350831 | 09:03 |
| dtantsur | but "task" object itself is not a thread, just a handle (optionally, with node lock) | 09:03 |
| dtantsur | mkrai: maybe it ^^^ helps | 09:03 |
| mkrai | dtantsur: yes | 09:04 |
| mkrai | I am trying to understand how the console is provided in Ironic | 09:04 |
| mkrai | Is it a new process that serves node's console? | 09:04 |
| dtantsur | mkrai: serial console? yeah, we lunch a process (socat or shellinabox) attached to node's BMC. | 09:05 |
| dtantsur | and we return its TCP endpoint to a user | 09:05 |
| mkrai | dtantsur: Can you point out the code to this ? | 09:06 |
| mkrai | yes serial console | 09:06 |
| dtantsur | lemme find | 09:06 |
| dtantsur | mkrai: I guess this https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ipmitool.py#L1224-L1409 | 09:07 |
| mkrai | dtantsur: Thank you. I will have a look :) | 09:09 |
| dtantsur | you're welcome :) | 09:09 |
| Nisha_Agarwal | dtantsur, http://logs.openstack.org/72/471072/5/experimental/gate-tempest-dsvm-ironic-ipa-partition-pxe_ipmitool-tinyipa-python3-ubuntu-xenial-nv/6b42b12/logs/screen-ir-cond.txt.gz#_Jun_07_06_54_09_031197 | 09:09 |
| Nisha_Agarwal | dtantsur, i am not sure if a VM created in py3 has some issues .... | 09:11 |
| dtantsur | this looks more concerning: http://logs.openstack.org/72/471072/5/experimental/gate-tempest-dsvm-ironic-ipa-partition-pxe_ipmitool-tinyipa-python3-ubuntu-xenial-nv/6b42b12/logs/screen-ir-cond.txt.gz#_Jun_07_06_56_29_472799 | 09:11 |
| dtantsur | I wonder if virtualbmc works on python 3 | 09:11 |
| dtantsur | lucasagomes: ^^^? | 09:11 |
| * lucasagomes looks | 09:12 | |
| Nisha_Agarwal | dtantsur, true | 09:12 |
| dtantsur | looks ok though http://logs.openstack.org/72/471072/5/experimental/gate-tempest-dsvm-ironic-ipa-partition-pxe_ipmitool-tinyipa-python3-ubuntu-xenial-nv/6b42b12/logs/ironic-bm-logs/virtualbmc.txt.gz | 09:12 |
| Nisha_Agarwal | i looked at that too | 09:12 |
| lucasagomes | dtantsur, oh, last time (it's been long ago) I tried pyghmi with py3 it didn't work | 09:12 |
| dtantsur | Nisha_Agarwal: we can start with addressing http://logs.openstack.org/72/471072/5/experimental/gate-tempest-dsvm-ironic-ipa-partition-pxe_ipmitool-tinyipa-python3-ubuntu-xenial-nv/6b42b12/logs/screen-ir-cond.txt.gz?level=WARNING#_Jun_07_06_56_39_029238 though | 09:13 |
| dtantsur | Nisha_Agarwal: oh, and it lucasagomes is right, we need to make sure to run virtualbmc on python 2 for now | 09:13 |
| dtantsur | s/it/if/ | 09:13 |
| *** FrankJ has joined #openstack-ironic | 09:14 | |
| Nisha_Agarwal | dtantsur, i didnt get....do u mean that but virtualbmc for py2 doesnt work? | 09:14 |
| Nisha_Agarwal | s/but// | 09:14 |
| dtantsur | sambetts|afk: I'm thinking of simplifying the tinyipa build. skip building wheels and just install the python packages in the finalise stage. what we have is too hard to do right in presence of source installation. | 09:14 |
| lucasagomes | Nisha_Agarwal, virtualbmc uses a library called pyghmi for the ipmi stuff | 09:14 |
| lucasagomes | Nisha_Agarwal, and I think that lib does not work with python3 atm | 09:15 |
| Nisha_Agarwal | lucasagomes, yes so we need to address issue for py3....and not for py2(as dtantsur said above it needs to be addresed for py2) | 09:16 |
| dtantsur | no, I said the same as lucasagomes :) | 09:16 |
| lucasagomes | Nisha_Agarwal, well he said that for now we should make sure we run vbmc on python2 | 09:16 |
| lucasagomes | until it's fixed for py3 | 09:16 |
| Nisha_Agarwal | lucasagomes, ok | 09:17 |
| Nisha_Agarwal | dtantsur, :) | 09:17 |
| Nisha_Agarwal | now i got it | 09:17 |
| Nisha_Agarwal | dtantsur, atleast we can have https://review.openstack.org/#/c/471072/ landed as this is required for the stack to come up in py3 | 09:19 |
| patchbot | patch 471072 - ironic - DNM: Experimental change to see if python3 gate pa... | 09:19 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: [imagebuild] support requirements referring to /opt/stack/new https://review.openstack.org/470918 | 09:19 |
| Nisha_Agarwal | i am just asking opinion | 09:19 |
| dtantsur | Nisha_Agarwal: sure, let's start with it (please un-WIP it) | 09:20 |
| Nisha_Agarwal | dtantsur, yes sure | 09:20 |
| Nisha_Agarwal | thanks | 09:20 |
| dtantsur | np | 09:20 |
| mkrai | The gate failure in https://review.openstack.org/#/c/462982/ is known? | 09:22 |
| patchbot | patch 462982 - ironic - Improve driver_info/redfish_verify_ca value valida... | 09:22 |
| dtantsur | mkrai: grenade has been quite unstable recently, but afaik nobody has looked into it. try rechecking. | 09:27 |
| *** aparnav has joined #openstack-ironic | 09:28 | |
| *** e0ne has joined #openstack-ironic | 09:31 | |
| *** hw_wutianwei has quit IRC | 09:45 | |
| mkrai | dtantsur: Ok | 09:48 |
| *** zhurong has quit IRC | 09:51 | |
| *** mvpnitesh has quit IRC | 09:55 | |
| *** mvpnitesh has joined #openstack-ironic | 09:55 | |
| *** ralonsoh has joined #openstack-ironic | 09:55 | |
| *** fragatina has joined #openstack-ironic | 10:00 | |
| *** mvpnitesh has quit IRC | 10:01 | |
| openstackgerrit | Merged openstack/bifrost master: Provide default for test-bifrost.yaml for clouds.yaml https://review.openstack.org/471333 | 10:01 |
| *** Nisha_Agarwal has quit IRC | 10:06 | |
| *** shuyingy_ has joined #openstack-ironic | 10:07 | |
| *** shuyingya has quit IRC | 10:11 | |
| *** lucasagomes has quit IRC | 10:15 | |
| *** mgoddard_ has quit IRC | 10:15 | |
| *** mgoddard has joined #openstack-ironic | 10:15 | |
| *** lucasagomes has joined #openstack-ironic | 10:19 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Improve endpoint interface handling https://review.openstack.org/469627 | 10:22 |
| *** gcb has quit IRC | 10:24 | |
| *** FrankJ has quit IRC | 10:24 | |
| openstackgerrit | Merged openstack/sushy master: Change assertTrue(isinstance()) by optimal assert https://review.openstack.org/471617 | 10:25 |
| *** sai has quit IRC | 10:27 | |
| *** rook has quit IRC | 10:29 | |
| *** rook has joined #openstack-ironic | 10:30 | |
| *** sai has joined #openstack-ironic | 10:30 | |
| *** rook is now known as Guest52590 | 10:30 | |
| sambetts|afk | dtantsur: the reason we build the wheels is to avoid needing to install gcc into the final image which adds >50mb to the final image size | 10:32 |
| *** sambetts|afk is now known as sambetts | 10:32 | |
| sambetts | dtantsur: I'm very confused about why your running into so many issues with it :/ | 10:33 |
| dtantsur | so am I | 10:33 |
| dtantsur | OMG it passed!!! | 10:33 |
| dtantsur | maybe I won't have to change the wheel building | 10:33 |
| sambetts | dtantsur: I'm just really surprised you've had to make any changes to the finalise script at all to make this, work I would have expected it should just have been making build look in a different location for IPA :/ | 10:37 |
| dtantsur | sambetts: building anything using pbr from source requires git | 10:39 |
| dtantsur | and I don't want to introduce git to the final image | 10:39 |
| openstackgerrit | M V P Nitesh proposed openstack/ironic master: Allow setting {provisioning,cleaning}_network in node driver_info https://review.openstack.org/460564 | 10:39 |
| dtantsur | sambetts: look at the hacks I ended up with: https://review.openstack.org/#/c/470918/ | 10:39 |
| patchbot | patch 470918 - ironic-python-agent - [imagebuild] support requirements referring to /op... | 10:39 |
| sambetts | dtantsur: yeah I'm looking, the building only happens in the build stage, surly it doesn't require pbr once it been built into the wheel??? | 10:40 |
| dtantsur | sambetts: yeah, except that -c upper-constraints.txt still makes it look for /opt/stack/new | 10:40 |
| dtantsur | and then when you drop it, you end up with a wheel called ironic_lib, while pip is looking for ironic-lib >_< | 10:41 |
| dtantsur | this is one big nightmare | 10:41 |
| dtantsur | but with this patch we finally can test ironic-lib changes inside IPA \o/ | 10:42 |
| sambetts | dtantsur: I'm surprised that we need to care about upperconstraints in finalise, because we're only installing what has been pre-built for us in the build container | 10:46 |
| dtantsur | sambetts: yes, we do not need it. but then, see point #2 | 10:46 |
| sambetts | the naming thing? | 10:46 |
| dtantsur | yep | 10:47 |
| dtantsur | I spent some time on internet, but I was unable to figure it out | 10:47 |
| dtantsur | but this is why we used to install ironic_python_agent, not ironic-python-agent | 10:47 |
| dtantsur | so it's not new | 10:47 |
| *** shuyingy_ has quit IRC | 10:47 | |
| *** shuyingya has joined #openstack-ironic | 10:48 | |
| dtantsur | very last call to review https://review.openstack.org/#/c/439907/ before I approve it | 10:54 |
| patchbot | patch 439907 - ironic-specs - OSC commands for ironic driver-related commands | 10:54 |
| *** e0ne has quit IRC | 11:04 | |
| *** lucasagomes is now known as lucas-hungry | 11:04 | |
| *** jkilpatr has joined #openstack-ironic | 11:10 | |
| *** jaganathan has quit IRC | 11:27 | |
| TheJulia | Good morning everyone | 11:28 |
| dtantsur | morning TheJulia | 11:31 |
| *** athomas has quit IRC | 11:33 | |
| *** athomas has joined #openstack-ironic | 11:34 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Add API and conductor methods for [un]rescue https://review.openstack.org/350831 | 11:35 |
| sambetts | dtantsur: just testing locally and the naming thing doesn't seem to be a problem, tbh I'm not sure why it pip install ironic_python_agent with underscores before, it works perfectly well with dashes | 11:35 |
| sambetts | dtantsur: pip automagically converts any dashes to underscores automagically | 11:35 |
| dtantsur | sambetts: well, it IS a problem under some conditions, which happen in CI | 11:35 |
| dtantsur | no, pip converts the other way around (at least that's what their documentation says) | 11:36 |
| openstackgerrit | Merged openstack/ironic-specs master: OSC commands for ironic driver-related commands https://review.openstack.org/439907 | 11:37 |
| *** causten has joined #openstack-ironic | 11:37 | |
| *** fragatina has quit IRC | 11:38 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Add rescuewait timeout periodic task https://review.openstack.org/353156 | 11:38 |
| *** litao__ has quit IRC | 11:38 | |
| dtantsur | sambetts: I think you only face this problem with --no-index and wheels saved to a directory. which is a very rare case, but that's how we build tinyipa.. | 11:39 |
| sambetts | dtantsur: hmmm it doesn't seem to be a problem for any of the other packages though there are other packages we install from the wheels which have a dash in the package name but an underscore in the wheel name | 11:39 |
| sambetts | dtantsur: rtslib-fb for example | 11:40 |
| sambetts | dtantsur: the wheel is called rtslib_fb-2.1.63-py2-none-any.whl | 11:40 |
| *** oanson has quit IRC | 11:40 | |
| dtantsur | I have no idea, to be honest | 11:40 |
| sambetts | just like ironic-lib is rtslib_fb-2.1.63-py2-none-any.whl | 11:40 |
| sambetts | ironic_lib-2.7.0-py2-none-any.whl | 11:40 |
| *** Guest52590 is now known as rook | 11:40 | |
| dtantsur | maybe the wheel is built in a strange way | 11:40 |
| *** milan has joined #openstack-ironic | 11:41 | |
| *** oanson has joined #openstack-ironic | 11:42 | |
| dtantsur | sambetts: this is the failure: http://logs.openstack.org/19/470919/1/check/gate-tempest-dsvm-ironic-lib-partition-agent_ipmitool-ubuntu-xenial/e13eae6/logs/devstacklog.txt.gz#_2017-06-06_13_04_04_450 | 11:42 |
| dtantsur | actually | 11:42 |
| dtantsur | maybe it considers it a dev version and hence ignores? | 11:42 |
| dtantsur | I need to check this version as well | 11:43 |
| sambetts | dtantsur: the --pre flag on the pip install should allow it to install dev versions | 11:43 |
| * dtantsur tries again | 11:43 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Agent driver rescue implementation https://review.openstack.org/400437 | 11:44 |
| sambetts | huh that flag is missing in this version for some reason | 11:44 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: [imagebuild] support requirements referring to /opt/stack/new https://review.openstack.org/470918 | 11:46 |
| dtantsur | wut? | 11:46 |
| dtantsur | you mean, in get-pip? | 11:46 |
| sambetts | dtantsur: yeah, you can do $CHROOT_CMD python /tmp/get-pip.py -c /tmp/upper-constraints.txt --no-wheel --no-index --find-links=file:///tmp/wheelhouse --pre ironic_python_agent | 11:47 |
| dtantsur | this, but without upper-constraints, they break everything and are not needed | 11:47 |
| sambetts | if this works that'll be awesome | 11:51 |
| *** tuanluong has quit IRC | 11:55 | |
| *** skazi_ is now known as skazi | 11:58 | |
| *** phuongnh has quit IRC | 11:59 | |
| *** chlong has joined #openstack-ironic | 12:03 | |
| *** causten has quit IRC | 12:03 | |
| *** rbudden has joined #openstack-ironic | 12:04 | |
| *** causten has joined #openstack-ironic | 12:04 | |
| *** causten has quit IRC | 12:04 | |
| *** causten has joined #openstack-ironic | 12:05 | |
| *** causten has quit IRC | 12:05 | |
| *** causten has joined #openstack-ironic | 12:05 | |
| *** ccamacho is now known as ccamacho|lunch | 12:06 | |
| *** causten has quit IRC | 12:06 | |
| *** causten has joined #openstack-ironic | 12:06 | |
| *** ricardoas has joined #openstack-ironic | 12:06 | |
| *** causten has quit IRC | 12:07 | |
| *** causten has joined #openstack-ironic | 12:07 | |
| *** mat128 has joined #openstack-ironic | 12:08 | |
| *** lucas-hungry is now known as lucasagomes | 12:09 | |
| *** causten has quit IRC | 12:12 | |
| *** fragatina has joined #openstack-ironic | 12:12 | |
| *** trown|outtypewww is now known as trown | 12:13 | |
| *** jaganathan has joined #openstack-ironic | 12:17 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Fedora 25 support https://review.openstack.org/471750 | 12:21 |
| TheJulia | jtanner: ^^^ | 12:24 |
| jtanner | morning =) | 12:26 |
| * jtanner is on a long call, but will look at patch after | 12:26 | |
| TheJulia | jtanner: okay, I'm firing up a fresh fc25 vm to test it on | 12:30 |
| TheJulia | sigh... lsb_release not present | 12:33 |
| *** nicodemos has joined #openstack-ironic | 12:34 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Fedora 25 support https://review.openstack.org/471750 | 12:36 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Fedora 25 support https://review.openstack.org/471750 | 12:37 |
| *** shuyingya has quit IRC | 12:42 | |
| *** aparnav has quit IRC | 12:43 | |
| *** jaganathan has quit IRC | 12:43 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Fedora 25 support https://review.openstack.org/471750 | 12:44 |
| TheJulia | jtanner: okay, a few quick edits later and it is running | 12:44 |
| *** vikrant has quit IRC | 12:46 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Fedora 25 support https://review.openstack.org/471750 | 12:47 |
| dtantsur | sambetts: https://review.openstack.org/#/c/470918/ seems to work indeed, please review | 12:49 |
| patchbot | patch 470918 - ironic-python-agent - [imagebuild] support requirements referring to /op... | 12:49 |
| dtantsur | this fixes ironic-lib CI ^^^ | 12:50 |
| sambetts | dtantsur: awesome! I will | 12:50 |
| *** hshiina has quit IRC | 12:52 | |
| jtanner | TheJulia: is github the primary repo for all this stuff or is there a sync going on to another repo? | 12:53 |
| TheJulia | primary repo is git.openstack.org | 12:54 |
| TheJulia | What is on GitHub is just a mirror | 12:54 |
| jtanner | k, that's what i assumed. just wanted to make sure | 12:55 |
| TheJulia | no problem :) | 12:55 |
| jtanner | if i have to patch something, i dont' want to send to the wrong place | 12:55 |
| *** jaganathan has joined #openstack-ironic | 12:56 | |
| *** sasha21 has joined #openstack-ironic | 12:56 | |
| *** ccamacho|lunch is now known as ccamacho | 12:57 | |
| TheJulia | jtanner: via review.openstack.org in that case :) | 12:57 |
| *** trown is now known as trown|brb | 12:58 | |
| *** sasha21 has quit IRC | 13:00 | |
| *** sasha21 has joined #openstack-ironic | 13:01 | |
| *** mrtenio has joined #openstack-ironic | 13:02 | |
| TheJulia | Well test-bifrost.sh exited with a return code of 0. Deployed, Inspected, etc. \o/ | 13:02 |
| *** fellypefca has joined #openstack-ironic | 13:03 | |
| *** e0ne has joined #openstack-ironic | 13:06 | |
| jtanner | TheJulia: you execute with sudo or not? | 13:10 |
| jtanner | btw, i tried ubuntu server 17.04 late last night and it seems to have forgotten/missed installing python-pymysql | 13:11 |
| jtanner | i manually installed it and got past the initial failure, but then it somehow didn't install any of the unit files for systemd | 13:12 |
| jtanner | it died on trying to start ironic-inspector | 13:12 |
| jtanner | fatal: [127.0.0.1]: FAILED! => {"changed": false, "failed": true, "invocation": {"module_args": {"arguments": "", "enabled": true, "name": "ironic-inspector", "pattern": null, "runlevel": "default", "sleep": null, "state": "restarted"}, "module_name": "service"}, "msg": "Error when trying to enable ironic-inspector: rc=1 Failed to enable unit: File ironic-inspector.service: No such file or directory\n"} | 13:12 |
| jtanner | i think i need to learn how to do a "clean" after a failed run | 13:13 |
| *** causten has joined #openstack-ironic | 13:13 | |
| *** trown|brb is now known as trown | 13:13 | |
| *** causten has joined #openstack-ironic | 13:13 | |
| TheJulia | jtanner: as root | 13:19 |
| jtanner | ah | 13:19 |
| TheJulia | so, I guess we could likely change the default for ubuntu to run with systemd | 13:19 |
| jtanner | so you do the checkout and run test as root in all cases then? | 13:19 |
| jtanner | would it make sense to add a Vagrantfile to the repo for people to get the "tried/tested goatpath" ? | 13:20 |
| TheJulia | jtanner: I do, but one should be able to without running as root, since the plays all have become embedded | 13:20 |
| TheJulia | jtanner: already present :) | 13:20 |
| TheJulia | tools/ | 13:20 |
| *** bcornec1 has joined #openstack-ironic | 13:21 | |
| jtanner | holy crap | 13:21 |
| jtanner | why didn't i do a find =( | 13:21 |
| *** baoli has joined #openstack-ironic | 13:23 | |
| *** e0ne has quit IRC | 13:26 | |
| *** e0ne has joined #openstack-ironic | 13:28 | |
| openstackgerrit | vinay kumar muddu proposed openstack/proliantutils master: Add HPE extensions support to redfish connection lib https://review.openstack.org/467912 | 13:28 |
| openstackgerrit | Merged openstack/ironic master: Improve driver_info/redfish_verify_ca value validation https://review.openstack.org/462982 | 13:28 |
| *** jcoufal has joined #openstack-ironic | 13:29 | |
| *** links has quit IRC | 13:30 | |
| *** r-daneel has joined #openstack-ironic | 13:31 | |
| *** jaganathan has quit IRC | 13:32 | |
| *** r-daneel has quit IRC | 13:33 | |
| *** r-daneel has joined #openstack-ironic | 13:33 | |
| TheJulia | dtantsur: vdrok: lucasagomes: did stickers ever arrive? | 13:36 |
| vdrok | good morning everyone! | 13:36 |
| dtantsur | TheJulia: I have an envelope on the reception, I'll try to pick it tomorrow | 13:36 |
| jtanner | TheJulia: another thing i forgot to mention. There's a task that generates an sshkey. It doesn't first check that ~/.ssh exists | 13:36 |
| vdrok | TheJulia: yeah, I did receive them last friday and forgot to write :( | 13:36 |
| vdrok | TheJulia: thank you! apart from the stickers you restored my faith into ukrainian post :) | 13:37 |
| TheJulia | jtanner: Oo | 13:37 |
| TheJulia | Excellent! | 13:37 |
| lucasagomes | TheJulia, hi there, oh I forgot to tell ya | 13:37 |
| lucasagomes | yes they did! thank you very much for them | 13:37 |
| jtanner | TASK [bifrost-openstack-ci-prep : Create an SSH key for Jenkins user if operating in OpenStack CI] *** | 13:38 |
| jtanner | "stderr": "Saving key \"/home/bifrostinstall/.ssh/id_rsa\" failed: No such file or directory" | 13:38 |
| TheJulia | oh, hmm | 13:38 |
| *** Goneri has joined #openstack-ironic | 13:39 | |
| TheJulia | Interesting, that role _should_ only be executing on openstack ci | 13:39 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Physical network aware VIF attachment https://review.openstack.org/470406 | 13:39 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: [WIP] Add tempest tests for physical networks https://review.openstack.org/470915 | 13:39 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Validate portgroup physical network consistency https://review.openstack.org/469932 | 13:39 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Expose ports' physical network attribute in API https://review.openstack.org/469933 | 13:39 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: PoC: Handle rolling upgrades for port ops https://review.openstack.org/471779 | 13:39 |
| TheJulia | I'll have to check. In the mean time, I'm going to go run into town so I can send anupn some stickers | 13:39 |
| *** chlong has quit IRC | 13:43 | |
| *** jaganathan has joined #openstack-ironic | 13:44 | |
| shvepsy | Hello again | 13:45 |
| TheJulia | greetings | 13:45 |
| shvepsy | Please say how i can fix the "GET /v1/v1/ports HTTP/1.1" request from ironic-inspector to ironic-api? | 13:47 |
| TheJulia | shvepsy: sounds like your keystone endpoint has /v1/ in the url? | 13:47 |
| *** rloo has joined #openstack-ironic | 13:47 | |
| shvepsy | v3 | 13:48 |
| shvepsy | or v2.0, both do not work | 13:48 |
| TheJulia | i mean, does the baremetal endpoint have the version in the url record in keystone | 13:48 |
| *** crushil has joined #openstack-ironic | 13:49 | |
| *** zhurong has joined #openstack-ironic | 13:54 | |
| TheJulia | shvepsy: it seems like somewhere along the way the version is defined, in either keystone or in a config file, which I believe is incorrect and may be the cause of what your seeing | 13:58 |
| shvepsy | yes, in endpoint list i see "http://10.0.2.110:6385/v1". However, it is registered under this address when installing | 14:02 |
| shvepsy | can i solve this in config files? | 14:03 |
| *** hjensas has quit IRC | 14:06 | |
| *** bfournie has quit IRC | 14:07 | |
| *** armax has joined #openstack-ironic | 14:10 | |
| *** bfournie has joined #openstack-ironic | 14:14 | |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Unit test framework for Ironic-UI API service https://review.openstack.org/469459 | 14:15 |
| dtantsur | shvepsy: I think it has to be fixed in the installer | 14:16 |
| *** zhurong has quit IRC | 14:17 | |
| yolanda | hi, question, what is the syntax for telling an ironic node to use an image different than overcloud-full.qcow2? | 14:21 |
| dtantsur | yolanda: you tell it to Nova, not to Ironic (well, in your case) | 14:22 |
| dtantsur | when doing 'nova boot' (or via THT, speaking of TripleO) | 14:23 |
| jtanner | TASK [Install Bifrost] ********************************************************* | 14:23 |
| jtanner | fatal: [bifrost]: FAILED! => {"changed": true, "cmd": "source /opt/stack/ansible/hacking/env-setup && ansible-playbook -vvvv -i inventory/localhost install.yaml -e network_interface=eth2", "delta": "0:00:00.003695", "end": "2017-06-07 14:07:59.232816", "failed": true, "rc": 127, "start": "2017-06-07 14:07:59.229121", "stderr": "/bin/bash: ansible-playbook: command not found", "stderr_lines": ["/bin/bash: | 14:23 |
| jtanner | ansible-playbook: command not found"], "stdout": "Sourcing this file is no longer needed! Ansible is always installed from PyPI", "stdout_lines": ["Sourcing this file is no longer needed! Ansible is always installed from PyPI"]} | 14:23 |
| jtanner | i guess the vagrantfile needs some work too? | 14:24 |
| yolanda | dtantsur, we need to tell ironic to use the new security hardened image instead of the normal one | 14:24 |
| *** cdearborn has joined #openstack-ironic | 14:24 | |
| yolanda | i normally just have renamed it to be overcloud-full , and follow the normal process, but i'd like to know if there is a better way | 14:24 |
| dtantsur | yolanda: I understand. My answer is still correct: in tripleo case it's not ironic who picks the image. | 14:24 |
| dtantsur | ironic is told which image to use by nova. and nova is told by heat from THT | 14:24 |
| *** causten has quit IRC | 14:26 | |
| yolanda | ah yes, seems that it can be done with THT | 14:26 |
| *** causten has joined #openstack-ironic | 14:26 | |
| dtantsur | ++ I remember some parameters related to that | 14:27 |
| *** causten_ has joined #openstack-ironic | 14:29 | |
| *** causten has quit IRC | 14:30 | |
| *** jaosorior is now known as jaosorior_away | 14:31 | |
| *** causten_ has quit IRC | 14:32 | |
| *** causten has joined #openstack-ironic | 14:33 | |
| bfournie | dtantsur, yolanda, yes you can change the image name as a property of OS::Nova::Server | 14:35 |
| bfournie | type: OS::Nova::Server | 14:36 |
| bfournie | properties: | 14:36 |
| bfournie | image: overcloud-full | 14:36 |
| *** causten has quit IRC | 14:37 | |
| amarao | vsaenk0: Can you please look into patch for brocade driver of generic switch? https://review.openstack.org/#/c/471800/ | 14:40 |
| patchbot | patch 471800 - networking-generic-switch - Brocade ICX driver: remove port from vlan prior to... | 14:40 |
| amarao | Oops, vsaienk0 | 14:41 |
| *** rbartal has quit IRC | 14:41 | |
| * dtantsur suspects he's on PTO | 14:41 | |
| *** causten has joined #openstack-ironic | 14:41 | |
| amarao | I hope he uses bouncer and see message later. | 14:42 |
| dtantsur | amarao: you can add him to the review (see "Add.." button) | 14:42 |
| amarao | Oh, thanks! | 14:42 |
| sambetts | amarao: can I ask is there a reason why you aren't using https://github.com/openstack/networking-brocade?? | 14:44 |
| amarao | @sambetts: Different hardware and different software inside switch | 14:45 |
| sambetts | amarao: shouldn't the changes to support that hardware be contributed back into networking-brocade? | 14:45 |
| amarao | @sambetts: Brocade ICX is hardware from FastIron. They bought it few years ago, so it's a separate product. And it's not supported by networking-brocade. | 14:46 |
| *** mtanino_ has joined #openstack-ironic | 14:46 | |
| amarao | @sambetts, in networking-brocade it's even stated: The following Mechanism Driver and Plugins are available in this repository: VDX ML2 Mechanism Driver (L2) VDX SVI Driver (L3). | 14:47 |
| amarao | And they costs about order of magnitude more than ICX. | 14:47 |
| *** causten has quit IRC | 14:49 | |
| sambetts | amarao: I'm sure they'd like contributions, they are openstack projects too, and generally networking-* projects have code much better designed for prod environments, vs n-g-s which was built for testing ironic | 14:50 |
| *** causten has joined #openstack-ironic | 14:50 | |
| *** causten has quit IRC | 14:50 | |
| *** causten has joined #openstack-ironic | 14:51 | |
| sambetts | amarao: just some thoughts, I find people tend to discard contributing to the networking-* projects since they were split out | 14:52 |
| amarao | @sambetts: VDX and ICX are absolutely separate products. It's like Microsoft mouse and Microsoft Phone - they have in common only vendor name. | 14:52 |
| sambetts | discribtion of the repo is Brocade plugins/drivers for ML2 not Brocade plugins/drivers for VDX | 14:53 |
| *** causten_ has joined #openstack-ironic | 14:53 | |
| amarao | @sambetts: yes, but ICX has no adequate API and NGN approach (netmiko and go) is much easier to maintain. | 14:55 |
| *** causten has quit IRC | 14:55 | |
| amarao | I'm not a fulltime openstack developer, I'm just an operator who runs public cloud with Ironic and NGN. I can write patch or two, but supporting monsters of networking-brocade is beyond my means. | 14:56 |
| amarao | So it's old school 'scratch your own itch' patch. Vendor does not want us to use ICX with openstack, and it wants us to buy VDX, and we want ICX with openstack. | 14:57 |
| *** causten_ has quit IRC | 14:57 | |
| *** causten has joined #openstack-ironic | 14:58 | |
| amarao | Outside of our case, generic switch approach is absolutely genius. Few lines and you have support for almost any equipment. No complicated things (like RabbitMQ, verisoned objects, schemas, etc) involved. | 14:58 |
| *** causten_ has joined #openstack-ironic | 14:59 | |
| *** rloo has quit IRC | 14:59 | |
| *** mat128 has quit IRC | 15:00 | |
| *** jmorgan1 has quit IRC | 15:00 | |
| *** rloo has joined #openstack-ironic | 15:01 | |
| *** jmorgan1 has joined #openstack-ironic | 15:02 | |
| pas-ha | dtantsur: re that long hunged up story with ssh drivers removal - it seems mitaka is going to be EOLed more or less now, so we need to tag the last commits accross all our projects with 'mitaka-eol' (and obviously stop accepting backports to mitaka) http://p.anticdent.org/logs/openstack-dev?dated=2017-06-07%2012:10:09.470144#39aY | 15:02 |
| pas-ha | althogh I'd clarify with folks first.. | 15:02 |
| dtantsur | I think we've stopped already | 15:02 |
| *** causten has quit IRC | 15:02 | |
| pas-ha | good | 15:02 |
| NobodyCam | Good Morning Ironic'ers | 15:04 |
| *** ralonsoh_ has joined #openstack-ironic | 15:04 | |
| *** rama_y has joined #openstack-ironic | 15:05 | |
| sambetts | amarao: sure... I get that... I don't mean to be coming across harsh it just something I'm pasionate about, I just wish the communities around the networking-* projects would be built up, and contributors that aren't from the vendors who started the project like yourself is what they need, people largely seem to belive that those projects are "owned" by the vendors, when they are just normal | 15:06 |
| sambetts | openstack projects waiting for contributors that are using that equipment to start pushing code into them | 15:06 |
| dtantsur | morning NobodyCam | 15:06 |
| NobodyCam | :) hey hey Good Morning dtantsur :) | 15:06 |
| *** hjensas has joined #openstack-ironic | 15:06 | |
| *** hjensas has joined #openstack-ironic | 15:06 | |
| *** ralonsoh has quit IRC | 15:07 | |
| *** ralonsoh_ is now known as ralonsoh | 15:07 | |
| *** rpioso has joined #openstack-ironic | 15:09 | |
| *** jaypipes has joined #openstack-ironic | 15:09 | |
| amarao | @sambetts: the key problem with most networking-* drivers is that they are mostly maintained by vendor. If vendor does not want something or have some strange ideas about how it should work, it really hard for users (operators/integrators) to add own pieces. For example, all networking-brocade is build around netconf protocol. And ICX does not support it. | 15:10 |
| amarao | (my day is over, good luck to everyone) | 15:11 |
| dtantsur | see you amarao | 15:11 |
| sambetts | amarao: cya, I'll try to review your n-g-s code | 15:11 |
| *** rcernin has quit IRC | 15:11 | |
| pas-ha | amaro: reviewed https://review.openstack.org/#/c/471800 | 15:13 |
| patchbot | patch 471800 - networking-generic-switch - Brocade ICX driver: remove port from vlan prior to... | 15:13 |
| pas-ha | it seems yours is 'brocade_fastiron' in netmiko, so just add an entry-point of netmiko_brocade_fastiron | 15:14 |
| *** amarao has quit IRC | 15:15 | |
| pas-ha | the patch as it is will not work (you can't specify this device in neutron ML2 plugin conf until it has a proper entry-point in NGS setup.cfg) | 15:15 |
| TheJulia | anupn: stickers are on their way! | 15:15 |
| *** mjura has quit IRC | 15:16 | |
| TheJulia | jtanner: hmm... yeah, guess so. If you kill the ansible playbook then you can just ssh. I'll see if I can get that running locally | 15:19 |
| TheJulia | jtanner: I think it would just be removing the source from the shell command in the playbook | 15:23 |
| jtanner | nothing installs ansible prior to that though | 15:24 |
| jtanner | so ansible-playbook is not in the path | 15:24 |
| TheJulia | ahh, very good point | 15:24 |
| TheJulia | I'm pulling down an updated ubuntu image at the moment | 15:25 |
| *** chlong has joined #openstack-ironic | 15:27 | |
| *** dsneddon has joined #openstack-ironic | 15:27 | |
| jtanner | i'm adding a pretask to checkout ansible to a new place and then have the failing task source env-setup from there | 15:31 |
| jtanner | just to see where it goes from there | 15:32 |
| jtanner | well, it completed | 15:34 |
| openstackgerrit | Merged openstack/ironic-python-agent master: [imagebuild] support requirements referring to /opt/stack/new https://review.openstack.org/470918 | 15:35 |
| jtanner | running sudo scripts/test-bifrost.sh as vagrant now | 15:35 |
| *** udesale has quit IRC | 15:36 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent stable/ocata: [imagebuild] support requirements referring to /opt/stack/new https://review.openstack.org/471830 | 15:38 |
| dtantsur | backport party ^^^ | 15:38 |
| jtanner | ok, that fails on missing python-pymysql too | 15:42 |
| TheJulia | jtanner: so env-setup takes care of ansible | 15:43 |
| TheJulia | what fails python-pymysql as well right now? | 15:43 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: PXE boot filtering drivers https://review.openstack.org/439603 | 15:43 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Introducing a dnsmasq PXE filter driver https://review.openstack.org/466448 | 15:43 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: TEST check dnsmasq PXE filter in devstack https://review.openstack.org/468712 | 15:43 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Refactoring the firewall https://review.openstack.org/471831 | 15:43 |
| milan | morning Ironic O:-) | 15:43 |
| jtanner | env-setup is a stub file now because i guess it's expected that ansible is installed via later processes | 15:44 |
| jtanner | "echo Sourcing this file is no longer needed! Ansible is always installed from PyPI" | 15:44 |
| jtanner | ^ content of /opt/stack/ansible/env-setup | 15:44 |
| milan | dtantsur, if you've got a minute for a brief check of the concept: https://review.openstack.org/#/c/439603/19/ironic_inspector/pxe_filter/base.py | 15:45 |
| patchbot | patch 439603 - ironic-inspector - PXE boot filtering drivers | 15:45 |
| TheJulia | jtanner: I'm referring to https://github.com/openstack/bifrost/blob/master/tools/vagrant_dev_env/vagrant.yml#L78 | 15:45 |
| jtanner | hrm | 15:46 |
| *** fragatina has quit IRC | 15:48 | |
| jtanner | for me, {{ansible_install_root}}/ansible/hacking/env-setup expanded to /opt/stack/ansible | 15:48 |
| jtanner | and that directory is not a real checkout | 15:48 |
| NobodyCam | dtantsur: quick question! | 15:50 |
| jtanner | https://github.com/openstack/bifrost/blob/master/scripts/env-setup.sh#L58-L60 | 15:50 |
| jtanner | that's why | 15:50 |
| dtantsur | fire! | 15:50 |
| NobodyCam | on patch to add disabled to add_ports, should the check just log a warning or should it raise and stop? | 15:51 |
| TheJulia | jtanner: yup, because we went to using pip | 15:51 |
| *** mrtenio has quit IRC | 15:51 | |
| TheJulia | looks like the path needs to be fixed for the vagrant playbook execution | 15:52 |
| dtantsur | NobodyCam: raise IMO | 15:52 |
| NobodyCam | :) ++ | 15:52 |
| dtantsur | this is configuration that does not make any sense | 15:52 |
| jtanner | this is how i'm working around | 15:52 |
| jtanner | https://www.irccloud.com/pastebin/tUePQqiT/ | 15:52 |
| NobodyCam | yea! :) | 15:52 |
| jtanner | that patch gets the vagrant play to complete | 15:53 |
| TheJulia | I guess that works, I'm testing fixing the path to where env-setup puts it | 15:53 |
| jtanner | so then i vagrant ssh, cd bifrost, sudo scripts/test-bifrost.sh ... that eventually fails on missing pymsql | 15:53 |
| *** mgoddard has quit IRC | 15:53 | |
| TheJulia | so pymysql should likely just be a range check on the OS version | 15:53 |
| * TheJulia ponders ripping out the range check and just installing it regardless at this point | 15:54 | |
| TheJulia | ugh, I know where this all broke at, urge to put the vagrant file in the ci path rising | 15:58 |
| *** Sukhdev has joined #openstack-ironic | 15:58 | |
| *** sambetts is now known as sambetts|afk | 16:00 | |
| jtanner | TheJulia: doo eet | 16:10 |
| jtanner | just doo eet | 16:10 |
| *** mrtenio has joined #openstack-ironic | 16:11 | |
| jtanner | https://media.giphy.com/media/wi8Ez1mwRcKGI/giphy.gif | 16:11 |
| TheJulia | yup, just hit pymysql. :( | 16:12 |
| *** causten_ has quit IRC | 16:13 | |
| *** causten has joined #openstack-ironic | 16:13 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic-inspector master: add disabled option to VALID_ADD_PORTS_VALUES https://review.openstack.org/471156 | 16:13 |
| *** causten_ has joined #openstack-ironic | 16:16 | |
| *** causten has quit IRC | 16:18 | |
| *** mat128 has joined #openstack-ironic | 16:27 | |
| *** mat128 has quit IRC | 16:29 | |
| *** trown is now known as trown|lunch | 16:30 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Hopefully fix Vagrant https://review.openstack.org/471851 | 16:34 |
| *** tesseract has quit IRC | 16:34 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Rework keystone auth for glance https://review.openstack.org/467728 | 16:35 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Improve endpoint interface handling https://review.openstack.org/469627 | 16:35 |
| *** jkilpatr has quit IRC | 16:36 | |
| *** milan has quit IRC | 16:36 | |
| TheJulia | jtanner: https://review.openstack.org/471851 | 16:37 |
| patchbot | patch 471851 - bifrost - Hopefully fix Vagrant | 16:37 |
| *** lucasagomes is now known as lucas-afk | 16:38 | |
| *** milan has joined #openstack-ironic | 16:40 | |
| *** jkilpatr has joined #openstack-ironic | 16:40 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Rework keystone auth for glance https://review.openstack.org/467728 | 16:41 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Improve endpoint interface handling https://review.openstack.org/469627 | 16:41 |
| *** Sukhdev has quit IRC | 16:42 | |
| openstackgerrit | Michael Turek proposed openstack/ironic master: Wire in storage interface attach/detach operations https://review.openstack.org/406290 | 16:46 |
| *** mgoddard has joined #openstack-ironic | 16:46 | |
| *** milan has quit IRC | 16:51 | |
| rloo | howdy and good morning folks, TheJulia, pas-ha, NobodyCam, dtantsur, jtanner | 16:54 |
| dtantsur | morning rloo | 16:54 |
| pas-ha | morninig rloo :) | 16:54 |
| NobodyCam | Good Morning rloo :) | 16:54 |
| rloo | mgoddard: hey, question about the physnet stuff -- what do you think about if it is none / unset for a port, if there are other ports in portgroup that have physnet set, that the unset ones also use the same physet as the others. is that too complicated/breaks user understanding? | 16:55 |
| *** derekh has quit IRC | 16:56 | |
| *** e0ne has quit IRC | 16:57 | |
| *** rama_y has quit IRC | 17:04 | |
| *** ralonsoh has quit IRC | 17:06 | |
| *** bcornec1 has quit IRC | 17:17 | |
| *** ChubYann has joined #openstack-ironic | 17:19 | |
| *** fragatina has joined #openstack-ironic | 17:20 | |
| *** ChubYann has quit IRC | 17:20 | |
| *** ChubYann has joined #openstack-ironic | 17:20 | |
| *** ChubYann has quit IRC | 17:21 | |
| *** ChubYann has joined #openstack-ironic | 17:21 | |
| *** ChubYann has quit IRC | 17:25 | |
| *** ChubYann has joined #openstack-ironic | 17:26 | |
| *** mat128 has joined #openstack-ironic | 17:27 | |
| mjturek | dtantsur: fyi - I added some replies to patchset 30 in response to some of your comments from 25. Would appreciate a review when you can https://review.openstack.org/406290 | 17:28 |
| patchbot | patch 406290 - ironic - Wire in storage interface attach/detach operations | 17:28 |
| dtantsur | sure, maybe not today though | 17:29 |
| *** rcernin has joined #openstack-ironic | 17:30 | |
| shvepsy | http://paste.openstack.org/show/611752/ - conductor uses credentials from inspector configuration ? | 17:31 |
| *** fragatina has quit IRC | 17:35 | |
| *** trown|lunch is now known as trown | 17:37 | |
| *** e0ne has joined #openstack-ironic | 17:39 | |
| *** rama_y has joined #openstack-ironic | 17:41 | |
| mjturek | dtantsur: no worries, just wanted to give you a heads up | 17:45 |
| openstackgerrit | Merged openstack/ironic-lib master: Updated from global requirements https://review.openstack.org/470097 | 17:47 |
| dtantsur | CI is alive \o/ | 17:50 |
| dtantsur | then good night everyone, see you tomorrow | 17:50 |
| *** dtantsur is now known as dtantsur|afk | 17:50 | |
| rloo | bye dtantsur|afk | 17:51 |
| openstackgerrit | Christopher Dearborn proposed openstack/ironic master: Wait until iDRAC is ready before out-of-band cleaning https://review.openstack.org/466086 | 17:54 |
| rloo | dtantsur|afk: are we ok with only one +2? Just for global requirements changes? https://review.openstack.org/#/c/470097/ | 17:56 |
| *** gsteinmuller has joined #openstack-ironic | 17:56 | |
| patchbot | patch 470097 - ironic-lib - Updated from global requirements (MERGED) | 17:56 |
| mgoddard | hi rloo, sorry I missed your call | 17:58 |
| rloo | mgoddard: no worries. i think i commented with my thoughts on one of your patches | 17:58 |
| rloo | mgoddard: do you want me to repeat it here? :-) Are you ready to discuss? | 17:58 |
| mgoddard | rloo: I'm just reading through the latest ones, give me a few and we can discuss | 17:58 |
| rloo | mgoddard: no worries | 17:59 |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Unit test framework for Ironic-UI API service https://review.openstack.org/469459 | 17:59 |
| *** e0ne has quit IRC | 18:02 | |
| * jtanner is back at it | 18:03 | |
| gsteinmuller | hey there... I was wondering if any of you guys could give me some more information to solve the problem that i've been passing through since last week, described here http://paste.openstack.org/show/611756/ Thanks in advance! | 18:04 |
| mgoddard | rloo: let's discuss :) | 18:05 |
| rloo | mgoddard: ok, got my boxing gloves on. ready... | 18:06 |
| mgoddard | rloo: round 1 | 18:06 |
| rloo | mgoddard: so what do you think of my idea about None == use portgroup's physnet if there is one | 18:06 |
| mgoddard | rloo: so by portgroup's physnet here you mean any port in the group with physnet != None? | 18:07 |
| rloo | mgoddard: right. | 18:07 |
| mgoddard | rloo: because portgroups themselves don't have a physical_network field | 18:07 |
| rloo | mgoddard: since if that is the case, all those ports will have the same physnet | 18:08 |
| rloo | mgoddard: yeah, but conceptually a portgroup's physnet is derived from... | 18:08 |
| mgoddard | rloo: it's a definite hole in the design currently - there is no way to 'bulk' set a portgroup's physnet | 18:08 |
| mgoddard | rloo: it was discussed previously and we talked about a 'virtual' portgroup API | 18:09 |
| mgoddard | rloo: I'm still not sure about that though | 18:09 |
| mgoddard | rloo: the main time when this will crop up is during upgrades. If we need to configure the physnet for an existing portgroup then we don't currently have a pleasant way to do it | 18:10 |
| mgoddard | rloo: we might be able to get away with the remove/change/add dance in other cases but following an upgrade seems too common to force that on people | 18:11 |
| rloo | mgoddard: it isn't just at upgrade - which i assume you mean upgrading to a release with this feature, not upgrades in general | 18:12 |
| mgoddard | rloo: yes, I mean an upgrade to Pike (fingers crossed) | 18:12 |
| mgoddard | rloo: it's not just at upgrade, but the other use cases are far less common | 18:13 |
| rloo | mgoddard: so my example still holds. someone uses pike. new cluster. creates nodes, portgroups, ports. | 18:13 |
| rloo | mgoddard: then they find out about physnets. now what. | 18:13 |
| rloo | mgoddard: they cannot change their ports while in the same portgroup. | 18:13 |
| mgoddard | rloo: I guess that's a reasonable point | 18:14 |
| mgoddard | rloo: I think we're agreed that remove/change/add sucks | 18:14 |
| rloo | mgoddard: yup. | 18:15 |
| rloo | mgoddard: so back to None... | 18:16 |
| mgoddard | rloo: I guess there are two times we check this. 1. When creating/updating a port. 2. When attaching a VIF | 18:16 |
| rloo | mgoddard: you want to treat that to mean 'None' as a valid phynet value. sort of. | 18:16 |
| rloo | mgoddard: not interested in 'checking' wrt None. | 18:16 |
| rloo | mgoddard: first need to know, what does it mean when it is None. | 18:16 |
| rloo | mgoddard: 1. valid value, all ports in portgroup must have same value, None or foobar. | 18:17 |
| rloo | mgoddard: 2. your 'unset' term == None. | 18:17 |
| mgoddard | rloo: it seems reasonable to allow a transient portgroup with None/not None during 1. | 18:17 |
| mgoddard | rloo: when a port's physnet is None, it is in 'physical network unaware mode' | 18:19 |
| *** e0ne has joined #openstack-ironic | 18:19 | |
| mgoddard | rloo: so it should exhibit the same behaviour as before physnet awareness was introduced | 18:20 |
| mgoddard | rloo: this allows us to continue to function following an upgrade to a newly physnet-aware ironic | 18:20 |
| rloo | mgoddard: and what does 'physical network unaware' mean in the context of a portgroup? | 18:20 |
| rloo | mgoddard: i mean, conceptually, is it wrong/doesn't make sense for some ports to be None, some ports to be Foobar, in same portgroup? | 18:21 |
| mgoddard | rloo: I don't think there's any distinction between ports and portgroups in this case. In my model, if all ports in the group have physical network = None, then the portgroup's physnet is None | 18:21 |
| mgoddard | rloo: I don't think it makes sense unless it's transient | 18:22 |
| rloo | mgoddard: how does one define 'transient' then? | 18:23 |
| mgoddard | rloo: so we could allow it during create_port/update_port to support changing a portgroup's physnet | 18:23 |
| mgoddard | rloo: but during VIF attachment we should ensure that all ports have the same physnet. Make sense? | 18:24 |
| rloo | mgoddard: makes sense i guess, but not going to be doable. | 18:24 |
| mgoddard | rloo: why not? | 18:24 |
| rloo | mgoddard: i mean, during vif attachment, what are you going to do, raise an error? or just skip those that don't have the same physnet. | 18:24 |
| mgoddard | rloo: yes, raise an error | 18:25 |
| mgoddard | rloo: it already does, only currently the scenario is "impossible" | 18:25 |
| mgoddard | rloo: sambetts suggested adding a check to the network driver's validate() method to check portgroup/port/physnet consistency | 18:29 |
| *** jaganathan has quit IRC | 18:29 | |
| TheJulia | NobodyCam: If you have a few minutes, please take a look at bifrost revs. I've been pushing up some fixes to help out jtanner | 18:29 |
| rloo | mgoddard: but the code you put in doesn't allow for inconsistencies at the create/update port level. | 18:29 |
| rloo | mgoddard: we cannot relax it at one point, and then error at another point. | 18:29 |
| NobodyCam | TheJulia: Ack... will do in a bit... :) | 18:29 |
| mgoddard | rloo: why not? you can set whatever driver info you like but when you deploy it has to be valid | 18:30 |
| * jtanner might be useful and submit patches at some point | 18:31 | |
| jtanner | we'll see | 18:31 |
| jtanner | still working on "goat path" | 18:31 |
| mgoddard | rloo: my code would need to be modified to support this | 18:31 |
| rloo | mgoddard: that is slightly different. i think. in this case, we're letting in inconsistencies. | 18:31 |
| rloo | driver-info is info that cannot be checked when set. | 18:31 |
| rloo | i mean, we check whatever we can easily do, when the data changes. | 18:31 |
| mgoddard | rloo: well yes, this is why I didn't initially allow any inconsistencies. The fact remains that users need to be able to change or set the physnet of their portgroups | 18:33 |
| rloo | mgoddard: would it make sense to add an overriding portgroup physnet value? | 18:34 |
| TheJulia | jtanner: patches are always appreciated :) | 18:35 |
| mgoddard | rloo: that was the spec's original design but I changed it once I started implementing as it didn't make sense. You've then got one physnet per port in the portgroup and the portgroup itself. It seems redundant | 18:35 |
| TheJulia | jtanner: just bumped the vagrant file to ubuntu xenial and letting it re-run, since I'm now getting a completing installation when doing vagrant up | 18:36 |
| TheJulia | jtanner: I'll rinse/repeat with fedora-25 and centos7.3 afterwards | 18:36 |
| mgoddard | rloo: I'm afraid I must have my dinner and call it a day. If you have any more thoughts your review comments are appreciated :) | 18:37 |
| rloo | mgoddard: hierarchical. overriding seems reasonable. | 18:37 |
| rloo | mgoddard: we need to agree on a good story on this before i can review :-) | 18:37 |
| mgoddard | rloo: having N+1 physnets seems confusing | 18:38 |
| *** Sukhdev has joined #openstack-ironic | 18:38 | |
| mgoddard | rloo: anyways, good night rloo and all | 18:38 |
| rloo | good night mgoddard | 18:38 |
| *** trown is now known as trown|mtg | 18:41 | |
| TheJulia | \o/ for vagrant images without python | 18:42 |
| *** mgoddard has quit IRC | 18:42 | |
| jtanner | hah | 18:48 |
| jtanner | TheJulia: which one? | 18:49 |
| TheJulia | ubuntu/xenial64, I'm sure there is likely python3, but vagrant firing ansible off fails and my immediate thought is just "wow...." | 18:49 |
| jtanner | interesting | 18:50 |
| jtanner | we claim to support py3 with ansible 2.3 | 18:50 |
| jtanner | modules are spotty | 18:50 |
| TheJulia | well, /usr/bin/python is attempting to be directly executed. I've not looked at the mechanisms in ansible as to how it is handled in more recent versions. | 18:51 |
| jtanner | is this ansible installed through pip3 ? | 18:51 |
| TheJulia | no, pip on a mac, so *shrugs* | 18:52 |
| jtanner | now i'm confused =/ | 18:52 |
| jtanner | oh ... | 18:52 |
| jtanner | so ansible fired off on the controlhost (mac) and connected into the box and failed to execute /usr/bin/python ? | 18:53 |
| TheJulia | yup | 18:53 |
| jtanner | k, the different between vagrant provision versus bifrost install is where i got lost | 18:53 |
| jtanner | is there -any- python on it? | 18:54 |
| TheJulia | no clue, I'd have to be able to get into the vagrant machine to be able to find out :) | 18:54 |
| jtanner | it didn't vagrant up? | 18:54 |
| TheJulia | it failed at customization | 18:55 |
| TheJulia | well, I guess I could likely still ssh in | 18:55 |
| * TheJulia will try again a little later with that box image | 18:55 | |
| jtanner | which is the guest extensions compile phase? | 18:55 |
| jtanner | and key management | 18:55 |
| TheJulia | I'm not sure if it got past key management or not to be honest, it is already long gone in my buffer | 18:56 |
| jtanner | no worries | 18:56 |
| *** adisky__ has quit IRC | 19:04 | |
| * jtanner is finally at node introspection with vagrantfile | 19:07 | |
| jtanner | argh, friggn /root/.ssh/known_hosts file missing | 19:14 |
| *** rcernin has quit IRC | 19:17 | |
| *** dsneddon has quit IRC | 19:18 | |
| * TheJulia raises an eyebrow | 19:19 | |
| TheJulia | NobodyCam: food for thought, getting rid of the extra helpers in the vagrant machine stand-up and just going with docs like the virsh example. So someone can just do `vagrant ssh` and roll forward | 19:20 |
| *** dsneddon has joined #openstack-ironic | 19:21 | |
| *** trown|mtg is now known as trown | 19:28 | |
| jtanner | what's the "right" way to clean up after a failed scripts/test-bifrost.sh ? | 19:29 |
| jtanner | should i expect to be able to remediate a failure and re-run or do i need to wipe the box and start over? | 19:30 |
| *** baoli_ has joined #openstack-ironic | 19:31 | |
| *** mat128 has quit IRC | 19:32 | |
| *** baoli has quit IRC | 19:33 | |
| TheJulia | jtanner: shortest path is to undefine any testvm instances in `virsh list` and drop the ironic database. You can add skip_install=true as a variable and the playbooks will skip reinstallation so you an jump right into the process | 19:33 |
| TheJulia | jtanner: ex: virsh undefine testvm1 | 19:34 |
| jtanner | that's what was doing, but i got concerned when the unit files weren't created on the subsequent runs | 19:35 |
| *** rcernin has joined #openstack-ironic | 19:35 | |
| *** rloo has quit IRC | 19:35 | |
| *** rloo has joined #openstack-ironic | 19:35 | |
| TheJulia | what OS? | 19:36 |
| TheJulia | well, distro | 19:36 |
| jtanner | that was my ubuntu 17.04 machine this morning | 19:36 |
| TheJulia | the unit file creation is an issue with service templating we can likely refactor at this point since I think everything is now shipping with systemd | 19:37 |
| jtanner | which is also why i moved to vagrant, so i could destroy/create easier | 19:37 |
| * TheJulia checks if a file exists for 17.04 | 19:37 | |
| jtanner | now i'm at "execute ping step to verify connectivity" | 19:38 |
| jtanner | making progress | 19:39 |
| TheJulia | we've not created a file for 17.04, largely because most of the folks in the community seem to only run LTS releases for the most part. | 19:39 |
| TheJulia | yeah, seems like time to flip that around :( | 19:40 |
| *** trown is now known as trown|brb | 19:42 | |
| *** trown|brb is now known as trown | 19:55 | |
| *** jcoufal has quit IRC | 19:56 | |
| jtanner | what is the credential that is supposed to be used for "cirros" on the test machine? | 19:57 |
| jtanner | the playbook fails to auth for me, and i see no sshpass being used or any keyfile specified | 19:58 |
| jtanner | root user also doesn't have /root/.ssh/id_rsa | 19:58 |
| *** Sukhdev has quit IRC | 20:02 | |
| -openstackstatus- NOTICE: The Gerrit service on review.openstack.org is being restarted now to clear some excessive connection counts while we debug the intermittent request failures reported over the past few minutes | 20:05 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: Add OSC 'baremetal driver property list' command https://review.openstack.org/381153 | 20:08 |
| TheJulia | if you create a public key it should roll that into a configuration drive. I'm trying to remember what cirros is.. something to do with the chicago cubs | 20:12 |
| TheJulia | cubswin:) https://docs.openstack.org/image-guide/obtain-images.html | 20:13 |
| jtanner | test-bifrost.sh builds some sort of image, pushes it to a testvm, and then tries to have ansible connect to it as teh "cirrus" user | 20:15 |
| *** e0ne_ has joined #openstack-ironic | 20:19 | |
| *** Goneri has quit IRC | 20:20 | |
| *** e0ne has quit IRC | 20:22 | |
| TheJulia | yeah, it expects to pull the user's ssh key. I could have sworn that it will create it if it is not present... | 20:23 |
| jtanner | possible that i've screwed up by doing sudo scripts/test-bifrost.sh as "vagrant" instead of root? | 20:24 |
| TheJulia | oh.. yeah | 20:24 |
| TheJulia | I just ran it as vagrant on ubuntu 16.04 without issues | 20:24 |
| jtanner | i failed to write it down, but i hit some other error when i ran as root | 20:24 |
| jtanner | and when i ran as vagrant without sudo | 20:25 |
| jtanner | i'll retry though | 20:25 |
| TheJulia | executing on fc25 as the vagrant user | 20:25 |
| TheJulia | oh, it is the playbook execution for fact collection that fails upon /usr/bin/python. I guess it is just guessing the same version or python path will be on the remote host, or it doesn't think to check over a command socket, or whatever. | 20:26 |
| * TheJulia guesses it just expects the same python version on the target | 20:27 | |
| jtanner | if not /usr/bin/python, we expect user to set ansible_python_interpreter for the host in inventory | 20:27 |
| TheJulia | ahhh! | 20:27 |
| TheJulia | good to know | 20:27 |
| TheJulia | NobodyCam: ^^^^ another good reason to drop the extra playbook steps :) | 20:28 |
| NobodyCam | hummm | 20:32 |
| jtanner | things that make you go ... | 20:34 |
| NobodyCam | :) | 20:36 |
| *** e0ne has joined #openstack-ironic | 20:40 | |
| *** e0ne has quit IRC | 20:42 | |
| *** e0ne_ has quit IRC | 20:42 | |
| *** e0ne has joined #openstack-ironic | 20:44 | |
| jtanner | ".../os_keyston_service.py: Permission denied" as the vagrant user | 20:50 |
| jtanner | digging through code now | 20:51 |
| * TheJulia blinks | 20:51 | |
| *** trown is now known as trown|outtypewww | 20:52 | |
| TheJulia | uhh... wow | 20:52 |
| *** gsteinmuller has quit IRC | 20:52 | |
| TheJulia | embarrassing typo maybe | 20:52 |
| TheJulia | err, no. os_keystone_service | 20:52 |
| jtanner | yeah, my typo | 20:53 |
| *** pcaruana has quit IRC | 20:53 | |
| *** pcaruana has joined #openstack-ironic | 20:53 | |
| TheJulia | I just ran test-bifrost.sh as vagrant on fc25, it had no python2, logged in, did "def install python2" and then "cd bifrost ; ./scripts/test-bifrost.sh" and it completed | 20:53 |
| * TheJulia steps away for a couple of minutes | 20:54 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-lib master: Support for NVMe drives for config drives https://review.openstack.org/471914 | 20:56 |
| *** priteau has quit IRC | 20:58 | |
| jtanner | interesting | 21:00 |
| jtanner | somehow /home/vagrant/bifrost/scripts/../playbooks/library/os_keystone_service.py is root:root 644 | 21:01 |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: Add OSC 'baremetal driver raid property list' cmd https://review.openstack.org/362047 | 21:02 |
| TheJulia | os umask? | 21:10 |
| TheJulia | no | 21:10 |
| TheJulia | hmm | 21:10 |
| TheJulia | I'm kind of stumped since it shouldn't be attempting direct execution as far as I remember | 21:11 |
| jtanner | not sure yet, but i fixed it by hand and reran ... now at node introspection | 21:11 |
| jtanner | oh, it's not "direct" execution i think | 21:11 |
| jtanner | i believe somehow ansible is triggered in the loop and can't load it | 21:11 |
| jtanner | but i dunno yet | 21:11 |
| TheJulia | yeah, it should be read and spit out in a single file to the remote host magic stuff | 21:13 |
| TheJulia | although, those modules are locally ran, so that might change things | 21:13 |
| TheJulia | since shade is required | 21:13 |
| *** pcaruana has quit IRC | 21:16 | |
| *** e0ne has quit IRC | 21:19 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: Add OSC 'baremetal driver property list' command https://review.openstack.org/381153 | 21:20 |
| jtanner | omg! goat path achieved! | 21:21 |
| *** fragatina has joined #openstack-ironic | 21:22 | |
| *** bfournie1 has joined #openstack-ironic | 21:23 | |
| *** betherly_ has joined #openstack-ironic | 21:24 | |
| *** causten_ has quit IRC | 21:25 | |
| *** Sukhdev has joined #openstack-ironic | 21:25 | |
| *** hwoarang_ has joined #openstack-ironic | 21:26 | |
| *** mcarden_ has joined #openstack-ironic | 21:26 | |
| *** JasonF has joined #openstack-ironic | 21:26 | |
| *** JayF has quit IRC | 21:27 | |
| *** JasonF is now known as JayF | 21:27 | |
| TheJulia | jtanner: https://i.imgflip.com/csze5.jpg | 21:27 |
| jtanner | i almost jumped like that | 21:27 |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: Add OSC 'baremetal driver raid property list' cmd https://review.openstack.org/362047 | 21:28 |
| *** causten has joined #openstack-ironic | 21:29 | |
| *** mtreinish has quit IRC | 21:29 | |
| *** betherly has quit IRC | 21:29 | |
| *** bfournie has quit IRC | 21:29 | |
| *** fbouliane has quit IRC | 21:29 | |
| *** hoonetorg has quit IRC | 21:29 | |
| *** tenobreg has quit IRC | 21:29 | |
| *** hwoarang has quit IRC | 21:29 | |
| *** mcarden has quit IRC | 21:29 | |
| *** betherly_ is now known as betherly | 21:29 | |
| TheJulia | awesome! Anyway, i'm going to go make some dinner | 21:30 |
| *** mtreinish has joined #openstack-ironic | 21:30 | |
| *** hoonetorg has joined #openstack-ironic | 21:30 | |
| TheJulia | or decide if I'm going to see the princess bride tonight | 21:30 |
| *** fbouliane has joined #openstack-ironic | 21:30 | |
| *** crushil has quit IRC | 21:31 | |
| *** crushil has joined #openstack-ironic | 21:31 | |
| *** causten has quit IRC | 21:32 | |
| NobodyCam | I loved the princess bride! | 21:32 |
| jtanner | is it back in theaters? | 21:33 |
| *** crushil has quit IRC | 21:36 | |
| TheJulia | jtanner: wakefield 12 in wake forest at 7pm tonight | 21:41 |
| TheJulia | I think they do another showing in 2 weeks | 21:41 |
| jtanner | i think i watched on netflix just like a month ago | 21:42 |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: Add OSC 'baremetal driver raid property list' cmd https://review.openstack.org/362047 | 21:42 |
| NobodyCam | :) | 21:42 |
| *** rcernin has quit IRC | 21:44 | |
| *** cdearborn has quit IRC | 21:50 | |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: added session_key and session_uri attributes for active session that maybe generated depending on platform. Also created functions to create the session_key and session_uri and delete them. Also created one to retrieve them. Added section in _op to check https://review.openstack.org/471930 | 21:57 |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: added functionality to main to create a session, delete a session based on an identifier url, and to delete the active session. Also added functionality to retrieve a session collection, and a session. https://review.openstack.org/471931 | 21:57 |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: added test cases for testing session connection information and for ensuring that if we have a key we are adding it to the headers properly. https://review.openstack.org/471932 | 21:57 |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: added to main test case unit tests for the functionality I provided for sessionservice/session/ and sessioncollection. https://review.openstack.org/471933 | 21:57 |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: added sessionservice resource and json_samples for unittesting of the session and sessionservice https://review.openstack.org/471934 | 21:57 |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: updated readme to include sessionmanagement to list of functionality https://review.openstack.org/471935 | 21:57 |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: added note on utilizing sessions and provided examples https://review.openstack.org/471936 | 21:57 |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: changed some wording in usage https://review.openstack.org/471937 | 21:57 |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: Created unittests for session, sessioncollection and sessionservice and modified json_samples for session and session_service https://review.openstack.org/471938 | 21:57 |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: added more to test cases https://review.openstack.org/471939 | 21:57 |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: added get_session_service function https://review.openstack.org/471940 | 21:57 |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: added oem data. https://review.openstack.org/471941 | 21:57 |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: checking if session_uri is our active one, if it is then we will reset the _conn key and uri. https://review.openstack.org/471942 | 21:57 |
| *** baoli_ has quit IRC | 22:12 | |
| *** jralph_ has joined #openstack-ironic | 22:18 | |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: Implemented Redfish Sessions because some vendor implementations have disabled basic auth and require a Redfish Session to access any resources. This was done by creating the SessionService, SessionCollection and Session functionality. An update to the c https://review.openstack.org/471942 | 22:20 |
| *** Sukhdev has quit IRC | 22:20 | |
| *** sasha21 has quit IRC | 22:25 | |
| *** chlong has quit IRC | 22:32 | |
| *** jkilpatr has quit IRC | 22:34 | |
| *** rpioso has quit IRC | 22:39 | |
| *** dsneddon has quit IRC | 22:42 | |
| *** fragatina has quit IRC | 23:09 | |
| *** openstack has joined #openstack-ironic | 23:14 | |
| *** jaypipes has quit IRC | 23:30 | |
| *** pmannidi has joined #openstack-ironic | 23:30 | |
| *** MattMan has quit IRC | 23:59 | |
| *** MattMan has joined #openstack-ironic | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
