| *** achanda has quit IRC | 00:03 | |
| *** cfarquhar has joined #openstack-ironic | 00:07 | |
| *** cfarquhar has quit IRC | 00:07 | |
| *** cfarquhar has joined #openstack-ironic | 00:07 | |
| *** smoriya_ has joined #openstack-ironic | 00:19 | |
| *** vishwanathj has joined #openstack-ironic | 00:27 | |
| *** vishwanathj has quit IRC | 00:27 | |
| *** baoli has joined #openstack-ironic | 00:31 | |
| *** baoli has quit IRC | 00:34 | |
| *** hoangcx has joined #openstack-ironic | 00:47 | |
| *** achanda has joined #openstack-ironic | 01:01 | |
| *** piet has joined #openstack-ironic | 01:09 | |
| *** penick has quit IRC | 01:13 | |
| lintan | jroll, I just see it. No problem, I will do this. This is what I want to do too, this is easier to get consensus. | 01:19 |
|---|---|---|
| *** baoli has joined #openstack-ironic | 01:24 | |
| *** Sukhdev has joined #openstack-ironic | 01:38 | |
| *** achanda has quit IRC | 01:39 | |
| *** piet has quit IRC | 01:40 | |
| *** piet has joined #openstack-ironic | 01:41 | |
| *** cfarquhar has quit IRC | 01:52 | |
| *** piet has quit IRC | 01:54 | |
| *** baoli has quit IRC | 02:06 | |
| *** baoli has joined #openstack-ironic | 02:13 | |
| *** achanda has joined #openstack-ironic | 02:26 | |
| *** baoli has quit IRC | 02:32 | |
| *** baoli has joined #openstack-ironic | 02:41 | |
| *** Sukhdev has quit IRC | 02:42 | |
| *** baoli has quit IRC | 02:44 | |
| *** vishwanathj has joined #openstack-ironic | 02:48 | |
| *** Sukhdev has joined #openstack-ironic | 02:50 | |
| *** Sukhdev has quit IRC | 02:51 | |
| *** baoli has joined #openstack-ironic | 02:52 | |
| *** baoli has quit IRC | 02:54 | |
| *** cfarquhar has joined #openstack-ironic | 02:55 | |
| *** cfarquhar has quit IRC | 02:55 | |
| *** cfarquhar has joined #openstack-ironic | 02:55 | |
| *** baoli has joined #openstack-ironic | 02:59 | |
| *** baoli has quit IRC | 03:02 | |
| *** baoli has joined #openstack-ironic | 03:11 | |
| *** Nisha has joined #openstack-ironic | 03:25 | |
| *** links has joined #openstack-ironic | 03:33 | |
| *** achanda has quit IRC | 03:44 | |
| *** saripurigopi has joined #openstack-ironic | 03:49 | |
| *** piet has joined #openstack-ironic | 03:51 | |
| *** yuanying has quit IRC | 03:53 | |
| *** hoangcx has quit IRC | 03:54 | |
| *** hoangcx has joined #openstack-ironic | 03:55 | |
| *** baoli has quit IRC | 03:58 | |
| *** piet has quit IRC | 04:07 | |
| *** piet has joined #openstack-ironic | 04:21 | |
| *** vishwanathj is now known as vishwanathj_zzz | 04:33 | |
| *** Sukhdev has joined #openstack-ironic | 04:43 | |
| *** yuanying has joined #openstack-ironic | 04:56 | |
| *** Goneri has quit IRC | 05:06 | |
| openstackgerrit | Debayan Ray proposed openstack/proliantutils: Adds test coverage in tox.ini https://review.openstack.org/293936 | 05:08 |
| *** piet has quit IRC | 05:20 | |
| *** saripurigopi has quit IRC | 05:21 | |
| *** AaronJohn has joined #openstack-ironic | 05:30 | |
| *** achanda has joined #openstack-ironic | 05:31 | |
| *** ishant has joined #openstack-ironic | 05:37 | |
| *** yuanying has quit IRC | 05:40 | |
| *** yuanying has joined #openstack-ironic | 05:48 | |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 05:48 |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic: Added operator documentation for ironic portgroups https://review.openstack.org/228496 | 05:49 |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic: Allow to build user image with DIB https://review.openstack.org/256363 | 05:49 |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic: refactor ironic enroll-node code https://review.openstack.org/256364 | 05:50 |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic: Add Link-Local-Connection info to ironic port https://review.openstack.org/256365 | 05:50 |
| *** harshs has quit IRC | 05:51 | |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic: Allow to use NETWORK_DRIVERS in devstack https://review.openstack.org/293520 | 05:52 |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic: Update Ironic VM network connection https://review.openstack.org/256366 | 05:53 |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic: Add configure_provision_network function https://review.openstack.org/256367 | 05:54 |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic: Add Ironic/Neutron integration documentation https://review.openstack.org/258596 | 05:54 |
| *** moshele has joined #openstack-ironic | 06:00 | |
| *** baoli has joined #openstack-ironic | 06:10 | |
| *** harlowja_at_home has quit IRC | 06:12 | |
| *** baoli has quit IRC | 06:15 | |
| *** harlowja_at_home has joined #openstack-ironic | 06:23 | |
| *** Marga_ has quit IRC | 06:32 | |
| *** pcaruana has joined #openstack-ironic | 06:49 | |
| *** vishwanathj_zzz has quit IRC | 06:52 | |
| *** divya_ has joined #openstack-ironic | 07:11 | |
| divya_ | Failed to get ironic network id, devstack fails in mitaka | 07:13 |
| divya_ | can anybody resolve this | 07:13 |
| *** stendulker has joined #openstack-ironic | 07:14 | |
| *** e0ne has joined #openstack-ironic | 07:15 | |
| *** e0ne has quit IRC | 07:18 | |
| Haomeng|2 | divya_: any more details, can you share the devstack log? | 07:18 |
| divya_ | yes haomeng..where shall i paste the logs | 07:20 |
| Haomeng|2 | divya_: paste.openstack.org if it is not too long:) | 07:21 |
| divya_ | http://paste.openstack.org/show/492009/ | 07:22 |
| divya_ | can you please check here | 07:23 |
| divya_ | @Haomengi, are you able to view the logs | 07:29 |
| Haomeng|2 | divya_: ok | 07:29 |
| Haomeng|2 | divya_: looks like the first error is "ailed to create neutron port" | 07:30 |
| Haomeng|2 | divya_: so need to check neutron log also | 07:30 |
| Haomeng|2 | divya_: why "Failed to create neutron port" | 07:30 |
| Haomeng|2 | divya_: and this one - Failed to get ironic network id | 07:31 |
| divya_ | can you please share neutron logs location | 07:31 |
| Haomeng|2 | divya_: did you follow this guide - docs.openstack.org/developer/ironic/dev/dev-quickstart.html#deploying-ironic-with-devstack | 07:32 |
| Haomeng|2 | divya_: by default for devstack, neutron server log should be q-svc.log, in your devstack folder, or it is set by localrc | 07:34 |
| *** Haomeng|2 has quit IRC | 07:36 | |
| *** phuongnh has joined #openstack-ironic | 07:39 | |
| divya_ | Failed to get ironic network id, devstack fails in mitaka | 07:43 |
| divya_ | can anybody resolve this | 07:43 |
| divya_ | http://paste.openstack.org/show/492009/ | 07:43 |
| *** Haomeng has joined #openstack-ironic | 07:48 | |
| *** Nisha has quit IRC | 07:54 | |
| *** Nisha has joined #openstack-ironic | 07:54 | |
| *** e0ne has joined #openstack-ironic | 08:14 | |
| divya_ | hi haomeng | 08:28 |
| divya_ | the neutron logs are huge | 08:28 |
| divya_ | where shal i paste | 08:28 |
| Haomeng | divya_: no | 08:29 |
| Haomeng | divya_: just search around the error about net creating failed, and check the error message | 08:29 |
| Haomeng | divya_: and suggest to clean your env(unstack.sh), and make sure the os is fresh and run devstack script again | 08:30 |
| divya_ | i even tried unstack and clean.. then run stack.sh got the same error | 08:30 |
| *** ndipanov has joined #openstack-ironic | 08:30 | |
| *** ndipanov has quit IRC | 08:30 | |
| divya_ | tried in different fresh machine.. got different error | 08:30 |
| *** yuanying has quit IRC | 08:31 | |
| divya_ | No matching distribution found for diskimage-builder===1.11.1 | 08:31 |
| *** yuanying has joined #openstack-ironic | 08:31 | |
| divya_ | this is the error in different server | 08:31 |
| *** yuanying has quit IRC | 08:32 | |
| divya_ | ERROR neutron.plugins.ml2.managers [^[[01;36mreq-cf01b105-a1ea-41f5-95c8-59f3c5953616 ^[[00;36mNone None^[[01;31m] ^[[01;35m^[[01;31mFailed to bind port 1a733a23-5d3c-4543-9950-31718e33b439 on host ub1404-32-206 for vnic_type normal using segments [{'segmentation_id': 674, 'physical_network': u'physnet1', 'id': u'70fe8316-eb7d-4618-b26f-16049f8050cb', 'network_type': u'vlan'}]^[[00m^M | 08:38 |
| *** yuanying has joined #openstack-ironic | 08:39 | |
| *** kromanenko has joined #openstack-ironic | 08:43 | |
| divya_ | haomeng..sincei am new to devstack..facing all issues | 08:44 |
| divya_ | can u hel[ | 08:44 |
| Haomeng | divya_: that is odd | 08:48 |
| Haomeng | divya_: what os did you run, ubuntu? | 08:48 |
| Haomeng | divya_: make sure it is supported by devstack and check the devstack wiki out | 08:48 |
| Haomeng | divya_: http://docs.openstack.org/developer/devstack/ | 08:48 |
| Haomeng | divya_: it works on the supported os | 08:49 |
| Haomeng | divya_: will go home and good luck to you:) | 08:49 |
| *** vinm213 has joined #openstack-ironic | 08:50 | |
| divya_ | i am using ubuntu 14.04 | 08:50 |
| Haomeng | divya_: I use it also, works fine:) | 08:51 |
| Haomeng | divya_: for such "o matching distribution found for diskimage-builder===1.11.1" issue, should depends on your apt source | 08:51 |
| divya_ | can you share your local.conf file | 08:51 |
| Haomeng | divya_: sure, my local.conf is same one with the guide - http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#deploying-ironic-with-devstack , just copy it from guide | 08:51 |
| divya_ | the same local.conf will work for physical baremental ? | 08:55 |
| Haomeng | divya_: no, it is devstack env | 08:56 |
| divya_ | we are not going to use VM for baremetal.. planning to use physical nodes.. so is there any configuration need to be added in local.conf | 08:57 |
| divya_ | for ironic | 08:57 |
| Haomeng | divya_: yes, have hardware options in ironic devstack script supports | 08:57 |
| divya_ | what is that configuration in local.conf | 08:58 |
| divya_ | Ironic_is_hardware=True ?? | 08:59 |
| *** Sukhdev has quit IRC | 08:59 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Move OpenStack client to test-requirements https://review.openstack.org/297730 | 08:59 |
| *** achanda has quit IRC | 09:04 | |
| *** achanda has joined #openstack-ironic | 09:11 | |
| pas-ha | morning Ironic | 09:12 |
| *** ndipanov has joined #openstack-ironic | 09:12 | |
| divya_ | haomeng:No matching distribution found for mox===0.5.3 (from -c /opt/stack/requirements/upper-constraints.txt (line 186)) | 09:13 |
| divya_ | when i tried with ironic website local.conf | 09:14 |
| *** stendulker has quit IRC | 09:14 | |
| *** yuanying has quit IRC | 09:16 | |
| *** shu-mutou has joined #openstack-ironic | 09:18 | |
| sergek | morning, pas-ha | 09:22 |
| sergek | morning Ironic | 09:22 |
| divya_ | morning pas-ha,sergek | 09:25 |
| sergek | o/ divya_ | 09:26 |
| *** achanda has quit IRC | 09:28 | |
| *** phuongnh has quit IRC | 09:34 | |
| *** hoangcx has quit IRC | 09:34 | |
| *** Nisha has quit IRC | 09:35 | |
| *** hoangcx has joined #openstack-ironic | 09:39 | |
| divya_ | devstack failed with die_if_not_set 565 port_id 'Failed to create neutron port | 09:39 |
| divya_ | can anybody help | 09:39 |
| *** pcaruana has quit IRC | 09:42 | |
| divya_ | No matching distribution found for pbr>=1.6 | 09:43 |
| divya_ | failed | 09:43 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 09:54 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 09:54 |
| *** pcaruana has joined #openstack-ironic | 09:55 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Check duplicate nodes in node verifying process https://review.openstack.org/298184 | 10:00 |
| divya_ | openstack object store account set --property Temp-URL-Key=password | 10:12 |
| divya_ | devstack failed with ironic | 10:12 |
| divya_ | with this reason | 10:12 |
| *** Nisha has joined #openstack-ironic | 10:13 | |
| *** AaronJohn has quit IRC | 10:17 | |
| *** vinm213 has quit IRC | 10:23 | |
| *** achanda has joined #openstack-ironic | 10:28 | |
| *** achanda has quit IRC | 10:33 | |
| *** mbound has joined #openstack-ironic | 10:35 | |
| *** lazy_prince has quit IRC | 10:35 | |
| *** achanda has joined #openstack-ironic | 10:36 | |
| *** shu-mutou is now known as shu-mutou-AFK | 10:40 | |
| *** hoangcx has quit IRC | 10:43 | |
| *** lazy_prince has joined #openstack-ironic | 10:44 | |
| divya_ | s-account failed to start" | tee "/opt/stack/status/stack/s-account.failure | 10:55 |
| *** Nisha has quit IRC | 11:00 | |
| *** pcaruana has quit IRC | 11:09 | |
| *** achanda has quit IRC | 11:11 | |
| *** trown|outtypewww is now known as trown|PTO | 11:22 | |
| *** pcaruana has joined #openstack-ironic | 11:23 | |
| TheJulia | Good morning everyone | 11:47 |
| divya_ | devstack fails,with s-account failed to start" | tee "/opt/stack/status/stack/s-account.failure | 11:47 |
| divya_ | can anybody help | 11:47 |
| TheJulia | did you look in the swift logs? | 11:51 |
| *** shu-mutou-AFK has quit IRC | 11:56 | |
| *** baoli has joined #openstack-ironic | 11:57 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 12:05 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 12:05 |
| *** jaypipes has joined #openstack-ironic | 12:12 | |
| *** achanda has joined #openstack-ironic | 12:12 | |
| *** afaranha has joined #openstack-ironic | 12:14 | |
| *** mbound has quit IRC | 12:15 | |
| *** thrash|pt0 is now known as thrash | 12:15 | |
| *** achanda has quit IRC | 12:17 | |
| *** dprince has joined #openstack-ironic | 12:24 | |
| *** awiddersheim has joined #openstack-ironic | 12:25 | |
| *** rbudden has joined #openstack-ironic | 12:29 | |
| *** spandhe has quit IRC | 12:29 | |
| *** _degorenko is now known as degorenko | 12:32 | |
| *** Nisha has joined #openstack-ironic | 12:35 | |
| *** mbound has joined #openstack-ironic | 12:36 | |
| *** pcaruana has quit IRC | 12:39 | |
| *** max_lobur has joined #openstack-ironic | 12:46 | |
| *** piet has joined #openstack-ironic | 12:47 | |
| *** pcaruana has joined #openstack-ironic | 12:52 | |
| *** mbound has quit IRC | 12:53 | |
| *** smoriya_ has quit IRC | 12:57 | |
| *** Nisha has quit IRC | 12:58 | |
| *** jcoufal has joined #openstack-ironic | 13:05 | |
| *** mtanino has quit IRC | 13:13 | |
| *** achanda has joined #openstack-ironic | 13:15 | |
| *** rcrit_ has quit IRC | 13:16 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add test for --wrap option for driver-properties output https://review.openstack.org/297077 | 13:16 |
| *** achanda has quit IRC | 13:19 | |
| *** mtanino has joined #openstack-ironic | 13:21 | |
| *** vishwanathj_zzz has joined #openstack-ironic | 13:22 | |
| *** absubram has quit IRC | 13:24 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Functional tests for portgroups in ironicclient https://review.openstack.org/297727 | 13:27 |
| *** mtanino has quit IRC | 13:27 | |
| *** piet has quit IRC | 13:28 | |
| *** ishant has quit IRC | 13:28 | |
| *** ppiela has joined #openstack-ironic | 13:29 | |
| *** alexpilotti has joined #openstack-ironic | 13:30 | |
| *** alexpilotti has quit IRC | 13:31 | |
| *** alexpilotti has joined #openstack-ironic | 13:31 | |
| mag009_ | hey guys | 13:33 |
| jroll | lintan: cool, thank you :) | 13:33 |
| jroll | morning everyone | 13:33 |
| jroll | hi mag009_ | 13:34 |
| mag009_ | im trying to get ipxe working with uefi | 13:34 |
| mag009_ | when I launch a node I get this :iPXE is enabled, but cannot be used configured to use UEFI boot | 13:34 |
| jroll | which version of ironic? | 13:35 |
| *** moshele has left #openstack-ironic | 13:35 | |
| *** moshele has joined #openstack-ironic | 13:35 | |
| *** links has quit IRC | 13:36 | |
| *** piet has joined #openstack-ironic | 13:36 | |
| *** alexpilotti has quit IRC | 13:36 | |
| mag009_ | 4.3 | 13:38 |
| mag009_ | the one with mitaka | 13:38 |
| jroll | 5.1.0 is the final mitaka release | 13:39 |
| jroll | I believe 5.0.0 is when we enabled ipxe+uefi, let me check | 13:39 |
| jroll | yep | 13:40 |
| jroll | http://docs.openstack.org/releasenotes/ironic/mitaka.html#id3 | 13:40 |
| jroll | about halfway down that list: "Adds support for using iPXE in UEFI mode." | 13:40 |
| *** Nisha has joined #openstack-ironic | 13:41 | |
| mag009_ | ironic-api 1:4.3.0-0ubuntu1~cloud0 | 13:42 |
| mag009_ | thats the version i have | 13:42 |
| mag009_ | guess it didn't hit the ubuntu repo yet | 13:42 |
| jroll | right, uefi+ipxe is not supported until 5.0.0 | 13:42 |
| jroll | probably not, was just released a few weeks ago :/ | 13:43 |
| mag009_ | i guess I will play with uefi later on | 13:43 |
| mag009_ | I'll enforce legacy bios for now | 13:43 |
| mag009_ | this uefi/legacy thing is a pain in the ass to be honest :) | 13:44 |
| mag009_ | especially for the partitioning.. | 13:44 |
| jroll | heh | 13:45 |
| mag009_ | I have another question if you dont mind | 13:45 |
| mag009_ | can I use ironic with nova without using neutron? | 13:46 |
| mag009_ | the thing is I have a ToR that manage the dhcp/network | 13:46 |
| mag009_ | so I dont need neutron in my guess but I'd like to have my node in openstack | 13:46 |
| jroll | hmmm | 13:47 |
| jroll | nova kind of expects neutron to be there | 13:47 |
| jroll | you can use ironic without nova and neutron, but I don't think there's a good way to use nova without neutron | 13:47 |
| jroll | you might be able to hack it out and then use static dhcp with ironic | 13:47 |
| jroll | (hack it out of nova) | 13:47 |
| *** openstackgerrit has quit IRC | 13:48 | |
| mag009_ | yes ok thats what I thought just needed the confirmation I can deal with it | 13:48 |
| NobodyCam | Good MOrning Ironic | 13:48 |
| mag009_ | and maybe another the idrac driver can I manage my raid config from it in 5.0 ? | 13:48 |
| *** openstackgerrit has joined #openstack-ironic | 13:49 | |
| mag009_ | is it ready for that yet ? | 13:49 |
| jroll | mag009_: don't look like the drac driver has support yet, no | 13:50 |
| jroll | morning NobodyCam :) | 13:50 |
| mag009_ | ok thx :) | 13:50 |
| jroll | not a problem | 13:50 |
| mag009_ | any chance to see ironic-inspector integrated/packaged in mitaka release? | 13:51 |
| mag009_ | for ubuntu exampel | 13:52 |
| mag009_ | example* | 13:52 |
| jroll | maybe? we don't really deal with packaging here | 13:52 |
| jroll | I believe debian will be packaging it | 13:52 |
| jroll | and ubuntu gets most packages from debian afaik | 13:52 |
| mag009_ | hm if we wait after debian... | 13:52 |
| mag009_ | will wait forever | 13:52 |
| NobodyCam | Morning jroll jlvillal devananda vdrok sinval sambetts Haomeng TheJulia dtantsur JayF and all others not listed here :) | 13:52 |
| TheJulia | I think they already packaged it fwiw | 13:53 |
| jroll | mag009_: well, debian packages things pretty quick | 13:53 |
| jroll | just gotta wait for ubuntu to pick it up | 13:53 |
| * jroll doesn't know who to talk with about ubuntu packages | 13:53 | |
| TheJulia | perhaps SpamapS knows.. | 13:54 |
| mag009_ | http://ftp.debian.org/debian/pool/main/i/ironic-inspector/ | 13:54 |
| mag009_ | your right | 13:54 |
| *** mbound has joined #openstack-ironic | 13:54 | |
| *** mbound has quit IRC | 13:59 | |
| *** ametts has joined #openstack-ironic | 13:59 | |
| sinval | morning NobodyCam and everyone else | 14:02 |
| NobodyCam | :) | 14:02 |
| TheJulia | Good morning sinval | 14:04 |
| jroll | heya sinval | 14:04 |
| SpamapS | Xenial is in final beta. They're not going to sync new packages... | 14:08 |
| SpamapS | But it would be a good candidate for backports | 14:08 |
| jroll | mag009_: ^ | 14:09 |
| NobodyCam | hey morning SpamapS :) | 14:09 |
| SpamapS | Howdy | 14:09 |
| SpamapS | Just popping in... I'm still a zombie. ;-) | 14:09 |
| NobodyCam | ;) same here.. here | 14:10 |
| TheJulia | good morning zombie SpamapS :) | 14:10 |
| * TheJulia thinks more coffee is needed | 14:10 | |
| NobodyCam | +++ | 14:10 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add flavor_id baremetal tempest option https://review.openstack.org/293020 | 14:12 |
| mag009_ | anyone knows if its possible to push partition during an deploy ? | 14:13 |
| jroll | vsaienko: any reason why we aren't just re-ordering devstack to run the plugins before tempest? | 14:15 |
| jroll | has that been talked about? | 14:15 |
| jroll | aha, it has | 14:16 |
| vsaienko | jroll, not sure that it will be accepted https://review.openstack.org/#/c/293954/ | 14:16 |
| jroll | yeah, I see it now | 14:17 |
| jroll | vsaienko: so I think we should set the normal tempest value, not introduce a new tempest config | 14:18 |
| jroll | vsaienko: we need this value to work when running things like tempest full, which runs other tests in tempest | 14:18 |
| jroll | iniset $TEMPEST_CONFIG compute flavor_ref $flavor_ref | 14:19 |
| * jroll comments in review | 14:19 | |
| vsaienko | jroll, It was done in pathset 4 please have look https://review.openstack.org/#/c/293020/4 | 14:20 |
| *** Nisha has quit IRC | 14:21 | |
| jroll | vsaienko: yeah, I agree with the comments there, I disagree with the solution | 14:22 |
| jroll | vsaienko: I think using IRONIC_FLAVOR_NAME is good, I think a new tempest config is bad | 14:23 |
| jroll | because it won't work with tests that are in the tempest tree | 14:23 |
| TheJulia | so... craziness... shouldn't we possibly strip \n/s out of things like driver_info? | 14:24 |
| jroll | heh | 14:25 |
| jroll | maybe? | 14:25 |
| TheJulia | I just accidently didn't close a string for a parameter and was like "oh... it actually took that... hmmmm | 14:25 |
| jroll | probably should do a strip(), idk about pulling them out of the middle | 14:25 |
| TheJulia | yeah | 14:25 |
| *** vishwana_ has joined #openstack-ironic | 14:26 | |
| *** vishwana_ has quit IRC | 14:26 | |
| vsaienko | jroll so we introduce new variable IRONIC_TEMPEST_FLAVOR_NAME which is unset by default, Update iniset $TEMPEST_CONFIG compute flavor_ref $flavor_ref if IRONIC_TEMPEST_FLAVOR_NAME is set. And update ironic jobs to set IRONIC_TEMPEST_FLAVOR_NAME right? | 14:27 |
| TheJulia | jroll: yeah, that might be a good idea espesically around usernames, passwords... or anything that gets executed... | 14:28 |
| jroll | vsaienko: no, I think it should default to baremetal | 14:28 |
| jroll | vsaienko: I like IRONIC_FLAVOR_NAME=${IRONIC_FLAVOR_NAME:-baremetal} | 14:28 |
| jroll | TheJulia: ooooooooooooo. this may be fun :| | 14:28 |
| jroll | I assume processutils.execute will quote them though | 14:28 |
| * jroll calms down | 14:29 | |
| *** vishwanathj_zzz has quit IRC | 14:29 | |
| TheJulia | jroll: ummm... I _think_ it does | 14:29 |
| *** vishwanathj has joined #openstack-ironic | 14:29 | |
| TheJulia | jroll: the real issue would be variable use ane expansion in something like a scripting invocation | 14:30 |
| * TheJulia thinks this might just be a good best practice thing to do | 14:30 | |
| jroll | I'm not sure I grok that | 14:32 |
| vsaienko | jroll, so if IRONIC_FLAVOR_NAME is not set we also do not create default baremetal flavor right? https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L790 | 14:32 |
| TheJulia | jroll: would be something we would want in case before say.,. letting an ansible driver that allows scripts/variables/data to be passed along | 14:33 |
| jroll | TheJulia: oh, before we template a playbook or something | 14:35 |
| TheJulia | yup | 14:35 |
| jroll | fwiw I tested execute() with this, with both shell=True/False | 14:35 |
| jroll | and it quotes things properly | 14:35 |
| TheJulia | sorry, I'm thinking far off in the future :) | 14:35 |
| jroll | so we should be ok for now | 14:35 |
| jroll | yeah :) | 14:35 |
| TheJulia | Yeah :) | 14:35 |
| jroll | vsaienko: if we default IRONIC_FLAVOR_NAME to baremetal, when would it be not set? | 14:36 |
| jroll | vsaienko: I'm not sure we even need IRONIC_FLAVOR_NAME unless someone asks for it | 14:36 |
| jroll | vsaienko: I think the patch to solve the bug at hand is just add the iniset | 14:37 |
| jroll | vsaienko: left another comment to that effect | 14:41 |
| * jroll steps away for breakfast | 14:42 | |
| *** max_lobur has left #openstack-ironic | 14:47 | |
| *** UForgotten has joined #openstack-ironic | 14:48 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add test for --wrap option for driver-properties output https://review.openstack.org/297077 | 14:48 |
| * NobodyCam needs more koffi | 14:56 | |
| *** garthb has joined #openstack-ironic | 15:06 | |
| *** alexpilotti has joined #openstack-ironic | 15:09 | |
| TheJulia | NobodyCam: about another gallon should do it :) | 15:10 |
| NobodyCam | ++++ | 15:10 |
| *** jcoufal has quit IRC | 15:12 | |
| *** Marga_ has joined #openstack-ironic | 15:12 | |
| *** Marga_ has quit IRC | 15:13 | |
| *** Marga_ has joined #openstack-ironic | 15:13 | |
| *** alexpilotti has quit IRC | 15:14 | |
| *** achanda has joined #openstack-ironic | 15:17 | |
| *** ChrisAusten has joined #openstack-ironic | 15:20 | |
| -openstackstatus- NOTICE: Gerrit is restarting on review.openstack.org in an attempt to address an issue reading an object from the ec2-api repository | 15:21 | |
| *** jaybeale has joined #openstack-ironic | 15:22 | |
| *** achanda has quit IRC | 15:22 | |
| *** absubram has joined #openstack-ironic | 15:23 | |
| NobodyCam | vdrok: thank you for the reply on 297745 ... DOH.. that was first review with very little coffee and only a quick grep :p | 15:25 |
| vdrok | No problem NobodyCam :) thanks for review. | 15:27 |
| vdrok | And morning everyone! | 15:27 |
| jroll | hey vdrok :) | 15:28 |
| NobodyCam | vote updated | 15:28 |
| *** causten_ has joined #openstack-ironic | 15:32 | |
| *** ChrisAusten has quit IRC | 15:36 | |
| NobodyCam | wow 5 fish bowl sessions | 15:37 |
| NobodyCam | if I'm reading correctly | 15:37 |
| jroll | yep | 15:37 |
| jroll | 5/5 | 15:37 |
| jroll | (fishbowl/workroom) | 15:37 |
| *** spandhe has joined #openstack-ironic | 15:38 | |
| *** keedya has joined #openstack-ironic | 15:39 | |
| *** mdorman has joined #openstack-ironic | 15:39 | |
| *** mtanino has joined #openstack-ironic | 15:43 | |
| *** jlk has left #openstack-ironic | 15:45 | |
| jlvillal | Good morning all :) | 15:52 |
| jroll | man summit wednesday is going to be jam packed | 15:53 |
| NobodyCam | morning jlvillal | 15:53 |
| *** keedya has quit IRC | 15:53 | |
| NobodyCam | ya | 15:54 |
| *** keedya has joined #openstack-ironic | 15:56 | |
| *** garthb_ has joined #openstack-ironic | 15:56 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 15:57 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 15:57 |
| *** garthb has quit IRC | 15:57 | |
| jroll | ohai jlvillal :) | 15:58 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add test for --wrap option for driver-properties output https://review.openstack.org/297077 | 16:01 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update tempest compute flavor_ref/flavor_ref_alt https://review.openstack.org/293020 | 16:03 |
| jlvillal | :) | 16:03 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update tempest compute flavor_ref/flavor_ref_alt https://review.openstack.org/293020 | 16:04 |
| jroll | thanks vsaienko \o/ | 16:05 |
| jroll | JayF: mind a look at https://review.openstack.org/#/c/297796/ when you're around? | 16:06 |
| vsaienko | you are welcome | 16:06 |
| *** garthb_ has quit IRC | 16:06 | |
| *** piet has quit IRC | 16:07 | |
| *** garthb has joined #openstack-ironic | 16:07 | |
| vsaienko | does anybody know why depends-on to python-ironicclient is not working from patches to Ironic repo? | 16:07 |
| JayF | jroll: we're going to have to do that somewhere else, too | 16:07 |
| *** piet has joined #openstack-ironic | 16:07 | |
| JayF | jroll: because full_trusty_build is only used by the post job | 16:07 |
| jroll | JayF: I think devstack must already install it, because the devstack gate is still good | 16:08 |
| jroll | install/mount | 16:08 |
| JayF | hmm | 16:08 |
| TheJulia | vsaienko: popssibly if pip is explicitly running against only requirements and not the actual source code of | 16:08 |
| jroll | but full_trusty_build doesn't use devstack | 16:08 |
| JayF | That's strange | 16:08 |
| jroll | vsaienko: TheJulia: yeah, ironic jobs run from pip, not source | 16:09 |
| JayF | +2'd | 16:09 |
| jroll | JayF: yeah, it is, the strangest part being it fails in the middle of the job, not at the start | 16:09 |
| jroll | but maybe that's only necessary for certain commands | 16:09 |
| *** alexpilotti has joined #openstack-ironic | 16:09 | |
| TheJulia | vsaienko: bifrost has logic to do the install from source so the dependency gets pulled in, _but_ bifrost only interacts with the python api and not the cli | 16:09 |
| JayF | jroll: isn't ironic meeting now/ | 16:10 |
| *** max_lobur has joined #openstack-ironic | 16:10 | |
| jroll | JayF: one hour | 16:10 |
| *** max_lobur has left #openstack-ironic | 16:10 | |
| JayF | jroll: oh, did I never update my invite? | 16:10 |
| jroll | JayF: dunno, I ended up deleting yours in favor of one under my control :) | 16:10 |
| JayF | I have outlook client installed on my mac now :) I can actually list it as UTC | 16:11 |
| JayF | now I just have to figure out what time it is utc, lol | 16:11 |
| JayF | 1700, bingo | 16:11 |
| *** Marga_ has quit IRC | 16:13 | |
| *** afaranha has quit IRC | 16:14 | |
| *** harshs has joined #openstack-ironic | 16:16 | |
| *** e0ne has quit IRC | 16:16 | |
| *** mannidi has joined #openstack-ironic | 16:16 | |
| *** cameron_c_ has joined #openstack-ironic | 16:19 | |
| *** piet has quit IRC | 16:21 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add test for --wrap option for driver-properties output https://review.openstack.org/297077 | 16:22 |
| *** [1]cdearborn has joined #openstack-ironic | 16:27 | |
| [1]cdearborn | hey guys, anyone know how to get a node out of the ERROR provisioning state? | 16:28 |
| *** Sukhdev has joined #openstack-ironic | 16:30 | |
| *** harshs has quit IRC | 16:34 | |
| TheJulia | [1]cdearborn: deleted or rebuild verbs | 16:34 |
| *** harshs has joined #openstack-ironic | 16:36 | |
| *** alexpilotti has quit IRC | 16:39 | |
| jroll | someone care to look at this? https://review.openstack.org/#/c/297796/ | 16:43 |
| dtantsur | hey-hey everyone! good UGT morning :) | 16:43 |
| jroll | still trying to fix ipa post jobs :| | 16:43 |
| jroll | welcome back dtantsur :) | 16:43 |
| dtantsur | 2 questions: do we have a meeting today and am I right assuming it's in 15 minutes? :) | 16:44 |
| dtantsur | jroll, o/ | 16:44 |
| *** david-lyle_ has joined #openstack-ironic | 16:44 | |
| jroll | dtantsur: yes, yes | 16:44 |
| *** david-lyle has quit IRC | 16:44 | |
| dtantsur | great :) | 16:44 |
| JayF | dtantsur: can you review and approve https://review.openstack.org/#/c/297796/ in the intervening 15m? So jroll can fix ipa post jobs | 16:45 |
| jroll | dtantsur: couple things to catch you up on: stable/kilo gate is green again, and there's an exciting discussion on https://review.openstack.org/#/c/295966/ (ironic stable/liberty release) going on in the release channel | 16:45 |
| dtantsur | thanks, will look at both | 16:46 |
| NobodyCam | jroll: I just did a minor update to the agenda.. :p | 16:46 |
| jroll | I see that, heh | 16:46 |
| jroll | good update | 16:46 |
| dtantsur | jroll, thanks for taking care of the inspector client while I was out | 16:47 |
| jroll | not a problem :) | 16:47 |
| *** david-lyle has joined #openstack-ironic | 16:48 | |
| *** david-lyle_ has quit IRC | 16:49 | |
| *** thiagop has joined #openstack-ironic | 16:50 | |
| thiagop | Good morning, Ironicers | 16:51 |
| thiagop | Happy new release! | 16:51 |
| NobodyCam | morning thiagop | 16:51 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 16:51 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 16:51 |
| jroll | vsaienko: btw, we could add the job that runs devstack with ironicclient from master as experimental, if that would help you | 16:53 |
| dtantsur | oh, interesting discussion indeed (re stable/liberty) | 16:53 |
| *** saripurigopi has joined #openstack-ironic | 16:53 | |
| jlvillal | JayF: dtantsur : jroll: I reviewed https://review.openstack.org/#/c/297796/ | 16:54 |
| vsaienko | jroll I did https://review.openstack.org/296432 workaround and it works | 16:54 |
| dtantsur | hehe, I was a few seconds late :) thanks jlvillal and good morning | 16:54 |
| jlvillal | :) | 16:54 |
| jroll | thanks jlvillal | 16:54 |
| jroll | vsaienko: ok, cool | 16:55 |
| *** UForgotten has quit IRC | 16:55 | |
| *** UForgotten has joined #openstack-ironic | 16:55 | |
| *** rpioso has joined #openstack-ironic | 16:56 | |
| *** alexpilotti has joined #openstack-ironic | 16:56 | |
| jroll | meeting in 90 seconds | 16:58 |
| *** alexpilotti has quit IRC | 16:59 | |
| *** piet has joined #openstack-ironic | 17:01 | |
| *** sivaramakrishna has joined #openstack-ironic | 17:02 | |
| TheJulia | vdrok: Thank you :) | 17:02 |
| *** harshs has quit IRC | 17:02 | |
| *** Marga_ has joined #openstack-ironic | 17:02 | |
| TheJulia | I posted my thoughts made my review a -1 | 17:02 |
| *** alexpilotti has joined #openstack-ironic | 17:05 | |
| openstackgerrit | Satoru Moriya proposed openstack/ironic-specs: Volume connection information for Ironic nodes https://review.openstack.org/200496 | 17:08 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 17:08 |
| TheJulia | ohhh | 17:08 |
| TheJulia | smoriya: Thank you :) | 17:08 |
| vdrok | TheJulia: thank you! I commented about that too, but decided that it can be done later as that had 2 +2s. I need to be more persistent :) | 17:10 |
| *** saripurigopi has quit IRC | 17:12 | |
| TheJulia | vdrok: :) | 17:13 |
| *** moshele has quit IRC | 17:15 | |
| *** piet has quit IRC | 17:21 | |
| *** MattMan has quit IRC | 17:21 | |
| *** MattMan has joined #openstack-ironic | 17:21 | |
| *** achanda has joined #openstack-ironic | 17:22 | |
| *** sivaramakrishna has quit IRC | 17:23 | |
| *** david-lyle has quit IRC | 17:24 | |
| dtantsur | jroll, before I forgot: if you need a stable branch cross-project liaison (https://wiki.openstack.org/wiki/CrossProjectLiaisons#Stable_Branch), feel free to add me :) | 17:24 |
| jroll | dtantsur: yes please, go ahead :) | 17:24 |
| jroll | dtantsur: I've already been treating you as that liaison ;) | 17:25 |
| *** david-lyle has joined #openstack-ironic | 17:25 | |
| dtantsur | heh, noted :) | 17:26 |
| *** achanda has quit IRC | 17:27 | |
| dtantsur | done | 17:27 |
| *** saripurigopi has joined #openstack-ironic | 17:28 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Mount cgroups in full_trusty_build https://review.openstack.org/297796 | 17:29 |
| *** spandhe_ has joined #openstack-ironic | 17:30 | |
| dtantsur | jroll, ^^ | 17:30 |
| jroll | dtantsur: yep, watching for post jobs now :) | 17:31 |
| *** spandhe has quit IRC | 17:31 | |
| *** spandhe_ is now known as spandhe | 17:31 | |
| *** penick has joined #openstack-ironic | 17:31 | |
| *** Goneri has joined #openstack-ironic | 17:34 | |
| *** suro-patz has joined #openstack-ironic | 17:34 | |
| *** jaybeale has quit IRC | 17:35 | |
| jroll | whee | 17:35 |
| * jroll stares at zuul status | 17:35 | |
| * jlvillal thinks jroll needs a new hobby ;) | 17:36 | |
| * TheJulia wonders if jroll is actually glaring yet | 17:36 | |
| dtantsur | It's a public holiday here, and I'll continue enjoying it :) see you tomorrow for real | 17:37 |
| jroll | hehe | 17:37 |
| TheJulia | JayF: based on the existing spec, it does look nicely detached, it will just need to be something that gets mixed in I think | 17:37 |
| jroll | nah just waiting for these post jobs to get queued up | 17:37 |
| jlvillal | dtantsur: You should take all of your holidays!!!! | 17:37 |
| TheJulia | dtantsur: have a good evening | 17:37 |
| jroll | later dtantsur \o | 17:39 |
| *** dtantsur is now known as dtantsur|afk | 17:39 | |
| *** kromanenko has quit IRC | 17:40 | |
| JayF | http://docs.openstack.org/developer/ironic-python-agent/#custom-hardwaremanagers-and-cleaning gotta love it when you try to document a feature | 17:40 |
| JayF | and find it was already documented. despite not being done. | 17:41 |
| JayF | for a long ass time too I think | 17:41 |
| JayF | lol | 17:41 |
| TheJulia | doh | 17:42 |
| *** piet has joined #openstack-ironic | 17:43 | |
| *** krtaylor has quit IRC | 17:44 | |
| *** Marga_ has quit IRC | 17:44 | |
| *** Marga_ has joined #openstack-ironic | 17:46 | |
| *** achanda has joined #openstack-ironic | 17:46 | |
| *** cameron_c_ has quit IRC | 17:47 | |
| saripurigopi | morning Ironic | 17:51 |
| thiagop | morning saripurigopi | 17:51 |
| saripurigopi | thiagop: o/ | 17:52 |
| *** krtaylor has joined #openstack-ironic | 17:56 | |
| *** piet has quit IRC | 17:56 | |
| *** Sukhdev has quit IRC | 17:57 | |
| *** e0ne has joined #openstack-ironic | 17:57 | |
| *** penick has quit IRC | 17:59 | |
| *** baoli has quit IRC | 18:00 | |
| jroll | what the https://jenkins05.openstack.org/job/ironic-python-agent-buildimage-coreos/60/console | 18:01 |
| *** baoli has joined #openstack-ironic | 18:01 | |
| jroll | urgh, apparently that doesn't exist in trusty | 18:02 |
| JayF | ugh | 18:02 |
| openstackgerrit | John L. Villalovos proposed openstack/python-ironicclient: Fix pep8 stderr warning regarding __all__ defined as list https://review.openstack.org/298355 | 18:02 |
| JayF | jroll: I'll trade you help with silly OS packages for help with silly python test problems | 18:02 |
| *** kwazar_ is now known as kwazar | 18:02 | |
| *** penick has joined #openstack-ironic | 18:03 | |
| *** rbrud has joined #openstack-ironic | 18:04 | |
| jroll | JayF: sure | 18:04 |
| *** moshele has joined #openstack-ironic | 18:05 | |
| *** piet has joined #openstack-ironic | 18:05 | |
| JayF | jroll: 99% sure rereading teh code that that suggestion to short circuit the extra call was wrong | 18:05 |
| JayF | first short circuit is "agent had no commands", what happens later is "agent running command" | 18:06 |
| jroll | JayF: or you're putting it in the wrong spot or something :) | 18:06 |
| JayF | I think I figured it out walking slowly through it | 18:06 |
| jroll | ok | 18:06 |
| JayF | no, but I'll still kill a call out to the agent | 18:06 |
| jroll | cool | 18:06 |
| jroll | no | 18:06 |
| JayF | this explains an old ass bug I was chasing like 2y ago | 18:06 |
| jroll | because that method starts with a short circuit | 18:06 |
| JayF | where it made extra calls out to the agent | 18:06 |
| jroll | if not commands: return | 18:07 |
| JayF | AFTER it refetches commands | 18:07 |
| jroll | no? | 18:07 |
| JayF | https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_base_vendor.py#L545 | 18:07 |
| jroll | yeah but your patch removes that | 18:07 |
| JayF | I'm going to have the agent_commands that the cleaning_reboot code fetches passed into that | 18:07 |
| JayF | yeah | 18:07 |
| JayF | oh | 18:07 |
| jroll | so moving the short-circuit as suggested just saves a push/pop on the stack | 18:08 |
| jroll | for the method call | 18:08 |
| jroll | it's fine to move it, but then if something else uses that method they'll also need to short circuit | 18:08 |
| jroll | so, meh | 18:08 |
| jroll | so 313 would be if not agent_commands: return | 18:09 |
| jroll | if you did move it | 18:09 |
| jroll | 313 in https://review.openstack.org/#/c/259135/17/ironic/drivers/modules/agent_base_vendor.py , I mean | 18:09 |
| JayF | yeah, but you still need an if not command: return later | 18:10 |
| jroll | ah yeah | 18:10 |
| JayF | because _get_completed_cleaning_command can return none if there are only commands in progress | 18:10 |
| jroll | yep | 18:10 |
| JayF | jroll: I suspect the package you'll want on trusty is something like "lxc" | 18:11 |
| *** david-lyle_ has joined #openstack-ironic | 18:12 | |
| *** david-lyle has quit IRC | 18:14 | |
| *** degorenko is now known as _degorenko|afk | 18:15 | |
| *** pcaruana has quit IRC | 18:16 | |
| jroll | JayF: hrm | 18:21 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: Add lxc to full_trusty_build deps https://review.openstack.org/298364 | 18:23 |
| jroll | JayF: here's a patch | 18:24 |
| JayF | speaking of... | 18:25 |
| JayF | (do your thing openstackgerrit, gogogo) | 18:25 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Support reboot_requested bool on agent clean_steps https://review.openstack.org/259135 | 18:25 |
| jroll | cool | 18:26 |
| JayF | I'm going to go make myself a sandwich, bbs | 18:28 |
| jroll | JayF: just one thing there and then it lgtm | 18:28 |
| openstackgerrit | Merged openstack/ironic-webclient: Added power button to node list https://review.openstack.org/274847 | 18:28 |
| openstackgerrit | Merged openstack/ironic-webclient: Removed account https://review.openstack.org/274848 | 18:28 |
| *** zer0c00l has joined #openstack-ironic | 18:30 | |
| *** pcaruana has joined #openstack-ironic | 18:30 | |
| * jroll lunches as well | 18:31 | |
| *** jaypipes has quit IRC | 18:31 | |
| *** baoli has quit IRC | 18:33 | |
| *** jaybeale has joined #openstack-ironic | 18:33 | |
| *** baoli has joined #openstack-ironic | 18:35 | |
| *** keedya has quit IRC | 18:36 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Support reboot_requested bool on agent clean_steps https://review.openstack.org/259135 | 18:37 |
| JayF | jroll: ^ | 18:38 |
| *** baoli has quit IRC | 18:40 | |
| *** baoli has joined #openstack-ironic | 18:41 | |
| *** baoli has quit IRC | 18:46 | |
| *** baoli has joined #openstack-ironic | 18:48 | |
| *** e0ne has quit IRC | 18:50 | |
| *** achanda has quit IRC | 18:51 | |
| *** achanda has joined #openstack-ironic | 18:54 | |
| *** penick has quit IRC | 18:57 | |
| *** baoli has quit IRC | 18:58 | |
| *** penick has joined #openstack-ironic | 18:59 | |
| *** baoli has joined #openstack-ironic | 18:59 | |
| *** baoli has quit IRC | 19:00 | |
| *** moshele has quit IRC | 19:02 | |
| *** piet has quit IRC | 19:02 | |
| *** moshele has joined #openstack-ironic | 19:03 | |
| jroll | JayF: +2, thanks | 19:10 |
| JayF | jroll: do you think I need a spec for the leave-nodes-powered-on-post-cleaning (aka which is now turning into, power on nodes as they enter AVAILABLE) | 19:11 |
| jroll | JayF: at least an RFE | 19:11 |
| JayF | jroll: just curious if you still think we need a big long-running-ramdisks spec and plan, or if just doing something like what we have downstream already is kosher for upstream | 19:12 |
| jroll | JayF: probably the former since we don't handle failure modes well etc | 19:12 |
| jroll | manageability is a big part | 19:12 |
| JayF | I mean, I'd argue we have managablility problems already with short running ramdisks too | 19:12 |
| jroll | especially when you consider driver_info indicates the ramdisk that should be running, how do you manage updates on that etc | 19:12 |
| JayF | but I can do that :( | 19:12 |
| JayF | oh shit | 19:13 |
| *** mannidi has quit IRC | 19:14 | |
| jroll | looking for review here: https://review.openstack.org/#/c/298364/ | 19:18 |
| *** e0ne has joined #openstack-ironic | 19:19 | |
| jroll | JayF: yeah, so it might end up needing a spec; if you file the RFE we can discuss it in meeting | 19:19 |
| *** e0ne has quit IRC | 19:21 | |
| *** penick has quit IRC | 19:23 | |
| *** e0ne has joined #openstack-ironic | 19:26 | |
| *** Sukhdev has joined #openstack-ironic | 19:27 | |
| *** moshele has quit IRC | 19:28 | |
| *** e0ne has quit IRC | 19:28 | |
| *** thiagop has quit IRC | 19:33 | |
| *** moshele has joined #openstack-ironic | 19:39 | |
| *** moshele has quit IRC | 19:42 | |
| *** clenimar has quit IRC | 19:45 | |
| NobodyCam | jroll: looking at 298364, LXC is so we get a good control-groups? | 19:45 |
| jroll | NobodyCam: so that cgroupfs is mounted | 19:46 |
| *** harshs has joined #openstack-ironic | 19:46 | |
| *** Sukhdev has quit IRC | 19:47 | |
| *** baoli has joined #openstack-ironic | 19:47 | |
| NobodyCam | not that I have tested but did you try the cgmanager package? | 19:48 |
| *** baoli has quit IRC | 19:48 | |
| *** baoli has joined #openstack-ironic | 19:49 | |
| JayF | Are root_device_hints per node or per deploy? i.e. are they tied to the instance or the node/ | 19:49 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 19:49 |
| TheJulia | node afaik | 19:50 |
| jroll | NobodyCam: no, I don't really have a good way to test this | 19:50 |
| JayF | node.properties, apparently | 19:51 |
| mrda | Morning Ironic | 19:51 |
| * JayF trying to think through pre-booted ramdisks | 19:51 | |
| * mrda ddin't have much success with device hinting | 19:52 | |
| NobodyCam | morning mrda | 19:52 |
| mrda | hey NobodyCam | 19:52 |
| TheJulia | mrda: what were you using as the hint? | 19:52 |
| * mrda can't rmember the name | 19:53 | |
| mrda | The UUID of the disk | 19:53 |
| TheJulia | mrda: good morning by the way | 19:53 |
| mrda | TheJulia: o/ | 19:53 |
| TheJulia | mrda: I don't think thats a property that is looked at afaik | 19:53 |
| TheJulia | the list of things is actually kind of short | 19:53 |
| mrda | Yeah, it was actually the GUID, which on my test Dell server is the first N bits of the UUID (can't remember the disk manufacturer) | 19:55 |
| mrda | (Also not sure if it's an abstarct GUID i.e. it's what the Dell preents the disk as rather than the actual disk) | 19:55 |
| mrda | But nonetheless, I didn't get it working | 19:56 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Make node manageable https://review.openstack.org/282458 | 19:56 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: IronicNodeProvision resource https://review.openstack.org/282457 | 19:56 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Basic node action hooks. https://review.openstack.org/281900 | 19:56 |
| TheJulia | mrda: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/utils.py#L31 if you ever look at it again | 19:57 |
| mrda | TheJulia: ahh, that's what it was. The WWN. | 19:58 |
| TheJulia | I recently had someone trying to use the long format wwn, once I go tthem to use the short format, it worked like a champ | 19:58 |
| TheJulia | we should likely extend that list in the future | 20:00 |
| * TheJulia files mental note in back of brain | 20:00 | |
| mrda | Yeah, I think I was fighting a heisenbug. I went back and forth between short and long wwn with no success, but then I switch to a different dell and didn't need to specify the wwn as it automagically got /dev/sda right without having to go to that level of specification | 20:00 |
| TheJulia | nice! | 20:00 |
| mrda | TheJulia: IIRC there was a patch to add an abstracted name to that list | 20:01 |
| TheJulia | oh, if the raid controller had a volume name? | 20:01 |
| mrda | yeah | 20:01 |
| mrda | specify the volume by name is the DRAC or iLO, and then use that in ipa | 20:02 |
| mrda | I saw a patch to add this, but it was still in review | 20:02 |
| mrda | since you pointed me to the code, it clearly didn't merge :) | 20:02 |
| TheJulia | :) | 20:02 |
| TheJulia | I seem to remember that, I kind of wonder what happened ot it | 20:03 |
| mrda | Also, it feels strange working from my regular desk this morning - haven't done this for a month as I've been at a customer's site :) | 20:03 |
| *** Sukhdev has joined #openstack-ironic | 20:04 | |
| TheJulia | mrda: :) | 20:06 |
| NobodyCam | welcome home mrda | 20:06 |
| mrda | \o/ | 20:06 |
| *** penick has joined #openstack-ironic | 20:08 | |
| *** baoli has quit IRC | 20:10 | |
| *** baoli has joined #openstack-ironic | 20:10 | |
| *** baoli has quit IRC | 20:11 | |
| *** baoli has joined #openstack-ironic | 20:11 | |
| jlvillal | jroll: Regarding https://review.openstack.org/#/c/297732/ Is that something for the future? | 20:13 |
| jlvillal | Hi mrda :) | 20:14 |
| jroll | jlvillal: it should be for now but the post job is still broken. | 20:14 |
| jroll | waiting on review for this still, to hopefully unbreak it https://review.openstack.org/#/c/298364/ | 20:14 |
| jlvillal | Ah, thanks. | 20:14 |
| mrda | hey jlvillal | 20:15 |
| jlvillal | jroll: Okay, +2 +A. I'll trust it will work in the future :) | 20:16 |
| *** dprince has quit IRC | 20:17 | |
| *** Nisha has joined #openstack-ironic | 20:17 | |
| *** rpioso has quit IRC | 20:17 | |
| NobodyCam | jroll: +2 with a comment ... giving other folks time to take a look before +a | 20:17 |
| jroll | thanks | 20:18 |
| NobodyCam | willing to hit the button if don't get any other lookers | 20:19 |
| NobodyCam | :) | 20:19 |
| JayF | NobodyCam: replied to your comment | 20:19 |
| JayF | NobodyCam: tl;dr docker used to dep on lxc, so it's not as extreme as it seems | 20:19 |
| JayF | devananda: got a sec to talk about https://review.openstack.org/#/c/263842/2 | 20:20 |
| *** piet has joined #openstack-ironic | 20:20 | |
| jroll | NobodyCam: what jay said, it's also a recommended package fwiw :) | 20:20 |
| JayF | devananda: for about 3 different things I wanna do, the dep chain resolves back to "fix taht problem first", so I wanted to see if you had a sec to talk about strategies for getting that code in without breaking things | 20:20 |
| JayF | devananda: I have a couple of ideas, but honestly would rather have a sync chat with you on it, because it's API design and you've got a bit more exp with that than I do | 20:21 |
| *** Sukhdev has quit IRC | 20:21 | |
| NobodyCam | ok then lets hit hte button | 20:21 |
| *** baoli has quit IRC | 20:22 | |
| NobodyCam | I was looking at cgmanager and cgmanager-utils packages | 20:22 |
| *** Nisha has quit IRC | 20:22 | |
| JayF | I'll approve it then, ty | 20:22 |
| jlvillal | NobodyCam: jroll JayF: I did a +A on https://review.openstack.org/#/c/298364/ | 20:22 |
| NobodyCam | jlvillal: :) | 20:22 |
| JayF | It's now +AA | 20:23 |
| JayF | lol | 20:23 |
| jlvillal | :) | 20:23 |
| NobodyCam | JayF: I also saw your comment here: https://review.openstack.org/#/c/297079/2/imagebuild/coreos/full_trusty_build.sh does that apply here? | 20:23 |
| jroll | thanks y'all | 20:23 |
| jroll | NobodyCam: it does not, we know that the build works in devstack because the IPA gate isn't broken in devstack :) | 20:24 |
| JayF | so full_trusty_build is based off trsuty, not devstack | 20:24 |
| JayF | devstack installs the proper stuff | 20:24 |
| NobodyCam | just saw the same line | 20:25 |
| NobodyCam | wanted to make sure | 20:25 |
| *** Sukhdev has joined #openstack-ironic | 20:28 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Add branched image download instructions to CoreOS readme https://review.openstack.org/297732 | 20:30 |
| mag009_ | hey | 20:35 |
| mag009_ | anyone can point me to the right direction | 20:35 |
| mag009_ | I want to create partitions(lvm) on my image that i install | 20:35 |
| mag009_ | i know that the python agent is not ready yet for that but how do you guys do that in the mean time? | 20:35 |
| *** alexpilotti has quit IRC | 20:38 | |
| JayF | oh you can absolutely do that today with full disk images | 20:38 |
| JayF | well... hmm, I wonder how that'd work with configdrive | 20:39 |
| JayF | I don't know how to put LVM inside your image using DIB though | 20:39 |
| mag009_ | ok lets say I want to partition without lvm | 20:40 |
| mag009_ | i can do that with disk build image ? | 20:40 |
| *** suro-patz has quit IRC | 20:40 | |
| mag009_ | this https://github.com/openstack/diskimage-builder | 20:40 |
| *** Goneri has quit IRC | 20:40 | |
| mag009_ | i did not found anywhere where I could do the partitioning from | 20:41 |
| JayF | I know it's possible; I don't know how :( I work in a giant cloud where images are given to me on a silver platter | 20:41 |
| JayF | if you hang out, others who know how to use it might come around | 20:41 |
| JayF | #tripleo folks might know a thing about it too | 20:41 |
| *** Sukhdev has quit IRC | 20:42 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Add lxc to full_trusty_build deps https://review.openstack.org/298364 | 20:48 |
| NobodyCam | not just wait for the post job :p | 20:52 |
| *** achanda has quit IRC | 20:52 | |
| *** baoli has joined #openstack-ironic | 20:52 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 20:53 |
| *** suro-patz has joined #openstack-ironic | 20:57 | |
| *** harshs has quit IRC | 20:58 | |
| *** alexpilotti has joined #openstack-ironic | 20:59 | |
| *** causten__ has joined #openstack-ironic | 20:59 | |
| *** causten_ has quit IRC | 20:59 | |
| *** Lyncos has joined #openstack-ironic | 21:00 | |
| *** baoli has quit IRC | 21:00 | |
| *** penick has quit IRC | 21:01 | |
| *** alexpilotti has quit IRC | 21:03 | |
| *** achanda has joined #openstack-ironic | 21:04 | |
| *** Sukhdev has joined #openstack-ironic | 21:04 | |
| *** alexpilotti has joined #openstack-ironic | 21:05 | |
| *** Sukhdev has quit IRC | 21:05 | |
| *** Marga_ has quit IRC | 21:06 | |
| *** Marga_ has joined #openstack-ironic | 21:06 | |
| *** baoli has joined #openstack-ironic | 21:07 | |
| *** Sukhdev has joined #openstack-ironic | 21:08 | |
| *** harshs has joined #openstack-ironic | 21:10 | |
| *** piet has quit IRC | 21:19 | |
| *** raildo is now known as raildo-afk | 21:23 | |
| *** jaybeale has quit IRC | 21:25 | |
| *** suro-patz has quit IRC | 21:33 | |
| devananda | JayF: looking ... | 21:33 |
| NobodyCam | morning /afternoon devananda :) | 21:34 |
| *** piet has joined #openstack-ironic | 21:34 | |
| *** Sukhdev has quit IRC | 21:36 | |
| *** achanda has quit IRC | 21:37 | |
| *** Sukhdev has joined #openstack-ironic | 21:37 | |
| JayF | devananda: lmk when you're back up to speed on the context; I just wanted to chat about the best way to implement that and get the patch unstuck | 21:42 |
| devananda | yea, I have an idea | 21:42 |
| JayF | devananda: one thing I ran past jroll in PM was just, for all implementations of that in the driver, have it check the conf in-method | 21:42 |
| devananda | JayF: the manage_boot param is always set to True in the code, except for where it inherits from the CONF option | 21:42 |
| devananda | JayF: so how about, rather than passing that option between methods and changing the base driver method signature, we move the logic around evaluating the CONF option | 21:43 |
| JayF | I'm not sure I follow | 21:43 |
| devananda | and put some warnings in place in case any out of tree drivers are passing that option right now | 21:43 |
| devananda | *passing that option to deploy_utils | 21:43 |
| devananda | JayF: https://review.openstack.org/#/c/263842/2/ironic/drivers/modules/deploy_utils.py | 21:44 |
| JayF | devananda: my idea was, have prepare_ramdisk() implementations themselves pull in the conf option, and act accordingly, rather than expect it to be passed in | 21:44 |
| JayF | that way you don't change the signature, you fix the behavior for in-band drivers, and have a template for out of band drivers to fix behavior if they want | 21:45 |
| *** cfarquhar has quit IRC | 21:45 | |
| devananda | JayF: I believe we're saying the same thing :) | 21:46 |
| JayF | okay, perfect then | 21:46 |
| JayF | I'll see if I can change it that way and get a new version of the patch up | 21:46 |
| devananda | that will move the CONF option out of the [agent] group | 21:47 |
| devananda | and change the function signature of deploy_utils.prepare_inband_cleaning | 21:47 |
| devananda | both of those will need deprecations | 21:47 |
| JayF | I know how to deprecate conf options | 21:47 |
| devananda | but it should have much less impact than changing base.prepare_ramdisk | 21:47 |
| devananda | yep. just going over the impact of that approach | 21:48 |
| JayF | for the method, what do I do? Look for that option; if it's passed use it and print a log | 21:48 |
| *** cfarquhar has joined #openstack-ironic | 21:48 | |
| *** cfarquhar has quit IRC | 21:48 | |
| *** cfarquhar has joined #openstack-ironic | 21:48 | |
| JayF | and then in O cycle we pull the kwarg from the signature altogether/ | 21:48 |
| devananda | i would say, leave the parameter on the function but then default it to None, and if any non-None value is passed, drop a warning in the logs and use the option instead | 21:48 |
| JayF | same idea, just fail-new instead of fail-old | 21:49 |
| devananda | yea | 21:49 |
| devananda | because there's no way to "use" that option if it's passed in | 21:49 |
| JayF | oh. OH. | 21:49 |
| JayF | makes perfect sense. Cool, I might be able to shake that out before the end of the day, nice | 21:49 |
| JayF | and then one more downstream patch will be dead, and I'll be on the way to crushing others | 21:49 |
| devananda | woot :) | 21:49 |
| * JayF paying off the Rackspace upstream credit card, then I can start saving up for shiny new features | 21:50 | |
| JayF | :P | 21:50 |
| devananda | hehehe | 21:50 |
| *** suro-patz has joined #openstack-ironic | 21:56 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 21:57 |
| openstackgerrit | Merged openstack/ironic: Fix typos in some source files. https://review.openstack.org/297576 | 21:57 |
| JayF | Hmm. Would deprecation inside a method be more like an API deprecation or an operator deprecation? https://specs.openstack.org/openstack/openstack-specs/specs/log-guidelines.html#operator-deprecation-warnings | 22:00 |
| * JayF treats it like an api one | 22:00 | |
| devananda | JayF: operator one, IMO | 22:01 |
| devananda | JayF: it's not a REST API deprecation at all | 22:01 |
| JayF | I think the distinction in the spec was more around "Does an operator or a dev do something about this?" | 22:02 |
| JayF | oh, it's more operator vs user | 22:02 |
| JayF | so warn does make sense for us | 22:02 |
| devananda | and I think this should be picked up by operators. "Hey, the system is going to behave differently than you might expect, because you're using an out of tree driver that doesn't support CONF.foo" | 22:02 |
| *** Sukhdev has quit IRC | 22:03 | |
| *** achanda has joined #openstack-ironic | 22:04 | |
| *** causten__ has quit IRC | 22:09 | |
| *** Marga_ has quit IRC | 22:19 | |
| *** penick has joined #openstack-ironic | 22:19 | |
| *** Marga_ has joined #openstack-ironic | 22:19 | |
| *** mdorman has quit IRC | 22:20 | |
| *** [1]cdearborn has quit IRC | 22:22 | |
| *** Sukhdev has joined #openstack-ironic | 22:25 | |
| *** harlowja has joined #openstack-ironic | 22:32 | |
| *** harlowja has quit IRC | 22:32 | |
| *** harlowja has joined #openstack-ironic | 22:33 | |
| *** mbound has joined #openstack-ironic | 22:35 | |
| *** jaybeale has joined #openstack-ironic | 22:35 | |
| jroll | devananda: JayF: oh, I guess that makes sense, nice | 22:36 |
| *** mbound has quit IRC | 22:39 | |
| *** causten__ has joined #openstack-ironic | 22:40 | |
| *** causten__ has quit IRC | 22:41 | |
| *** piet has quit IRC | 22:50 | |
| NobodyCam | jlvillal: just posted a comment to a OLD patch https://review.openstack.org/#/c/227581 | 22:53 |
| jlvillal | NobodyCam: Good idea :) | 22:54 |
| *** harlowja has quit IRC | 22:54 | |
| NobodyCam | lol :) | 22:54 |
| *** harlowja has joined #openstack-ironic | 22:54 | |
| *** david-lyle_ is now known as david-lyle | 22:57 | |
| openstackgerrit | Merged openstack/ironic: Assign valid values to UUIDFields in unit tests https://review.openstack.org/297745 | 23:01 |
| jroll | JayF: btw, that lxc install didn't help :/ http://logs.openstack.org/7a/7a0184d47dee635eb5dcc174fc3d87af72fef753/post/ironic-python-agent-buildimage-coreos/30dc4cc/console.html | 23:06 |
| jroll | I'm inclined to just find the automation that builds these images and build my own | 23:06 |
| JayF | what the hell man | 23:08 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: [WIP] Add notification base and example to ironic https://review.openstack.org/298461 | 23:08 |
| jroll | right? | 23:08 |
| *** yuanying has joined #openstack-ironic | 23:11 | |
| *** absubram has quit IRC | 23:15 | |
| *** penick has quit IRC | 23:15 | |
| *** harshs has quit IRC | 23:20 | |
| *** ametts has quit IRC | 23:22 | |
| jroll | fyi to anyone curious: tools/build-image.sh should be a rough equivalent to the gate images, given the update I made to it in 298463 | 23:23 |
| *** harlowja has quit IRC | 23:25 | |
| *** harlowja has joined #openstack-ironic | 23:25 | |
| openstackgerrit | Merged openstack/ironic-webclient: Updated eslint plugins https://review.openstack.org/293953 | 23:27 |
| *** shakamunyi has joined #openstack-ironic | 23:30 | |
| openstackgerrit | Merged openstack/ironic-webclient: Updated application dependencies https://review.openstack.org/293963 | 23:31 |
| *** barra204 has quit IRC | 23:32 | |
| *** achanda has quit IRC | 23:41 | |
| *** rbudden has quit IRC | 23:41 | |
| *** NobodyCam has quit IRC | 23:46 | |
| *** davideagnello has quit IRC | 23:51 | |
| JayF | https://review.openstack.org/#/c/259135/ is up, has 1x+2, and has survived many rounds of back and forth already; if someone has some spare time would love to get more reviews on it and get it merged up :) | 23:54 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!