| *** mat128 has joined #openstack-ironic | 00:00 | |
| openstackgerrit | Anup Navare proposed openstack/ironic master: Config drive support for ceph radosgw https://review.openstack.org/407639 | 00:02 |
|---|---|---|
| *** crushil has quit IRC | 00:11 | |
| *** mat128 has quit IRC | 00:13 | |
| *** Fdaisuke has quit IRC | 00:36 | |
| *** joanna__ has quit IRC | 00:37 | |
| *** causten has joined #openstack-ironic | 00:54 | |
| *** causten has quit IRC | 00:54 | |
| *** causten has joined #openstack-ironic | 00:55 | |
| *** phuongnh has joined #openstack-ironic | 01:04 | |
| *** rbudden has joined #openstack-ironic | 01:21 | |
| *** crushil has joined #openstack-ironic | 01:41 | |
| *** crushil has quit IRC | 02:01 | |
| *** mat128 has joined #openstack-ironic | 02:05 | |
| *** mat128 has quit IRC | 02:05 | |
| *** harlowja has quit IRC | 02:06 | |
| *** rloo has quit IRC | 02:06 | |
| *** Sukhdev_ has quit IRC | 02:10 | |
| *** jaganathan has joined #openstack-ironic | 02:26 | |
| *** sayantan_ has joined #openstack-ironic | 02:31 | |
| *** tuanluong has joined #openstack-ironic | 02:44 | |
| *** harlowja has joined #openstack-ironic | 02:50 | |
| *** crushil has joined #openstack-ironic | 02:58 | |
| *** rbudden has quit IRC | 03:01 | |
| *** causten has quit IRC | 03:01 | |
| *** sayanta__ has joined #openstack-ironic | 03:01 | |
| *** causten has joined #openstack-ironic | 03:01 | |
| *** sayantan_ has quit IRC | 03:04 | |
| *** causten_ has joined #openstack-ironic | 03:06 | |
| *** causten has quit IRC | 03:06 | |
| *** links has joined #openstack-ironic | 03:08 | |
| *** causten_ has quit IRC | 03:22 | |
| *** vikrant has joined #openstack-ironic | 03:32 | |
| *** amotoki has joined #openstack-ironic | 03:34 | |
| *** causten has joined #openstack-ironic | 03:43 | |
| *** causten has quit IRC | 03:50 | |
| *** amotoki has quit IRC | 03:50 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Rescue implementation for iLO Virtual Media drivers https://review.openstack.org/453012 | 03:51 |
| *** aparnav has joined #openstack-ironic | 03:54 | |
| *** gkadam has joined #openstack-ironic | 03:55 | |
| *** harlowja has quit IRC | 03:56 | |
| *** Sukhdev has joined #openstack-ironic | 03:59 | |
| *** amotoki has joined #openstack-ironic | 04:01 | |
| *** sabeen has quit IRC | 04:07 | |
| *** harlowja has joined #openstack-ironic | 04:07 | |
| *** sabeen has joined #openstack-ironic | 04:09 | |
| *** nmathew has joined #openstack-ironic | 04:18 | |
| *** amotoki has quit IRC | 04:19 | |
| *** amotoki has joined #openstack-ironic | 04:28 | |
| *** harlowja has quit IRC | 04:33 | |
| *** rcernin has joined #openstack-ironic | 04:45 | |
| *** gkadam has quit IRC | 04:51 | |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic-specs master: Replace WSME with jsonschema for api validation https://review.openstack.org/415894 | 04:55 |
| *** g3ek has quit IRC | 04:55 | |
| *** nmathew- has joined #openstack-ironic | 05:04 | |
| *** g3ek has joined #openstack-ironic | 05:04 | |
| *** vishwana_ has joined #openstack-ironic | 05:05 | |
| *** cvstealth has joined #openstack-ironic | 05:06 | |
| *** alezil_ has joined #openstack-ironic | 05:06 | |
| *** gkadam has joined #openstack-ironic | 05:07 | |
| *** betherly_ has joined #openstack-ironic | 05:07 | |
| *** tuanluong_ has joined #openstack-ironic | 05:09 | |
| *** rcernin has quit IRC | 05:09 | |
| *** rcernin has joined #openstack-ironic | 05:11 | |
| *** harlowja has joined #openstack-ironic | 05:11 | |
| *** crushil has quit IRC | 05:12 | |
| *** bl0m1_ has joined #openstack-ironic | 05:12 | |
| *** patriciadomin_ has joined #openstack-ironic | 05:13 | |
| *** xhku_ has joined #openstack-ironic | 05:13 | |
| *** nmathew has quit IRC | 05:14 | |
| *** vikrant has quit IRC | 05:14 | |
| *** tuanluong has quit IRC | 05:14 | |
| *** ccamacho has quit IRC | 05:14 | |
| *** vishwanathj has quit IRC | 05:14 | |
| *** fbouliane has quit IRC | 05:14 | |
| *** dtantsur|afk has quit IRC | 05:14 | |
| *** retr0h has quit IRC | 05:14 | |
| *** bl0m1 has quit IRC | 05:14 | |
| *** SpamapS has quit IRC | 05:14 | |
| *** betherly has quit IRC | 05:14 | |
| *** xavierr has quit IRC | 05:14 | |
| *** zhenguo has quit IRC | 05:14 | |
| *** cvstealt1 has quit IRC | 05:14 | |
| *** patriciadomin has quit IRC | 05:14 | |
| *** thomasem has quit IRC | 05:14 | |
| *** alezil has quit IRC | 05:14 | |
| *** dtantsur has joined #openstack-ironic | 05:15 | |
| *** adisky_ has joined #openstack-ironic | 05:15 | |
| *** alezil_ is now known as alezil | 05:15 | |
| *** thomasem has joined #openstack-ironic | 05:15 | |
| *** betherly_ is now known as betherly | 05:16 | |
| *** udesale has joined #openstack-ironic | 05:17 | |
| *** sasha2 has quit IRC | 05:18 | |
| *** jaganathan has quit IRC | 05:20 | |
| *** xavierr has joined #openstack-ironic | 05:20 | |
| *** vikrant has joined #openstack-ironic | 05:21 | |
| *** retr0h has joined #openstack-ironic | 05:22 | |
| *** ccamacho has joined #openstack-ironic | 05:22 | |
| *** SpamapS has joined #openstack-ironic | 05:23 | |
| *** ChubYann has quit IRC | 05:23 | |
| *** amotoki has quit IRC | 05:26 | |
| *** amotoki has joined #openstack-ironic | 05:30 | |
| *** Sukhdev has quit IRC | 05:37 | |
| *** tuanluong_ has quit IRC | 05:38 | |
| *** madgoat has joined #openstack-ironic | 05:41 | |
| *** e0ne has joined #openstack-ironic | 05:41 | |
| *** madgoat has left #openstack-ironic | 05:41 | |
| *** udesale__ has joined #openstack-ironic | 05:42 | |
| *** sayanta__ has quit IRC | 05:42 | |
| *** udesale has quit IRC | 05:43 | |
| openstackgerrit | Merged openstack/ironic master: Call clean_up_instance() during node teardown for Agent deploy https://review.openstack.org/450146 | 05:46 |
| *** mtreinish has quit IRC | 05:49 | |
| *** mtreinish has joined #openstack-ironic | 05:50 | |
| *** mtreinish has quit IRC | 05:52 | |
| *** mtreinish has joined #openstack-ironic | 05:56 | |
| *** mjura has joined #openstack-ironic | 05:57 | |
| *** jaosorior_away is now known as jaosorior | 06:01 | |
| *** harlowja has quit IRC | 06:02 | |
| *** sabeen has quit IRC | 06:02 | |
| *** tuanluong has joined #openstack-ironic | 06:06 | |
| *** e0ne has quit IRC | 06:08 | |
| *** udesale__ has quit IRC | 06:08 | |
| *** yogi has joined #openstack-ironic | 06:12 | |
| *** mjura has quit IRC | 06:17 | |
| *** jaganathan has joined #openstack-ironic | 06:19 | |
| *** rbartal has joined #openstack-ironic | 06:23 | |
| pas-ha | morning ironic :) | 06:24 |
| *** tesseract has joined #openstack-ironic | 06:26 | |
| *** dyasny has joined #openstack-ironic | 06:38 | |
| *** Serlex has joined #openstack-ironic | 06:40 | |
| *** mjura has joined #openstack-ironic | 06:43 | |
| *** jcoufal has joined #openstack-ironic | 07:00 | |
| *** amotoki has quit IRC | 07:00 | |
| *** makowals has joined #openstack-ironic | 07:00 | |
| *** mjura has quit IRC | 07:03 | |
| *** amotoki has joined #openstack-ironic | 07:03 | |
| *** athomas has joined #openstack-ironic | 07:05 | |
| *** aarefiev_afk is now known as aarefiev | 07:07 | |
| aarefiev | morning! | 07:07 |
| *** phuongnh has quit IRC | 07:11 | |
| *** tuanluong has quit IRC | 07:11 | |
| *** zhenguo has joined #openstack-ironic | 07:15 | |
| *** pcaruana has joined #openstack-ironic | 07:19 | |
| *** udesale has joined #openstack-ironic | 07:27 | |
| *** jcoufal has quit IRC | 07:29 | |
| *** amotoki has quit IRC | 07:35 | |
| *** amotoki has joined #openstack-ironic | 07:36 | |
| *** udesale__ has joined #openstack-ironic | 07:48 | |
| *** udesale has quit IRC | 07:50 | |
| *** amoralej|off is now known as amoralej | 07:53 | |
| *** ianychoi has quit IRC | 07:54 | |
| *** lucas-afk is now known as lucasagomes | 07:59 | |
| lucasagomes | morning all | 07:59 |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| *** nmathew has joined #openstack-ironic | 08:11 | |
| *** nmathew- has quit IRC | 08:12 | |
| *** snecklifter has quit IRC | 08:13 | |
| *** ralonsoh has joined #openstack-ironic | 08:14 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Use new oslo db EngineFacade https://review.openstack.org/453057 | 08:15 |
| *** ianychoi has joined #openstack-ironic | 08:17 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic-specs master: API Evolution - ETAG identifiers https://review.openstack.org/381991 | 08:19 |
| *** kromanenko_ has joined #openstack-ironic | 08:21 | |
| *** jaianshu has joined #openstack-ironic | 08:24 | |
| *** mgoddard has joined #openstack-ironic | 08:33 | |
| *** derekh has joined #openstack-ironic | 08:40 | |
| *** vills has joined #openstack-ironic | 08:41 | |
| *** erolg has joined #openstack-ironic | 08:46 | |
| dtantsur | morning ironic | 08:47 |
| *** vills has quit IRC | 08:51 | |
| *** priteau has joined #openstack-ironic | 08:57 | |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: [WIP]inventory: Add support for defining custom host groups https://review.openstack.org/450970 | 08:59 |
| *** amarao has joined #openstack-ironic | 08:59 | |
| *** stendulker has joined #openstack-ironic | 09:05 | |
| vdrok | morning ironic, pas-ha aarefiev lucasagomes and dtantsur | 09:05 |
| lucasagomes | o/ | 09:06 |
| aarefiev | morning lucasagomes, dtantsur, vdrok | 09:06 |
| vdrok | jroll: sure, that's not my rfe, I just moved it from blueprint :) | 09:06 |
| *** Nisha_Agarwal has joined #openstack-ironic | 09:07 | |
| *** nmathew has quit IRC | 09:08 | |
| *** tuanluong has joined #openstack-ironic | 09:12 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: Add /baremetal path instead of port 6385 https://review.openstack.org/444337 | 09:13 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: Add /baremetal path instead of port 6385 https://review.openstack.org/444337 | 09:16 |
| *** athomas has quit IRC | 09:23 | |
| *** e0ne has joined #openstack-ironic | 09:24 | |
| openstackgerrit | Merged openstack/ironic master: Updated from global requirements https://review.openstack.org/452471 | 09:29 |
| *** athomas has joined #openstack-ironic | 09:31 | |
| *** pmannidi has quit IRC | 09:31 | |
| *** dtantsur is now known as dtantsur|relocat | 09:35 | |
| *** dtantsur|relocat is now known as dtantsur|bbl | 09:35 | |
| *** nmathew has joined #openstack-ironic | 09:38 | |
| *** garyk has joined #openstack-ironic | 09:41 | |
| garyk | is there a hook script to run before services start on the over cloud deployments? | 09:42 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Refactor iLO drivers code to clean 'boot' and 'deploy' operations https://review.openstack.org/450002 | 09:46 |
| *** Nisha_Agarwal has quit IRC | 09:49 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Use standard deploy interfaces for iscsi_ilo and agent_ilo https://review.openstack.org/450012 | 09:49 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Changes 'deploy' and 'boot' interface for 'pxe_ilo' driver https://review.openstack.org/450013 | 09:53 |
| *** sambetts|afk is now known as sambetts | 10:04 | |
| sambetts | Morning all | 10:04 |
| *** snecklifter has joined #openstack-ironic | 10:07 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Functional tests: add API microversion tests https://review.openstack.org/451685 | 10:10 |
| aarefiev | morning sambetts | 10:10 |
| *** priteau has quit IRC | 10:11 | |
| vdrok | morning sambetts | 10:14 |
| *** rook is now known as rook|mobile | 10:15 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic master: Remove unused methods in common/paths and common/rpc https://review.openstack.org/453094 | 10:15 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi https://review.openstack.org/437299 | 10:15 |
| *** MattMan has quit IRC | 10:18 | |
| *** MattMan has joined #openstack-ironic | 10:18 | |
| *** tuanluong has quit IRC | 10:20 | |
| openstackgerrit | Merged openstack/ironic master: Updates to RPC and object version pinning https://review.openstack.org/452806 | 10:21 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [Devstack] Increase default number of interfaces to 2 https://review.openstack.org/453105 | 10:24 |
| *** mgould|afk is now known as mgould | 10:25 | |
| *** jaganathan has quit IRC | 10:25 | |
| *** priteau has joined #openstack-ironic | 10:30 | |
| *** udesale__ has quit IRC | 10:30 | |
| *** jaganathan has joined #openstack-ironic | 10:32 | |
| *** jkilpatr has quit IRC | 10:38 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Rescue implementation for iLO Virtual Media drivers https://review.openstack.org/453012 | 10:47 |
| garyk | does anyone have a link to an example of using rsync to update information on a controller/compute node? | 10:48 |
| mgould | garyk: what information are you trying to update? | 10:53 |
| garyk | mgould: if possible i would like to try and do two things: 1. copy a package to the controller if possible, then install it. This is the python code for our plugin. 2. overwrite some configuration variiables of neutron | 10:54 |
| garyk | mgould: i am not too savvy with puppet and trying to that to all work is pretty complicated | 10:54 |
| *** lucasagomes is now known as lucas-hungry | 10:56 | |
| *** vikrant has quit IRC | 10:56 | |
| *** dtantsur|bbl is now known as dtantsur | 10:57 | |
| mgould | garyk: hand-managing nodes that are controlled by a configuration management system is a short path to madness | 10:57 |
| *** jaganathan has quit IRC | 10:57 | |
| mgould | but: rsync -az file.rpm node: && ssh root@node yum install file.rpm :-) | 10:58 |
| mgould | for 2, I guess you'd ssh into the node, edit /etc/neutron/neutron.conf and restart neutron | 10:59 |
| mgould | but yeah, expect all this to be overwritten by puppet at some point in the future | 10:59 |
| mgould | wait, is this on an overcloud node? | 11:00 |
| mgould | you'll probably get a better answer in #tripleo | 11:00 |
| garyk | mgould: yes, this is overcloud. i'll ask on tripleo | 11:01 |
| mgould | cool, good luck | 11:02 |
| mgould | I'm pretty sure you can pass config parameters to tripleo-heat-templates | 11:02 |
| mgould | yeah, you create a .yaml file with the parameters you want to set and pass it to `openstack overcloud deploy` with the -e flag | 11:03 |
| TheJulia | I'll second the note that it is a very short path to madness. Puppet is easy if you start with a small problem and just iterate | 11:03 |
| openstackgerrit | Anton Arefiev proposed openstack/python-ironic-inspector-client master: Do not depend on python-openstackclient https://review.openstack.org/453125 | 11:04 |
| TheJulia | Good morning everyone | 11:05 |
| *** mat128 has joined #openstack-ironic | 11:05 | |
| mgould | morning TheJulia | 11:07 |
| TheJulia | Good morning mat128 | 11:07 |
| garyk | TheJulia: do you by any chance have an example of how to copy a file from under cloud to over cloud with puppet? | 11:07 |
| mat128 | hi TheJulia | 11:07 |
| vdrok | morning mgould TheJulia | 11:09 |
| mgould | morning vdrok | 11:09 |
| mgould | garyk: is the file you want to copy an OS package file (rpm/deb)? | 11:11 |
| mgould | because if so, I think puppet has better ways of doing that | 11:12 |
| garyk | mgould: yes. following a suggestion from derekh i see that this can be done by swift. i am checking that out now | 11:13 |
| mgould | garyk: good luck | 11:14 |
| garyk | thanks | 11:15 |
| mgould | garyk: is this a special custom package file, not available via the usual channels? | 11:16 |
| *** jkilpatr has joined #openstack-ironic | 11:18 | |
| *** stendulker has quit IRC | 11:18 | |
| garyk | mgould: at the moment we are between a rock and a hard place. trying to get this working with RHEL, but do not have a RPM built yet. So I am using a tar.gz built from git. | 11:18 |
| garyk | i would like to copy that and then use it | 11:19 |
| mgould | OK; what's in the tar.gz? | 11:19 |
| *** mat128 has quit IRC | 11:19 | |
| *** mat128 has joined #openstack-ironic | 11:21 | |
| sambetts | garyk: you might be able to use a nova user script to curl/wget the file into the instance on first boot, but I'm not sure how to customise that in tripleo | 11:25 |
| TheJulia | garyk: copy no, but one generally writes manifests to execute locally. I mean you could just curl as well, but the file would need to be in place. Puppet you want it all to be hands off in the end | 11:29 |
| garyk | thanks | 11:29 |
| mgould | garyk: are you trying to install a patched neutron on your overcloud nodes? | 11:34 |
| mgould | I know tripleo has support for doing that on the undercloud: https://docs.openstack.org/developer/tripleo-docs/developer/in_progress_review.html | 11:34 |
| mgould | OTOH, I don't remember having much success with that approach when I tried it | 11:34 |
| garyk | mgould: thanks, will check it out | 11:35 |
| mgould | bbl | 11:35 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: WIP: Move netowrk interface calls from deploy interfaces to conductor https://review.openstack.org/453139 | 11:43 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: WIP: Move netowrk interface calls from deploy interfaces to conductor https://review.openstack.org/453139 | 11:45 |
| *** amarao has quit IRC | 11:46 | |
| *** ccamacho is now known as ccamacho|lunch | 11:46 | |
| *** amarao has joined #openstack-ironic | 11:46 | |
| *** jaganathan has joined #openstack-ironic | 11:49 | |
| *** baoli has joined #openstack-ironic | 11:50 | |
| *** baoli has quit IRC | 11:50 | |
| *** baoli has joined #openstack-ironic | 11:50 | |
| *** vills has joined #openstack-ironic | 11:52 | |
| jroll | vdrok: ah right, ok. thanks! | 11:52 |
| jroll | morning everyone | 11:52 |
| TheJulia | Good morning jroll | 11:54 |
| *** vills has quit IRC | 11:56 | |
| vdrok | morning jroll | 11:57 |
| sambetts | o/ jroll vdrok aarefiev mgould | 11:57 |
| dtantsur | morning jroll, TheJulia, sambetts, mgould, vdrok | 11:58 |
| *** jaianshu has quit IRC | 12:01 | |
| *** lucas-hungry is now known as lucasagomes | 12:02 | |
| sambetts | hey dtantsur | 12:02 |
| *** dtantsur is now known as dtantsur|mtg | 12:03 | |
| *** trown|outtypewww is now known as trown | 12:03 | |
| *** amoralej is now known as amoralej|lunch | 12:08 | |
| *** chlong has joined #openstack-ironic | 12:10 | |
| jroll | dtantsur|mtg: fyi, 8.0.0 release https://review.openstack.org/#/c/450954/ | 12:16 |
| dtantsur|mtg | k | 12:18 |
| *** srobert has joined #openstack-ironic | 12:20 | |
| *** dprince has joined #openstack-ironic | 12:22 | |
| *** jkilpatr has quit IRC | 12:24 | |
| *** bfournie has joined #openstack-ironic | 12:25 | |
| *** rbudden has joined #openstack-ironic | 12:27 | |
| *** srobert has quit IRC | 12:32 | |
| *** srobert has joined #openstack-ironic | 12:32 | |
| *** baoli has quit IRC | 12:35 | |
| *** ralonsoh has quit IRC | 12:36 | |
| *** aparnav has quit IRC | 12:36 | |
| *** baoli has joined #openstack-ironic | 12:39 | |
| *** jkilpatr has joined #openstack-ironic | 12:40 | |
| sambetts | TheJulia: hey having issues installing latest bifrost, it appears that bifrost installs the python deps for Ironic into a location not accessible/configured for the Ironic service user | 12:41 |
| sambetts | TheJulia: any ideas? | 12:41 |
| *** nicodemos has joined #openstack-ironic | 12:41 | |
| TheJulia | sambetts: what install options/distro? | 12:42 |
| sambetts | centos7, just following the standard install from the README (with some minor changes like the network interface in the global_vars) | 12:42 |
| TheJulia | Works on ubuntu xenial though... :( | 12:43 |
| sambetts | TheJulia: paramiko is getting installed in <the user I ran the playbooks as>/.local/ and then ironic-api and conductor are being run as the ironic user which can't see the other users .local | 12:44 |
| sambetts | TheJulia: do you run with venv on or off? | 12:44 |
| TheJulia | off | 12:44 |
| sambetts | yeah we're doing the same :/ | 12:44 |
| TheJulia | uhh, paramiko shouldn't be a use requirement for the api and conductor services | 12:44 |
| TheJulia | unless your trying to use agent_ssh? | 12:44 |
| *** rook|mobile is now known as rook-tower | 12:45 | |
| sambetts | hmmm our enabled_drivers is only agent_ipmitool | 12:46 |
| TheJulia | where are you getting the dependency error? | 12:46 |
| sambetts | ironic-api is throwing it | 12:46 |
| TheJulia | funky | 12:46 |
| TheJulia | any chance we can get a backtrace? | 12:47 |
| jroll | O_o | 12:47 |
| TheJulia | and is the staging drivers repo turned on? | 12:47 |
| jroll | oh, we do import paramiko regardless of drivers loaded | 12:48 |
| jroll | in ironic/common/utils.py | 12:48 |
| sambetts | yeah thats were its blowing up | 12:48 |
| TheJulia | gah... | 12:48 |
| TheJulia | why?!? | 12:48 |
| jroll | that said I would expect bifrost to install everything from requirements.txt in the same place, dunno why paramiko would be skipped | 12:48 |
| TheJulia | paramiko is not in requirements.txt | 12:49 |
| sambetts | it is | 12:49 |
| jroll | it is | 12:49 |
| TheJulia | doh, accidently clicked test-req | 12:49 |
| jroll | well, ironic's reqs.txt | 12:49 |
| sambetts | this is traceback I'm seeing: http://paste.openstack.org/show/605393/ | 12:50 |
| mgould | o/ sambetts jroll | 12:50 |
| *** fellypefca has joined #openstack-ironic | 12:50 | |
| jroll | \o | 12:50 |
| TheJulia | Interesting, in a kolla container I take it? | 12:51 |
| sambetts | TheJulia: nope, we're using bifrost to install K8S for kolla hence the name :) | 12:51 |
| TheJulia | heh, okay. I was slightly hopeful there | 12:51 |
| jroll | https://github.com/openstack/bifrost/blob/master/playbooks/roles/bifrost-ironic-install/tasks/install.yml#L110 | 12:52 |
| jroll | there's absolutely no reason that shouldn't ensure paramiko is installed | 12:52 |
| sambetts | jroll: problem is, it is installed for the user that is doing the install | 12:52 |
| sambetts | jroll: but not the service user | 12:52 |
| sambetts | that the services are run undr | 12:53 |
| TheJulia | and now that makes sense... weird | 12:53 |
| jroll | hm, right | 12:53 |
| jroll | how are you installing bifrost's requirements? pip install --user? | 12:53 |
| sambetts | it gets installed into the weird {HOME}/.local directory | 12:53 |
| jroll | I understand that, I'm trying to understand how that's happening :) | 12:54 |
| TheJulia | I bet the pip that is being executed is the one in ~/.local | 12:54 |
| TheJulia | pas-ha: ^^^ fyi | 12:54 |
| jroll | ah, env-setup.sh mucks with that | 12:54 |
| * jroll screams always use virtualenvs | 12:55 | |
| *** jcoufal has joined #openstack-ironic | 12:55 | |
| sambetts | paramiko is one of ansibles requirements I guess so it gets installed in there as part of that step, and then when it reachs the install Ironic part it just skips installing it | 12:55 |
| TheJulia | sambetts: any chance I can get an install console output/log? | 12:56 |
| jroll | yeah, because it's running from ~/.local, I assume you're using env-setup.sh? | 12:56 |
| jroll | s/running/running ansible-playbook/ | 12:56 |
| sambetts | yup we use the env-setup.sh | 12:56 |
| jroll | ya makes sense | 12:57 |
| TheJulia | frack, that was to prevent other issues elsewhere, I just don't remember what | 12:57 |
| jroll | I'm surprised user packages are included but not site packages | 12:57 |
| * TheJulia logs into her xenial test machine to check something | 12:57 | |
| TheJulia | I'm also kind of surprised opnvf ci didn't catch this, but they might already have paramiko on a system level before we even execute | 12:58 |
| *** ccamacho|lunch is now known as ccamacho | 12:58 | |
| TheJulia | heh, I installed it as root so /root/.local is blocked off from anything touching it based on file system permissions | 13:00 |
| TheJulia | and the only thing in /root/.local is ansible | 13:01 |
| *** dtantsur|mtg is now known as dtantsur | 13:01 | |
| TheJulia | like.. the literal only thing | 13:01 |
| sambetts | TheJulia: we initally attempted to install bifrost using the root user, but install.yml fails every time because it says the SUDO_USER variable doesn't exist | 13:01 |
| * pas-ha reading scroll log | 13:01 | |
| TheJulia | heh | 13:02 |
| TheJulia | yeah, i hit that error on this machine too | 13:02 |
| TheJulia | I logged it, but have not fixed it yet | 13:02 |
| pas-ha | sambetts: the install.yaml playbook forces become on all tasks | 13:02 |
| pas-ha | that was actually the the roadmap - to use become more sparringly | 13:02 |
| *** lhinds|afk is now known as lhinds | 13:03 | |
| jroll | oh, I'm dumb, "no site packages" is a virtualenv thing, not a pip thing | 13:03 |
| TheJulia | sambetts: fwiw, that error is just from the creation of clouds.yaml | 13:03 |
| *** yogi has quit IRC | 13:04 | |
| *** links has quit IRC | 13:04 | |
| TheJulia | sambetts: exact command line you executed ansible-playbook with would be useful | 13:05 |
| TheJulia | for the initial install that is | 13:05 |
| TheJulia | or whatever install your working with right now | 13:05 |
| sambetts | TheJulia: ansible-playbook -K -vvvv -i inventory/target install.yaml | 13:05 |
| sambetts | TheJulia: straight out of the README :) | 13:06 |
| TheJulia | awesome | 13:06 |
| TheJulia | and this is the moment I wish I had 32GB of ram in my laptop | 13:07 |
| *** rloo has joined #openstack-ironic | 13:07 | |
| jroll | TheJulia: if you need to borrow a vm, happy to give you one | 13:07 |
| pas-ha | TheJulia: it seems on OPNFV they already have "some" ansible installed on nodes (I think I remember seeing this) so paramiko must be already there | 13:08 |
| TheJulia | jroll: I just wish I had one already running on my laptop so I could go poke around inside real quick. The whole instant gratification desire | 13:09 |
| *** Goneri has joined #openstack-ironic | 13:09 | |
| * jroll stands by ready to build a 30gb vm | 13:09 | |
| pas-ha | installing into .local was to install ansible from pip w/o venv but not to squash a possibly existing version installed in the system | 13:09 |
| jroll | ah, ok | 13:09 |
| jroll | they only take 2 minutes ¯\_(ツ)_/¯ | 13:09 |
| * pas-ha restoring a clean xenial snapshot | 13:10 | |
| TheJulia | jroll: and coffee, breakfast is needed | 13:10 |
| jroll | fair enough :P | 13:10 |
| vsaienk0 | sambetts: looks like we have an mtu issue after switching to tap interfaces http://logs.openstack.org/19/440719/18/check/gate-tempest-dsvm-ironic-inspector-ubuntu-xenial-nv/9c096db/logs/syslog.txt.gz#_Mar_30_14_43_13 | 13:14 |
| *** amoralej|lunch is now known as amoralej | 13:14 | |
| *** amotoki has quit IRC | 13:15 | |
| *** ricardoas has left #openstack-ironic | 13:17 | |
| sambetts | vsaienk0: :/ do we set the MTU of the brbm bridge? | 13:18 |
| *** ricardoas has joined #openstack-ironic | 13:18 | |
| vsaienk0 | sambetts: we do | 13:18 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Node should reflect what was saved https://review.openstack.org/452930 | 13:20 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-inspector master: Provide correct MTU via DHCP for clients https://review.openstack.org/453166 | 13:21 |
| sambetts | vsaienk0: we set the MTU on the tap interfaces too... | 13:22 |
| sambetts | vsaienk0: i've never hit an MTU issue when I put this patch together because I ensured that the MTU was set everywhere :/ | 13:23 |
| vsaienk0 | sambetts: the only one difference I found is that neutron provides MTU via DHCP to clients | 13:23 |
| sambetts | vsaienk0: that should be set using the PUBLIC_BRIDGE_MTU though :/ | 13:24 |
| vsaienk0 | sambetts: inspector job was always red on that patch | 13:24 |
| sambetts | vsaienk0: oooh I bet we need to set the MTU on the br-inspector | 13:24 |
| sambetts | vsaienk0: in my full patch I made the inspector plumb into the same bridge as the nodes and set the MTU: https://review.openstack.org/#/c/421288/5/devstack/plugin.sh | 13:25 |
| *** jaganathan has quit IRC | 13:25 | |
| vsaienk0 | sambetts: br-inspector is just veth pair, but setting MTU on it might help | 13:25 |
| sambetts | I think that should resolve it | 13:26 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [Devstack] Increase default NIC numbers for VMs to 2 https://review.openstack.org/453105 | 13:27 |
| rloo | hi and good morning all, sambetts, vsaienk0, pas-ha, TheJulia, jroll | 13:31 |
| mgould | hi rloo | 13:32 |
| rloo | hi mgould! | 13:32 |
| jroll | morning \o | 13:32 |
| dtantsur | morning rloo | 13:32 |
| *** crushil has joined #openstack-ironic | 13:33 | |
| rloo | galyna: it might be good to get your opinion on this, since you had left a TODO in https://review.openstack.org/#/c/452930/2/ironic/objects/node.py | 13:33 |
| rloo | hi dtantsur. wrt trello, since no one seems to be keeping it updated, i think we shouldn't use it any more. we forgot to vote on it. do we need to? if so, i can add to next week's agenda. | 13:34 |
| vsaienk0 | morning rloo | 13:35 |
| dtantsur | rloo, yes please | 13:35 |
| rloo | dtantsur: yes to which part? (teach me to mention too much stuff at once) | 13:35 |
| jroll | rloo: dtantsur: here's an idea. I was doing a trello board per cycle. nobody has made one for pike. let's say if they want one, they can step up and make it. | 13:36 |
| dtantsur | rloo, "i can add to next week's agenda" part (sorry, I should have read more carefully) | 13:36 |
| jroll | can't vote to kill a trello board that doesn't exist :) | 13:36 |
| *** jaosorior is now known as jaosorior_away | 13:36 | |
| dtantsur | jroll, right :) | 13:36 |
| *** litao has joined #openstack-ironic | 13:36 | |
| rloo | jroll, dtantsur: ok, i'll put an item in the agenda about that. the problem is that the trello link exists and is in some of our subteam status but no idea if they are up to date or not so somewhat useless :-( | 13:37 |
| jroll | I'm happy to delete everything from trello :) | 13:37 |
| dtantsur | rloo, I think removing wrong links is something we should do right now | 13:37 |
| jroll | rloo: like docs, I tend to think it's better to not exist than be wrong | 13:38 |
| rloo | jroll, dtantsur: i agree. but give me a few secs before you remove the trello links, there are some useful ones there for rolling upgrades, i will copy/paste first... | 13:39 |
| dtantsur | I'm not removing anything, I'm buried in meetings today.. | 13:39 |
| *** sabeen has joined #openstack-ironic | 13:39 | |
| galyna | ruby: yes, I wanted to erase this and test . this fix seems to be not needed since this patch was merged https://review.openstack.org/#/c/354744/ | 13:39 |
| jroll | rloo: I'm not removing the links, I will delete the trello artifacts when you're ready though | 13:39 |
| galyna | thanks for the remind | 13:39 |
| *** fellypefca has quit IRC | 13:45 | |
| rloo | jroll: the trello stuff can wait, i only updated the rolling upgrades stuff. which i should really focus on. so i'll deal with the trello stuff next week... | 13:46 |
| jroll | rloo: okay | 13:46 |
| thomasem | Good morning! | 13:47 |
| *** garyk has quit IRC | 13:47 | |
| rloo | galyna: i think 354744 addresses a diff issue. | 13:47 |
| yuriyz | hi ironicers rloo I change my mind about db reconnect on getters, comment here https://review.openstack.org/#/c/430351/ | 13:48 |
| galyna | rloo: but it may cover updated_at issue... if no, it can be left as it is unless better solution is implemented | 13:48 |
| rloo | galyna: please comment in the patch so we can keep track of it. otherwise, i'll have to link to this irc conversation | 13:50 |
| *** amotoki has joined #openstack-ironic | 13:50 | |
| galyna | rloo: pk, no problem | 13:50 |
| rloo | yuriyz: ok, i'm fine with that :) we just need to document the reasoning; i'll comment in the patch. thx. | 13:51 |
| *** nmathew has quit IRC | 13:53 | |
| galyna | rloo: done, thanks for this update, I think, it works fine | 13:58 |
| *** amotoki has quit IRC | 13:59 | |
| *** cdearborn has joined #openstack-ironic | 13:59 | |
| rloo | thx galyna! | 14:00 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-inspector master: WIP: testing https://review.openstack.org/453166 | 14:00 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: WIP: Initial neutron task changes https://review.openstack.org/452515 | 14:07 |
| *** causten has joined #openstack-ironic | 14:11 | |
| *** causten has quit IRC | 14:11 | |
| *** rbartal has quit IRC | 14:11 | |
| *** causten has joined #openstack-ironic | 14:11 | |
| openstackgerrit | Merged openstack/ironic master: Add assertion of name to test_list_portgroups test https://review.openstack.org/451796 | 14:12 |
| *** rama_y_ has joined #openstack-ironic | 14:14 | |
| *** jaganathan has joined #openstack-ironic | 14:17 | |
| *** amotoki has joined #openstack-ironic | 14:17 | |
| *** gkadam has quit IRC | 14:19 | |
| vdrok | morning rloo | 14:23 |
| *** e0ne_ has joined #openstack-ironic | 14:23 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: WIP: Move netowrk interface calls from deploy interfaces to conductor https://review.openstack.org/453139 | 14:23 |
| *** jaganathan has quit IRC | 14:24 | |
| *** e0ne has quit IRC | 14:25 | |
| rloo | hi vdrok! | 14:26 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: WIP: SSL TEST https://review.openstack.org/442402 | 14:26 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Provide fallback variable if sudo is not used https://review.openstack.org/453196 | 14:27 |
| TheJulia | good morning rloo | 14:28 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-inspector master: [Devstack]: pick correct MTU for br-inspector veth https://review.openstack.org/453166 | 14:29 |
| *** crushil has quit IRC | 14:32 | |
| *** ralonsoh has joined #openstack-ironic | 14:32 | |
| ricardoas | morning! | 14:34 |
| ricardoas | i got a quick question wrt oneview drivers... | 14:35 |
| TheJulia | ricardoas: We might be able to answer :) | 14:36 |
| ricardoas | according to https://github.com/openstack/ironic-specs/blob/master/specs/approved/oneview-drivers-dynamic-allocation.rst we should remove the deprecated pre-allocation model from our driver in this cycle | 14:36 |
| *** fellypefca has joined #openstack-ironic | 14:36 | |
| ricardoas | how should i proceed with this? open a rfe? | 14:37 |
| *** crushil has joined #openstack-ironic | 14:37 | |
| vdrok | I'd say, just remove it :) | 14:37 |
| vdrok | and morning ricardoas | 14:37 |
| *** ElCoyote_ has left #openstack-ironic | 14:37 | |
| jroll | yeah, no need for RFE IMO | 14:38 |
| dtantsur | folks, these warnings seem suspicious: http://logs.openstack.org/04/450104/5/check/gate-ironic-dsvm-standalone-ubuntu-xenial-nv/5692107/logs/screen-ir-cond.txt.gz?#_2017-03-31_12_53_56_654 | 14:38 |
| vdrok | hrm, list_vifs | 14:38 |
| ricardoas | ok, vdrok... :) | 14:38 |
| *** sasha2 has joined #openstack-ironic | 14:38 | |
| dtantsur | similar thing in non-standalone jobs: http://logs.openstack.org/04/450104/5/check/gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-ubuntu-xenial/183d04b/logs/screen-ir-cond.txt.gz?#_2017-03-31_12_50_11_940 | 14:38 |
| dtantsur | jroll, we may want to figure out the source of ^^^ and fix it before the release | 14:39 |
| jroll | O_o | 14:39 |
| vdrok | maybe we need to convert VIF to a proper ovo | 14:39 |
| vdrok | but, it's just a dict, should be serializablr | 14:39 |
| ricardoas | thanks, jroll | 14:40 |
| *** ElCoyote_ has joined #openstack-ironic | 14:40 | |
| jroll | yeah, that's strange, looking at the code it seems like it should be fine | 14:41 |
| jroll | >>> isinstance(u'hi', str) | 14:42 |
| jroll | False | 14:42 |
| jroll | I wonder if that's the problem :) | 14:42 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: Add /baremetal path instead of port 6385 https://review.openstack.org/444337 | 14:42 |
| * jroll puts up test patch | 14:43 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi https://review.openstack.org/437299 | 14:43 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Fix serialization warnings for unicode strings https://review.openstack.org/453205 | 14:45 |
| jroll | vdrok: dtantsur: ^ let's see how that behaves | 14:46 |
| openstackgerrit | Stenio Araujo proposed openstack/python-oneviewclient master: Reenable the dynamic allocation validation methods https://review.openstack.org/453206 | 14:46 |
| dtantsur | jroll, thanks! | 14:46 |
| jroll | np | 14:46 |
| jroll | dtantsur: shall I WIP the release? | 14:46 |
| dtantsur | jroll, I'd say yes | 14:46 |
| jroll | ok | 14:46 |
| jroll | done | 14:47 |
| * jroll steps away for a bit | 14:47 | |
| *** srobert_ has joined #openstack-ironic | 14:47 | |
| *** srobert_ has quit IRC | 14:47 | |
| *** srobert has quit IRC | 14:49 | |
| *** crushil has quit IRC | 14:53 | |
| *** causten has quit IRC | 14:56 | |
| *** crushil has joined #openstack-ironic | 14:57 | |
| *** causten has joined #openstack-ironic | 14:58 | |
| sambetts | Am I correct when I say that Ironic doesn't (currently) have any say about what kernel options are passed in when we install grub2 for local boot? | 15:01 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Detailed sequence-diagram for deploy process https://review.openstack.org/414512 | 15:01 |
| *** rpioso has joined #openstack-ironic | 15:04 | |
| *** causten has quit IRC | 15:05 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: WIP: Move netowrk interface calls from deploy interfaces to conductor https://review.openstack.org/453139 | 15:07 |
| vdrok | good night everyone! | 15:07 |
| sambetts | cya vdrok o/ | 15:08 |
| *** causten has joined #openstack-ironic | 15:08 | |
| dtantsur | sambetts, this is correct, and a pain point for certain folks. yolanda was looking into it as part of the deploy steps work. | 15:11 |
| sambetts | dtantsur: ok cool, I'm just reviewing a patch which add docs for how to change the kernel parameters for enabling console support, and it currently only talks about the netboot case because thats the only case we can influence | 15:12 |
| dtantsur | yep, which is another #sadpanda point | 15:13 |
| *** jaganathan has joined #openstack-ironic | 15:14 | |
| sambetts | :( | 15:17 |
| *** [1]cdearborn has joined #openstack-ironic | 15:17 | |
| *** baoli has quit IRC | 15:18 | |
| *** ralonsoh_ has joined #openstack-ironic | 15:19 | |
| *** jaganathan has quit IRC | 15:21 | |
| *** joanna__ has joined #openstack-ironic | 15:22 | |
| *** ralonsoh has quit IRC | 15:23 | |
| joanna | morning rloo, vdrok , jroll, dtantsur, sambetts, TheJulia :) | 15:25 |
| *** ralonsoh_ is now known as ralonsoh | 15:25 | |
| jroll | hey joanna :) | 15:25 |
| *** e0ne_ has quit IRC | 15:25 | |
| rloo | morning joanna! | 15:25 |
| sambetts | o/ joanna | 15:25 |
| dtantsur | morning joanna | 15:27 |
| mgould | morning joanna | 15:28 |
| joanna | morning mgould | 15:28 |
| *** rama_y_ has quit IRC | 15:29 | |
| *** sayantan_ has joined #openstack-ironic | 15:30 | |
| dtantsur | vdrok, sambetts, re https://review.openstack.org/#/c/451506/ it seems like all the checks get revered order now, no? | 15:31 |
| dtantsur | IIRC decorators are applied from inner to outer, not from top to bottom | 15:31 |
| *** cdearborn has quit IRC | 15:32 | |
| mgould | dtantsur: yep: http://stackoverflow.com/questions/27342149/decorator-execution-order | 15:34 |
| mgould | decorators wrap the function below them | 15:35 |
| mgould | so @d1 @d2 f() means d1(d2(f)) | 15:35 |
| dtantsur | ah, but it's ok in this case, because we do expect d1 to be called first | 15:35 |
| dtantsur | sorry for the noise | 15:35 |
| TheJulia | good morning joanna | 15:36 |
| dtantsur | mariojv, JayF, jroll, TheJulia, re https://review.openstack.org/#/c/448661/: did we really agree to change the default version for Python API? | 15:39 |
| dtantsur | I remember quite the opposite, to be honest.. | 15:39 |
| *** baha has joined #openstack-ironic | 15:39 | |
| dtantsur | this make ~ every release of ironicclient potentially breaking.. | 15:39 |
| TheJulia | I don't remember exactly, but the pin is a pain because it is at client initialization, not on a per call level | 15:40 |
| * mariojv looks at notes | 15:40 | |
| TheJulia | It really, truly, creates more pain and discourages use | 15:40 |
| dtantsur | well, it's by design | 15:40 |
| TheJulia | it is bad design | 15:40 |
| TheJulia | Truthfully | 15:40 |
| dtantsur | I've been talking about it since the beginning ;) | 15:41 |
| TheJulia | We've just never, as a group, come to any consensus | 15:41 |
| dtantsur | it's a bad design, but that's the design we have, and I don't remember a lot of people disagreeing as much as I did | 15:41 |
| mariojv | it's only breaking if you don't specify the API version | 15:41 |
| TheJulia | Lets put it this way, I tried to start hacking on code for shade to do ports, and realized I was in for a world of hurt because of the pin | 15:41 |
| *** amarao has quit IRC | 15:42 | |
| TheJulia | ports/vifs | 15:42 |
| dtantsur | I hear you, it's fun for inspector too (I think we have to create version-dependent client, which is also against the design) | 15:42 |
| mariojv | i am not strongly opinionated on whether to default to latest for the python api, though | 15:42 |
| jroll | dtantsur: I remember CLI only | 15:42 |
| mariojv | although now i'm wondering whether the strategy in https://review.openstack.org/#/c/442153/ will also automatically log the deprecation for the python api | 15:43 |
| jroll | folks, joanna mentioned multinode CI is down, is anyone working on this? vsaienk0 maybe? | 15:43 |
| TheJulia | really we should if we're going to start nuking the server's ability to support older microversions | 15:43 |
| mariojv | might be a little strange implementation wise if we did client and not python api, but we could figure it out | 15:43 |
| joanna | http://ci-watch.tintri.com/project?project=ironic | 15:44 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: WIP: SSL TEST https://review.openstack.org/442402 | 15:44 |
| joanna | gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial-nv and gate-tempest-dsvm-ironic-inspector-ubuntu-xenial-nv seem to be problematic | 15:44 |
| *** vinsh has quit IRC | 15:44 | |
| TheJulia | mariojv: I was thinking, we really need to go through the docs and change everything to be =latest | 15:45 |
| jroll | TheJulia: once we can agree on the deprecation mechanism for microversions, we should just warn based on that | 15:45 |
| mariojv | TheJulia: yeah that's a good point | 15:45 |
| *** Nisha_Agarwal has joined #openstack-ironic | 15:45 | |
| TheJulia | That was a headache for me as well this past weekend, once I realized what I was doing I was like "oh, ugh......", but that requires intimate knowledge of the microversions | 15:46 |
| dtantsur | TheJulia, mariojv, I'm even fine with what the spec suggests, but I suspect we should have another round of ML :( I'm pretty sure some people are going to feel strongly about that. And then decide how and when we treat new ironicclient versions as requiring a major version bump. | 15:46 |
| *** srobert has joined #openstack-ironic | 15:47 | |
| TheJulia | I kind of suspect those who were strong for it may have learned the pain of the pin in the client :) | 15:47 |
| mariojv | dtantsur: i'm fine with that, if it's going to break tooling it's not a change that should be done lightly | 15:47 |
| *** Nisha_Agarwal has quit IRC | 15:47 | |
| dtantsur | to be fair, I'd start with fixing how painful our microversions are in the client | 15:47 |
| TheJulia | +1 | 15:48 |
| dtantsur | e.g. for my pet SDK project, I implemented the ability for a developer to specify several supported versions, and then learn which one was picked | 15:48 |
| dtantsur | there may be more ways to improve it, this is just my personal pain point | 15:48 |
| TheJulia | jroll: I don't disagree with you there, but perhaps we should also be warning if one is not specified by the caller | 15:48 |
| mariojv | TheJulia: that's what the spec proposes | 15:49 |
| TheJulia | right | 15:49 |
| dtantsur | I nearly think that we should split the spec, and approve the CLI one already.. | 15:49 |
| jroll | TheJulia: warning about what, specifically? my thought was, if we see a "next minimum version", warn if a deprecated version is being used, else let it go | 15:50 |
| mariojv | dtantsur: i agree, i think these are separate questions | 15:50 |
| mariojv | i'll update the spec later today + remove reference to the python api for now | 15:51 |
| TheJulia | mariojv: sorry :( | 15:51 |
| mariojv | it's all good, these are good questions to bring up :) | 15:51 |
| dtantsur | for the python API part, I'd go as far as just making the version mandatory.. | 15:51 |
| dtantsur | ouch, wait, mariojv. I realized one thing | 15:52 |
| TheJulia | jroll: invocation of the client api lib without a version for now, then once we have a plan for version deprecation, we can change it | 15:52 |
| mariojv | ? | 15:52 |
| dtantsur | mariojv, if we pin any version to the last version that *client* knows, we make ironicclient stop working with all older Ironic servers | 15:52 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: WIP: SSL TEST https://review.openstack.org/442402 | 15:52 |
| dtantsur | this is why what we have to do is taking min(version client knows, version server knows) or something | 15:53 |
| mariojv | so throw version negotiation in there ? | 15:53 |
| sambetts | +1000 client should negotiate | 15:53 |
| jroll | TheJulia: what's the warning, though? "someday this will be deprecated"? | 15:53 |
| dtantsur | mariojv, unfortunately... | 15:54 |
| mariojv | hmm ok | 15:54 |
| mariojv | that'll be a bit tricky but the deprecation message patch can stay the same at least | 15:54 |
| dtantsur | yep, this bit is ok | 15:54 |
| *** jlvillal_pto is now known as jlvillal | 15:54 | |
| mariojv | i think it's better ux to do it that way though | 15:54 |
| TheJulia | jroll: Basically, that the ironic community recommends explicitly invoking with a version. Once there is agreement on deprecation, then the message can be changed :) | 15:54 |
| mariojv | a bit harder to explain to client users, but doable | 15:55 |
| * mariojv needs to go get food | 15:55 | |
| TheJulia | food +++++ | 15:55 |
| jroll | TheJulia: fair enough | 15:56 |
| * TheJulia foods | 15:56 | |
| *** aparnav has joined #openstack-ironic | 15:57 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-inspector-specs master: Collect Bios, Memory and Processor Info from Dmidecode https://review.openstack.org/406790 | 15:59 |
| *** aparna has joined #openstack-ironic | 16:00 | |
| *** lucasagomes is now known as lucas-afk | 16:00 | |
| *** trown is now known as trown|lunch | 16:00 | |
| *** pcaruana has quit IRC | 16:01 | |
| *** aparnav has quit IRC | 16:02 | |
| jroll | dtantsur: we didn't put bulk enrollment tools in ironic CLI, did we? | 16:03 |
| *** aparna has quit IRC | 16:03 | |
| jroll | oh wait, docs | 16:03 |
| jroll | woo | 16:03 |
| dtantsur | jroll, we did | 16:03 |
| dtantsur | 'openstack baremetal create' | 16:03 |
| jroll | yeah, found it now. thanks! | 16:03 |
| dtantsur | did you end up needing it in the end? ;) | 16:03 |
| jroll | shhhhhhh | 16:04 |
| jroll | I'm not sure, just people asking questions :D | 16:04 |
| dtantsur | okie, okie :) | 16:04 |
| jlvillal | vsaienk0, joanna pointed out that the multi-node grenade job is failing. Not sure if you already noticed that. | 16:06 |
| jlvillal | vsaienk0, I'm going to look at it after some meetings. | 16:06 |
| alezil | ls | 16:07 |
| *** Goneri has quit IRC | 16:08 | |
| TheJulia | Error: No such file or directory | 16:08 |
| TheJulia | :) | 16:08 |
| alezil | TheJulia: :p | 16:09 |
| *** Goneri has joined #openstack-ironic | 16:09 | |
| joanna | alezil: try sl | 16:09 |
| alezil | joanna: I did not knew about this 'sl'. :) | 16:12 |
| dtantsur | fish: Unknown command 'sl' :( | 16:13 |
| joanna | alezil: I hope you like it :) | 16:13 |
| joanna | dtantsur: apt is your friend! :) | 16:13 |
| NobodyCam | Good Morning IRonic'ers | 16:13 |
| joanna | (or yum) | 16:13 |
| joanna | (or brew) | 16:13 |
| dtantsur | dnf! | 16:13 |
| dtantsur | :) | 16:13 |
| *** derekh has quit IRC | 16:13 | |
| joanna | morning NobodyCam | 16:13 |
| NobodyCam | dpkg | 16:13 |
| NobodyCam | morning joanna | 16:13 |
| dtantsur | morning NobodyCam :) | 16:13 |
| NobodyCam | morning dtantsur :) | 16:14 |
| alezil | he he | 16:14 |
| *** aparnav has joined #openstack-ironic | 16:19 | |
| *** aarefiev is now known as aarefiev_afk | 16:23 | |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Make Ironic devstack simulation less dependent on OVS https://review.openstack.org/440721 | 16:25 |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Rework Ironic devstack baremetal network simulation https://review.openstack.org/392959 | 16:25 |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Handle multinode VXLAN tunnel inside ironic devstack plugin https://review.openstack.org/440718 | 16:25 |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Grenade source correct Ironic devstack plugin version https://review.openstack.org/440717 | 16:25 |
| *** litao has quit IRC | 16:29 | |
| *** sabeen has quit IRC | 16:39 | |
| *** mtanino has joined #openstack-ironic | 16:39 | |
| *** sabeen has joined #openstack-ironic | 16:44 | |
| thomasem | Ahhh, the familiar wall of cyan, yellow, and green of Devstack. | 16:47 |
| *** aparnav has quit IRC | 16:51 | |
| *** sayantan_ has quit IRC | 16:51 | |
| openstackgerrit | Anup Navare proposed openstack/ironic master: Config drive support for ceph radosgw https://review.openstack.org/407639 | 16:54 |
| *** sayantan_ has joined #openstack-ironic | 16:55 | |
| *** vinsh has joined #openstack-ironic | 16:57 | |
| *** vinsh_ has joined #openstack-ironic | 16:58 | |
| *** vinsh has quit IRC | 16:58 | |
| *** athomas has quit IRC | 16:59 | |
| sambetts | dtantsur: did we decide we should or should not have "dhcp" in the ipxe file for inspection? I see my server DHCP 3 times during pxe boot, first for normal pxe, then once for iPXE and then a second time for iPXE once it loads the ipxe file | 16:59 |
| vdrok | dtantsur: nope, in direct order. It's mock who adds arguments to function in reverse | 16:59 |
| dtantsur | sambetts, we dropped it from puppet-ironic fwiw | 17:01 |
| sambetts | dtantsur: hmm ok, install with bifrost still adds it | 17:02 |
| dtantsur | I think we dropped it back in Newton, and nobody complained | 17:03 |
| *** tesseract has quit IRC | 17:04 | |
| *** trown|lunch is now known as trown | 17:05 | |
| *** mgoddard has quit IRC | 17:06 | |
| *** ChubYann has joined #openstack-ironic | 17:06 | |
| dtantsur | see you tomorrow | 17:07 |
| *** dtantsur is now known as dtantsur|afk | 17:07 | |
| openstackgerrit | Merged openstack/python-oneviewclient master: Reenable the dynamic allocation validation methods https://review.openstack.org/453206 | 17:07 |
| *** dyasny has quit IRC | 17:09 | |
| sambetts | cya dtantsur|afk | 17:15 |
| openstackgerrit | Anup Navare proposed openstack/ironic master: Config drive support for ceph radosgw https://review.openstack.org/407639 | 17:16 |
| *** ElCoyote_ has quit IRC | 17:19 | |
| *** ElCoyote_ has joined #openstack-ironic | 17:20 | |
| *** amoralej is now known as amoralej|off | 17:20 | |
| *** baoli has joined #openstack-ironic | 17:25 | |
| *** mgould is now known as mgould|afk | 17:26 | |
| * sambetts -> home | 17:26 | |
| sambetts | night all | 17:26 |
| *** sambetts is now known as sambetts|afk | 17:27 | |
| thomasem | Cheers | 17:28 |
| jroll | vdrok: dtantsur|afk: fyi, this does get rid of that serialization warning but isn't py3-happy :/ https://review.openstack.org/#/c/453205/ | 17:32 |
| *** jvgrant_ has quit IRC | 17:35 | |
| *** jvgrant has joined #openstack-ironic | 17:35 | |
| vdrok | jroll: we can have an or in this elif, checking idinstance six.text_types? | 17:36 |
| *** dsneddon is now known as dsneddon_afk | 17:36 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Fix serialization warnings for unicode strings https://review.openstack.org/453205 | 17:37 |
| jroll | vdrok: yep, just did that :) | 17:37 |
| vdrok | :) thanks | 17:37 |
| jroll | I just wanted to mention it before a meeting but then decided to just go ahead and do it | 17:37 |
| *** e0ne has joined #openstack-ironic | 17:38 | |
| vdrok | jroll: oh, also we might want to do the same for int types | 17:38 |
| vdrok | Just in case | 17:38 |
| *** vgadiraj_ has joined #openstack-ironic | 17:39 | |
| jroll | ooo | 17:39 |
| jroll | good point | 17:39 |
| *** vgadiraj_ has quit IRC | 17:40 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Fix base object serialization checks https://review.openstack.org/453205 | 17:41 |
| jroll | vdrok: ^ there we go | 17:41 |
| vdrok | Yup, will take a look when I'm somewhere near PC | 17:42 |
| jroll | no worries | 17:43 |
| openstackgerrit | Anup Navare proposed openstack/ironic master: Config drive support for ceph radosgw https://review.openstack.org/407639 | 17:44 |
| *** d0ugal has quit IRC | 17:48 | |
| *** d0ugal has joined #openstack-ironic | 17:49 | |
| aNuposic | vdrok, if you could take a look at ^ when possible :) | 17:50 |
| vdrok | aNuposic: yup, sorry for the delay (again), will take a look today | 17:53 |
| *** mgoddard has joined #openstack-ironic | 17:53 | |
| aNuposic | vdrok, cool, thnx | 17:53 |
| *** sayantan_ has quit IRC | 17:58 | |
| *** tuan_luong has joined #openstack-ironic | 18:02 | |
| *** tuanla_ has joined #openstack-ironic | 18:02 | |
| *** ralonsoh has quit IRC | 18:04 | |
| *** sayantan_ has joined #openstack-ironic | 18:08 | |
| *** e0ne has quit IRC | 18:14 | |
| *** ijw_ has joined #openstack-ironic | 18:14 | |
| rajinir | The tempest test BaremetalBasicOps fails with this error ValidationError: Ironic node Node xyz does not have any port associated with it. Any idea? | 18:17 |
| *** Sukhdev has joined #openstack-ironic | 18:20 | |
| *** g3ek has quit IRC | 18:22 | |
| rajinir | sambetts|afk: hi | 18:23 |
| *** yatinkarel has quit IRC | 18:24 | |
| *** dprince has quit IRC | 18:27 | |
| *** tuanla_ has quit IRC | 18:31 | |
| *** tuan_luong has quit IRC | 18:31 | |
| *** g3ek has joined #openstack-ironic | 18:31 | |
| *** snecklifter has quit IRC | 18:32 | |
| *** crushil has quit IRC | 18:32 | |
| *** amotoki has quit IRC | 18:34 | |
| *** yatinkarel has joined #openstack-ironic | 18:37 | |
| TheJulia | rajinir: Was a VIF attachment posted to the node? | 18:40 |
| TheJulia | Err | 18:41 |
| TheJulia | No | 18:41 |
| TheJulia | It needs a port, it sounds like the port was never posted | 18:41 |
| TheJulia | It needs a port to relate back to the node in the database, else you can't validate the node | 18:41 |
| TheJulia | rajinir: ^^^ and forget the VIF attachment question, the error would be explicit. | 18:41 |
| rajinir | TheJulia: Is this bug? https://bugs.launchpad.net/ironic/+bug/1678995 | 18:42 |
| openstack | Launchpad bug 1678995 in Ironic "In flat network scenario with port groups, neutron port mac address isn't updated" [Medium,Triaged] | 18:42 |
| rajinir | TheJulia: VIF 6d056d1a-3013-4198-b325-954898b18907 isn't attached to Ironic node | 18:43 |
| TheJulia | rajinir: Unlikely, since that is a VIF issue | 18:44 |
| rajinir | TheJulia: We are using flat networks as well | 18:44 |
| TheJulia | rajinir: Maybe paste your log into paste.openstack.org for additional context? | 18:44 |
| *** dsneddon_afk has quit IRC | 18:44 | |
| TheJulia | Are you trying to run the tempest test against a configured operating environment? | 18:44 |
| ricardoas | hey, ironic'ers... another question regarding oneview drivers :) | 18:44 |
| ricardoas | We currently maintain two oneview drivers for ironic and an oneview ml2 driver. The problem for us is that the ml2 driver makes use of https://pypi.python.org/pypi/hpOneView which is being more actively maintained as a lib to access oneview than python-oneviewclient. We would like that *pxe_oneview drivers at ironic tree to use the hpOneView too, but as this will require to deprecate python-oneviewclient I'm not sure how to procee | 18:44 |
| TheJulia | ricardoas: Deprecating the entire library... that is a really good question, but the first step would be to begin to change over the usage in ironic to use the hponeview python library | 18:46 |
| *** snecklifter has joined #openstack-ironic | 18:46 | |
| *** joanna__ has quit IRC | 18:47 | |
| TheJulia | ricardoas: I _think_ an RFE bug would be useful for at least tracking purposes, I doubt there would be objections to switching over to a more actively maintained library | 18:47 |
| rajinir | https://www.irccloud.com/pastebin/4frV9wAU/BareMetalOps | 18:47 |
| rajinir | TheJulia: See the paste https://www.irccloud.com/pastebin/4frV9wAU/BareMetalOps | 18:47 |
| ricardoas | TheJulia: that makes sense... one point I'm not sure is that we currently maintain some validations at python-oneviewclient which do not fit into hponeview | 18:48 |
| ricardoas | TheJulia: do you think migrating these code to upstream drivers is an option? | 18:48 |
| ricardoas | TheJulia: for example, https://github.com/openstack/python-oneviewclient/blob/master/oneview_client/client.py#L614 | 18:50 |
| TheJulia | rajinir: Interesting.... Also that the vif data indicates it is a tap interface.... | 18:51 |
| TheJulia | rajinir: That feels like something server side configuration wise, possibly the test is getting scheduled by nova compute on to a VM somehow and not an ironic baremetal node... Which seems rather odd. | 18:53 |
| TheJulia | ricardoas: So... commit/contribution to the hponeview library from outside hp's walls was never sorted out? | 18:54 |
| TheJulia | S/hp/hpe/ | 18:55 |
| rajinir | TheJulia: Its a baremetal node, ironic node-list shows the info and it looks correct | 18:55 |
| rajinir | TheJulia: I thought since we use flat networks, it could be related to this https://bugs.launchpad.net/ironic/+bug/1678995. This error is new, was working fine in the last release | 18:56 |
| openstack | Launchpad bug 1678995 in Ironic "In flat network scenario with port groups, neutron port mac address isn't updated" [Medium,Triaged] | 18:56 |
| TheJulia | Sounds more like a misconfiguration to me. jroll any thoughts? | 18:56 |
| *** dprince has joined #openstack-ironic | 18:57 | |
| TheJulia | ricardoas: I don't know how the community would feel about extra validations moving into ironic, but perhaps a meeting agenda item for monday to raise the question. If you could point to an explicit example, that would help for discussion purposes? | 18:57 |
| TheJulia | S/?// | 18:57 |
| jroll | rajinir: is this a real deployment, or set up with devstack, or something else? | 18:58 |
| ricardoas | TheJulia I guess it is starting to be... we've been encouraged to start contributing to this lib instead, as it is more actively keeping track of changes in oneview api. But in this case, some of these validations are directly related to ironic entities (port, node_info)... | 18:58 |
| jroll | TheJulia: rajinir: it doesn't sound like the node has a port associated at all | 18:58 |
| rajinir | TheJulia: Thirdparty CI using baremetal, so devstack gate | 18:58 |
| rajinir | jroll:Thirdparty CI using baremetal, so devstack gate | 18:59 |
| *** mjturek has quit IRC | 19:00 | |
| ricardoas | TheJulia, that sounds great... I'll make sure to raise it in the next meeting :) | 19:00 |
| ricardoas | TheJulia: thanks a lot! | 19:00 |
| TheJulia | ricardoas: so that does actually kind of make sense to have in the driver code, as long as it would cleanly cross revisions/move forward, but we should discuss as a group. | 19:00 |
| TheJulia | ricardoas: Thank you! | 19:01 |
| jroll | rajinir: do the nodes have a mac address in your devstack's hardware info file? | 19:01 |
| jroll | oh wait wat, did we break all third party CI | 19:02 |
| jroll | https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L1645 | 19:02 |
| *** mgoddard has quit IRC | 19:02 | |
| jroll | $interface_info seems to only be built if IRONIC_IS_HARDWARE is false | 19:02 |
| jroll | @_@ | 19:02 |
| jroll | rajinir: looks like this broke it https://github.com/openstack/ironic/commit/4ec88c9b8acb1229c30bb39cb5b92bb3131da6d2 | 19:05 |
| jroll | sigh | 19:05 |
| jroll | sambetts|afk: ^^ fyi, that broke use with IRONIC_IS_HARDWARE==True | 19:05 |
| TheJulia | Looks one level higher, but I'm on a tablet right ow | 19:05 |
| EmilienM | hey ironic folks | 19:06 |
| Sukhdev | jroll : Hey Jim - QQ: Once a node moves into Active state (after the network flip), for the final boot or subsequent boots (issued manually by power cycle) is there any specific DHCP options required for it to boot from the downloaded image? | 19:06 |
| jroll | TheJulia: it's used higher, but it's set only directly after lines with: if [[ "$IRONIC_IS_HARDWARE" == "False" ]]; then | 19:06 |
| EmilienM | tripleo upgrade jobs are broken because it seems ironic db-sync doesn't work fine | 19:06 |
| EmilienM | http://logs.openstack.org/78/453078/2/check/gate-tripleo-ci-centos-7-undercloud-upgrades-nv/ac0887c/logs/var/log/undercloud_upgrade.txt.gz#_2017-04-04_12_38_04_000 | 19:06 |
| EmilienM | jroll: have you seen that before? ^ | 19:06 |
| slagle | hi, can anyone help with https://bugs.launchpad.net/ironic/+bug/1679759? | 19:06 |
| openstack | Launchpad bug 1679759 in tripleo "Undercloud upgrade failing on ironic dbsync" [Critical,Triaged] | 19:06 |
| EmilienM | we deploy from trunk | 19:07 |
| jroll | EmilienM: that's fixed in master | 19:07 |
| * jroll finds commit | 19:07 | |
| jroll | EmilienM: https://github.com/openstack/ironic/commit/d31d984871a03c3e7cc4be85a0f2fb12bda5dc7e | 19:07 |
| TheJulia | Yeah, I see it in the diff :( | 19:07 |
| rajinir | jroll: Is there a workaround? | 19:07 |
| jroll | rajinir: doesn't seem so, the code needs to be fixed. this landed just this week | 19:07 |
| EmilienM | jroll: we have this commit I think | 19:08 |
| EmilienM | jroll: see the version of ironic on http://logs.openstack.org/78/453078/2/check/gate-tripleo-ci-centos-7-undercloud-upgrades-nv/ac0887c/logs/rpm-qa.txt.gz | 19:08 |
| EmilienM | openstack-ironic-common-7.0.1-0.20170404112528.7f1639e.el7.centos.noarch | 19:08 |
| EmilienM | openstack-ironic-inspector-5.1.1-0.20170401215607.8dc2d00.el7.centos.noarch | 19:08 |
| EmilienM | both are build from very recent commits | 19:09 |
| TheJulia | jroll: I think we just need to revert the devstack lib change | 19:09 |
| TheJulia | It can be fixed in the revert revert | 19:09 |
| rajinir | jroll: TheJulia: I'm glad, I know the reason now. We are trying to get the builds to working again, a revert would help | 19:10 |
| EmilienM | jroll: oh wait | 19:10 |
| EmilienM | jroll: it seems like 7f1639e is older, but why the timestamp says 04/04 | 19:10 |
| EmilienM | slagle: it sounds like we deploy old ironic (I don't know why though) | 19:10 |
| jroll | TheJulia: rajinir: yes, a revert would also work, y'all have the power :P | 19:11 |
| EmilienM | slagle: see https://github.com/openstack/ironic/commit/7f1639e ( 11 days ago ) | 19:11 |
| EmilienM | jroll: sorry for noise. We might have an issue with our tooling | 19:11 |
| * TheJulia goes and mashes a revert button | 19:11 | |
| jroll | EmilienM: :) | 19:11 |
| jroll | Sukhdev: it should just work - however, using dhcp for the user image would require that the tenant network can reach the dhcp/tftp services. I'd recommend using local boot for user images | 19:13 |
| jroll | Sukhdev: e.g. https://docs.openstack.org/project-install-guide/baremetal/draft/advanced.html#local-boot-with-partition-images | 19:13 |
| *** baoli has quit IRC | 19:14 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Revert "[Devstack] Rework VMs connection logic" https://review.openstack.org/453300 | 19:14 |
| jroll | TheJulia: I hate to nit on this but can you wrap that commit message? :P | 19:15 |
| jroll | actually I'll just get it | 19:15 |
| Sukhdev | jroll : cool - let me switch to the local boot (I think this is missing in my setup) - thanks | 19:15 |
| TheJulia | Thank you, I'm on a tablet and used the web ui | 19:15 |
| *** dsneddon has joined #openstack-ironic | 19:15 | |
| * TheJulia has to go, bbl | 19:15 | |
| jroll | np, have a good night :) | 19:16 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Revert "[Devstack] Rework VMs connection logic" https://review.openstack.org/453300 | 19:16 |
| jroll | and a +2 if someone wants to land that | 19:17 |
| jroll | mariojv: ^ I know you're around | 19:17 |
| jroll | rajinir: sorry, which CI system(s) do you run? | 19:17 |
| rajinir | jroll: pxe_drac for dell | 19:17 |
| jroll | rajinir: looks like it's reporting NOT_REGISTERED on the revert | 19:18 |
| *** crushil has joined #openstack-ironic | 19:18 | |
| rajinir | jroll: Will fix that and retrigger a build | 19:18 |
| jroll | cool, thanks! | 19:19 |
| *** sayantan_ has quit IRC | 19:21 | |
| mariojv | jroll: TheJulia: L1634 here was the change that broke 3rd party ci? https://review.openstack.org/#/c/453300/2/devstack/lib/ironic | 19:23 |
| mariojv | trying to wrap my head around why this broke it | 19:23 |
| *** harlowja has joined #openstack-ironic | 19:24 | |
| jroll | mariojv: yes, because $interface_info is only defined in the case where IRONIC_IS_HARDWARE==False | 19:24 |
| jroll | which is True for all of our third party CI | 19:25 |
| jroll | thus undefined, thus no ports created | 19:25 |
| mariojv | ah, i see it | 19:25 |
| mariojv | yeah, this is good, +2+W | 19:26 |
| jroll | it's fairly obvious reading the code in tree, less so in the diff | 19:26 |
| jroll | thanks! | 19:26 |
| jroll | was considering waiting for dell CI to come back, but pretty clear this is broken anyway ¯\_(ツ)_/¯ | 19:26 |
| jroll | rajinir: ^^ when that comes down you should be up and running again | 19:26 |
| *** dprince has quit IRC | 19:26 | |
| rajinir | TheJulia: Thanks julia will watch for it | 19:27 |
| *** dprince has joined #openstack-ironic | 19:29 | |
| *** bfournie has quit IRC | 19:30 | |
| *** baoli has joined #openstack-ironic | 19:32 | |
| *** cdearborn has joined #openstack-ironic | 19:40 | |
| *** krtaylor has quit IRC | 19:45 | |
| *** [1]cdearborn has quit IRC | 19:53 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-inspector-specs master: Collect Bios, Memory and Processor Info from Dmidecode https://review.openstack.org/406790 | 19:57 |
| *** Sukhdev has quit IRC | 20:15 | |
| *** causten has quit IRC | 20:19 | |
| *** ElCoyote_1 has joined #openstack-ironic | 20:23 | |
| *** ElCoyote_ has quit IRC | 20:23 | |
| *** ElCoyote_1 is now known as ElCoyote_ | 20:24 | |
| rajinir | TheJulia, jroll, the revert helped https://review.openstack.org/#/c/453300/ | 20:29 |
| jroll | rajinir: excellent! | 20:29 |
| jroll | and in before jenkins :) | 20:29 |
| vdrok | TheJulia: an rfe you asked about - https://bugs.launchpad.net/ironic/+bug/1679834, mariojv you might be interested too, as rescue stuff also falls into that category I suppose | 20:30 |
| openstack | Launchpad bug 1679834 in Ironic "[RFE] Moving network interface calls out of the deployment interface" [Wishlist,New] - Assigned to Vladyslav Drok (vdrok) | 20:30 |
| vdrok | jroll: would be good to have your opinion too :) ^^ | 20:31 |
| mariojv | vdrok on a first pass without having thought about it a lot, i think this makes a decent amount of sens | 20:32 |
| mariojv | *sense | 20:32 |
| jroll | vdrok: for the last bit, you could just split "finishing the deployment" into pieces, call deploy.shutdown_server(), network.flip_things(), deploy.finish() | 20:32 |
| jroll | but yeah makes sense, I think | 20:32 |
| mariojv | i am struggling a bit with trying to piece together all the network logic for rescue in the agent driver, since it's scattered all over the place | 20:33 |
| mariojv | abstracting that away into the conductor would make this quite a bit easier | 20:33 |
| vdrok | jroll: yeah, might be, tho I wanted to keep the conductor changes minimal :) | 20:33 |
| vdrok | but that would be more clear way | 20:33 |
| jroll | vdrok: yeah, we might have to just roll with it. maybe if a driver doesn't have the new method(s), fall back to the old way | 20:33 |
| jroll | I don't love out of tree deploy interfaces, fwiw :) | 20:34 |
| vdrok | jroll: oh, tell me about it, I hate them more :D | 20:34 |
| jroll | heh | 20:34 |
| vdrok | mariojv: here is a patch doing everything apart the deploy finish -- https://review.openstack.org/453139 | 20:34 |
| *** shobhar_ has joined #openstack-ironic | 20:44 | |
| *** jkilpatr has quit IRC | 20:45 | |
| *** racedo has joined #openstack-ironic | 20:51 | |
| *** racedo has quit IRC | 20:53 | |
| *** racedo has joined #openstack-ironic | 20:53 | |
| TheJulia | jroll: vdrok: I do like network.flip_things() from a naming standpoint already. | 20:53 |
| jroll | heh | 20:53 |
| jroll | network.do_a_barrel_roll() | 20:53 |
| TheJulia | That is another option :) | 20:53 |
| vdrok | :) | 20:54 |
| *** trown is now known as trown|outtypewww | 20:55 | |
| *** rcernin has quit IRC | 20:55 | |
| TheJulia | vdrok: thank you :) | 20:55 |
| vdrok | TheJulia: why? np :D | 20:56 |
| TheJulia | :) | 20:58 |
| *** krtaylor has joined #openstack-ironic | 20:59 | |
| vdrok | aNuposic: just one thing in that patch, apart from that looks great :) | 20:59 |
| *** racedo has quit IRC | 21:00 | |
| *** dprince has quit IRC | 21:01 | |
| openstackgerrit | Emilien Macchi proposed openstack/ironic master: Add sem-ver flag so pbr generates correct version https://review.openstack.org/453331 | 21:07 |
| EmilienM | slagle, jroll : ^ that should help our tripleo issue with upgrades for ironic ^ | 21:07 |
| jroll | EmilienM: oh dang, we even document we should do that | 21:08 |
| EmilienM | yup | 21:08 |
| jroll | EmilienM: so we're about to release 8.0.0, waiting on one patch, should we just go ahead and make that bump the major (whatever the flag is)? | 21:08 |
| EmilienM | so: | 21:09 |
| EmilienM | "sem-ver: api-break" bumps X aka major version | 21:09 |
| jroll | nod | 21:09 |
| * jroll just didn't want to find the docs | 21:09 | |
| EmilienM | "sem-ver: feature" bumps Y | 21:09 |
| EmilienM | what I did is Y | 21:09 |
| jroll | right | 21:09 |
| EmilienM | but if you prefer X, I'm fine | 21:09 |
| jroll | which is fine for you, and fine for us, I guess I'm just wondering if api-break is more "right" | 21:10 |
| *** Goneri has quit IRC | 21:10 | |
| * jroll +2s with a similar note | 21:10 | |
| *** racedo has joined #openstack-ironic | 21:11 | |
| *** baha has quit IRC | 21:12 | |
| *** bcornec2 has quit IRC | 21:15 | |
| vdrok | good night! (again) | 21:16 |
| *** racedo has quit IRC | 21:17 | |
| jroll | \o vdrok | 21:17 |
| TheJulia | I thought pbr was supposed to look at the latest tag as it relates to whatever head is to version :( | 21:17 |
| mrda | Morning Ironic | 21:18 |
| jroll | TheJulia: it does, here's the problem: | 21:19 |
| jroll | when we cut a branch and land a couple patches on stable | 21:19 |
| jroll | stable/foo is now 10.0.0.dev5 | 21:19 |
| jroll | and master is 10.0.0.dev3 or whatever | 21:19 |
| jroll | and packaging tools get uber confused | 21:19 |
| *** Serlex has quit IRC | 21:19 | |
| *** snecklifter has quit IRC | 21:20 | |
| TheJulia | Yeah | 21:21 |
| TheJulia | Because the branch is only for humans | 21:21 |
| *** racedo has joined #openstack-ironic | 21:21 | |
| jroll | that Sem-Ver commit message tag tells PBR to expect to bump the version, and turns it into 10.0.99999.dev3 or something | 21:21 |
| jroll | pbr spec because I can't find docs https://specs.openstack.org/openstack/oslo-specs/specs/juno/pbr-semver.html | 21:21 |
| *** erolg has quit IRC | 21:21 | |
| TheJulia | :) | 21:22 |
| *** srobert has quit IRC | 21:22 | |
| *** jkilpatr has joined #openstack-ironic | 21:23 | |
| *** racedo has quit IRC | 21:26 | |
| *** racedo has joined #openstack-ironic | 21:31 | |
| TheJulia | mrda: Good morning! Your Morning message didn't register until now | 21:31 |
| * TheJulia feels bad | 21:31 | |
| jroll | guh, same | 21:32 |
| jroll | morning mrda :) | 21:32 |
| mrda | o/ | 21:33 |
| *** shobhar_ has quit IRC | 21:33 | |
| mrda | But your return welcomes are still very much appreciated :) | 21:33 |
| *** racedo has quit IRC | 21:37 | |
| *** rfc1918 has joined #openstack-ironic | 21:37 | |
| *** rfc1918 has quit IRC | 21:37 | |
| *** racedo has joined #openstack-ironic | 21:41 | |
| *** racedo has quit IRC | 21:43 | |
| *** mat128 has quit IRC | 21:44 | |
| thomasem | Time for me to run. See ya tomorrow! | 21:50 |
| *** racedo has joined #openstack-ironic | 21:52 | |
| aNuposic | vdrok: Thanks :) will address your comments | 21:58 |
| *** racedo has quit IRC | 21:58 | |
| jlvillal | jroll, TheJulia rloo: Any objections to doing +A on https://review.openstack.org/#/c/355625/ ? | 21:59 |
| jlvillal | mariojv, ^^ | 22:00 |
| jroll | jlvillal: I'm not waiting for anything | 22:00 |
| rloo | jlvillal: dunno. i haven't looked at it. can it be used as is, or is it part of some work. | 22:00 |
| mariojv | jlvillal: none from me | 22:00 |
| jlvillal | Okay thanks. | 22:00 |
| rloo | jlvillal: i see a config added? does it work? | 22:00 |
| mariojv | rloo: it's a prerequisite for the cinder storage driver | 22:01 |
| jlvillal | rloo, It is the first patch in a series | 22:01 |
| rloo | jlvillal: so does the config work? | 22:01 |
| rloo | jroll: have we released ironic yet? if not, i'd prefer to hold off on this jlvillal | 22:01 |
| mariojv | ah that's a good catch, the config won't be used at all if just that lands | 22:02 |
| jroll | rloo: ah, good point. we're waiting on https://review.openstack.org/#/c/453205/ | 22:02 |
| jlvillal | rloo, Define work? You can set the config and it is used by the code. But the code isn't yet being used. If that makes sense. | 22:02 |
| rloo | mariojv, jlvillal: I ask cuz i just went through that with rolling upgrades. one of the patches landed and it is part of the release. so i updated the config help to say 'work in progress'. | 22:02 |
| rloo | jlvillal: so it doesn't really work. from the user's perspective. | 22:02 |
| *** Sukhdev has joined #openstack-ironic | 22:03 | |
| mariojv | yeah, so we can either add the note in sample config & land, or just wait until after release | 22:03 |
| mariojv | i vote just wait | 22:03 |
| rloo | jlvillal: i'd just say hold off until we do a release. and maybe we should get in the habit of putting warnings in our configs, and remove them when the feature lands. | 22:03 |
| jlvillal | joanna, ^^^^ | 22:03 |
| jlvillal | rloo, Okay. I'll wait. | 22:04 |
| rloo | jlvillal: maybe put a -2 for now in case someone else thinks to land it...you're around tomorrow? | 22:04 |
| jlvillal | I am | 22:04 |
| *** priteau has quit IRC | 22:04 | |
| rloo | jlvillal: that release better be done by tomorrow :) | 22:05 |
| jlvillal | rloo, Done | 22:05 |
| jlvillal | rloo, I'm here all week (except Friday) ;) | 22:05 |
| rloo | jlvillal: thx. ciao for now. gotta find some food for dinner... | 22:05 |
| *** shobhar_ has joined #openstack-ironic | 22:08 | |
| *** shobhar_ has quit IRC | 22:15 | |
| *** shobhar_ has joined #openstack-ironic | 22:18 | |
| *** jcoufal has quit IRC | 22:20 | |
| *** baoli has quit IRC | 22:23 | |
| *** Goneri has joined #openstack-ironic | 22:25 | |
| openstackgerrit | Anup Navare proposed openstack/ironic master: Config drive support for ceph radosgw https://review.openstack.org/407639 | 22:33 |
| openstackgerrit | Merged openstack/ironic-python-agent master: Updated from global requirements https://review.openstack.org/451012 | 22:34 |
| openstackgerrit | Joanna Taryma proposed openstack/ironic master: Remove translation of log messages from ironic/api https://review.openstack.org/453360 | 22:37 |
| *** racedo has joined #openstack-ironic | 22:37 | |
| aNuposic | vdrok, jlvillal: addressed the review comments https://review.openstack.org/407639 | 22:41 |
| *** hoonetorg has quit IRC | 22:43 | |
| *** mat128 has joined #openstack-ironic | 22:43 | |
| *** hoonetorg has joined #openstack-ironic | 22:44 | |
| *** hoonetorg has quit IRC | 22:47 | |
| *** racedo has quit IRC | 22:50 | |
| *** rpioso has quit IRC | 22:56 | |
| *** hoonetorg has joined #openstack-ironic | 22:58 | |
| *** Goneri has quit IRC | 22:59 | |
| *** mat128 has quit IRC | 23:03 | |
| openstackgerrit | Merged openstack/ironic master: Revert "[Devstack] Rework VMs connection logic" https://review.openstack.org/453300 | 23:04 |
| openstackgerrit | Merged openstack/ironic master: Fix base object serialization checks https://review.openstack.org/453205 | 23:06 |
| *** causten has joined #openstack-ironic | 23:22 | |
| *** watanabe_isao has joined #openstack-ironic | 23:26 | |
| *** causten has quit IRC | 23:28 | |
| *** watanabe_isao has quit IRC | 23:30 | |
| *** bfournie has joined #openstack-ironic | 23:36 | |
| *** pmannidi has joined #openstack-ironic | 23:38 | |
| *** pmannidi is now known as pmannidi|brb | 23:38 | |
| *** pmannidi|brb has quit IRC | 23:38 | |
| *** pmannidi has joined #openstack-ironic | 23:39 | |
| *** mtanino has quit IRC | 23:39 | |
| *** rook-tower is now known as rook | 23:49 | |
| *** xavierr has quit IRC | 23:49 | |
| Sukhdev | Hey Ironic experts, I used diskimage-builder utility to create Ubuntu image to install using Ironic - I can not login to the server when it boots up -How/what is used as default username/password ? | 23:50 |
| *** xavierr has joined #openstack-ironic | 23:52 | |
| *** xavierr has quit IRC | 23:53 | |
| *** yuanying_ has quit IRC | 23:53 | |
| *** xavierr has joined #openstack-ironic | 23:55 | |
| *** xavierr has quit IRC | 23:55 | |
| *** shobhar_ has quit IRC | 23:58 | |
| *** xavierr has joined #openstack-ironic | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!