| *** aslezil has quit IRC | 00:00 | |
| *** BadCub has quit IRC | 00:00 | |
| *** cppforlife_ has quit IRC | 00:00 | |
| *** betherly has quit IRC | 00:00 | |
| *** BadCub has joined #openstack-ironic | 00:00 | |
| *** amit213 has quit IRC | 00:01 | |
| *** comstud has quit IRC | 00:01 | |
| *** patrickeast has quit IRC | 00:02 | |
| *** natorious has quit IRC | 00:02 | |
| *** melwitt has quit IRC | 00:02 | |
| *** bl0m1_ has quit IRC | 00:02 | |
| *** jmccrory has quit IRC | 00:02 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Use longer names for screen- log files https://review.openstack.org/438082 | 00:04 |
|---|---|---|
| *** patrickeast has joined #openstack-ironic | 00:06 | |
| *** amit213 has joined #openstack-ironic | 00:06 | |
| *** bl0m1 has joined #openstack-ironic | 00:07 | |
| *** aslezil_ has joined #openstack-ironic | 00:07 | |
| *** jcoufal has quit IRC | 00:09 | |
| *** melwitt has joined #openstack-ironic | 00:09 | |
| *** jmccrory has joined #openstack-ironic | 00:09 | |
| *** comstud has joined #openstack-ironic | 00:09 | |
| *** melwitt is now known as Guest68076 | 00:09 | |
| *** natorious has joined #openstack-ironic | 00:10 | |
| *** betherly has joined #openstack-ironic | 00:11 | |
| *** Guest68076 is now known as melwitt | 00:12 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: WIP/DNM: Testing multi-tenant multi-node grenade job https://review.openstack.org/422149 | 00:18 |
| *** rajinir_ has quit IRC | 00:21 | |
| *** yuanying has quit IRC | 00:28 | |
| *** ijw has quit IRC | 00:29 | |
| *** mtanino_ has joined #openstack-ironic | 00:29 | |
| *** ijw has joined #openstack-ironic | 00:30 | |
| *** ijw has quit IRC | 00:30 | |
| *** mtanino has quit IRC | 00:30 | |
| *** hoangcx has joined #openstack-ironic | 00:36 | |
| *** natorious has quit IRC | 00:43 | |
| *** natorious has joined #openstack-ironic | 00:43 | |
| *** rpioso has quit IRC | 00:52 | |
| *** rloo has quit IRC | 00:52 | |
| *** fragatina has quit IRC | 01:01 | |
| *** SerenaFeng has joined #openstack-ironic | 01:02 | |
| TheJulia | goodnight everyone | 01:03 |
| *** phuongnh has joined #openstack-ironic | 01:03 | |
| *** sambetts|afk has quit IRC | 01:04 | |
| *** Syed__ has quit IRC | 01:05 | |
| *** zhugaoxiao has quit IRC | 01:06 | |
| *** Administrator_ has quit IRC | 01:06 | |
| *** sambetts_ has joined #openstack-ironic | 01:07 | |
| NobodyCam | Good Night TheJulia | 01:11 |
| *** yuanying has joined #openstack-ironic | 01:24 | |
| *** baoli has quit IRC | 01:28 | |
| *** sdake_ has joined #openstack-ironic | 01:30 | |
| *** zhangjl has joined #openstack-ironic | 01:32 | |
| *** sdake has quit IRC | 01:33 | |
| *** fragatina has joined #openstack-ironic | 01:36 | |
| *** sdake_ has quit IRC | 01:40 | |
| *** cppforlife_ has joined #openstack-ironic | 02:00 | |
| *** baoli has joined #openstack-ironic | 02:02 | |
| *** goldyfruit has quit IRC | 02:02 | |
| *** goldyfruit has joined #openstack-ironic | 02:03 | |
| *** gcb has joined #openstack-ironic | 02:06 | |
| *** goldyfruit has quit IRC | 02:07 | |
| *** bfournie has joined #openstack-ironic | 02:11 | |
| *** mtanino_ has quit IRC | 02:17 | |
| *** goldyfruit has joined #openstack-ironic | 02:18 | |
| *** goldyfruit has quit IRC | 02:19 | |
| *** goldyfruit has joined #openstack-ironic | 02:19 | |
| *** baoli has quit IRC | 02:32 | |
| *** goldyfruit has quit IRC | 02:35 | |
| *** goldyfruit has joined #openstack-ironic | 02:36 | |
| *** awiddersheim has quit IRC | 03:20 | |
| *** vikrant has joined #openstack-ironic | 03:35 | |
| *** zhenguo has joined #openstack-ironic | 03:37 | |
| *** goldyfruit has quit IRC | 03:50 | |
| *** baoli has joined #openstack-ironic | 03:51 | |
| *** aparnav has joined #openstack-ironic | 03:52 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 03:55 | |
| *** nmathew has joined #openstack-ironic | 03:56 | |
| *** fragatin_ has joined #openstack-ironic | 03:59 | |
| *** SerenaFeng has quit IRC | 04:00 | |
| *** fragati__ has joined #openstack-ironic | 04:01 | |
| *** fragatina has quit IRC | 04:03 | |
| *** fragatin_ has quit IRC | 04:04 | |
| *** fragati__ has quit IRC | 04:05 | |
| *** gcb has quit IRC | 04:22 | |
| *** crushil has quit IRC | 04:22 | |
| *** gcb has joined #openstack-ironic | 04:34 | |
| *** sasha2 has quit IRC | 04:36 | |
| *** tiendc has joined #openstack-ironic | 04:38 | |
| *** jaganathan has joined #openstack-ironic | 04:52 | |
| *** baoli has quit IRC | 04:54 | |
| *** crushil has joined #openstack-ironic | 04:54 | |
| *** vmud213 has joined #openstack-ironic | 05:00 | |
| *** sdake has joined #openstack-ironic | 05:03 | |
| *** sdake_ has joined #openstack-ironic | 05:07 | |
| *** sdake has quit IRC | 05:08 | |
| *** udesale has joined #openstack-ironic | 05:19 | |
| *** karthiks has joined #openstack-ironic | 05:20 | |
| *** fragatina has joined #openstack-ironic | 05:20 | |
| *** fragatina has quit IRC | 05:21 | |
| *** fragatina has joined #openstack-ironic | 05:21 | |
| *** udesale__ has joined #openstack-ironic | 05:33 | |
| *** udesale has quit IRC | 05:35 | |
| *** SerenaFeng has joined #openstack-ironic | 05:38 | |
| *** jaosorior has joined #openstack-ironic | 05:41 | |
| *** sdake_ has quit IRC | 05:47 | |
| *** sdake has joined #openstack-ironic | 05:47 | |
| *** hshiina has joined #openstack-ironic | 05:52 | |
| *** phuongnh has quit IRC | 05:59 | |
| *** phuongnh has joined #openstack-ironic | 05:59 | |
| *** phuongnh has quit IRC | 06:10 | |
| *** sdake_ has joined #openstack-ironic | 06:10 | |
| *** phuongnh has joined #openstack-ironic | 06:11 | |
| *** fragatina has quit IRC | 06:12 | |
| *** sdake has quit IRC | 06:13 | |
| *** tuanluong has joined #openstack-ironic | 06:13 | |
| *** sacharya has quit IRC | 06:13 | |
| *** sacharya has joined #openstack-ironic | 06:14 | |
| *** sacharya has quit IRC | 06:19 | |
| *** vsaienko has joined #openstack-ironic | 06:20 | |
| *** SerenaFeng has quit IRC | 06:22 | |
| furb | hi | 06:32 |
| *** vsaienko has quit IRC | 06:32 | |
| furb | I am confused whether ironic can be used in multi tenant environments or not | 06:33 |
| *** gk-1wm-su has joined #openstack-ironic | 06:33 | |
| *** gk-1wm-su has left #openstack-ironic | 06:33 | |
| *** udesale__ has quit IRC | 06:35 | |
| *** dyasny has quit IRC | 06:42 | |
| *** SerenaFeng has joined #openstack-ironic | 06:45 | |
| *** mjura has joined #openstack-ironic | 06:46 | |
| *** dyasny has joined #openstack-ironic | 06:57 | |
| *** fxpester has joined #openstack-ironic | 07:00 | |
| *** fxpester has quit IRC | 07:00 | |
| *** karthiks has quit IRC | 07:05 | |
| *** ijw has joined #openstack-ironic | 07:07 | |
| *** rcernin has joined #openstack-ironic | 07:11 | |
| *** ijw has quit IRC | 07:12 | |
| *** e0ne has joined #openstack-ironic | 07:12 | |
| *** ChubYann has quit IRC | 07:15 | |
| *** sacharya has joined #openstack-ironic | 07:15 | |
| *** karthiks has joined #openstack-ironic | 07:18 | |
| *** yuanying_ has joined #openstack-ironic | 07:25 | |
| *** Nisha_ has joined #openstack-ironic | 07:25 | |
| *** Nisha_Agarwal has quit IRC | 07:26 | |
| *** yuanying has quit IRC | 07:27 | |
| *** makowals_ has quit IRC | 07:27 | |
| *** rbartal has joined #openstack-ironic | 07:28 | |
| *** makowals has joined #openstack-ironic | 07:28 | |
| openstackgerrit | Jun Yao proposed openstack/ironic-inspector master: Adds node state to the API response data https://review.openstack.org/436415 | 07:32 |
| *** vsaienk0 has joined #openstack-ironic | 07:32 | |
| *** tesseract has joined #openstack-ironic | 07:34 | |
| *** e0ne has quit IRC | 07:34 | |
| *** parlos has joined #openstack-ironic | 07:34 | |
| *** crushil has quit IRC | 07:36 | |
| Nisha_ | Hello Ironic | 07:52 |
| *** sdake_ is now known as sdake | 07:56 | |
| *** vsaienk0 has quit IRC | 08:02 | |
| *** ccamacho has joined #openstack-ironic | 08:07 | |
| *** stendulker has joined #openstack-ironic | 08:09 | |
| Nisha_ | furb, https://docs.openstack.org/developer/ironic/deploy/multitenancy.html#multitenancy | 08:10 |
| *** pcaruana has joined #openstack-ironic | 08:17 | |
| *** athomas has joined #openstack-ironic | 08:21 | |
| pas-ha | morning ironic | 08:21 |
| furb | thanks nisha | 08:26 |
| *** moshele has joined #openstack-ironic | 08:29 | |
| *** amarao has joined #openstack-ironic | 08:29 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests https://review.openstack.org/423556 | 08:35 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add standalone tests using direct HTTP links https://review.openstack.org/437549 | 08:35 |
| *** priteau has joined #openstack-ironic | 08:44 | |
| *** racedo has joined #openstack-ironic | 08:45 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-specs master: Out-of-band hardware inspection support for Redfish driver https://review.openstack.org/438859 | 08:53 |
| *** yuanying_ has quit IRC | 08:53 | |
| *** parlos has quit IRC | 08:59 | |
| *** vsaienk0 has joined #openstack-ironic | 08:59 | |
| *** zzzeek has quit IRC | 09:00 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-specs master: Out-of-band hardware inspection support for Redfish driver https://review.openstack.org/438859 | 09:00 |
| *** jmccarthy has joined #openstack-ironic | 09:00 | |
| furb | nisha_ : being able to put a node's interface in different tenant networks sounds good and a major problem seem to be solvable therewith | 09:01 |
| *** aarefiev_afk is now known as aarefiev | 09:02 | |
| aarefiev | morning all! | 09:02 |
| furb | using pxe for deployment sounds fine as well, what i don't get are devices with IPMI, I do not even know all hardware we may need | 09:03 |
| furb | what I get are pdus to switch on/off nodes | 09:04 |
| jmccarthy | Hi all ! Gotta question, I have my a proxy I must use in my build environment, for me I had to add -E to: | 09:04 |
| jmccarthy | INSTALLER_CMD="sudo -H ${PKG_MANAGER} -y install" | 09:04 |
| jmccarthy | I see it is that way elsewhere in the script, is this a known issue ? | 09:04 |
| jmccarthy | --> INSTALLER_CMD="sudo -H -E ${PKG_MANAGER} -y install" | 09:04 |
| jmccarthy | I meant to build bifrost | 09:05 |
| vsaienk0 | morning Ironic! | 09:09 |
| *** mkrai1 has quit IRC | 09:09 | |
| *** retr0h has quit IRC | 09:09 | |
| *** csmart has quit IRC | 09:09 | |
| jmccarthy | This script: bifrost/scripts/install-deps.sh | 09:10 |
| openstackgerrit | yuhui proposed openstack/ironic master: Correct typos in doc_files https://review.openstack.org/437776 | 09:10 |
| *** zzzeek has joined #openstack-ironic | 09:12 | |
| *** sdake has quit IRC | 09:13 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Update HTTP API docs with missing 1.9 API microversion https://review.openstack.org/438870 | 09:15 |
| jmccarthy | Ok, I'll do up a review for it :) | 09:15 |
| *** udesale has joined #openstack-ironic | 09:17 | |
| *** SerenaFeng has quit IRC | 09:19 | |
| Nisha_ | vsaienk0, morning | 09:21 |
| Nisha_ | furb, what is pdus? | 09:21 |
| furb | https://en.wikipedia.org/wiki/Power_distribution_unit | 09:24 |
| furb | I do not know another term | 09:24 |
| *** tiendc has quit IRC | 09:25 | |
| *** SerenaFeng has joined #openstack-ironic | 09:25 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 09:29 | |
| *** Nisha_ has quit IRC | 09:29 | |
| *** Nisha_ has joined #openstack-ironic | 09:34 | |
| *** Nisha_Agarwal has quit IRC | 09:34 | |
| *** e0ne has joined #openstack-ironic | 09:35 | |
| *** derekh has joined #openstack-ironic | 09:37 | |
| furb | to summarize what we want to do... we want to use openstack to run a test-cluster. what we aim to are isolated tenant environments where users can run there tests | 09:39 |
| furb | since we have to test network appliances (virtual and in hardware)... our main interest is to have tenant networking and integration of a multitude of hardware nodes into the environment | 09:39 |
| *** jaosorior has quit IRC | 09:39 | |
| *** jaosorior has joined #openstack-ironic | 09:40 | |
| *** mgoddard has joined #openstack-ironic | 09:50 | |
| *** lucas-afk is now known as lucasagomes | 09:50 | |
| lucasagomes | morning all | 09:50 |
| openstackgerrit | James McCarthy proposed openstack/bifrost master: Add '-E' options to pass proxy env for sudo https://review.openstack.org/438884 | 09:52 |
| *** e0ne has quit IRC | 09:53 | |
| *** e0ne has joined #openstack-ironic | 09:54 | |
| *** MattMan has quit IRC | 09:58 | |
| *** MattMan has joined #openstack-ironic | 09:59 | |
| *** gcb has quit IRC | 10:00 | |
| *** sdake has joined #openstack-ironic | 10:05 | |
| *** pmannidi has quit IRC | 10:07 | |
| *** SerenaFeng has quit IRC | 10:08 | |
| *** hoangcx has quit IRC | 10:10 | |
| openstackgerrit | James McCarthy proposed openstack/bifrost master: Add '-E' options to pass proxy env for sudo https://review.openstack.org/438884 | 10:10 |
| *** Nisha_Agarwal has joined #openstack-ironic | 10:14 | |
| *** Nisha_ has quit IRC | 10:14 | |
| sambetts_ | furb: there is a SNMP driver | 10:17 |
| *** sambetts_ is now known as sambetts | 10:17 | |
| *** Nisha_Agarwal has quit IRC | 10:20 | |
| *** zhangjl has quit IRC | 10:20 | |
| *** SerenaFeng has joined #openstack-ironic | 10:22 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 10:23 | |
| Nisha_Agarwal | hey lucasagomes....morning :) | 10:23 |
| *** SerenaFeng has quit IRC | 10:24 | |
| lucasagomes | o/ | 10:25 |
| *** Nisha_ has joined #openstack-ironic | 10:26 | |
| vsaienk0 | Once node driver is set to one of dynamic drivers there is no way to change it back to classic is it expected behavior? | 10:27 |
| openstackgerrit | Merged openstack/ironic master: Fix BaseBaremetalTest._assertExpected docstring https://review.openstack.org/438569 | 10:28 |
| *** sacharya has quit IRC | 10:29 | |
| *** Nisha_Agarwal has quit IRC | 10:29 | |
| *** mkrai has joined #openstack-ironic | 10:30 | |
| openstackgerrit | zack chen proposed openstack/ironic-python-agent master: Collected logs is 'bytes' type. https://review.openstack.org/438901 | 10:31 |
| *** SerenaFeng has joined #openstack-ironic | 10:33 | |
| openstackgerrit | M V P Nitesh proposed openstack/ironic master: ipmi_address is validated https://review.openstack.org/436945 | 10:35 |
| *** yuanying has joined #openstack-ironic | 10:40 | |
| *** phuongnh has quit IRC | 10:41 | |
| pas-ha | furb, there also was an iBoot power driver in ironic, was removed in Ocata, you can try that out (YMMV as it was and is officially unsupported) if your PDUs do not support SNMP | 10:41 |
| pas-ha | furb, also you might be interested in MoltenIron https://github.com/openstack/molteniron | 10:42 |
| *** sdake has quit IRC | 10:42 | |
| *** udesale has quit IRC | 10:43 | |
| *** ijw has joined #openstack-ironic | 10:47 | |
| *** ijw has quit IRC | 10:47 | |
| *** dtantsur|afk is now known as dtantsur | 10:47 | |
| dtantsur | morning ironic | 10:47 |
| *** SerenaFeng has quit IRC | 10:55 | |
| *** SerenaFeng has joined #openstack-ironic | 10:55 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Add fsm dot diagram generator https://review.openstack.org/437037 | 10:57 |
| vsaienk0 | morning dtantsur | 11:03 |
| vsaienk0 | dtantsur: Once node driver is set to one of dynamic drivers there is no way to change it back to classic is it expected behavior? | 11:03 |
| dtantsur | vsaienk0, no, I don't think so. what is the error? | 11:04 |
| vsaienk0 | dtantsur: http://logs.openstack.org/49/437549/9/experimental/gate-ironic-dsvm-standalone-ubuntu-xenial-nv/be1645e/console.html#_2017-02-25_02_22_52_012482 | 11:04 |
| dtantsur | huh, interesting.. maybe we should check that the fields do not change.. or set them to null automatically | 11:05 |
| dtantsur | vsaienk0, mind filing a bug please? | 11:05 |
| vsaienk0 | dtantsur: Ok will do | 11:05 |
| dtantsur | thnx | 11:06 |
| vdrok | late morning everyone | 11:13 |
| pas-ha | morning vdrok dtantsur | 11:15 |
| vsaienk0 | dtantsur: so we need to add here a check to verify that update change only driver without other interfaces? https://github.com/openstack/ironic/blob/350dec88f765ffb12c46e34dafc07088b5418442/ironic/common/driver_factory.py#L267 | 11:16 |
| dtantsur | vsaienk0, yeah, we need to check that these fields are not changed AND reset them to None for classic drivers | 11:17 |
| vsaienk0 | dtantsur: so what if interface field was changed to not default value should we override it to None also or fail? | 11:18 |
| *** stendulker has quit IRC | 11:18 | |
| vsaienk0 | I mean if user set driver to dynamic, changed some interface to non-default value, and then trying to update driver to classic | 11:18 |
| *** mvpnitesh has joined #openstack-ironic | 11:19 | |
| dtantsur | vsaienk0, if only the driver is updated - everything should go to None. if a user tries to update any interfaces as well - this should result in a failure | 11:19 |
| *** jaganathan is now known as jaganathan_afk | 11:19 | |
| vsaienk0 | ok will prepare a fix | 11:20 |
| furb | thanks pas-ha | 11:20 |
| dtantsur | vsaienk0, cool, thanks | 11:22 |
| *** mvpnitesh has quit IRC | 11:24 | |
| vdrok | ugh, gerrit issues again | 11:25 |
| *** mvpnitesh has joined #openstack-ironic | 11:26 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Refactor waiters in our tempest plugin https://review.openstack.org/410684 | 11:27 |
| *** retr0h has joined #openstack-ironic | 11:28 | |
| *** csmart has joined #openstack-ironic | 11:28 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Fix valid provision states for introspection https://review.openstack.org/438651 | 11:28 |
| *** ntpttr has quit IRC | 11:30 | |
| Nisha_ | dtantsur, lucasagomes wanted to ask on https://github.com/openstack/sushy | 11:32 |
| *** SerenaFeng has quit IRC | 11:32 | |
| lucasagomes | Nisha_, hi there | 11:32 |
| Nisha_ | lucasagomes, hi | 11:32 |
| *** SerenaFeng has joined #openstack-ironic | 11:32 | |
| Nisha_ | lucasagomes, i was thinking to add the redfish inspect support to the sushy library, | 11:33 |
| lucasagomes | Nisha_, that would be great! It's something we need to do, I'm not sure if dtantsur is working on it already or not | 11:34 |
| dtantsur | no, I did not have time for it | 11:35 |
| Nisha_ | dtantsur, lucasagomes great | 11:38 |
| *** ntpttr has joined #openstack-ironic | 11:39 | |
| Nisha_ | dtantsur, lucasagomes i wanted to know how we want to add the support back in ironic for RedfishInspect Interface....could it be something like we have IloInspect today? i.e. sushy returns the essential properties and supported capabilities to ironic in form of dictionary.(similar to IloInspect implementation) | 11:40 |
| *** nmathew has quit IRC | 11:41 | |
| *** jkilpatr has quit IRC | 11:41 | |
| Nisha_ | dtantsur, lucasagomes, Or we want to create these dictionaries in ironic and call each rest end point for each property in sushy from ironic | 11:41 |
| Nisha_ | dtantsur, lucasagomes FYI, i filed the RFE for the same today | 11:42 |
| Nisha_ | https://bugs.launchpad.net/ironic/+bug/1668487 | 11:42 |
| openstack | Launchpad bug 1668487 in Ironic "[RFE] Add Redfish Inspect Interface to Ironic" [Undecided,In progress] - Assigned to Shivanand Tendulker (stendulker) | 11:42 |
| *** jaganathan_afk is now known as jaganathan | 11:43 | |
| lucasagomes | Nisha_, the 2nd, I want to keep sushy conservative when sending requests to the BMC. For example, some things like total memory you already have at the system resource representation, but others like etherned interfaces you need extra requests to get. The way I see it is the System object in sushy having a method to fetch those | 11:44 |
| TheJulia | Good morning | 11:44 |
| lucasagomes | System().get_ethernet_interfaces() ... or something like that | 11:44 |
| Nisha_ | lucasagomes, ok... | 11:45 |
| dtantsur | morning TheJulia | 11:45 |
| Nisha_ | lucasagomes, so we want ironic to call each of these end points? | 11:46 |
| lucasagomes | Nisha_, yeah | 11:46 |
| Nisha_ | lucasagomes, ok. | 11:46 |
| *** karthiks has quit IRC | 11:47 | |
| Nisha_ | lucasagomes, who are the core members of sushy? | 11:47 |
| lucasagomes | I know it sounds like more work, but I really prefer sushy to be cautions when talking to BMCs, like a building block. And the software around it can then wrap it all up to make it easier | 11:47 |
| lucasagomes | Nisha_, only me at the moment | 11:47 |
| lucasagomes | the library is 1 week old +- | 11:47 |
| Nisha_ | lucasagomes, :) | 11:48 |
| lucasagomes | so it's not "ready" yet | 11:48 |
| Nisha_ | lucasagomes, yeah i saw | 11:48 |
| Nisha_ | lucasagomes, we would want to contribute to library in general also | 11:48 |
| lucasagomes | Nisha_, that's great! So far it was well accepted at the PTG I talked to some folks from other companies that are also interested | 11:49 |
| lucasagomes | well received* | 11:49 |
| Nisha_ | yes...redfish is new specification every vendor looking at | 11:49 |
| lucasagomes | Nisha_, we just need to pay attention to the scope of it. For now we are limiting sushy to what ironic supports | 11:50 |
| lucasagomes | so it's not a general redfish library | 11:50 |
| Nisha_ | lucasagomes, yes we also want to do only what ironic needs | 11:50 |
| Nisha_ | :) | 11:50 |
| lucasagomes | good stuff then :D | 11:50 |
| Nisha_ | lucasagomes, :D | 11:50 |
| Nisha_ | lucasagomes, i hope to post the patch for sushy in a day or two for this | 11:51 |
| lucasagomes | Nisha_, cool | 11:51 |
| lucasagomes | yesterday I merged a patch from dtantsur that allows you to start the libvirt simulator with a single tox command | 11:51 |
| lucasagomes | so it's quite easy to develop it :-) | 11:51 |
| Nisha_ | :) | 11:52 |
| *** jmccarthy has quit IRC | 11:52 | |
| lucasagomes | see http://sushy.readthedocs.io/en/latest/usage.html#libvirt-mockup | 11:52 |
| *** betherly has quit IRC | 11:53 | |
| *** athomas has quit IRC | 11:53 | |
| Nisha_ | lucasagomes, so for redfish support in ironic general power, deploy/boot interfaces are in progress? | 11:54 |
| *** betherly has joined #openstack-ironic | 11:55 | |
| Nisha_ | lucasagomes, or its yet to be started? | 11:55 |
| lucasagomes | Nisha_, we still need the spec merged, but I will put a patch with a PoC for ironic soon | 11:55 |
| lucasagomes | shouldn't take long | 11:55 |
| * lucasagomes has it own his list | 11:55 | |
| Nisha_ | lucasagomes, thats why asked | 11:55 |
| Nisha_ | lucasagomes, if you are fine i can also take up that :) | 11:55 |
| Nisha_ | :D | 11:55 |
| lucasagomes | up to you, but it kinda helps me designing the sushy interface | 11:56 |
| lucasagomes | so I will try to do it anyway | 11:56 |
| Nisha_ | lucasagomes, no prob :) i just proposed :) | 11:56 |
| lucasagomes | thanks :-) | 11:56 |
| Nisha_ | dtantsur, one question i had on this...do we need spec for the redfish Inspect Interface? | 11:58 |
| Nisha_ | stendulker has already proposed one but i thought it might not be required | 11:59 |
| Nisha_ | lucasagomes, ^^^ | 11:59 |
| *** athomas has joined #openstack-ironic | 12:00 | |
| *** sergek_ has quit IRC | 12:01 | |
| *** ildikov has quit IRC | 12:01 | |
| *** sergek_ has joined #openstack-ironic | 12:02 | |
| *** cppforlife_ has quit IRC | 12:02 | |
| *** ildikov has joined #openstack-ironic | 12:03 | |
| *** karthiks has joined #openstack-ironic | 12:04 | |
| *** johnthetubaguy has quit IRC | 12:04 | |
| dtantsur | Nisha_, I think after the main redfish spec is approved, only an RFE might be needed | 12:04 |
| *** deray has joined #openstack-ironic | 12:07 | |
| *** sergek_ has quit IRC | 12:11 | |
| *** fungi has quit IRC | 12:11 | |
| *** jmccarthy has joined #openstack-ironic | 12:11 | |
| *** tuanluong has quit IRC | 12:12 | |
| *** aparnav has quit IRC | 12:12 | |
| *** aparnav has joined #openstack-ironic | 12:12 | |
| *** sergek_ has joined #openstack-ironic | 12:12 | |
| deray | g'morning, ironicers | 12:12 |
| *** jkilpatr has joined #openstack-ironic | 12:13 | |
| deray | lucasagomes, hi | 12:14 |
| lucasagomes | deray, hi there | 12:14 |
| *** johnthetubaguy has joined #openstack-ironic | 12:14 | |
| lucasagomes | deray, btw, I didn't have time to see your patch yet, but it's on my list | 12:14 |
| *** yhvh has joined #openstack-ironic | 12:14 | |
| deray | lucasagomes, cool .. https://review.openstack.org/#/c/438929/ | 12:15 |
| patchbot_ | patch 438929 - sushy - Refactor resource collection | 12:15 |
| lucasagomes | deray, I just wonder whether we really need chassis and managers now (see the talk about about the scope) | 12:15 |
| *** Nisha_ has quit IRC | 12:15 | |
| deray | lucasagomes, I have -1ed that one .. https://review.openstack.org/#/c/438627 | 12:15 |
| patchbot_ | patch 438627 - sushy - Adding backbone structure for Chassis, Managers | 12:15 |
| *** Nisha_ has joined #openstack-ironic | 12:15 | |
| lucasagomes | deray, a-ha gotcha! | 12:16 |
| yhvh | hey, will ironic discard project/user when creating ports in neutron? Or am I missing some config? | 12:16 |
| deray | lucasagomes, **see the talk about about the scope** - where? | 12:16 |
| lucasagomes | <lucasagomes> Nisha_, we just need to pay attention to the scope of it. For now we are limiting sushy to what ironic supports | 12:16 |
| lucasagomes | <lucasagomes> so it's not a general redfish library | 12:16 |
| lucasagomes | deray, ^ | 12:16 |
| deray | lucasagomes, oh kay .. makes sense :) | 12:17 |
| lucasagomes | deray, it's in the docs as well, to make it more maintainable I wanted to limit it's scope so, whenever ironic adds more stuff we can go and extend the library too | 12:17 |
| deray | lucasagomes, that's fine | 12:17 |
| lucasagomes | otherwise it would be a big burden for us to have to keep up with the whole redfish specifications, it changes a lot from one version to another | 12:18 |
| *** fungi has joined #openstack-ironic | 12:18 | |
| lucasagomes | so we are less ambitious than other libraries for it :-) | 12:18 |
| deray | lucasagomes, agreed | 12:18 |
| deray | lucasagomes, and fine tuned (for ironic) as well | 12:18 |
| lucasagomes | yeah | 12:18 |
| deray | btw, the new patch I have prosed is regarding refactoring the resource collection class (trick: ``resourceType`` attribute) | 12:19 |
| deray | https://review.openstack.org/#/c/438929/ | 12:20 |
| patchbot_ | patch 438929 - sushy - Refactor resource collection | 12:20 |
| deray | lucasagomes, ^^^ | 12:20 |
| *** SerenaFeng has quit IRC | 12:20 | |
| lucasagomes | cool I will take a look soon-ish! | 12:20 |
| lucasagomes | deray, and thanks for collaborating | 12:20 |
| deray | ah!! .. I see it in merge-conflict | 12:20 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: Adds wsgi to ironic-api in devstack https://review.openstack.org/430851 | 12:20 |
| deray | lucasagomes, my pleasure | 12:21 |
| TheJulia | yhvh: I believe it now does, previously there was a bug that doing so would encounter since normal users are not allowed to perform certain actions/updates against the neutron api | 12:21 |
| deray | wait I will rebase and post new patch set soon | 12:21 |
| lucasagomes | right on | 12:21 |
| yhvh | TheJulia: Thanks, I will investigate further | 12:22 |
| TheJulia | If anyone is interested in time windows for an ironic-ui irc meeting, the doodle to add yourself to is at -> http://doodle.com/poll/edbuhpr8nvs8n5u9 | 12:24 |
| Nisha_ | dtantsur, thanx | 12:26 |
| *** ijw has joined #openstack-ironic | 12:31 | |
| *** ccamacho has quit IRC | 12:34 | |
| *** amotoki has joined #openstack-ironic | 12:35 | |
| *** ijw has quit IRC | 12:36 | |
| bradjones | Thanks TheJulia | 12:39 |
| *** bfournie has quit IRC | 12:39 | |
| *** csmart has quit IRC | 12:39 | |
| *** csmart has joined #openstack-ironic | 12:40 | |
| deray | lucasagomes, done uploading.. | 12:45 |
| TheJulia | yhvh: Quick question for our clarification, what version of ironic are you running? | 12:46 |
| deray | as a matter of fact ``test_base`` class, which I also proposed, was also added as part of the recent commit | 12:46 |
| *** ccamacho has joined #openstack-ironic | 12:50 | |
| *** vikrant has quit IRC | 12:51 | |
| *** retr0h has quit IRC | 12:53 | |
| deray | lucasagomes, another one related to coverage: https://review.openstack.org/#/c/438806/ | 12:53 |
| patchbot_ | patch 438806 - sushy - Refining test coverage | 12:53 |
| lucasagomes | deray, a-ha nice | 12:54 |
| deray | lucasagomes, :-) | 12:55 |
| *** retr0h has joined #openstack-ironic | 12:55 | |
| *** dprince has joined #openstack-ironic | 13:02 | |
| deray | lucasagomes, coverage results (on the branch of refactor patch): http://paste.openstack.org/show/600766/ | 13:02 |
| deray | lucasagomes, 95% | 13:03 |
| lucasagomes | yeah we should try to keep it > 90% | 13:04 |
| lucasagomes | I want to have a job in the gate to check that | 13:04 |
| * lucasagomes adds to the list | 13:04 | |
| deray | lucasagomes, k | 13:05 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: WIP/DNM: Testing multi-tenant multi-node grenade job https://review.openstack.org/436957 | 13:06 |
| -openstackstatus- NOTICE: restarting gerrit to address performance problems | 13:06 | |
| *** ChanServ changes topic to "restarting gerrit to address performance problems" | 13:06 | |
| *** hshiina has quit IRC | 13:15 | |
| openstackgerrit | M V P Nitesh proposed openstack/ironic master: ipmi_address is validated https://review.openstack.org/436945 | 13:15 |
| *** bfournie has joined #openstack-ironic | 13:16 | |
| jroll | morning y'all | 13:17 |
| openstackgerrit | Merged openstack/ironic master: Add testcases for iLO drivers https://review.openstack.org/436785 | 13:18 |
| TheJulia | good morning jroll | 13:19 |
| *** trown|outtypewww is now known as trown | 13:20 | |
| *** lucasagomes is now known as lucas-hungry | 13:21 | |
| lucas-hungry | jroll, TheJulia morning | 13:21 |
| *** Nisha_Agarwal has joined #openstack-ironic | 13:22 | |
| deray | jroll, TheJulia g'morning | 13:22 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Add tests for node resource class https://review.openstack.org/437963 | 13:23 |
| *** Nisha_brb has joined #openstack-ironic | 13:24 | |
| *** Nisha_ has quit IRC | 13:24 | |
| *** Nisha_Agarwal has quit IRC | 13:28 | |
| *** awiddersheim has joined #openstack-ironic | 13:29 | |
| *** Nisha_brb has quit IRC | 13:29 | |
| *** deray has quit IRC | 13:32 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 13:32 | |
| *** amoralej is now known as amoralej|lunch | 13:32 | |
| *** vmud213 has quit IRC | 13:33 | |
| -openstackstatus- NOTICE: ok gerrit is back to normal | 13:35 | |
| *** ChanServ changes topic to "ok gerrit is back to normal" | 13:35 | |
| *** mkrai has quit IRC | 13:35 | |
| vdrok | morning jroll TheJulia and derekh | 13:36 |
| *** aparna has joined #openstack-ironic | 13:37 | |
| *** aparnav has quit IRC | 13:39 | |
| *** mkrai has joined #openstack-ironic | 13:40 | |
| *** ChanServ changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic" | 13:42 | |
| -openstackstatus- NOTICE: gerrit is back to normal and I don't know how to use the openstackstaus bot | 13:42 | |
| *** hoangcx has joined #openstack-ironic | 13:46 | |
| *** mjturek has joined #openstack-ironic | 13:51 | |
| *** zhurong has joined #openstack-ironic | 13:52 | |
| *** baoli has joined #openstack-ironic | 13:52 | |
| *** aparna has quit IRC | 13:53 | |
| *** baoli_ has joined #openstack-ironic | 13:54 | |
| *** goldyfruit has joined #openstack-ironic | 13:56 | |
| *** vsaienko has joined #openstack-ironic | 13:56 | |
| *** sasha2 has joined #openstack-ironic | 13:57 | |
| *** baoli has quit IRC | 13:58 | |
| *** retr0h has quit IRC | 14:00 | |
| *** jjohnson2 has joined #openstack-ironic | 14:00 | |
| *** retr0h has joined #openstack-ironic | 14:01 | |
| TheJulia | heh | 14:01 |
| *** openstackgerrit has quit IRC | 14:03 | |
| *** openstackgerrit has joined #openstack-ironic | 14:07 | |
| openstackgerrit | M V P Nitesh proposed openstack/ironic master: Now displays maintenance and maintenance_reason https://review.openstack.org/433029 | 14:07 |
| *** cppforlife_ has joined #openstack-ironic | 14:09 | |
| *** fyxim has quit IRC | 14:09 | |
| *** fyxim has joined #openstack-ironic | 14:12 | |
| *** Nisha_ has joined #openstack-ironic | 14:13 | |
| *** rloo has joined #openstack-ironic | 14:14 | |
| *** lucas-hungry is now known as lucasagomes | 14:15 | |
| *** jaypipes has joined #openstack-ironic | 14:16 | |
| *** jaganathan is now known as jaganathan_afk | 14:16 | |
| *** Nisha_Agarwal has quit IRC | 14:16 | |
| *** crushil has joined #openstack-ironic | 14:18 | |
| *** milan has joined #openstack-ironic | 14:20 | |
| *** fxpester has joined #openstack-ironic | 14:22 | |
| *** goldyfruit has quit IRC | 14:24 | |
| *** sdake has joined #openstack-ironic | 14:25 | |
| *** goldyfruit has joined #openstack-ironic | 14:25 | |
| *** sacharya has joined #openstack-ironic | 14:26 | |
| *** sacharya has quit IRC | 14:30 | |
| *** chlong has joined #openstack-ironic | 14:31 | |
| *** mvpnitesh has quit IRC | 14:33 | |
| *** amoralej|lunch is now known as amoralej | 14:38 | |
| *** vsaienko has quit IRC | 14:38 | |
| lucasagomes | first node (VM) deployed with redfish :-) | 0c0b44fb-4c8b-4001-9e9e-be0947cd4d7f | node-0 | 21e76ce2-a2e9-4606-ab09-2ba7c485eaf0 | power on | active | False | | 14:41 |
| sambetts | lucasagomes: :O nice!!! | 14:42 |
| * lucasagomes will submit the patch in a few | 14:42 | |
| jroll | niiiiice | 14:42 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic master: Add redfish driver https://review.openstack.org/438982 | 14:47 |
| lucasagomes | sambetts, jroll ^ | 14:48 |
| * lucasagomes needs to add unittests | 14:48 | |
| pas-ha | TheJulia: mind taking a look at https://review.openstack.org/#/c/438076/ ? seems to solve/work around libvirt networking glitches in bifrost | 14:48 |
| patchbot_ | patch 438076 - bifrost - Workaround for network start race condition | 14:48 |
| pas-ha | and good morning :) | 14:48 |
| jroll | lucasagomes: sweet | 14:48 |
| TheJulia | pas-ha: sure! I'm just going to make a latte first and then I'll look at it | 14:48 |
| pas-ha | thanks, and bon appetit | 14:49 |
| *** vsaienko has joined #openstack-ironic | 14:49 | |
| pas-ha | :) | 14:49 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Fix missing _ import in driver_factory https://review.openstack.org/438986 | 14:49 |
| *** vsaienko has quit IRC | 14:50 | |
| *** dyasny has quit IRC | 14:53 | |
| *** dyasny has joined #openstack-ironic | 14:53 | |
| mariojv | good morning ironic | 14:54 |
| mariojv | nice job lucasagomes :D | 14:54 |
| *** retr0h has quit IRC | 14:54 | |
| *** retr0h has joined #openstack-ironic | 14:56 | |
| *** fxpester has quit IRC | 14:57 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 14:57 | |
| *** fxpester has joined #openstack-ironic | 14:57 | |
| lucasagomes | mariojv, morning! Cheers man | 15:00 |
| *** mgould|afk is now known as mgould | 15:00 | |
| mgould | lucasagomes: wow, that was fast | 15:00 |
| *** Nisha_ has quit IRC | 15:00 | |
| lucasagomes | mgould, yeah, it's kinda fun to code too | 15:00 |
| des_consolado | hey I used ironic to provision a host and then I destroyed the openstack environment and then when rebooting the host I provisioned it cannot boot from disk | 15:03 |
| des_consolado | like it's just a blank black prompt, it doesn't even error/fail or try the next boot option in the boot sequence | 15:03 |
| des_consolado | does ironic not format/copy the disks so that hosts can be rebooted independently? | 15:04 |
| TheJulia | pas-ha: If we never see the ipv4 error again, your officially my hero. :) | 15:06 |
| TheJulia | des_consolado: sounds like your nodes are deployed using netboot with partition images | 15:07 |
| TheJulia | des_consolado: the default, is, or was until very recently netboot. You have to explicitly set it to local boot or use a whole disk image | 15:08 |
| des_consolado | TheJulia: is there a diskimage-builder element for that? | 15:10 |
| *** zhurong has quit IRC | 15:10 | |
| TheJulia | des_consolado: the horribly named vm element | 15:10 |
| *** chlong has quit IRC | 15:11 | |
| des_consolado | ty :)) | 15:12 |
| *** andyjones2001 has joined #openstack-ironic | 15:12 | |
| *** hoangcx has quit IRC | 15:13 | |
| openstackgerrit | Merged openstack/bifrost master: Workaround for network start race condition https://review.openstack.org/438076 | 15:15 |
| pas-ha | TheJulia: not me, it was Mark Goddard :) | 15:16 |
| TheJulia | mgoddard: Your my hero! | 15:16 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Add functional api tests for node resource class https://review.openstack.org/437963 | 15:17 |
| mgoddard | TheJulia: hooray! | 15:17 |
| mgoddard | TheJulia: nothing worse than CI flakiness. wastes peoples' time and reduces confidence in the testing infrastructure | 15:19 |
| TheJulia | Oh agreed, I think we've thought we worked around it a few times now, but... ugh | 15:20 |
| mgoddard | TheJulia: I'm sure it won't be the last though :) | 15:20 |
| TheJulia | nope | 15:20 |
| mgoddard | only time will tell how this bit of sticky tape holds up | 15:20 |
| openstackgerrit | Merged openstack/ironic-ui master: Source code documentation updates https://review.openstack.org/429916 | 15:23 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Add functional tests for VIFs with ironicclient commands https://review.openstack.org/430881 | 15:24 |
| *** trown is now known as trown|brb | 15:26 | |
| *** sacharya has joined #openstack-ironic | 15:27 | |
| NobodyCam | Good Morning Ironic'ers :) it's Coffee Time! | 15:27 |
| *** sacharya has quit IRC | 15:27 | |
| *** sacharya has joined #openstack-ironic | 15:28 | |
| *** trown|brb is now known as trown | 15:28 | |
| *** karthiks has quit IRC | 15:31 | |
| *** jcoufal has joined #openstack-ironic | 15:31 | |
| *** mtanino has joined #openstack-ironic | 15:33 | |
| rloo | hi and morning NobodyCam, mgoddard, TheJulia, pas-ha, des_consolado, lucasagomes, mgould, mariojv, jroll and everyone else! | 15:34 |
| openstackgerrit | Trung Nguyen proposed openstack/python-ironicclient master: Should upgrade tox version up to 1.9 to test configuration https://review.openstack.org/439022 | 15:34 |
| mgoddard | morning rloo | 15:35 |
| NobodyCam | Good Morning rloo :) | 15:35 |
| rloo | lucasagomes: great stuff wrt redfish driver. just wondering, is it clear upstream, that you are working on it? (don't want to duplicate effort by others that might be doing that.) | 15:35 |
| rloo | jroll and others: did we actually clarify how 'strict' we want to review release notes? eg: https://review.openstack.org/#/c/394413/ | 15:36 |
| patchbot_ | patch 394413 - ironic - Add support for DBDeadlock handling | 15:36 |
| rloo | ^^ Can we use British spelling? | 15:36 |
| lucasagomes | rloo, morning! Not sure if it's clear, we commented on the redfish spec but we didn't get a reply yet (I'm waiting on it) | 15:36 |
| NobodyCam | morning lucasagomes :) | 15:37 |
| dtantsur | morning rloo, mgoddard, NobodyCam | 15:37 |
| NobodyCam | morning dtantsur :) | 15:37 |
| rloo | hiya dtantsur! | 15:37 |
| mgoddard | morning dtantsur | 15:37 |
| jroll | rloo: this is what we have for "how we do release notes" so far: https://docs.openstack.org/developer/ironic/dev/faq.html#know-if-a-release-note-is-needed-for-my-change | 15:37 |
| rloo | lucasagomes: ok, did we make it clear in the redfish spec that the python-redfish package was ... undesirable? | 15:38 |
| rloo | jroll: ok, nothing about british spelling so i assume that we're good with that. | 15:38 |
| *** Syed__ has joined #openstack-ironic | 15:38 | |
| lucasagomes | rloo, we talked on the PTG and dtantsur/jroll have left a comment in the current spec | 15:38 |
| jroll | rloo: my primary goal is that: 1) most technical people can understand them, and 2) they look well-written | 15:38 |
| rloo | lucasagomes: ok, that should be good enough. | 15:39 |
| jroll | rloo: so I don't really care about english vs british, personally, not sure if anyone else does | 15:39 |
| rloo | jroll: i don't care either, just wanted to get clarification cuz of jlvillal's comment. | 15:39 |
| rloo | jroll: thx :) | 15:39 |
| jroll | sure | 15:39 |
| dtantsur | rloo, I'm fine if we don't allow Russian spelling, variants of English are fine ;) | 15:40 |
| rloo | dtantsur: ha ha, ++ | 15:40 |
| * dtantsur convinces himself to get back to writing a PTG summary | 15:41 | |
| *** retr0h has quit IRC | 15:41 | |
| jroll | dtantsur: we should allow "ок" though :D | 15:42 |
| rloo | dtantsur: if it makes it easier, you could split it up into several emails. | 15:42 |
| dtantsur | rloo, I'm going to end up with a blog post (copy-pasted into a email), but good point about splitting it | 15:42 |
| *** chlong has joined #openstack-ironic | 15:43 | |
| rloo | dtantsur: ok, whatever works for you. and ++ to copy-pasting to email, that's what jroll did the first time I believe. | 15:43 |
| jroll | yeah, I can't draft long things in gmail's compose window :P | 15:43 |
| dtantsur | true :) | 15:43 |
| *** rbartal has quit IRC | 15:44 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Fix updating node.driver to classic https://review.openstack.org/439026 | 15:44 |
| *** karthiks has joined #openstack-ironic | 15:44 | |
| NobodyCam | Morning jroll :) | 15:45 |
| jroll | \o NobodyCam | 15:45 |
| NobodyCam | :) | 15:46 |
| openstackgerrit | Trung Nguyen proposed openstack/python-ironicclient master: add further information for tox.ini https://review.openstack.org/439029 | 15:47 |
| rloo | xek, vdrok, joanna: do you want to update this or do a followup to clean up the release notes? https://review.openstack.org/#/c/394413/ | 15:47 |
| patchbot_ | patch 394413 - ironic - Add support for DBDeadlock handling | 15:47 |
| *** fxpester has quit IRC | 15:48 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: Adds wsgi to ironic-api in devstack https://review.openstack.org/430851 | 15:49 |
| *** retr0h has joined #openstack-ironic | 15:49 | |
| joanna | morning, rloo! | 15:50 |
| TheJulia | good morning rloo, NobodyCam | 15:51 |
| joanna | rloo: I'm afraid of losing +2s :) | 15:51 |
| NobodyCam | Good Morning TheJulia :) | 15:52 |
| joanna | but if you think it's necessary - will do | 15:52 |
| NobodyCam | morning joanna :) | 15:52 |
| joanna | morning NobodyCam, TheJulia | 15:52 |
| rloo | joanna: no, it isn't necessary. that's why i gave you all a choice :) | 15:53 |
| *** sayantan_ has joined #openstack-ironic | 15:53 | |
| rloo | joanna: I'll +A and you can push up a followup patch | 15:53 |
| joanna | ok - will do! | 15:53 |
| rloo | joanna: done :) | 15:54 |
| *** ijw has joined #openstack-ironic | 15:54 | |
| joanna | awesome, thank yoou | 15:54 |
| joanna | s/yoou/you/, sorry :) | 15:54 |
| rloo | joanna: i see a comment from vasyl on that patch, so please address it in follow up too. thx. | 15:55 |
| joanna | ok, I will :) | 15:56 |
| *** rajinir_ has joined #openstack-ironic | 15:56 | |
| *** vsaienko has joined #openstack-ironic | 15:56 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: Adds wsgi to ironic-api in devstack https://review.openstack.org/430851 | 15:57 |
| dtantsur | jroll and others: part 1 of my notes is online, please take a quick look before I send it: http://dtantsur.github.io/posts/ironic-ptg-atlanta-2017-1.html | 15:57 |
| dtantsur | next parts will probably follow tomorrow, I'm tired of long English texts :) | 15:58 |
| openstackgerrit | Merged openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/433518 | 15:58 |
| openstackgerrit | Merged openstack/ironic-ui master: Maintenance processing code cleanup https://review.openstack.org/431573 | 15:58 |
| *** sdake has quit IRC | 15:58 | |
| openstackgerrit | Merged openstack/ironic-ui master: Add support for manual cleaning of nodes https://review.openstack.org/431642 | 15:58 |
| openstackgerrit | Merged openstack/ironic-ui master: Replace ironic with ironic-ui in contributing doc https://review.openstack.org/433401 | 15:59 |
| jroll | dtantsur: in meeting, will need some time | 15:59 |
| *** amarao has quit IRC | 15:59 | |
| dtantsur | sure, no hurry | 15:59 |
| vsaienk0 | dtanstur, jroll please add to your review queue https://review.openstack.org/#/c/438907 a patch to add networking-baremetal project | 16:02 |
| patchbot_ | patch 438907 - openstack-infra/project-config - Add networking-baremetal project | 16:02 |
| *** retr0h has quit IRC | 16:03 | |
| *** jaosorior has quit IRC | 16:03 | |
| *** aparnav has joined #openstack-ironic | 16:05 | |
| NobodyCam | oh wow, ^^ :) | 16:07 |
| *** vsaienko has quit IRC | 16:08 | |
| *** sdake has joined #openstack-ironic | 16:08 | |
| *** rcernin has quit IRC | 16:08 | |
| *** baha has joined #openstack-ironic | 16:09 | |
| *** rpioso has joined #openstack-ironic | 16:11 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Use longer names for screen- log files https://review.openstack.org/438082 | 16:16 |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic master: Fix directories permission for tftpboot https://review.openstack.org/427077 | 16:20 |
| *** retr0h has joined #openstack-ironic | 16:21 | |
| jroll | dtantsur: blog looks fine to me | 16:23 |
| dtantsur | thanks | 16:24 |
| jroll | np | 16:25 |
| *** dyasny has quit IRC | 16:25 | |
| vdrok | morning rloo NobodyCam and joanna | 16:26 |
| joanna | morning vdrok :) | 16:26 |
| NobodyCam | morning vdrok :) | 16:26 |
| rloo | hiya vdrok! | 16:26 |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Rework Ironic devstack baremetal network simulation https://review.openstack.org/392959 | 16:26 |
| *** stendulker has joined #openstack-ironic | 16:26 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: WIP/DNM: Testing multi-tenant multi-node grenade job https://review.openstack.org/436957 | 16:26 |
| *** baoli_ has quit IRC | 16:28 | |
| *** baoli has joined #openstack-ironic | 16:29 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Use longer names for screen- log files https://review.openstack.org/438082 | 16:31 |
| *** pcaruana has quit IRC | 16:34 | |
| *** e0ne has quit IRC | 16:36 | |
| *** baoli has quit IRC | 16:37 | |
| vsaienk0 | jlvillal: do you remember why you added this? https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L32 it is blocking update local.conf on subnode | 16:37 |
| jlvillal | vsaienk0: Yes I do. It is because otherwise we have the plugin getting enabled twice. | 16:38 |
| jlvillal | vsaienk0: How is it breaking the subnode? | 16:39 |
| vsaienk0 | jlvillal: at the moment on multinode grenade ironic is not enabled in local.conf on subnode | 16:39 |
| *** jaypipes has quit IRC | 16:40 | |
| jlvillal | vsaienk0: Oh, not sure how to fix that off the top of my head. | 16:41 |
| *** csmart has quit IRC | 16:42 | |
| *** cppforlife_ has quit IRC | 16:47 | |
| *** dyasny has joined #openstack-ironic | 16:48 | |
| *** csmart has joined #openstack-ironic | 16:51 | |
| vsaienk0 | jlvillal: me either, and it looks like a blocker for grenade mulitnode | 16:51 |
| sambetts | do other projects do multinode grenade? | 16:52 |
| dtantsur | jroll, did you have a change to release ironicclient stable/ocata after the freeze? | 16:52 |
| vsaienk0 | sambetts: not those who has a devstack plugin I think | 16:52 |
| jroll | dtantsur: it's on my list, I can probably do it today | 16:52 |
| jroll | also ironic-lib for pike | 16:52 |
| jlvillal | vsaienk0: Maybe we remove enabling the ironic plugin from grenade and add it back into the local.conf? | 16:53 |
| jroll | dtantsur: I have a change up, just needs rebase | 16:53 |
| jlvillal | vsaienk0: Sounds ugly though :( | 16:53 |
| dtantsur | k thanks | 16:53 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: doc: update FAQ for release notes https://review.openstack.org/439054 | 16:53 |
| rloo | jlvillal: ^^ as a followup to the question of british spelling in release notes | 16:53 |
| vsaienk0 | jlvillal I don't remember why it was done | 16:53 |
| *** igordcar1 has quit IRC | 16:54 | |
| *** ijw_ has joined #openstack-ironic | 16:54 | |
| *** igordcard has joined #openstack-ironic | 16:54 | |
| jlvillal | vsaienk0: Initially we added enable plugin to grenade as that is what the docs say we should do. | 16:54 |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Rework Ironic devstack baremetal network simulation https://review.openstack.org/392959 | 16:57 |
| *** ijw has quit IRC | 16:57 | |
| jlvillal | rloo: So does that mean it is perfectly acceptable to use. "Colours" in one sentence and then "colors" in the next sentence? | 16:57 |
| rloo | jlvillal: what do you think? if it is the same paragraph, i would say no. | 16:58 |
| vsaienk0 | jlvillal: it looks like we missed multinode grenade case | 16:58 |
| jlvillal | rloo: I would vote no also | 16:58 |
| rloo | jlvillal: to be consistent, if the word occurs in the release notes, it should be consistent throughout but i honestly don't expect everyone to compare their reno with the others. | 16:58 |
| jlvillal | rloo: True | 16:59 |
| jlvillal | vsaienk0: I'm not sure I understand. | 16:59 |
| jlvillal | vsaienk0: I have to start a meeting in 1 minute :( | 16:59 |
| *** mgoddard has quit IRC | 16:59 | |
| *** sayanta__ has joined #openstack-ironic | 17:00 | |
| *** sayantan_ has quit IRC | 17:00 | |
| *** sdake_ has joined #openstack-ironic | 17:03 | |
| *** crushil has quit IRC | 17:05 | |
| JayF | jroll: dtantsur: Just +2'd this, it needs another stable to land https://review.openstack.org/#/c/433974/ | 17:06 |
| patchbot_ | patch 433974 - ironic (stable/ocata) - Update documentation links to Ocata equivalents | 17:06 |
| JayF | jroll: are you still stable core even not as PTL anymore/ | 17:06 |
| *** ijw has joined #openstack-ironic | 17:06 | |
| dtantsur | good call ;) | 17:06 |
| *** gcb has joined #openstack-ironic | 17:06 | |
| *** sdake has quit IRC | 17:07 | |
| *** moshele has quit IRC | 17:09 | |
| *** tesseract has quit IRC | 17:09 | |
| *** ijw_ has quit IRC | 17:09 | |
| *** csmart has quit IRC | 17:11 | |
| *** mkrai has quit IRC | 17:11 | |
| *** retr0h has quit IRC | 17:11 | |
| openstackgerrit | Merged openstack/ironic master: Trivial: Change hardcoded values in tempest plugin https://review.openstack.org/437704 | 17:13 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost master: Use virtualbmc for virtual BM nodes https://review.openstack.org/427376 | 17:13 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost master: Use JSON baremetal data in testing https://review.openstack.org/430898 | 17:13 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost master: Use Ansible to create test VMs https://review.openstack.org/427071 | 17:13 |
| rloo | JayF: regardless of whether jroll is ptl or not, he has to be stable core. if nothing else, as punishment for not being ptl :) | 17:14 |
| openstackgerrit | Merged openstack/ironic master: Add support for DBDeadlock handling https://review.openstack.org/394413 | 17:14 |
| dtantsur | lol | 17:15 |
| *** jaypipes has joined #openstack-ironic | 17:16 | |
| TheJulia | that is either crafty or evil, I can't determine which | 17:16 |
| rloo | keep them guessin' ... | 17:16 |
| *** jaganathan_afk has quit IRC | 17:18 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Update HTTP API docs with missing 1.9 API microversion https://review.openstack.org/438870 | 17:20 |
| jroll | JayF: don't see why not | 17:23 |
| *** sdake_ is now known as sdake | 17:24 | |
| *** stendulker has quit IRC | 17:26 | |
| *** ijw has quit IRC | 17:28 | |
| *** gcb has quit IRC | 17:30 | |
| *** trown is now known as trown|lunch | 17:30 | |
| *** ccamacho has quit IRC | 17:33 | |
| *** ijw has joined #openstack-ironic | 17:36 | |
| *** mgoddard has joined #openstack-ironic | 17:38 | |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector master: [WIP] Put logic in place for using new Ironic network simulation https://review.openstack.org/421288 | 17:38 |
| *** strigazi has quit IRC | 17:39 | |
| *** strigazi has joined #openstack-ironic | 17:40 | |
| *** ijw has quit IRC | 17:41 | |
| *** mkrai has joined #openstack-ironic | 17:43 | |
| *** derekh has quit IRC | 17:44 | |
| *** dsneddon has joined #openstack-ironic | 17:47 | |
| *** sdake_ has joined #openstack-ironic | 17:48 | |
| *** sdake has quit IRC | 17:50 | |
| *** aparnav has quit IRC | 17:50 | |
| *** crushil has joined #openstack-ironic | 17:54 | |
| *** fragatina has joined #openstack-ironic | 17:55 | |
| *** fragatina has quit IRC | 17:57 | |
| *** fragatina has joined #openstack-ironic | 17:58 | |
| *** sdake_ has quit IRC | 18:02 | |
| *** moshele has joined #openstack-ironic | 18:02 | |
| lucasagomes | folks I'm calling it a day | 18:05 |
| lucasagomes | have a great evening all | 18:05 |
| lucasagomes | talk soon | 18:05 |
| *** lucasagomes is now known as lucas-afk | 18:05 | |
| *** sdake has joined #openstack-ironic | 18:05 | |
| NobodyCam | have a good night lucas-afk | 18:06 |
| *** chlong has quit IRC | 18:06 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost master: Use virtualbmc for virtual BM nodes https://review.openstack.org/427376 | 18:15 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost master: Use JSON baremetal data in testing https://review.openstack.org/430898 | 18:15 |
| * mgould -> home; good night! | 18:20 | |
| *** mgould is now known as mgould|afk | 18:20 | |
| NobodyCam | night mgould|afk | 18:20 |
| *** sdake has quit IRC | 18:21 | |
| *** mgoddard has quit IRC | 18:23 | |
| *** sdake has joined #openstack-ironic | 18:23 | |
| *** vsaienko has joined #openstack-ironic | 18:26 | |
| *** milan has quit IRC | 18:27 | |
| *** krtaylor has quit IRC | 18:32 | |
| *** sambetts is now known as sambetts|afk | 18:33 | |
| sambetts|afk | night NobodyCam | 18:33 |
| sambetts|afk | night all | 18:33 |
| NobodyCam | night sambetts|afk | 18:34 |
| dtantsur | calling it a day as well, see you tomorrow | 18:34 |
| NobodyCam | night dtantsur | 18:34 |
| *** dtantsur is now known as dtantsur|afk | 18:34 | |
| *** Nisha_Agarwal has quit IRC | 18:34 | |
| *** dprince has quit IRC | 18:38 | |
| *** baoli has joined #openstack-ironic | 18:41 | |
| *** milan has joined #openstack-ironic | 18:44 | |
| *** cppforlife_ has joined #openstack-ironic | 18:45 | |
| *** csmart has joined #openstack-ironic | 18:47 | |
| *** retr0h has joined #openstack-ironic | 18:47 | |
| *** trown|lunch is now known as trown | 18:47 | |
| *** moshele has quit IRC | 18:48 | |
| *** milan has quit IRC | 18:50 | |
| *** vsaienko has quit IRC | 18:50 | |
| *** ChubYann has joined #openstack-ironic | 18:51 | |
| *** ijw has joined #openstack-ironic | 18:51 | |
| *** vsaienko has joined #openstack-ironic | 18:52 | |
| *** ijw has quit IRC | 18:57 | |
| *** rloo has quit IRC | 18:58 | |
| *** rloo has joined #openstack-ironic | 18:58 | |
| *** baoli has quit IRC | 19:00 | |
| *** baoli has joined #openstack-ironic | 19:01 | |
| *** baoli has quit IRC | 19:01 | |
| *** UForgotten has quit IRC | 19:05 | |
| *** UForgotten has joined #openstack-ironic | 19:05 | |
| *** sayanta__ has quit IRC | 19:07 | |
| *** dprince has joined #openstack-ironic | 19:08 | |
| *** gfhellma has joined #openstack-ironic | 19:10 | |
| *** baoli has joined #openstack-ironic | 19:11 | |
| *** patchbot_ is now known as patchbot | 19:12 | |
| *** amoralej is now known as amoralej|off | 19:13 | |
| *** krtaylor has joined #openstack-ironic | 19:15 | |
| *** fragatina has quit IRC | 19:17 | |
| *** krtaylor has quit IRC | 19:18 | |
| *** krtaylor has joined #openstack-ironic | 19:19 | |
| *** sdake has quit IRC | 19:19 | |
| *** krtaylor has quit IRC | 19:23 | |
| *** chlong has joined #openstack-ironic | 19:25 | |
| *** ppiela_ has quit IRC | 19:25 | |
| *** sdake has joined #openstack-ironic | 19:27 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests https://review.openstack.org/423556 | 19:28 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add ironic standlaone test with ipmi dynamic driver https://review.openstack.org/439126 | 19:28 |
| *** krtaylor has joined #openstack-ironic | 19:36 | |
| *** krtaylor has quit IRC | 19:39 | |
| *** krtaylor has joined #openstack-ironic | 19:40 | |
| *** krtaylor has quit IRC | 19:42 | |
| *** fragatina has joined #openstack-ironic | 19:42 | |
| *** baoli has quit IRC | 19:42 | |
| *** baoli has joined #openstack-ironic | 19:43 | |
| *** krtaylor has joined #openstack-ironic | 19:44 | |
| *** ppiela has joined #openstack-ironic | 19:45 | |
| *** krtaylor has quit IRC | 19:46 | |
| *** fragatina has quit IRC | 19:47 | |
| *** krtaylor has joined #openstack-ironic | 19:48 | |
| *** krtaylor has quit IRC | 19:50 | |
| *** fragatina has joined #openstack-ironic | 19:50 | |
| *** fragatina has quit IRC | 19:50 | |
| *** fragatina has joined #openstack-ironic | 19:51 | |
| *** cdearborn has joined #openstack-ironic | 19:52 | |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Cleanup code associated with the ironic REST api https://review.openstack.org/432295 | 19:54 |
| *** ijw has joined #openstack-ironic | 19:55 | |
| *** baoli has quit IRC | 19:56 | |
| cdearborn | Hey guys, I'm running into a problem using manual cleaning. The error is "No VIFs found for node". A paste of the log is here: http://paste.openstack.org/show/600832/. Any ideas? | 19:57 |
| cdearborn | I verified that the cleaning network is configured. I'm digging through the code but haven't found anything yet | 19:58 |
| *** ijw has quit IRC | 19:59 | |
| *** baoli has joined #openstack-ironic | 20:03 | |
| *** dyasny has quit IRC | 20:05 | |
| *** dyasny has joined #openstack-ironic | 20:06 | |
| *** sdake has quit IRC | 20:07 | |
| *** sdake has joined #openstack-ironic | 20:07 | |
| *** vsaienko has quit IRC | 20:10 | |
| *** sayantan_ has joined #openstack-ironic | 20:11 | |
| *** sdake has quit IRC | 20:13 | |
| TheJulia | cdearborn: using ocata? | 20:14 |
| *** sdake has joined #openstack-ironic | 20:14 | |
| *** baoli has quit IRC | 20:15 | |
| *** baoli has joined #openstack-ironic | 20:15 | |
| cdearborn | newton i believe | 20:15 |
| *** vsaienko has joined #openstack-ironic | 20:15 | |
| *** athomas has quit IRC | 20:15 | |
| *** harlowja has quit IRC | 20:15 | |
| *** harlowja has joined #openstack-ironic | 20:16 | |
| TheJulia | I feel like there was a bug around that at one point, but I thought that got fixed a long time ago. | 20:17 |
| *** vsaienko has quit IRC | 20:18 | |
| *** vsaienko has joined #openstack-ironic | 20:20 | |
| jroll | rpioso had this, it was that the node had no ports defined | 20:22 |
| * jroll gets eavesdrop link | 20:22 | |
| jroll | cdearborn: http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2016-12-09.log.html#t2016-12-09T16:36:12 | 20:22 |
| TheJulia | Good memory jroll! | 20:23 |
| jroll | TheJulia: grep is the closest thing to memory I have. | 20:23 |
| *** baoli has quit IRC | 20:25 | |
| *** baoli has joined #openstack-ironic | 20:25 | |
| cdearborn | jroll: thx! taking a look... | 20:26 |
| cdearborn | jroll, TheJulia: thx, that was it - no ports on the ironic nodes. I believe I'll be able to hunt it down from here. Thx much for your help! | 20:29 |
| jroll | cdearborn: woot, you're welcome | 20:29 |
| *** baoli has quit IRC | 20:30 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: doc: update FAQ for release notes https://review.openstack.org/439054 | 20:31 |
| *** vsaienko has quit IRC | 20:34 | |
| *** vsaienko has joined #openstack-ironic | 20:35 | |
| rpioso | jroll: How did you so quickly find that IRC log? | 20:38 |
| jroll | rpioso: I also log locally, so grep | 20:38 |
| jroll | and then I have the date for link | 20:38 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-ui master: Correct Release Notes theme https://review.openstack.org/439144 | 20:41 |
| *** baoli has joined #openstack-ironic | 20:43 | |
| rpioso | jroll: Ahhh, locally. I was hoping it was available on the web :-( | 20:44 |
| rpioso | jroll: ty | 20:44 |
| jroll | rpioso: it is indexed, so you could google something like site:eavesdrop.openstack.org jroll grep | 20:44 |
| jroll | or even site:eavesdrop.openstack.org/irclogs/%23openstack-ironic/ | 20:45 |
| *** sdake_ has joined #openstack-ironic | 20:47 | |
| *** athomas has joined #openstack-ironic | 20:47 | |
| rpioso | jroll: Nice! Thank you so much. | 20:49 |
| jroll | :) | 20:50 |
| *** sdake has quit IRC | 20:50 | |
| mrda | Morning Ironic | 20:51 |
| *** e0ne has joined #openstack-ironic | 20:52 | |
| *** aslezil_ has quit IRC | 20:53 | |
| *** aslezil_ has joined #openstack-ironic | 20:53 | |
| NobodyCam | morning mrda :) | 20:54 |
| *** aslezil_ has quit IRC | 20:55 | |
| mrda | o/ | 20:55 |
| *** aslezil_ has joined #openstack-ironic | 20:55 | |
| *** ijw has joined #openstack-ironic | 20:56 | |
| *** aslezil_ has quit IRC | 20:56 | |
| *** aslezil_ has joined #openstack-ironic | 20:57 | |
| *** baoli has quit IRC | 20:58 | |
| NobodyCam | :) | 20:58 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: WIP: Index toc cleanup https://review.openstack.org/439151 | 20:59 |
| *** ijw has quit IRC | 21:01 | |
| *** baoli has joined #openstack-ironic | 21:03 | |
| *** moshele has joined #openstack-ironic | 21:05 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests https://review.openstack.org/423556 | 21:06 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add ironic standlaone test with ipmi dynamic driver https://review.openstack.org/439126 | 21:06 |
| *** bfournie has quit IRC | 21:24 | |
| *** crushil has quit IRC | 21:24 | |
| *** moshele has quit IRC | 21:25 | |
| *** mjura has quit IRC | 21:25 | |
| *** chlong has quit IRC | 21:27 | |
| *** dsneddon has quit IRC | 21:35 | |
| TheJulia | So I was glancing at the dev docs that we have to figure out how to notate consensus from friday on exceptions, and now I feel that it is silly to do so. Figures :) | 21:38 |
| *** crushil has joined #openstack-ironic | 21:38 | |
| *** ijw has joined #openstack-ironic | 21:39 | |
| *** chlong has joined #openstack-ironic | 21:39 | |
| *** e0ne has quit IRC | 21:41 | |
| jroll | lol | 21:42 |
| jroll | TheJulia: you have another volunteer for BFV code in the form of joanna | 21:42 |
| * jroll throws her under the bus | 21:42 | |
| jroll | it's public, now it has to happen :) | 21:42 |
| TheJulia | lol | 21:42 |
| * TheJulia wonders if anyone has responded to her post to the ml | 21:43 | |
| jroll | TheJulia: I worry about multiple subteams conflicting | 21:43 |
| joanna | :) | 21:43 |
| jroll | oh, mjturek has good questions | 21:43 |
| joanna | I like the idea of having a sync meeting on that | 21:44 |
| jroll | ya, I'm +1000 on BFV meeting, idk about general purpose thing | 21:45 |
| *** sdake_ has quit IRC | 21:45 | |
| TheJulia | jroll: I agree that could be an issue, we do fairly well about wiki page editing with the existing ironic project, I'm just worried even adding anything that could take 10+ minutes might derail other things in that existing meeting | 21:45 |
| jroll | TheJulia: yep, same | 21:45 |
| * TheJulia is getting distracted by her fiancee who is trying to convince her of where to go on vacation | 21:45 | |
| jroll | mmmm vacation. | 21:46 |
| * jroll has march 10-12 off to hang around here with friends from portlandia | 21:46 | |
| TheJulia | for some silly reason, the thought of visiting japan again has crossed our minds | 21:46 |
| joanna | TheJulia: I heard Poland is really nice | 21:47 |
| joanna | there are pierogi in Poland :D | 21:47 |
| jroll | nom nom pierogi | 21:47 |
| * TheJulia suddenly wants dinner | 21:47 | |
| jroll | today is paczki day around here, lots of polish folks and fat tuesday | 21:47 |
| openstackgerrit | Anup Navare proposed openstack/ironic master: Allow logical name in port creation https://review.openstack.org/405586 | 21:48 |
| joanna | actually - Poland has fat Thursday which was last Thursday :) | 21:48 |
| JayF | jroll: I can provide grilled meats if you wanna drive north about 90 minutes :P | 21:48 |
| jroll | heh, I learned that today! | 21:48 |
| jroll | JayF: portland people coming here, not vice versa, but thanks :D | 21:48 |
| jroll | joanna: even if we have the wrong day, we have paczki so all is well | 21:49 |
| JayF | jroll: aha | 21:49 |
| joanna | jroll: nice! | 21:49 |
| jlvillal | I'm perplexed on why unit tests show '_' as a defined method and I can't figure out where it comes from... https://review.openstack.org/#/c/438986/ | 21:49 |
| patchbot | patch 438986 - ironic - Fix missing _ import in driver_factory | 21:49 |
| jroll | people were lined up at 3am | 21:49 |
| * jlvillal is confused | 21:49 | |
| JayF | jroll: in the case, s/north about 90 minutes/west about 3 days/ :P | 21:49 |
| JayF | jlvillal: _ in python is basically "ignore this" aiui | 21:49 |
| joanna | jroll: Tueasday makes more sense, at least you eat them after you try to fit your dress for the last weekend before the Lent :) | 21:49 |
| jroll | joanna: lol | 21:50 |
| JayF | jlvillal: so if you have a method that returns (stdout, stderr), and you only care about stderr, you could do _, err = processRun(myprocess) | 21:50 |
| jlvillal | JayF: Right. But this is a different issue | 21:50 |
| jlvillal | I see it as: <bound method NullTranslations.gettext of <gettext.NullTranslations object at 0x7f87f0386e48>> | 21:51 |
| jroll | JayF: there's also ironic.common.i18n._ for translation | 21:51 |
| JayF | jroll: ooh, good point | 21:51 |
| * jroll wonders if jlvillal has time to sit by the fire and hear stories about the old days | 21:51 | |
| jlvillal | And I have no idea where the hell that came from | 21:51 |
| jlvillal | jroll: :) | 21:51 |
| *** ijw has quit IRC | 21:52 | |
| TheJulia | jroll: so... for this sitting by the fire, will there be beer? | 21:52 |
| *** ijw has joined #openstack-ironic | 21:52 | |
| jroll | lol if you think beer will handle this | 21:52 |
| *** jkilpatr has quit IRC | 21:52 | |
| openstackgerrit | Merged openstack/bifrost master: Add '-E' options to pass proxy env for sudo https://review.openstack.org/438884 | 21:52 |
| TheJulia | Well "how much" is one of those variables | 21:53 |
| * jlvillal likes to hear stories. Usually... | 21:53 | |
| jroll | fair enough | 21:53 |
| jroll | I'm failing to find sources and such | 21:53 |
| jroll | but tl;dr it's pretty standard in python to monkey patch a _ into globals for translations | 21:53 |
| jroll | and we (do|used to do) this in openstack | 21:54 |
| jroll | this has some monkey patching but it's unclear if that's happening there or elsewhere https://github.com/openstack/oslo.i18n/blob/master/oslo_i18n/fixture.py | 21:54 |
| *** athomas has quit IRC | 21:54 | |
| *** bfournie has joined #openstack-ironic | 21:55 | |
| jroll | I kind of suspect i18n.install('ironic') might do that thing | 21:55 |
| jlvillal | Oh and yuck... | 21:55 |
| jroll | although that shouldn't run in unit tests | 21:55 |
| jroll | at any rate, might be an oslo question | 21:55 |
| jroll | because I'm failing :( | 21:56 |
| jlvillal | jroll: Thanks. That is useful to know | 21:56 |
| JayF | the one thing I hear a pythonism I know the answer to, it's still somehow wrong | 21:56 |
| srart | https://shahriar.svbtle.com/underscores-in-python | 21:56 |
| JayF | lol | 21:56 |
| *** ijw has quit IRC | 21:56 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: WIP/DNM: Debugging stuff https://review.openstack.org/439183 | 21:56 |
| jroll | how are there no rants on this in google | 21:56 |
| srart | jroll: see the link I just posted | 21:57 |
| *** trown is now known as trown|outtypewww | 21:57 | |
| jroll | srart: yeah, was waiting for it to load | 21:57 |
| srart | your google-fu is lacking ;-) | 21:57 |
| jroll | seems that explains it but doesn't rant | 21:57 |
| jroll | I want a "STOP DOING THIS GDI" link | 21:57 |
| *** chlong has quit IRC | 21:58 | |
| srart | nope, python docs still have it there. https://docs.python.org/3/library/gettext.html#gettext.gettext | 22:04 |
| *** krtaylor has joined #openstack-ironic | 22:08 | |
| *** baoli has quit IRC | 22:13 | |
| *** jjohnson2 has quit IRC | 22:13 | |
| *** dsneddon has joined #openstack-ironic | 22:13 | |
| *** baoli has joined #openstack-ironic | 22:13 | |
| *** baoli has quit IRC | 22:14 | |
| openstackgerrit | Merged openstack/ironic master: Add ironic port group CRUD notifications https://review.openstack.org/427281 | 22:22 |
| *** baoli has joined #openstack-ironic | 22:24 | |
| mariojv | jlvillal: you cool with me landing https://review.openstack.org/#/c/438986/1 or did you want to look at it more? | 22:26 |
| patchbot | patch 438986 - ironic - Fix missing _ import in driver_factory | 22:26 |
| jlvillal | mariojv: Yeah, I think it is good to be landed. Strange on the unit tests passing... | 22:27 |
| mariojv | i think somehow the _() from gettext is being used in the pre-patch code https://docs.python.org/2/library/gettext.html | 22:27 |
| mariojv | https://docs.python.org/2/library/gettext.html#gettext.NullTranslations.install is more specific | 22:27 |
| *** sayantan_ has quit IRC | 22:28 | |
| mariojv | i wonder if this is doing it https://github.com/openstack/ironic/blob/77b86f26c4e42eca65ba7baaf469e10c69f1cd1f/ironic/tests/unit/objects/test_objects.py#L31 | 22:28 |
| jlvillal | mariojv: Thanks. Not sure. | 22:29 |
| jlvillal | mariojv: ding ding ding! :) | 22:30 |
| jlvillal | mariojv: Commenting out that line made it fail | 22:31 |
| mariojv | ah. i wonder if it serves any other important purpose | 22:31 |
| * mariojv needs to git blame it and find out | 22:31 | |
| jlvillal | mariojv: Me too | 22:31 |
| mariojv | jlvillal: you want me to file a bug about this? or should we just fix it | 22:32 |
| mariojv | i've seen missing _ before so i don't think the issue was limited to driver_factory.py | 22:32 |
| *** jkilpatr has joined #openstack-ironic | 22:33 | |
| jlvillal | mariojv: Not sure on bug or just fix it. If easy to fix then just do it. I guess a bug doesn't hurt though :) | 22:35 |
| mariojv | k | 22:35 |
| jlvillal | mariojv: Trying a full tox run with that line commented out. See what fails | 22:35 |
| mariojv | nice, then we can fix anywhere else it's missing, too | 22:36 |
| jlvillal | mariojv: Sweet if found a total of four missing '_' | 22:36 |
| jlvillal | s/if/it/ | 22:36 |
| *** amotoki has quit IRC | 22:37 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic master: Update the Ironic Upgrade guide https://review.openstack.org/439196 | 22:37 |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic master: Update the Ironic Upgrade guide https://review.openstack.org/439196 | 22:40 |
| *** amotoki has joined #openstack-ironic | 22:42 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Fix missing _ import in driver_factory https://review.openstack.org/438986 | 22:42 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Remove gettext.install() for unit tests https://review.openstack.org/439197 | 22:42 |
| jlvillal | mariojv: ^^^ | 22:42 |
| * jlvillal made a mistake by cherry-picking vsaienk0's patch :( | 22:43 | |
| *** baoli has quit IRC | 22:43 | |
| mariojv | jlvillal: https://bugs.launchpad.net/ironic/+bug/1668789 | 22:44 |
| openstack | Launchpad bug 1668789 in Ironic "Missing "_" import isn't caught in unit tests" [Undecided,New] | 22:44 |
| mariojv | jlvillal: dang you're fast :P | 22:44 |
| jlvillal | mariojv: I'll update patch :) | 22:45 |
| mariojv | thx | 22:45 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Remove gettext.install() for unit tests https://review.openstack.org/439197 | 22:45 |
| jlvillal | mariojv: ^^ done. And thanks for the bug :) | 22:45 |
| *** amotoki has quit IRC | 22:46 | |
| *** priteau has quit IRC | 22:47 | |
| jlvillal | mariojv: Thanks. It was fun to find the root cause of the issue. | 22:47 |
| *** crushil has quit IRC | 22:48 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Remove gettext.install() for unit tests https://review.openstack.org/439197 | 22:50 |
| soliosg | jroll, jlvillal: for the QA/Tempest Team request to host a copy of tempest.scenarios.manager.py in ironic, should I create a bug or just a summary what the change is about in the commit message? | 22:52 |
| jlvillal | soliosg: I would vote for a bug, to track it. | 22:52 |
| *** ijw has joined #openstack-ironic | 22:53 | |
| soliosg | jroll, jlvillal: a bug may not apply, but I wonder what's the best for record purposes | 22:53 |
| jlvillal | soliosg: And a good summary in the commit message :) | 22:53 |
| jroll | +1 for a bug | 22:53 |
| soliosg | jroll, jlvillal: great! thanks | 22:53 |
| jroll | np | 22:54 |
| *** baha has quit IRC | 22:56 | |
| *** ijw has quit IRC | 22:58 | |
| *** zackf has quit IRC | 22:59 | |
| *** zackf has joined #openstack-ironic | 23:03 | |
| *** baoli has joined #openstack-ironic | 23:07 | |
| *** zackf has quit IRC | 23:07 | |
| *** zackf has joined #openstack-ironic | 23:08 | |
| *** cdearborn has quit IRC | 23:09 | |
| *** jcoufal has quit IRC | 23:10 | |
| *** zackf has quit IRC | 23:12 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/439217 | 23:17 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/439218 | 23:17 |
| *** crushil has joined #openstack-ironic | 23:20 | |
| *** jjohnson2 has joined #openstack-ironic | 23:22 | |
| *** baoli has quit IRC | 23:22 | |
| *** pmannidi has joined #openstack-ironic | 23:22 | |
| *** zackf has joined #openstack-ironic | 23:25 | |
| *** zackf has quit IRC | 23:29 | |
| *** dprince has quit IRC | 23:29 | |
| *** Fdaisuke has joined #openstack-ironic | 23:33 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic master: Update the Ironic Upgrade guide https://review.openstack.org/439196 | 23:37 |
| *** sacharya has quit IRC | 23:47 | |
| *** sayantan_ has joined #openstack-ironic | 23:48 | |
| *** zackf has joined #openstack-ironic | 23:49 | |
| *** sayantan_ has quit IRC | 23:49 | |
| *** amotoki has joined #openstack-ironic | 23:53 | |
| *** crushil has quit IRC | 23:53 | |
| *** zackf has quit IRC | 23:54 | |
| *** ijw has joined #openstack-ironic | 23:54 | |
| *** sayantan_ has joined #openstack-ironic | 23:55 | |
| *** zackf has joined #openstack-ironic | 23:56 | |
| *** ijw has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!