| *** Haomeng has quit IRC | 00:00 | |
| *** wajdi has joined #openstack-ironic | 00:11 | |
| *** amotoki has joined #openstack-ironic | 00:12 | |
| *** rama_y has quit IRC | 00:13 | |
| *** wajdi_ has joined #openstack-ironic | 00:13 | |
| *** wajdi has quit IRC | 00:16 | |
| *** sdake_ is now known as sdake | 00:19 | |
| *** ayoung has joined #openstack-ironic | 00:22 | |
| *** joprovost has joined #openstack-ironic | 00:25 | |
| *** wajdi_ has quit IRC | 00:30 | |
| *** amotoki has quit IRC | 00:32 | |
| *** hoangcx has joined #openstack-ironic | 00:41 | |
| *** joprovost has quit IRC | 00:45 | |
| chenke | jroll, tku. i want to know what kind of tenant network this feature(Tenant networking support for Ironic driver) refer to? | 00:51 |
|---|---|---|
| *** YiboCai has joined #openstack-ironic | 00:52 | |
| chenke | i treat the tenant network as vlan or vxlan, but in the end of description it point out it is not vlan or vxlan(in the future we'll want to support multiple networks via VLAN or VXLAN over a pair of bonded NICs ) | 00:52 |
| *** phuongnh has joined #openstack-ironic | 00:59 | |
| *** Sukhdev has quit IRC | 01:13 | |
| *** Sukhdev has joined #openstack-ironic | 01:14 | |
| *** amotoki has joined #openstack-ironic | 01:15 | |
| *** Sukhdev has quit IRC | 01:19 | |
| *** joprovost has joined #openstack-ironic | 01:20 | |
| *** amotoki_ has joined #openstack-ironic | 01:24 | |
| *** amotoki has quit IRC | 01:26 | |
| *** moshele has joined #openstack-ironic | 01:31 | |
| *** amit213 has joined #openstack-ironic | 01:39 | |
| *** amit213 has quit IRC | 01:41 | |
| *** amit213 has joined #openstack-ironic | 01:44 | |
| *** joprovost has quit IRC | 01:45 | |
| *** sdake has quit IRC | 01:49 | |
| *** joprovost has joined #openstack-ironic | 01:56 | |
| *** sacharya has joined #openstack-ironic | 01:57 | |
| *** PollyZ_ has joined #openstack-ironic | 01:59 | |
| *** vishwanathj has joined #openstack-ironic | 02:03 | |
| *** joprovost1 has joined #openstack-ironic | 02:04 | |
| *** joprovost has quit IRC | 02:06 | |
| *** joprovost1 is now known as joprovost | 02:06 | |
| *** spandhe has quit IRC | 02:08 | |
| *** sdake has joined #openstack-ironic | 02:16 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Add a missing unit test to create node with tags https://review.openstack.org/323680 | 02:18 |
| *** sdake has quit IRC | 02:18 | |
| *** joprovost1 has joined #openstack-ironic | 02:41 | |
| *** joprovost has quit IRC | 02:43 | |
| *** joprovost1 is now known as joprovost | 02:43 | |
| *** yuanying has quit IRC | 02:51 | |
| *** sabeen has joined #openstack-ironic | 02:59 | |
| *** vishwanathj has quit IRC | 03:01 | |
| *** rloo has quit IRC | 03:07 | |
| *** spandhe has joined #openstack-ironic | 03:07 | |
| *** moshele has quit IRC | 03:15 | |
| *** spandhe_ has joined #openstack-ironic | 03:21 | |
| *** ChrisAusten has joined #openstack-ironic | 03:22 | |
| *** spandhe has quit IRC | 03:23 | |
| *** spandhe_ is now known as spandhe | 03:23 | |
| *** rbudden has quit IRC | 03:25 | |
| *** stevebaker has quit IRC | 03:27 | |
| *** stevebaker has joined #openstack-ironic | 03:29 | |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Added functions to translate between uuid and uri https://review.openstack.org/301877 | 03:33 |
| *** links has joined #openstack-ironic | 03:35 | |
| openstackgerrit | Merged openstack/ironic-lib: Updated from global requirements https://review.openstack.org/323066 | 03:47 |
| *** yuanying has joined #openstack-ironic | 03:48 | |
| *** Sukhdev has joined #openstack-ironic | 03:48 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/323889 | 04:04 |
| *** moshele has joined #openstack-ironic | 04:09 | |
| *** moshele has quit IRC | 04:14 | |
| *** tangchen has quit IRC | 04:16 | |
| *** ayoung has quit IRC | 04:22 | |
| *** anush_ has quit IRC | 04:23 | |
| *** rama_y has joined #openstack-ironic | 04:24 | |
| *** anush_ has joined #openstack-ironic | 04:27 | |
| *** tangchen has joined #openstack-ironic | 04:29 | |
| *** lipt has joined #openstack-ironic | 04:31 | |
| *** bsilverman has joined #openstack-ironic | 04:38 | |
| *** Sukhdev has quit IRC | 04:48 | |
| *** joprovost has quit IRC | 04:52 | |
| *** sacharya has quit IRC | 04:57 | |
| *** spandhe has quit IRC | 04:58 | |
| lazy_prince | @chenke: its more of VLAN network for the time being.. | 05:05 |
| *** spandhe has joined #openstack-ironic | 05:12 | |
| *** jtomasek has joined #openstack-ironic | 05:16 | |
| *** amotoki_ is now known as amotoki | 05:19 | |
| *** ChubYann has quit IRC | 05:19 | |
| *** bharathk has joined #openstack-ironic | 05:47 | |
| *** phuongnh has quit IRC | 05:48 | |
| *** rcernin has joined #openstack-ironic | 05:48 | |
| *** appu103_ has joined #openstack-ironic | 05:48 | |
| *** phuongnh has joined #openstack-ironic | 05:48 | |
| *** appu103_ is now known as aparnav | 05:49 | |
| *** sacharya has joined #openstack-ironic | 05:57 | |
| *** greghaynes has quit IRC | 05:58 | |
| *** vmud213 has joined #openstack-ironic | 05:59 | |
| *** sacharya has quit IRC | 06:02 | |
| *** greghaynes has joined #openstack-ironic | 06:09 | |
| *** mjura has joined #openstack-ironic | 06:11 | |
| *** e0ne has joined #openstack-ironic | 06:15 | |
| *** mjura has quit IRC | 06:19 | |
| *** rama_y has quit IRC | 06:21 | |
| *** mjura has joined #openstack-ironic | 06:31 | |
| *** spandhe has quit IRC | 06:35 | |
| *** SiRiuS__ has joined #openstack-ironic | 06:38 | |
| *** e0ne has quit IRC | 06:38 | |
| *** ChrisAusten has quit IRC | 06:41 | |
| openstackgerrit | Vaishnavi proposed openstack/ironic: Config variable to configure [glance] section https://review.openstack.org/266803 | 06:44 |
| *** _vs has joined #openstack-ironic | 06:50 | |
| *** _vsaienko has joined #openstack-ironic | 06:50 | |
| *** ishant|away is now known as ishant | 06:52 | |
| *** tesseract- has joined #openstack-ironic | 06:53 | |
| *** praneshp has quit IRC | 06:56 | |
| *** sdake has joined #openstack-ironic | 06:57 | |
| *** _vsaienko has quit IRC | 06:59 | |
| *** _vs has quit IRC | 06:59 | |
| *** hoangcx has quit IRC | 07:00 | |
| *** hoangcx has joined #openstack-ironic | 07:01 | |
| *** _vsaienko has joined #openstack-ironic | 07:03 | |
| *** _vs has joined #openstack-ironic | 07:03 | |
| *** _vs has quit IRC | 07:06 | |
| *** _vs has joined #openstack-ironic | 07:06 | |
| *** vishwanathj has joined #openstack-ironic | 07:15 | |
| *** _vs has quit IRC | 07:15 | |
| *** _vsaienko has quit IRC | 07:15 | |
| *** vishwana_ has joined #openstack-ironic | 07:15 | |
| *** _vsaienko has joined #openstack-ironic | 07:17 | |
| *** _vs has joined #openstack-ironic | 07:17 | |
| *** fragatina has quit IRC | 07:18 | |
| *** fragatina has joined #openstack-ironic | 07:19 | |
| *** vishwanathj has quit IRC | 07:19 | |
| *** phuongnh has quit IRC | 07:20 | |
| *** fragatina has quit IRC | 07:23 | |
| *** vishwana_ is now known as vishwanathj | 07:26 | |
| *** fragatina has joined #openstack-ironic | 07:26 | |
| *** _vs has quit IRC | 07:26 | |
| *** _vsaienko has quit IRC | 07:26 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Patch to test Grenade. https://review.openstack.org/311109 | 07:30 |
| *** fragatina has quit IRC | 07:31 | |
| *** vishwanathj has quit IRC | 07:34 | |
| *** jtomasek_ has joined #openstack-ironic | 07:37 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Improve _check_depolying_status to handle nodes in maintenance https://review.openstack.org/324269 | 07:37 |
| *** hoonetorg has quit IRC | 07:38 | |
| *** sdake has quit IRC | 07:39 | |
| *** kromanenko__ has joined #openstack-ironic | 07:39 | |
| *** kromanenko_ has joined #openstack-ironic | 07:39 | |
| *** Romanenko_K has joined #openstack-ironic | 07:39 | |
| *** fragatina has joined #openstack-ironic | 07:39 | |
| *** lipt has quit IRC | 07:41 | |
| *** jtomasek_ has quit IRC | 07:44 | |
| *** fragatina has quit IRC | 07:44 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Refactor enroll-node code https://review.openstack.org/323190 | 07:46 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: TEMP PATCH. https://review.openstack.org/323770 | 07:46 |
| *** lipt has joined #openstack-ironic | 07:46 | |
| chenke | @lazy_prince, tku. so it supports not only the vlan network, also vxlan is supported in this feature? | 07:47 |
| *** hoonetorg has joined #openstack-ironic | 07:50 | |
| *** lipt has joined #openstack-ironic | 07:51 | |
| *** lipt has joined #openstack-ironic | 07:51 | |
| *** lipt has joined #openstack-ironic | 07:56 | |
| *** lipt has quit IRC | 07:56 | |
| *** hoangcx has quit IRC | 07:57 | |
| *** dtantsur|afk is now known as dtantsur | 07:58 | |
| dtantsur | Morning Ironic | 07:58 |
| *** sacharya has joined #openstack-ironic | 07:59 | |
| aarefiev | morning | 07:59 |
| *** zzzeek has quit IRC | 08:00 | |
| vsaienko | morning dtantsur, aarefiev chenke and all Ironic'ers | 08:00 |
| *** hoangcx has joined #openstack-ironic | 08:00 | |
| aarefiev | morning vsaienko, dtantsur | 08:00 |
| chenke | Morning | 08:01 |
| aarefiev | dtantsur: any suggestions about new tempest job name in inspector? | 08:01 |
| aarefiev | which runs discovery and other tests | 08:02 |
| *** zzzeek has joined #openstack-ironic | 08:02 | |
| *** sacharya has quit IRC | 08:03 | |
| openstackgerrit | Merged openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/323890 | 08:03 |
| dtantsur | morning vsaienko, aarefiev | 08:04 |
| dtantsur | aarefiev, hmm, you make me think too ealier in the morning :D | 08:04 |
| aarefiev | dtantsur: :), me too, because I can't come up a good name | 08:06 |
| aarefiev | extended, feature test | 08:07 |
| *** mbound has joined #openstack-ironic | 08:08 | |
| alineb | morning all | 08:08 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 08:10 |
| *** e0ne has joined #openstack-ironic | 08:13 | |
| *** vishwanathj has joined #openstack-ironic | 08:14 | |
| *** ifarkas has joined #openstack-ironic | 08:15 | |
| *** lipt has joined #openstack-ironic | 08:15 | |
| *** ohamada has joined #openstack-ironic | 08:17 | |
| lucasagomes | morning all | 08:17 |
| *** lipt has joined #openstack-ironic | 08:19 | |
| *** lipt has quit IRC | 08:19 | |
| *** lipt has joined #openstack-ironic | 08:20 | |
| *** lipt has quit IRC | 08:20 | |
| *** lipt has joined #openstack-ironic | 08:21 | |
| *** lipt has joined #openstack-ironic | 08:22 | |
| *** lipt has quit IRC | 08:22 | |
| vsaienko | morning lucasagomes | 08:28 |
| *** MattMan has quit IRC | 08:31 | |
| *** MattMan has joined #openstack-ironic | 08:31 | |
| *** lipt has joined #openstack-ironic | 08:34 | |
| *** lipt has joined #openstack-ironic | 08:34 | |
| *** lipt has quit IRC | 08:34 | |
| dtantsur | morning alineb, lucasagomes | 08:36 |
| *** mgould|afk is now known as mgould | 08:36 | |
| mgould | morning dtantsur alineb lucasagomes | 08:36 |
| dtantsur | hey mgould | 08:36 |
| lucasagomes | dtantsur, vsaienko alineb mgould hey there :-) | 08:39 |
| alineb | o/ dtantsur mgould lucasagomes | 08:40 |
| lucasagomes | dtantsur, just wondering what you think about https://review.openstack.org/#/c/324015/ ? | 08:41 |
| dtantsur | lucasagomes, did not look yet, but my first comment would be "bring it to ironicclient please" :) | 08:41 |
| lucasagomes | dtantsur, sure, it's part of the ultimate idea | 08:42 |
| dtantsur | I've started the whole discussion because I hate when people try to sneak in some scripts that nobody outside of ironic will benefit | 08:42 |
| lucasagomes | dtantsur, gathering more opninions on things like how the syntax of the file looks like? should it be json instead? or maybe yaml AND json? | 08:42 |
| lucasagomes | things like that | 08:42 |
| *** lipt has joined #openstack-ironic | 08:43 | |
| *** lipt has quit IRC | 08:43 | |
| dtantsur | lucasagomes, yaml and json which both parse to the same python structure | 08:43 |
| dtantsur | thus making it not too important | 08:43 |
| lucasagomes | yeah, I'm honestly think we should support or or another, not both... just more work to keep updating schema and so on | 08:44 |
| lucasagomes | s/I'm/I | 08:44 |
| dtantsur | lucasagomes, it's line 2 more lines of code :/ | 08:44 |
| *** lipt has joined #openstack-ironic | 08:44 | |
| lucasagomes | dtantsur, + the schema file | 08:44 |
| *** lipt has joined #openstack-ironic | 08:45 | |
| dtantsur | lucasagomes, schema validating that 3 top-level keys exist: nodes, ports and chassis? | 08:45 |
| dtantsur | because I hope you don't imply we will validate "node" object on client side | 08:45 |
| dtantsur | firm -2 to it | 08:45 |
| lucasagomes | dtantsur, yeah the schema does that | 08:45 |
| lucasagomes | or should at least (/me having tested much since the last time I played with that script) | 08:46 |
| * dtantsur leaves a review then | 08:47 | |
| dtantsur | so as soon as you drop it, validation will be very simple :) | 08:47 |
| *** vdrok has joined #openstack-ironic | 08:47 | |
| vdrok | morning ironic | 08:47 |
| mgould | morning vdrok | 08:48 |
| vdrok | morning mgould ! | 08:48 |
| dtantsur | lucasagomes, left a review | 08:49 |
| *** mbound has quit IRC | 08:49 | |
| *** lipt has joined #openstack-ironic | 08:50 | |
| dtantsur | lucasagomes, btw I don't know why you need pykwalify there, but its documentation says it does support both json and yaml, so no modifications are needed :) | 08:50 |
| lazy_prince | chenke: vxlan is not in plan yet... | 08:52 |
| lucasagomes | dtantsur, thanks | 08:52 |
| *** lipt has joined #openstack-ironic | 08:52 | |
| *** lipt has quit IRC | 08:52 | |
| lucasagomes | dtantsur, I think that was what came up on google for schema validations to yaml at the time | 08:52 |
| dtantsur | lucasagomes, I would use JSON schema instead, it's already used by ironic and it's in g-r | 08:53 |
| dtantsur | and it does not depend on file format actually, as it works on a dict | 08:53 |
| dtantsur | lemme find an example | 08:53 |
| dtantsur | lucasagomes, schema: https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/node.py#L50-L79 usage: https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/node.py#L570 | 08:54 |
| lucasagomes | dtantsur, cool, will take a look | 08:54 |
| chenke | @lazy_prince, tks! | 08:56 |
| chenke | @lazy_prince, so i could get the ironic vlan support from mitaka? right? | 08:58 |
| *** lipt has joined #openstack-ironic | 08:59 | |
| lazy_prince | chenke: its not yet merged in mitaka Ironic, as per PTL, its going to merge in Newton for sure. If you need it for Mitaka, you can backport in-flight patches to try it out.. | 09:00 |
| *** dtantsur is now known as dtantsur|bbl | 09:00 | |
| lazy_prince | but be warned that it still getting reviewed.. | 09:00 |
| *** lipt has joined #openstack-ironic | 09:02 | |
| divya | morning mgould, vdrok | 09:02 |
| vdrok | morning divya dtantsur|bbl lucasagomes | 09:02 |
| openstackgerrit | Merged openstack/ironic-python-agent: Remove unused SUPPORTED_ROOT_DEVICE_HINTS https://review.openstack.org/323050 | 09:04 |
| chenke | lazy_prince, if refer to tenant work, it need to ironic driver to set up physical switch while deploy? how can we resolve the diversity of the phy switch issue? | 09:04 |
| *** lipt has joined #openstack-ironic | 09:04 | |
| *** bharathk has quit IRC | 09:06 | |
| *** bharathk has joined #openstack-ironic | 09:07 | |
| *** chenke has quit IRC | 09:09 | |
| *** lipt has joined #openstack-ironic | 09:09 | |
| lazy_prince | chenke: each vendor is supposed to provide baremetal support in neutron ml2 driver. You will need to use one which supports your switches. | 09:10 |
| lazy_prince | there is a Generic switch too which can work for your switches using ssh. | 09:11 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add restrictions for changing portgroup-node association https://review.openstack.org/281169 | 09:12 |
| *** sdake has joined #openstack-ironic | 09:13 | |
| *** chenke has joined #openstack-ironic | 09:18 | |
| divya | hi mgould, No compute node record for host Ubuntu14 | 09:18 |
| divya | this error is observed | 09:19 |
| divya | n-cpu.log | 09:19 |
| divya | and hypervisor-list is not listing the details | 09:19 |
| divya | what could be issue | 09:19 |
| divya | plz help | 09:19 |
| *** vmud213 has quit IRC | 09:29 | |
| *** vmud213 has joined #openstack-ironic | 09:30 | |
| divya | No compute node record for host Ubuntu14 | 09:33 |
| divya | how to resolve this issue in ironic | 09:34 |
| openstackgerrit | Merged openstack/ironic: NOTIFICATION_TRANSPORT should be global https://review.openstack.org/323526 | 09:34 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add restrictions for changing portgroup-node association https://review.openstack.org/281169 | 09:39 |
| *** sdake has quit IRC | 09:40 | |
| *** e0ne has quit IRC | 09:41 | |
| *** e0ne has joined #openstack-ironic | 09:41 | |
| *** bharathk has quit IRC | 09:43 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Mock globals on unittests https://review.openstack.org/324332 | 09:46 |
| sambetts|afk | Morning all | 09:51 |
| *** sambetts|afk is now known as sambetts | 09:51 | |
| *** vishwanathj has quit IRC | 09:57 | |
| *** blinky_ghost has joined #openstack-ironic | 09:59 | |
| blinky_ghost | hi all, can anybody tell me if dell poweredge servers r430 supprt pxe_ipmitool driver or which one should I use? | 09:59 |
| *** sacharya has joined #openstack-ironic | 10:00 | |
| *** vmud213 has quit IRC | 10:00 | |
| sambetts | blinky_ghost: does that server support ipmi power on and off? | 10:00 |
| *** vmud213 has joined #openstack-ironic | 10:00 | |
| *** amotoki has quit IRC | 10:01 | |
| blinky_ghost | sambetts: I don't know, how do I test it? ipmitool? | 10:01 |
| sambetts | try something like: ipmitool -I lanplus -H myserver.example.com -P mypass chassis status | 10:02 |
| *** dtantsur|bbl is now known as dtantsur | 10:02 | |
| dtantsur | morning vdrok, sambetts, ifarkas | 10:02 |
| ifarkas | morning dtantsur and all! | 10:02 |
| vdrok | morning sambetts and ifarkas | 10:02 |
| blinky_ghost | sambetts: I get an error: unable to establish IPMI V2 / RMCP+ session | 10:03 |
| *** amotoki has joined #openstack-ironic | 10:04 | |
| *** sacharya has quit IRC | 10:04 | |
| *** derekh has joined #openstack-ironic | 10:05 | |
| lucasagomes | blinky_ghost, you may want to ask cdearborn (not online at the moment) he works for dell | 10:05 |
| lucasagomes | blinky_ghost, but as a guess, it should support ipmi yes, that would be odd not to | 10:05 |
| *** vishwanathj has joined #openstack-ironic | 10:07 | |
| lucasagomes | blinky_ghost, http://www.dell.com/ie/business/p/poweredge-r430/pd it says: "IPMI 2.0 compliant" | 10:07 |
| lucasagomes | you may need to enable it somewhere then | 10:07 |
| blinky_ghost | lucasagomes: I see it on the bios but it doesn't work | 10:09 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 10:10 |
| lucasagomes | blinky_ghost, :-/ firmware bug maybe? | 10:10 |
| *** YiboCai has quit IRC | 10:12 | |
| ifarkas | blinky_ghost, you can also try the pxe_drac driver, the dell page says it has a drac card in it | 10:15 |
| *** hshiina has joined #openstack-ironic | 10:16 | |
| blinky_ghost | ifarkas: testing that also | 10:18 |
| ifarkas | blinky_ghost, for pxe_ipmi you can also check the ipmi settings on the web ui of the management card | 10:20 |
| *** vishwanathj has quit IRC | 10:21 | |
| blinky_ghost | ifarkas: I only see ipmi version 2.0 | 10:21 |
| divya_ | mFixed interval looping call 'nova.virt.ironic.driver.IronicDriver._wait_for_active' failed^[[00m ^[[01;31m2016-06-02 15:49:38.960 TRACE oslo.service.loopingcall ^[[01;35m^[[00mTraceback (most recent call last): ^[[01;31m2016-06-02 15:49:38.960 TRACE oslo.service.loopingcall ^[[01;35m^[[00m File "/usr/local/lib/python2.7/dist-packages/oslo_service/loopingcall.py", line 136, in _run_loop ^[[01;31m2016-06-02 15:49:38.960 TRA | 10:24 |
| divya_ | deploy failing.. with error above | 10:24 |
| divya_ | help plz | 10:24 |
| divya_ | InstanceDeployFailure: Failed to provision instance d5a9aed8-0718-4d63-b253-14336d80b87a: Failed to deploy. Error: 500 Internal Server Error: The server has either erred or is incapable of performing the requested operation. (HTTP 500) ^[[01;31m2016-06-02 15:49:38.960 TRACE oslo.service.loopingcall ^[[01;35m^[[00m | 10:25 |
| *** spandhe has joined #openstack-ironic | 10:25 | |
| *** hoangcx has quit IRC | 10:26 | |
| lucasagomes | divya_, these errors are generic, check ironic-conductor (or nova-compute). Also note it's a 500 error, probably is a server configuration problem | 10:26 |
| divya_ | http://paste.openstack.org/show/507239/ | 10:30 |
| divya_ | ir-cond | 10:30 |
| divya_ | placed in above location | 10:30 |
| blinky_ghost | farkas: found it, I had to activate ipmi over the lan on settings, thanks :) | 10:30 |
| divya_ | http://paste.openstack.org/show/507240/ ir-cond logs | 10:32 |
| divya_ | can you tell know what conf issue | 10:33 |
| *** electrofelix has joined #openstack-ironic | 10:36 | |
| *** amotoki has quit IRC | 10:37 | |
| *** amotoki has joined #openstack-ironic | 10:39 | |
| openstackgerrit | Sam Betts proposed openstack/python-ironicclient: Catch RetriableConnectionFailures from KAuth and retry https://review.openstack.org/323851 | 10:41 |
| sambetts | divya_: it seems like Ironic failed to download the image it needs from glance | 10:42 |
| divya_ | the image is loaded in glance | 10:43 |
| divya_ | stack@Ubuntu14:~/devstack$ glance image-list +--------------------------------------+---------------------------------+ | ID | Name | +--------------------------------------+---------------------------------+ | d344209b-6599-4f5c-967c-87d5d42d46b5 | cirros-0.3.4-x86_64-disk | | 5416536a-9a37-4d05-b257-d53e14fd9301 | cirros-0.3.4-x86_64-uec | | 9fb7c | 10:43 |
| lucasagomes | sambetts, ++ yeah it's failing on download-image | 10:44 |
| lucasagomes | divya_, see the glance services logs | 10:44 |
| divya_ | http://paste.openstack.org/show/507250/ | 10:44 |
| lucasagomes | divya_, logs | 10:45 |
| divya_ | http://paste.openstack.org/show/507251/ | 10:47 |
| divya_ | glance logs are updated here | 10:47 |
| sambetts | check the swift service logs | 10:50 |
| lucasagomes | divya, apparently you configured glance to use swift as backend and swift is not configured | 10:50 |
| *** trown|outtypewww is now known as trown | 10:51 | |
| divya_ | can we use glance without swift configuration | 10:51 |
| lucasagomes | divya_, yes | 10:52 |
| lucasagomes | check the glance docs | 10:52 |
| divya_ | after devstack install.. can i disable swift | 10:52 |
| sambetts | dtantsur: WRT adding ipa- to the kernel params, is it a bug that lldp-timeout doesn't have ipa- prefixed on it? | 10:54 |
| lucasagomes | divya_, you can manually reconfigure it, http://docs.openstack.org/developer/glance/configuring.html#configuring-glance-storage-backends | 10:54 |
| divya_ | thanks lucasagomes.. let me try and get back | 10:57 |
| lucasagomes | divya_, also, please try to dig more into the errors when you see them. Follow the traceback code path, until you get the real cause of it | 10:57 |
| lucasagomes | divya_, you will only get better at reading logs if you actually do it | 10:58 |
| openstackgerrit | Sam Betts proposed openstack/ironic-python-agent: Support LLDP data as part of interfaces in inventory https://review.openstack.org/320584 | 10:59 |
| *** trown is now known as trown|outtypewww | 11:00 | |
| divya_ | sure lucasagomes | 11:00 |
| lucasagomes | cool, thanks | 11:01 |
| *** itamarl has joined #openstack-ironic | 11:05 | |
| *** itamarl has quit IRC | 11:07 | |
| *** amotoki has quit IRC | 11:07 | |
| *** ishant is now known as ishant|away | 11:11 | |
| openstackgerrit | Sam Betts proposed openstack/python-ironicclient: Catch RetriableConnectionFailures from KAuth and retry https://review.openstack.org/323851 | 11:11 |
| *** amotoki has joined #openstack-ironic | 11:15 | |
| *** vdrok has quit IRC | 11:19 | |
| *** sabeen has quit IRC | 11:19 | |
| dtantsur | sambetts, I think so. at least all the other arguments have this prefix | 11:28 |
| sambetts | dtantsur: that what I thought when I looked at it, I'll add a bug | 11:29 |
| sambetts | dtantsur: does IPA have its own lp ? | 11:29 |
| *** aparnav has quit IRC | 11:29 | |
| dtantsur | sambetts, yes | 11:29 |
| sambetts | cool :) | 11:29 |
| vmud213 | Morning all | 11:33 |
| vmud213 | I am having trouble with configuring the grub for fedora secure boot | 11:34 |
| vmud213 | What is the default location fedora looks for a grub config file | 11:35 |
| vmud213 | Should the efi mount point need to be added to /etc/fstab, to make sure the system gets the grub.cfg from the efi partition? | 11:38 |
| *** tangchen has quit IRC | 11:40 | |
| openstackgerrit | Sam Betts proposed openstack/python-ironicclient: Catch RetriableConnectionFailures from KAuth and retry https://review.openstack.org/323851 | 11:40 |
| *** ifarkas has quit IRC | 11:42 | |
| *** ifarkas has joined #openstack-ironic | 11:42 | |
| *** tangchen has joined #openstack-ironic | 11:43 | |
| *** baoli has joined #openstack-ironic | 11:43 | |
| *** baoli has quit IRC | 11:44 | |
| *** baoli has joined #openstack-ironic | 11:44 | |
| *** baoli_ has joined #openstack-ironic | 11:46 | |
| vmud213 | there seems to be an element "ubuntu-signed", while building the images for secure boot using DIB. | 11:46 |
| vmud213 | Is there something on fedora side? | 11:47 |
| dtantsur | vmud213, the only thing I know is that some people were using fedora images with secure boot | 11:47 |
| vmud213 | dtantsur :) | 11:48 |
| *** baoli has quit IRC | 11:49 | |
| vmud213 | dtantsur: I am trying to build an image that has uefi secure boot support. | 11:50 |
| vmud213 | I have created a efi partition and tried to install the related packages. | 11:51 |
| vmud213 | I know that the system while booting looks for the grub configuration file in some path which is on EFI system partition. | 11:52 |
| lucasagomes | vmud213, "shim" should do it | 11:53 |
| vmud213 | lucasagomes: i tried loading shim and shim-signed | 11:53 |
| vmud213 | The problem is when the system is booting, it fall on to grub prompt | 11:54 |
| vmud213 | lucasagomes,dtantsur: Does this mean it's not getting the grub configuration file? | 11:54 |
| *** rloo has joined #openstack-ironic | 11:54 | |
| vmud213 | If so, i placed the file at every place that i can think off :) | 11:55 |
| lucasagomes | vmud213, possibly, or it can't find stage 1.5/2.0 | 11:55 |
| lucasagomes | I think it may very, needs some troubleshooting to figure out why it's dropping to the prompt | 11:55 |
| vmud213 | lucasagomes: what is the best way to debug them..does grub prompt helps any way | 11:56 |
| divya_ | Error: Failed to connect to Glance to get the properties of the image | 11:56 |
| divya_ | http://paste.openstack.org/show/507268/ | 11:56 |
| divya_ | ironic.conf is placed here | 11:56 |
| vmud213 | But the same image works for UEFI non-secure boot | 11:56 |
| vmud213 | lucasagomes,dtantsur: may be i am missing a basic point here, but clear me | 11:59 |
| lucasagomes | vmud213, oh :-/ I'm not really sure tbh | 11:59 |
| * lucasagomes googles a bit | 11:59 | |
| *** sacharya has joined #openstack-ironic | 12:00 | |
| vmud213 | If the node boots up, to be able to read the grub configuration file the file system needs to be mounted. | 12:01 |
| vmud213 | If that is true, and the system looks for the bootloader or configuration file in EFI system partition, then that partition also needs to be mounted by the time it looks for those files | 12:02 |
| *** dprince has joined #openstack-ironic | 12:02 | |
| vmud213 | Am i missing anything here? | 12:02 |
| *** sacharya has quit IRC | 12:05 | |
| lucasagomes | vmud213, that's correct. The only distinction is that, there's no OS running at the time, so grub has the code to read these files from a given filesystem | 12:09 |
| vsaienko | sambetts: morning | 12:10 |
| sambetts | Morning vsaienko | 12:10 |
| *** trown|outtypewww is now known as trown | 12:10 | |
| lucasagomes | which is the stage2 of grub afaict | 12:10 |
| *** jjohnson2 has joined #openstack-ironic | 12:10 | |
| vsaienko | sambetts: looks like I finally fixed cisco CI in enroll node refactoring https://review.openstack.org/#/c/323770/ | 12:10 |
| *** amotoki has quit IRC | 12:11 | |
| sambetts | vsaienko: heh, yeah I'm still working to get the Depends-On tag working in my CI, for some reason zuul wasn't processing it correctly and didn't think your patch had any dependancies | 12:11 |
| vsaienko | dtantsur do we have a spec for importing ironic nodes from file by ironicclient? | 12:12 |
| *** hoonetorg has quit IRC | 12:12 | |
| vmud213 | lucasagmoes: hmm, thanks.. | 12:13 |
| vsaienko | sambetts: I have created a temp patch, and add devstack functions directly in ironic tree | 12:13 |
| *** nico-afk is now known as nicodemos | 12:13 | |
| dtantsur | vsaienko, I don't think we need a spec for it | 12:13 |
| nicodemos | morning, ironic | 12:14 |
| sambetts | vsaienko: I can see, so at least we know it should work :-P | 12:14 |
| *** mgoddard_ has joined #openstack-ironic | 12:14 | |
| *** mgoddard has quit IRC | 12:14 | |
| vsaienko | dtantsur: why not? at least we should discuss file format | 12:15 |
| dtantsur | vsaienko, ironic already defines the format, we should not invent anything | 12:16 |
| dtantsur | i.e. file format: list of things accepted by the node create API | 12:16 |
| dtantsur | here's your spec :) | 12:16 |
| jroll | good morning everyone | 12:16 |
| dtantsur | morning jroll | 12:16 |
| vsaienko | morning jroll | 12:17 |
| lucasagomes | hey jroll | 12:17 |
| dtantsur | I definitely agree we need an RFE for that though | 12:17 |
| jroll | divya_: I'm sorry, but this is a dev channel, not a support channel. a question here and there is okay, but dumping incomplete errors and asking us to solve them is not. you might try #openstack, or the openstack users mailing list, or ask.openstack.org | 12:17 |
| vsaienko | dtantsur: I mean file format it may be stored in yaml or json | 12:17 |
| *** jcoufal has joined #openstack-ironic | 12:18 | |
| *** amotoki has joined #openstack-ironic | 12:18 | |
| *** amotoki has quit IRC | 12:18 | |
| dtantsur | vsaienko, sorry, I'm not getting it. file format is json or yaml. the exact data format is "nodes": [list of node objects accepted by ironic API] | 12:19 |
| TheJulia | Good morning! | 12:19 |
| dtantsur | morning TheJulia | 12:19 |
| lucasagomes | dtantsur, we should also be able to represent ports and chassis and the relation between these resources | 12:19 |
| lucasagomes | I think that's what vsaienko means | 12:19 |
| lucasagomes | how to I represent that node X is a node under chassis Y with the ports A and B | 12:19 |
| lucasagomes | things like that | 12:19 |
| dtantsur | I don't care much about chassis, but ports can be a list inside the node object. | 12:19 |
| lucasagomes | well it's part of the api, I think we should represent it | 12:20 |
| dtantsur | these things have a clear one-to-many relationship, so I still don't see anything complex in it. do you want me to create an RFE? | 12:20 |
| lucasagomes | dtantsur, vsaienko I also think a RFE should be enough for this, the format should be simple enough | 12:20 |
| dtantsur | lemme write an RFE, then fix it as you find convenient | 12:20 |
| vsaienko | dtantsur, lucasagomes https://bugs.launchpad.net/ironic/+bug/1588339 | 12:21 |
| openstack | Launchpad bug 1588339 in Ironic "[RFE] Import nodes from file" [Undecided,New] | 12:21 |
| dtantsur | ah good | 12:21 |
| dtantsur | vsaienko, I'll extend it if you don't mind | 12:22 |
| vsaienko | I think it should go to openstackclient | 12:22 |
| vsaienko | dtantsur sure | 12:22 |
| *** lindycoder has joined #openstack-ironic | 12:23 | |
| jroll | openstackclient++ | 12:24 |
| *** appu103_ has joined #openstack-ironic | 12:25 | |
| dtantsur | jroll, vsaienko, lucasagomes: https://bugs.launchpad.net/ironic/+bug/1588339 | 12:26 |
| openstack | Launchpad bug 1588339 in Ironic "[RFE] Import nodes from file" [Undecided,New] | 12:27 |
| *** hoonetorg has joined #openstack-ironic | 12:27 | |
| dtantsur | it's like a dozen or two lines of code in Python IMO | 12:27 |
| *** thiagop has joined #openstack-ironic | 12:29 | |
| thiagop | Good morning Ironic | 12:29 |
| TheJulia | I would lean towards two dozen, some assembly and data validation required | 12:29 |
| TheJulia | :) | 12:29 |
| TheJulia | Good morning thiagop | 12:29 |
| dtantsur | TheJulia, what do you mean by data validation here? | 12:30 |
| lucasagomes | dtantsur, off-topic, can you take a look at JayF's comment here https://review.openstack.org/#/c/248832/9/ironic_python_agent/extensions/log.py | 12:32 |
| lucasagomes | since it affects inspector I want to see if OK with that idea | 12:32 |
| lucasagomes | if you're* | 12:32 |
| * dtantsur looks | 12:33 | |
| dtantsur | also morning TheJulia, thiagop | 12:33 |
| thiagop | TheJulia dtantsur o/ | 12:33 |
| TheJulia | dtantsur: well, likely a client doing an import should at least check with the API and make sure the node does not already exist before trying to send it. The fallback being what does the enroll process then do, update instead of a completely new post, ignore the node, etc. I'm only thinking of this because of the ansible os_ironic module | 12:34 |
| dtantsur | TheJulia, does node-create do what you describe? treat this new command is a short-cut for node-create + port-create when you have a file available | 12:35 |
| dtantsur | TheJulia, FWIW I've done what you describe in tripleo (re finding a node and updating it) and it's horrible. I would not want us to try do it in ironicclient. | 12:35 |
| *** amotoki has joined #openstack-ironic | 12:36 | |
| lucasagomes | yeah node-create only requires a driver | 12:36 |
| TheJulia | dtantsur: I don't think so, I seem to remember the client just ships the request off to the api and it either fails or does not fail | 12:36 |
| lucasagomes | so it does not check if a node already exist, unless the UUID is passed as part of the command | 12:37 |
| TheJulia | Well, you can post a lot more upon creation, but yeah | 12:37 |
| dtantsur | TheJulia, then that's what we should do here. | 12:37 |
| dtantsur | I would love to have create-or-update semantics, but that's a huge can of worms | 12:37 |
| jroll | dtantsur: ++ | 12:37 |
| TheJulia | dtantsur: ++ | 12:37 |
| lucasagomes | dtantsur, PUT :-) | 12:37 |
| lucasagomes | but yeah... | 12:37 |
| TheJulia | doing it in the ansible modules required thinking through a lot of cases... and the jsonpatch module :) | 12:37 |
| dtantsur | TheJulia, we can probably start sharing this logic between tripleo and bifrost somewhere, but this place is unlikely to be ironicclient :) | 12:38 |
| *** Goneri has joined #openstack-ironic | 12:39 | |
| mgould | morning TheJulia thiagop | 12:39 |
| dtantsur | and I'm pushing tripleo folks to get rid of this magic at least partially tbh | 12:39 |
| thiagop | mgould: o/ | 12:39 |
| dtantsur | so folks, is https://bugs.launchpad.net/ironic/+bug/1588339 something we can agree on doing and then using in our devstack as well? | 12:40 |
| openstack | Launchpad bug 1588339 in Ironic "[RFE] Import nodes from file" [Wishlist,Confirmed] | 12:40 |
| TheJulia | dtantsur: I think I ended up putting most of it in the os_ironic ansible module, but it wouldn't be hard to reproduce in shade as a common working ground. I may have actually put some of it in shade now that I think about it | 12:40 |
| lucasagomes | dtantsur, TheJulia (not proposing it but...) it would be cool actually to be able to pass a whole node representation to the client and let it create the json patches needed to the node to get to that state, basically a diff. And jsonpatch does support it | 12:41 |
| jroll | dtantsur: I still like the sound of it being in openstackclient | 12:41 |
| TheJulia | lucasagomes: that is exactly what I did for ansible support | 12:41 |
| TheJulia | also, there is the problem of not being able to compare passwords, so I had to add a switch for detecting that the password was the only difference :( | 12:42 |
| lucasagomes | TheJulia, heh yeah | 12:42 |
| dtantsur | jroll, do you mean OSC repo itself or our OSC plugin? | 12:43 |
| lucasagomes | TheJulia, the only way around that which I can think of, is having ironic itself to do it via PUT (create the patches internally and call node_update) | 12:43 |
| lucasagomes | but yeah... | 12:43 |
| TheJulia | yeah :( | 12:43 |
| jroll | dtantsur: probably the plugin | 12:44 |
| dtantsur | lucasagomes, it's a good feature, but that's a different feature :) | 12:44 |
| dtantsur | jroll, so just to clarify: only create such command for an OSC plugin, but not our "ironic" CLI? | 12:44 |
| lucasagomes | dtantsur, sure, I made sure I was not proposing it :-) | 12:44 |
| jroll | dtantsur: that's what I lean to, but I'm not sure what we've said about the future of ironic cli | 12:45 |
| dtantsur | I don't think we've said anything, but the whole openstack seems to move the OSC | 12:45 |
| dtantsur | with keystone being the 1st to drop their specific cli | 12:45 |
| jroll | yeah | 12:45 |
| lucasagomes | ++ | 12:45 |
| dtantsur | now: anyone has suggestion for a name for the OSC command then? | 12:46 |
| lucasagomes | yeah I think that's the ultimate goal, all CLI bits in the ironic client goes to OSC | 12:46 |
| dtantsur | I'd love 'openstack baremetal import', but it's occupied by tripleo :( | 12:46 |
| jroll | node import? | 12:46 |
| jroll | dunno | 12:46 |
| jroll | names are the worst | 12:46 |
| lucasagomes | dtantsur, openstack real-baremetal import #jk | 12:46 |
| TheJulia | node import sounds good truthfully | 12:46 |
| * dtantsur grumbles about tripleo hijacking into baremetal namespace | 12:46 | |
| lucasagomes | jroll, TheJulia I really would like it to be more than node | 12:47 |
| TheJulia | because that is what your importing, nodes with possibly attached list of nics | 12:47 |
| dtantsur | that's not only about nodes, is it? | 12:47 |
| lucasagomes | ports, chassis | 12:47 |
| sambetts | seeing as it does the same thing can't we stomp on the tripleo on? | 12:47 |
| jroll | >.> | 12:47 |
| dtantsur | sambetts, well, we'll break tripleo then :) ofc they were asking for a trouble, but it's still a bit... | 12:47 |
| TheJulia | dtantsur: is it just a file formatting thing? | 12:48 |
| * dtantsur looks if any OSC plugin has a similar command | 12:48 | |
| sambetts | I'm thinking that if we make sure we're backwards compatible with the tripleo format for a time with a deprecation warning, then stomping on it would be ok right? | 12:49 |
| dtantsur | "we're backwards compatible with the tripleo format" NOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOO | 12:49 |
| * dtantsur runs away | 12:49 | |
| jroll | hahahaha | 12:49 |
| TheJulia | lol | 12:49 |
| lucasagomes | heh | 12:49 |
| * TheJulia slides a beverage over to dtantsur :) | 12:50 | |
| lucasagomes | more philosophical question, shouldn't "baremetal" the name space be reserved to Ironic project only? | 12:50 |
| dtantsur | sambetts, actually no, I think OSC will get into an undefined state if 2 plugins define the same namespace | 12:50 |
| dtantsur | lucasagomes, I'd say yes, but sigh... | 12:50 |
| dtantsur | remember how many mails have I sent to the ML about it? :) | 12:50 |
| lucasagomes | I find it hard if any project can just create whatever commands within whatever namespace | 12:50 |
| TheJulia | lucasagomes: I have to agree 110% with dtantsur on both of his statements | 12:50 |
| lucasagomes | dtantsur, yeah, well just saying | 12:50 |
| lucasagomes | we need to get a final decision on that | 12:50 |
| dtantsur | we do, but that's a deeper problem that one we're solving now :) | 12:51 |
| dtantsur | 'openstack baremetal load'? | 12:51 |
| TheJulia | that could work, but if the import command exists... then that might get confusing | 12:51 |
| dtantsur | TheJulia, confusing? oh you don't know what confusing mean :D | 12:52 |
| dtantsur | TheJulia, now imagine that tripleo has 'openstack baremetal bulk introspection' command which is VERY different from inspector | 12:52 |
| TheJulia | seems like dual file or data structure format support is going to be required for import. It adds a little code, but it is not that hard to figure out if something is a csv, or yaml, or whatever | 12:52 |
| dtantsur | inspector's 'openstack baremetal introspection start' | 12:52 |
| dtantsur | TheJulia, JSON vs YAML itself is trivial; supporting tripleo format is not at all | 12:53 |
| TheJulia | true | 12:53 |
| *** amotoki has quit IRC | 12:53 | |
| dtantsur | I've changed https://bugs.launchpad.net/ironic/+bug/1588339 to use 'baremetal load'. any better ideas or objections? | 12:54 |
| openstack | Launchpad bug 1588339 in Ironic "[RFE] Import nodes from file" [Wishlist,Confirmed] | 12:54 |
| *** ohamada has quit IRC | 12:57 | |
| *** ohamada has joined #openstack-ironic | 13:01 | |
| xavierr | load is ok :) | 13:01 |
| xavierr | good morning Ironic | 13:01 |
| *** [1]cdearborn has joined #openstack-ironic | 13:02 | |
| dtantsur | morning xavierr | 13:02 |
| rloo | morning all | 13:02 |
| dtantsur | morning rloo :) do you like naming problems? | 13:02 |
| rloo | dtantsur: is it *only* nodes that you want to create from info in a file? | 13:02 |
| rloo | dtantsur: hate naming problems. | 13:02 |
| rloo | dtantsur: hate problems :) | 13:02 |
| dtantsur | rloo, no, nodes, ports and chassis | 13:02 |
| dtantsur | rloo, see the bug for details | 13:03 |
| rloo | dtantsur: and it is all creations? | 13:03 |
| lucasagomes | rloo, xavierr morning | 13:03 |
| dtantsur | rloo, only creations, yes | 13:03 |
| rloo | dtantsur: well, the title sez 'import nodes' :) | 13:03 |
| * dtantsur fixes | 13:03 | |
| dtantsur | fixed: https://bugs.launchpad.net/ironic/+bug/1588339 | 13:03 |
| openstack | Launchpad bug 1588339 in Ironic "[RFE] A command to import Ironic resources from a file" [Wishlist,Confirmed] | 13:03 |
| *** joprovost has joined #openstack-ironic | 13:04 | |
| rloo | dtantsur: i think we have to support CLI and OSC until OSC covers everything that CLI does. | 13:04 |
| dtantsur | jroll, what's the reason to not have it in ironic CLI? | 13:04 |
| rloo | dtantsur: we cannot NOT have it in CLI because it is a bad user experience. we're asking people to do 'ironic X' and 'openstack baremetal Y' | 13:05 |
| rloo | dtantsur: sorry, we can omit it from cli, but bad UX | 13:05 |
| dtantsur | I'm fine with both | 13:06 |
| TheJulia | perhaps this is something to discuss with piet | 13:06 |
| dtantsur | with whom, sorry? | 13:06 |
| TheJulia | since UX is his thing | 13:06 |
| TheJulia | UX PTL | 13:06 |
| dtantsur | we're going a bit too far for a trivial helper command :( | 13:06 |
| rloo | I don't know if that is necessary. as a user, i can imagine the frustration trying to figure out which ironic commands exist where :-( | 13:07 |
| dtantsur | jroll, do you have hard objections to also having 'ironic load' command? | 13:07 |
| TheJulia | I'm suggesting that perhaps it might be good to gain some context beause there were discussions and a study performed at the summit on CLIs | 13:07 |
| dtantsur | I don't want this to involve a cross-project spec or something like that :D | 13:07 |
| TheJulia | oh, jeeze, no | 13:07 |
| * lucasagomes grab popcorn | 13:07 | |
| rloo | TheJulia: i would be fine only OSC, except that our OSC is currently broken and doesn't provide what our CLI does. | 13:07 |
| rloo | lucasagomes loves popcorn as much as beer ;) | 13:08 |
| TheJulia | rloo: I think I would be as well, it would just be... different | 13:08 |
| lucasagomes | rloo, ++ | 13:08 |
| TheJulia | See, I'm jealous, I can't eat popcorn | 13:08 |
| lucasagomes | TheJulia, oh, that's sad :-/ why's that? | 13:09 |
| TheJulia | corn allergy | 13:09 |
| dtantsur | jroll, I have a good idea: if you just quickly approve this RFE we'll maybe stop arguing about it :) | 13:09 |
| jroll | dtantsur: I'm okay with both CLIs, I suppose | 13:09 |
| jroll | lol | 13:09 |
| TheJulia | jroll: trouble maker ;) | 13:09 |
| rloo | dtantsur: hey wait, i'm still thinking about the rfe! | 13:09 |
| jroll | TheJulia: ruby makes good points | 13:09 |
| rloo | dtantsur: the rfe talks about import and export | 13:09 |
| rloo | dtantsur: so it isn't just about creation of resources? | 13:10 |
| dtantsur | wait, where? | 13:10 |
| TheJulia | jroll: indeed :) | 13:10 |
| rloo | dtantsur: first line | 13:10 |
| dtantsur | ouch, that was written by vsaienko | 13:10 |
| * rloo got caught in this trap of looking at an rfe that she didn't plan on doing when she got up this morning | 13:10 | |
| dtantsur | lets have export separate for now | 13:10 |
| * lucasagomes thinks: "someone please fix it" | 13:10 | |
| * dtantsur fixed | 13:11 | |
| dtantsur | we have an export command, it's called ironic node-show :) | 13:11 |
| dtantsur | after getting the --json flag it does the thing | 13:11 |
| *** mbound has joined #openstack-ironic | 13:11 | |
| lucasagomes | dtantsur, ish, no ports *runs* | 13:12 |
| * dtantsur shakes a fist at lucasagomes | 13:12 | |
| rloo | dtantsur: so this only creates resources? what is wrong with 'baremetal create' ? | 13:12 |
| dtantsur | rloo, this is a helper for people who have a file (like tripleo or vsaienko's patch to our devstack plugin) | 13:12 |
| rloo | dtantsur: right, 'openstack baremetal create <filename>' ? | 13:13 |
| *** mbound has quit IRC | 13:13 | |
| rloo | dtantsur: or is 'load' a common term for that? | 13:13 |
| *** mbound has joined #openstack-ironic | 13:13 | |
| dtantsur | rloo, I'd be fine with it, but we already have the deprecated 'baremetal create' :( | 13:13 |
| dtantsur | which is the same as baremetal node create | 13:13 |
| rloo | dtantsur: OH CRAP | 13:13 |
| dtantsur | indeed | 13:13 |
| * rloo sorry that we didn't get a spec for OSC before any code landed | 13:14 | |
| * dtantsur too | 13:14 | |
| sambetts | if we are supporting this sort of bulk operation, can/do we support other bulk opts like ironic port-delete <uuid> <uuid> <uuid> ? | 13:14 |
| dtantsur | sambetts, we already do, at least for node-delete | 13:15 |
| sambetts | cool :) | 13:15 |
| lucasagomes | sambetts, what dtantsur said, it works | 13:15 |
| rloo | while I'm thinking about this, maybe someone(s) could review the first OSC patch: https://review.openstack.org/#/c/284160/ | 13:16 |
| dtantsur | rloo, it's on my radar, as soon as we stop arguing here :) | 13:16 |
| rloo | dtantsur: who's arguing? | 13:17 |
| rloo | dtantsur: am looking for a list of existing OSC commands to see what verbs are being used already | 13:17 |
| dtantsur | well, we are :) not really arguing, rather figuring out naming (which is even worse) :D | 13:17 |
| *** Goneri has quit IRC | 13:17 | |
| rloo | dtantsur: and here I thought we were refining the rfe ;) | 13:17 |
| dtantsur | heh | 13:18 |
| * TheJulia just gets out the can of purple paint and smiles | 13:18 | |
| *** mjturek1 has joined #openstack-ironic | 13:18 | |
| *** PollyZ_ has quit IRC | 13:18 | |
| *** vmud213 has quit IRC | 13:19 | |
| mgould | dtantsur: one of the two hardest problems in computing, along with cache invalidation and off-by-one errors | 13:22 |
| dtantsur | definitely :) | 13:22 |
| rloo | dtantsur: so as far as i can tell, existing OSC commands use 'create' with --file <filename> | 13:24 |
| dtantsur | rloo, we can update baremetal node create with --file too and not accept chassis (for now) | 13:25 |
| rloo | dtantsur: sorry, w/o --filename (was looking at a diff thing) eg: http://docs.openstack.org/developer/python-openstackclient/command-objects/policy.html#policy-create | 13:25 |
| *** appu103_ has quit IRC | 13:25 | |
| *** e0ne has quit IRC | 13:25 | |
| rloo | dtantsur: but yeah, can we use --filename? | 13:25 |
| rloo | dtantsur: can we do 'baremetal create --filename'? | 13:26 |
| *** rbudden has joined #openstack-ironic | 13:26 | |
| dtantsur | IIRC --file is more standard, otherwise I'm fine with that | 13:26 |
| dtantsur | rloo, baremetal create or baremetal node create? | 13:26 |
| rloo | dtantsur: it is for creating more than nodes, right? | 13:27 |
| *** e0ne has joined #openstack-ironic | 13:27 | |
| rloo | dtantsur: i thought you said ironic resources | 13:27 |
| rloo | dtantsur: so baremetal create | 13:27 |
| dtantsur | "baremetal create" creates nodes right now, hence my question | 13:27 |
| rloo | dtantsur: yeah, but we're going to deprecate baremetal create SOON | 13:27 |
| rloo | dtantsur: i'm fine with folks being confused until we delete baremetal create | 13:27 |
| dtantsur | we already did, but we're not removing it for 2 more cycles, right? | 13:27 |
| dtantsur | lol | 13:27 |
| rloo | dtantsur: once that patch lands that i mentioned above, *then* we will have deprecated it. | 13:28 |
| dtantsur | I don't know, what do others think? | 13:28 |
| rloo | dtantsur: if it lands in this cycle, then we delete in p* cycle | 13:28 |
| rloo | dtantsur: this is to create ironic resources, so 'create' is the right verb | 13:29 |
| dtantsur | right. We're probably overcomplicating a simple thing, but I'm fine with whatever name which works for folks | 13:29 |
| rloo | dtantsur: if i had spent this much time when the first osc command went into ironic ... | 13:29 |
| dtantsur | true | 13:29 |
| rloo | dtantsur: fwiw, here's a list of osc commands: http://docs.openstack.org/developer/python-openstackclient/command-list.html | 13:30 |
| *** vdrok has joined #openstack-ironic | 13:30 | |
| dtantsur | lucasagomes, jroll, objections to (re)using 'openstack baremetal create --file <name>'? | 13:30 |
| dtantsur | rloo, btw inspector has separate import and create commands | 13:31 |
| * rloo doesn't hear dtantsur | 13:31 | |
| lucasagomes | dtantsur, hmm idk. Wtihout --file what baremetal create do? | 13:32 |
| lucasagomes | create ? ... a node ? ... a port? | 13:32 |
| dtantsur | lucasagomes, it's a deprecated variant of node create (requires --driver) | 13:32 |
| lucasagomes | dtantsur, if it's already deprecated we shouldn't use/extend it | 13:32 |
| * lucasagomes haven't read the whole scrollback | 13:33 | |
| rloo | lucasagomes: it will be deprecated, because it *should* have been 'baremetal node create' :-( | 13:33 |
| *** xhku has quit IRC | 13:33 | |
| rloo | dtantsur: you want me to update the rfe with my suggestions? | 13:33 |
| dtantsur | rloo, if you can convince lucasagomes, then yes please :) | 13:33 |
| lucasagomes | rloo, I mean, if we are going to deprecate that command why we are using it for this? | 13:34 |
| * dtantsur has found a flaw in our OSC spec in the meanwhile | 13:34 | |
| *** PollyZ_ has joined #openstack-ironic | 13:34 | |
| rloo | lucasagomes: cuz for this particular operation, the creation of baremetal resources, it actually is correct. or the more correct might be 'baremetal create <filename>' | 13:34 |
| *** crinkle has quit IRC | 13:35 | |
| rloo | lucasagomes: the reason why we are going to deprecate 'baremetal create' was cuz it was creating a node only. | 13:35 |
| lucasagomes | rloo, I agree with the version using a positional argument | 13:35 |
| rloo | lucasagomes: and I shake my fist at the cores that approved that :) | 13:35 |
| lucasagomes | rloo, because if we do --file, what happens without it? Shouldn't be mandatory, looks odd | 13:35 |
| lucasagomes | rloo, right yeah "create" == "node create" is def bad | 13:36 |
| rloo | lucasagomes: without it, while 'baremetal create' is deprecated, it will create a node | 13:36 |
| lucasagomes | rloo, what's wrong with "load" that dtantsur suggested? | 13:36 |
| rloo | lucasagomes: 'load' doesn't mean 'create' does it? | 13:36 |
| rloo | lucasagomes: it is a verb used to indicate that something is being loaded. | 13:36 |
| *** crinkle has joined #openstack-ironic | 13:36 | |
| lucasagomes | rloo, it loads a file describing the reosurces into ironic's inventory | 13:37 |
| lucasagomes | rloo, but idk, I don't think "baremetal create --file" is great, I may be missing somehting obvious | 13:37 |
| rloo | lucasagomes: can it load a file describing existing resources? it is a shortcut for creating resources | 13:37 |
| lucasagomes | rloo, nop, only new... yeah :-/ | 13:38 |
| rloo | lucasagomes: cuz our OSC will use 'create' as the verb for creating resources. baremetal node create, baremetal port create, ... this is baremetal create <from file> | 13:38 |
| lucasagomes | rloo, right, "baremetal create <file>" +1 / "baremetal create --file" -1 | 13:39 |
| dtantsur | having an explicit --file is kind of common IIRC | 13:41 |
| rloo | lucasagomes, dtantsur: so can we overload 'baremetal create'. If 'baremetal create <string>' with no --driver, we assume it is a file? | 13:41 |
| dtantsur | --file <file> | 13:41 |
| lucasagomes | dtantsur, right, but I wonder what will be of the "baremetal create" command without it | 13:42 |
| lucasagomes | afaiui, it will be deprecated right? | 13:42 |
| dtantsur | hmm, maybe I'm wrong, wait | 13:42 |
| rloo | lucasagomes: right. so i think you have a good point. i added --file cuz of existing command. | 13:42 |
| rloo | lucasagomes: but maybe we can have both, cuz the existing one needs to have --driver specified. | 13:43 |
| dtantsur | rloo, lucasagomes is right, --file is not used by other commands (like object create) | 13:43 |
| dtantsur | I've confused it with a 'save' command, sorry | 13:43 |
| lucasagomes | rloo, dtantsur baremetal resources import ? | 13:43 |
| lucasagomes | baremetal add ? | 13:43 |
| dtantsur | STAAHP :D lets settle on create then | 13:43 |
| dtantsur | create is OSC standard | 13:43 |
| lucasagomes | ok | 13:43 |
| dtantsur | example: http://docs.openstack.org/developer/python-openstackclient/command-objects/object.html#object-create | 13:43 |
| lucasagomes | resource screate? | 13:43 |
| lucasagomes | resources create* | 13:44 |
| dtantsur | lucasagomes, we don't have a notion of 'baremetal resource', so no | 13:44 |
| rloo | lucasagomes: if 'baremetal create' didn't already exist, what would you *think* this new command should be. | 13:44 |
| sambetts | openstack baremental inventory import ? | 13:44 |
| lucasagomes | rloo, if it didn't already exist I would just use it | 13:45 |
| rloo | lucasagomes: ok, lets just use it then. | 13:45 |
| lucasagomes | baremetal create <file> (as a positional argument) | 13:45 |
| lucasagomes | ok | 13:45 |
| lucasagomes | I'm good with it | 13:45 |
| rloo | lucasagomes: i think that's the right thing to do. i assume we can code it appropriately. | 13:45 |
| dtantsur | rloo, please update the RFE if you don't mind and we'll check if jroll has already run away from us | 13:45 |
| rloo | dtantsur: ^ you OK with that? I think it means we should land the OSC patch first. | 13:45 |
| *** mgoddard__ has joined #openstack-ironic | 13:46 | |
| rloo | dtantsur: ok, will update. | 13:46 |
| dtantsur | rloo, I've approved the OSC patch already | 13:46 |
| rloo | dtantsur: YAY | 13:46 |
| vdrok | morning rloo and jroll | 13:47 |
| NobodyCam | good morning Ironicers | 13:47 |
| vdrok | morning NobodyCam | 13:47 |
| NobodyCam | morning rloo lucasagomes dtantsur vdrok | 13:47 |
| NobodyCam | :) | 13:47 |
| rloo | hi vdrok, morning NobodyCam | 13:47 |
| *** mgoddard_ has quit IRC | 13:49 | |
| dtantsur | morning NobodyCam | 13:49 |
| NobodyCam | :) | 13:49 |
| *** Goneri has joined #openstack-ironic | 13:52 | |
| jroll | dtantsur: sorry, was doing some things around home :) | 13:52 |
| *** ametts has joined #openstack-ironic | 13:53 | |
| * jroll still made it back by 10am, woot | 13:53 | |
| NobodyCam | morning jroll :) | 13:53 |
| dtantsur | jroll, I was imaging how you got fed up by this discussion and ran away :D | 13:54 |
| lucasagomes | NobodyCam, morning | 13:55 |
| NobodyCam | :) | 13:55 |
| jroll | dtantsur: I didn't say why I went off to do other things :) | 13:56 |
| jroll | morning NobodyCam :D | 13:56 |
| dtantsur | lol | 13:56 |
| jroll | dtantsur: so, do you need me to do anything about it, or waiting for rloo to update? | 13:58 |
| rloo | jroll, dtantsur: I've added my comments | 13:58 |
| dtantsur | rloo, please update the description instead | 13:58 |
| dtantsur | otherwise it's unclear what we've settled upon | 13:59 |
| dtantsur | (and apparently we've settled on 'baremetal create <file>') | 13:59 |
| dtantsur | rloo, as to "I wonder if it makes more sense to stop at the first error.", we don't do it with deletion, and anyway creating several resources it not atomic | 14:00 |
| *** mjturek1 has quit IRC | 14:00 | |
| rloo | dtantsur: so deletion is easier, you try to delete if it doesn't work, it doesn't affect the other deletes. | 14:00 |
| *** ohamada has quit IRC | 14:00 | |
| dtantsur | neither does creating, no? | 14:00 |
| rloo | dtantsur: with creation, if you can't create one of the ports, should the node be created? | 14:00 |
| sambetts | you need to node created to create the port right? | 14:01 |
| dtantsur | rloo, node is already created at that moment. so your question is: should the node be deleted? | 14:01 |
| dtantsur | I'd answer "no" | 14:01 |
| rloo | dtantsur: it might be simple / easy, cuz i hadn't thought much about it. but maybe someone wouldn't want something created cuz something else failed. dunno. | 14:01 |
| *** sacharya has joined #openstack-ironic | 14:01 | |
| rloo | dtantsur: oh, right, forgot the order. | 14:01 |
| rloo | dtantsur: so node is created, a port fails. not sure what the reason is. should we try to create the rest of the ports, or stop doing anything with that node? | 14:02 |
| mgould | morning NobodyCam | 14:02 |
| NobodyCam | morning mgould :) | 14:02 |
| dtantsur | rloo, for simplicity and consistency I would just go ahead with other ports | 14:03 |
| dtantsur | rloo, background here: I got a complain from UI folks that tripleo import command stops at first failure, so they end up with some random number of resources created | 14:04 |
| *** mjturek1 has joined #openstack-ironic | 14:05 | |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal power state commands https://review.openstack.org/172517 | 14:05 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Add maintenance mode commands https://review.openstack.org/216727 | 14:05 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal port create https://review.openstack.org/172461 | 14:05 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Add provision state commands https://review.openstack.org/206119 | 14:05 |
| rloo | and it would be better for the import to contineu? | 14:05 |
| *** ayoung has joined #openstack-ironic | 14:05 | |
| *** sacharya has quit IRC | 14:06 | |
| dtantsur | rloo, they'd prefer it, yes. so that the result is "import whatever is possible", not "import something" | 14:06 |
| rloo | dtantsur: ok, then that is a good reason for continuing :) | 14:06 |
| rloo | dtantsur: I've updated the description. Do you want me to discuss the overlap of 'baremetal create' in detail in the description? | 14:07 |
| *** rama_y has joined #openstack-ironic | 14:07 | |
| * dtantsur reads | 14:07 | |
| dtantsur | rloo, it's detailed enough to my taste | 14:08 |
| rloo | dtantsur: OK. we can always update later if need be. | 14:08 |
| dtantsur | jroll, now we're ready to summon you for evaluating https://bugs.launchpad.net/ironic/+bug/1588339 :) | 14:08 |
| openstack | Launchpad bug 1588339 in Ironic "[RFE] A command to import Ironic resources from a file" [Wishlist,Confirmed] | 14:08 |
| * jroll refreshes | 14:09 | |
| jroll | rloo: what if I have a file named "node" - "openstack baremetal create node" | 14:10 |
| jroll | this is gonna get confusing :) | 14:11 |
| rloo | jroll: i thought the file needed an extension. | 14:11 |
| jroll | nope | 14:11 |
| rloo | jroll: and even if it doesn't need an extension, it needs somewhat correct info inside it | 14:11 |
| jroll | $ cat node | 14:11 |
| jroll | hi rloo | 14:11 |
| jroll | sure | 14:11 |
| jroll | just pointing out some ambiguity that's going to be difficult to handle | 14:11 |
| rloo | jroll: the rfe sez '...accepting a JSON or YAML file (detected by extension)' | 14:12 |
| jroll | ah | 14:12 |
| *** links has quit IRC | 14:12 | |
| jroll | O-o | 14:12 |
| jroll | O_o | 14:12 |
| rloo | dtantsur: ^^ maybe you need to be explicit and mention what the valid extensions are | 14:13 |
| dtantsur | oh, that's full stop | 14:13 |
| jroll | I totally missed the extension bit btw | 14:13 |
| dtantsur | ah no, wait | 14:13 |
| lucasagomes | JayF, around? | 14:13 |
| dtantsur | we have baremetal node create, not baremetal create node | 14:13 |
| dtantsur | so from OSC side it's fine | 14:13 |
| jroll | ohhh | 14:13 |
| jroll | ok | 14:13 |
| lucasagomes | JayF, you remmeber the name of the service you have to run to proxy the journal ? | 14:13 |
| rloo | dtantsur: RIGHT! | 14:14 |
| dtantsur | otherwise we would not be able to create the new command at all | 14:14 |
| *** _vsaienko has joined #openstack-ironic | 14:14 | |
| *** _vs has joined #openstack-ironic | 14:14 | |
| dtantsur | I've just detected that it's not possible to create baremetal node create port, which we agreed on in this spec | 14:14 |
| dtantsur | * the spec | 14:14 |
| *** mjturek1 has quit IRC | 14:14 | |
| dtantsur | because of the conflict with baremetal node create | 14:14 |
| dtantsur | but here it should be fine | 14:14 |
| jroll | so 'if driver in args: create a node; else: assume a file' | 14:14 |
| dtantsur | jroll, simply speaking, yes. maybe we can even express it in terms of argparse (they have mutual exclusive arguments) | 14:15 |
| jroll | nod | 14:16 |
| jroll | ok I'm good with this | 14:16 |
| *** ohamada has joined #openstack-ironic | 14:16 | |
| * jroll leaves a comment | 14:16 | |
| openstackgerrit | Merged openstack/python-ironicclient: Bring OSC plugin inline with approved spec https://review.openstack.org/284160 | 14:16 |
| *** appu103_ has joined #openstack-ironic | 14:17 | |
| dtantsur | woohoo | 14:17 |
| dtantsur | jroll, does it count as rfe-approved? :) | 14:18 |
| jroll | if lucasagomes is happy let's approve it | 14:18 |
| lucasagomes | what's the final decision ? baremetal create <file> ? | 14:19 |
| lucasagomes | if so, +1 | 14:19 |
| dtantsur | yes | 14:19 |
| lucasagomes | ok | 14:19 |
| *** ayoung has quit IRC | 14:19 | |
| rloo | dtantsur: what's the response for that? | 14:19 |
| dtantsur | rloo, sorry, for what? | 14:20 |
| rloo | dtantsur: the new command | 14:20 |
| lucasagomes | rloo, return code? | 14:20 |
| lucasagomes | 0 success, != 0 error | 14:20 |
| dtantsur | rloo, nothing for success, error messages for any error | 14:20 |
| lucasagomes | I wonder tho, if _some_resources fail to be create, what is the ret code? | 14:20 |
| lucasagomes | since we are going to continue creating the rest | 14:21 |
| lucasagomes | dtantsur, ^ ? | 14:21 |
| dtantsur | lucasagomes, error code, just as with deletion already | 14:21 |
| lucasagomes | ack | 14:21 |
| lucasagomes | yeah | 14:21 |
| dtantsur | we can return different status codes in these cases, but that's all | 14:21 |
| rloo | dtantsur, lucasagomes: when we create something now, do we return the uuid of the created resource? | 14:21 |
| dtantsur | rloo, we return the whole representation | 14:22 |
| lucasagomes | rloo, in ironic the whole object repr is returned | 14:22 |
| rloo | ok, that's my question for this new command. | 14:22 |
| lucasagomes | but that's only create 1 resources, this new api can create many | 14:22 |
| rloo | what do we return? | 14:22 |
| jroll | I feel like we should at least list UUIDs for things we create | 14:22 |
| dtantsur | jroll, UUIDs to stdout, errors to stderr, right? | 14:22 |
| lucasagomes | so, I would say we shouldn't return anything? Only log the errors | 14:22 |
| lucasagomes | or UUIDs yeah, that's good | 14:22 |
| jroll | dtantsur: makes sense to me | 14:23 |
| dtantsur | ok | 14:23 |
| lucasagomes | {'nodes': [uuid, uuid], 'ports': [uuid, uuid, ...} | 14:23 |
| lucasagomes | like this ^ ? | 14:23 |
| jroll | I was thinking 'created node <uuid>\ncreated port <uuid> for node <uuid>' | 14:23 |
| jroll | or something | 14:23 |
| jroll | I'm not opinionated | 14:23 |
| dtantsur | yeah, lets keep it human-readable maybe | 14:23 |
| betherly | jroll: sorry only just getting round to amending the ironic docs. wrt http://docs.openstack.org/developer/ironic/ shall i add 'Ironic UI docs' to the Administrators guide list? | 14:24 |
| jroll | betherly: sounds good to me | 14:24 |
| rloo | betherly: what does 'UI' mean again? | 14:24 |
| jroll | dtantsur: ye, scripts can just hit node-list again or something | 14:24 |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Added get_server_profile_by_uuid https://review.openstack.org/302225 | 14:24 |
| betherly | rloo: user interface :) | 14:25 |
| rloo | betherly: which means what? | 14:25 |
| betherly | jroll: wanted to wait until things had merged to create the docs for ironic-ui first then totally forgot to update the ironic docs *facepalm | 14:25 |
| rloo | betherly: is it web UI? | 14:25 |
| jroll | betherly: no worries | 14:25 |
| *** rama_y has quit IRC | 14:26 | |
| lucasagomes | dtantsur, jroll +1 I was thinking about dict/list because at least in the ironic client we pass it to prettytable and it makes it human readable | 14:26 |
| jroll | sure, the method can return a dict, I'm not concerned with internals | 14:26 |
| rloo | betherly: it is the webclient, right? | 14:26 |
| * lucasagomes also don't know if --json is a thing on osc either | 14:26 | |
| jroll | (oh man, are we adding this to python sdk too) | 14:26 |
| dtantsur | lucasagomes, it is | 14:26 |
| betherly | rloo: yes. it allows you to use ironic using a more user friendly interface | 14:26 |
| jroll | lucasagomes: it is, -f json | 14:26 |
| lucasagomes | jroll, dtantsur ah, nice | 14:26 |
| lucasagomes | ok :-) all good | 14:27 |
| rloo | betherly: would it be clear to put web client then? | 14:27 |
| rloo | betherly: this is in the admin part, not the developer part. i don't know what terminology would be clearer for admins, but i don't think we should use abbreviations like 'ui' or at least spell it out with (UI) | 14:28 |
| betherly | rloo: no because the webclient is a different thing | 14:28 |
| rloo | betherly: OH. so what is the UI then? | 14:28 |
| *** _vsaienko has quit IRC | 14:28 | |
| *** _vs has quit IRC | 14:28 | |
| betherly | rloo: the webclient is the standalone for ironic that krotscheck focuses on | 14:28 |
| betherly | rloo: the ironic ui is a horizon plugin that enables users to use ironic within horizon | 14:29 |
| rloo | betherly: what UI docs are you going to add in? | 14:29 |
| *** spandhe_ has joined #openstack-ironic | 14:29 | |
| rloo | betherly: OH. | 14:29 |
| rloo | betherly: is it *only* a plugin for horizon? | 14:29 |
| betherly | rloo: it will be a link to http://docs.openstack.org/developer/ironic-ui/ which has further changes submitted for review here https://review.openstack.org/#/c/324353/ | 14:30 |
| betherly | rloo: yes. the ironic-ui is a project that is designed solely as a plugin for horizon. | 14:30 |
| *** spandhe has quit IRC | 14:30 | |
| *** spandhe_ is now known as spandhe | 14:30 | |
| betherly | rloo: the webclient is the standalone interface for users who want an interface without horizon | 14:31 |
| jroll | maybe call it "ironic horizon plugin" then? | 14:31 |
| jroll | er, 'ironic horizon plugin docs' | 14:31 |
| rloo | betherly: sorry, I'm wondering why you called it 'ironic-ui' but that's fine, it is a project/repo name. | 14:31 |
| *** _vsaienko has joined #openstack-ironic | 14:31 | |
| *** _vs has joined #openstack-ironic | 14:32 | |
| rloo | betherly: I looked at the governance, horizon is a 'Dashboard'. So in the admin section, put 'Dashboard' | 14:32 |
| betherly | rloo: ye i just refer to it as that so theres no confusion between the webclient and the ui and such. its slightly confusing as it is :) | 14:33 |
| jroll | lucasagomes: dtantsur: going to email about it later today, but thought I'd show you https://trello.com/b/ROTxmGIc/ironic-newton-priorities | 14:33 |
| rloo | betherly: well, how about 'Dashboard (horizon) plugin'. Docs folks can let us know if they don't like seeing horizon there but that index page isn't official documentation. | 14:33 |
| rloo | betherly: yeah, you can see I was confused too. but horizon plugin clarifies it :) | 14:34 |
| rloo | betherly: but docs folks don't like us using project names, we should use official service names, for user-facing docs. hence the 'dashboard' part. | 14:34 |
| jroll | dtantsur: also, I want to release all the stable things today, any objections? (other than the client release you did this week already) | 14:34 |
| betherly | jroll: rloo: are you happy for me to keep the naming of the ironic-ui docs as 'Ironic UI Documentation' or should i be changing that also? Also can you decide what you want me to definitely call the link in the ironic docs :) | 14:35 |
| *** amotoki has joined #openstack-ironic | 14:35 | |
| *** rloo has quit IRC | 14:35 | |
| betherly | rloo: ah ok that makes sense :) | 14:35 |
| *** rloo has joined #openstack-ironic | 14:35 | |
| *** ayoung has joined #openstack-ironic | 14:36 | |
| rloo | betherly: i'd prefer if you clarified in your docs, what it means. a webclient is also UI. | 14:37 |
| rloo | betherly: wrt the description of the link from the admin section, i think 'Dashboard (horizon) plugin' is ok | 14:37 |
| betherly | rloo: ok cool i will do that. | 14:38 |
| rloo | thx betherly | 14:39 |
| * krotscheck really needs to find time to do ironic-webclient things :( :( | 14:39 | |
| krotscheck | argh infra taking up all my time. | 14:39 |
| lucasagomes | jroll, cool | 14:40 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal power state commands https://review.openstack.org/172517 | 14:40 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Add maintenance mode commands https://review.openstack.org/216727 | 14:40 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal port create https://review.openstack.org/172461 | 14:40 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Add provision state commands https://review.openstack.org/206119 | 14:40 |
| openstackgerrit | Beth Elwell proposed openstack/ironic: Added ironic-ui horizon dashboard plugin to ironic docs https://review.openstack.org/324574 | 14:40 |
| * lucasagomes likes trello | 14:41 | |
| betherly | rloo: ive left the title for the ironic-ui docs as it is so the name doesnt become ridiculously long. the first line of the introduction explains that it is the horizon dashboard plugin. | 14:42 |
| *** mtanino has joined #openstack-ironic | 14:42 | |
| rloo | betherly: do what you think is right/better. I honestly don't have time to look at your docs. | 14:42 |
| betherly | rloo: sure. just wanted to let you know what i had decided since we had discussed. | 14:43 |
| rloo | betherly: :) thx | 14:43 |
| *** _vsaienko has quit IRC | 14:43 | |
| dtantsur | jroll, wow for trello, ack for release all the things | 14:43 |
| *** _vs has quit IRC | 14:43 | |
| dtantsur | jroll, oh, one thing about IPA | 14:44 |
| jroll | dtantsur: running to meeting for 10m but fire away | 14:44 |
| dtantsur | jroll, could you weigh in on this backport first: https://review.openstack.org/323723 ? | 14:44 |
| dtantsur | if we agree to do it, I'd love it to be released too | 14:44 |
| jroll | oh yeah, sure | 14:44 |
| *** david-lyle has joined #openstack-ironic | 14:44 | |
| jroll | agree new config isn't auto-reject | 14:44 |
| *** devananda has quit IRC | 14:45 | |
| dtantsur | jroll, yeah, also tony told me it's possible | 14:45 |
| jroll | right | 14:45 |
| dtantsur | jroll, btw how do I join https://trello.com/b/ROTxmGIc/ironic-newton-priorities (and should I)? | 14:46 |
| *** appu103_ has quit IRC | 14:47 | |
| jroll | dtantsur: I'll add at least specs cores today | 14:49 |
| dtantsur | good | 14:49 |
| jroll | probably all ironic core | 14:50 |
| *** adu has joined #openstack-ironic | 14:50 | |
| NobodyCam | ++ | 14:51 |
| *** _vsaienko has joined #openstack-ironic | 14:52 | |
| *** _vs has joined #openstack-ironic | 14:52 | |
| JayF | lucasagomes: journal-to-syslog or something like that | 14:53 |
| JayF | lucasagomes: then you run a syslogger | 14:53 |
| JayF | lucasagomes: the reason I hedged on all my comments is I'm slightly afraid the coreos pxe might not have the service in it | 14:53 |
| lucasagomes | JayF, oh right, yeah I saw that you can configure journald to foward to syslog | 14:53 |
| lucasagomes | and then use rsyslog etc | 14:53 |
| JayF | yeah exactly | 14:53 |
| JayF | and then logging is "normal" | 14:53 |
| lucasagomes | yup | 14:53 |
| JayF | I just really want to keep distro/ramdisk specific code out of IPA python | 14:53 |
| JayF | there's another bugfix review I have a -1 on for the same reason | 14:54 |
| lucasagomes | JayF, problem being, this will still change inspector behavior cause right now they only have one file "journal" that contains the logs from the whole system there | 14:54 |
| lucasagomes | (not arguing if that's good or bad) | 14:54 |
| JayF | how is that file created? | 14:54 |
| JayF | journalctl > file? | 14:54 |
| lucasagomes | JayF, running journaldctl | 14:54 |
| lucasagomes | yes | 14:54 |
| * JayF cringes | 14:54 | |
| JayF | We should replace that with proper syslog integration and deprecate it. You could configure a syslogger to create that file to keep the interface | 14:55 |
| dtantsur | JayF, hi! re https://review.openstack.org/323723 new config options are not blockers for stable, changing existing options is | 14:55 |
| lucasagomes | JayF, https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/inspector.py#L336 | 14:55 |
| lucasagomes | dtantsur, ^ would you be OK with it? | 14:55 |
| dtantsur | lucasagomes, I'm fine as soon as I still get 1. system logs, 2. IPA logs | 14:56 |
| dtantsur | keeping the same file name is nice, though not absolutely required | 14:56 |
| JayF | dtantsur: I had a backport -1'd for adding a config option | 14:56 |
| *** devananda has joined #openstack-ironic | 14:56 | |
| JayF | dtantsur: not just for modifying one | 14:56 |
| JayF | dtantsur: so there's something inconsistent there, we should check the policy | 14:56 |
| dtantsur | JayF, we've clarified policy since then; I've also asked stable folks | 14:57 |
| lucasagomes | dtantsur, I mean, you will get everything but the services logs will be each one on it's own file | 14:57 |
| jroll | JayF: yeah, stable PTL clarified it's okay for us recently | 14:57 |
| dtantsur | lucasagomes, I think we can live with it | 14:57 |
| jroll | JayF: I'd be curious which backport that was | 14:57 |
| JayF | dtantsur: re: logs -- it would make the logs be in "normal sysloggy places" if that makes sense | 14:57 |
| lucasagomes | dtantsur, cause I'm going to proxy the journal log to /var/log/ and when collect I will zip the whole /var/log directory | 14:57 |
| JayF | jroll: I honestly don't remember; it's something I cared about a little but might not have been my patch | 14:57 |
| jroll | right on | 14:57 |
| JayF | jroll: just carried over the memory of that | 14:57 |
| lucasagomes | dtantsur, ok, so you don't depend on a tar.gz having a file called "journal" there? | 14:57 |
| dtantsur | lucasagomes, there is no code dependency on the tar.gz contents. inspector does not even unpack it | 14:58 |
| lucasagomes | dtantsur, perfect | 14:58 |
| lucasagomes | ok thanks | 14:58 |
| lucasagomes | I will see if I can get the coreos to do it | 14:58 |
| *** mjura has quit IRC | 14:59 | |
| *** appu103_ has joined #openstack-ironic | 14:59 | |
| *** spandhe has quit IRC | 15:00 | |
| *** appu103_ has quit IRC | 15:00 | |
| *** appu103_ has joined #openstack-ironic | 15:01 | |
| jroll | JayF: to clarify further, any backward-compatible config change is okay | 15:02 |
| JayF | jroll: yeah; I'm 100% fine with that, I just misunderstood the policy | 15:03 |
| *** spandhe has joined #openstack-ironic | 15:03 | |
| jroll | totally | 15:04 |
| jroll | I just wanted to clarify changing them can be okay | 15:04 |
| jroll | dtantsur: +W | 15:04 |
| jroll | I'll wait for that to land and release all the things | 15:04 |
| dtantsur | thanks! | 15:05 |
| rloo | vsaienko: where does DEVSTACK_GATE_TEMPEST_BAREMETAL_BUILD_TIMEOUT come from, wrt https://review.openstack.org/#/c/315496/8/devstack-vm-gate.sh | 15:05 |
| *** pas-ha has quit IRC | 15:05 | |
| jroll | some specs cores mind reviewing the rescue spec (which I'm about to do)? it's been sitting for a while: https://review.openstack.org/#/c/171878/ | 15:05 |
| jroll | ditto for https://review.openstack.org/#/c/321153/ | 15:06 |
| jroll | (hasn't been sitting as long but is small) | 15:06 |
| *** _vsaienko has quit IRC | 15:06 | |
| *** _vs has quit IRC | 15:06 | |
| vsaienko | rloo: https://review.openstack.org/#/c/315545 | 15:06 |
| rloo | vsaienko: heh, that one. any reason for using BAREMETAL instead of IRONIC? | 15:07 |
| *** pas-ha has joined #openstack-ironic | 15:07 | |
| *** ildikov has quit IRC | 15:07 | |
| *** mgkwill has quit IRC | 15:07 | |
| *** agordeev has quit IRC | 15:07 | |
| *** vdrok-afk has quit IRC | 15:07 | |
| *** patrickeast has quit IRC | 15:08 | |
| vsaienko | rloo: in tempest.conf it is an option from baremetal section | 15:08 |
| vsaienko | that was the reason why I use baremetal and not ironic | 15:08 |
| *** mgkwill has joined #openstack-ironic | 15:09 | |
| *** ildikov has joined #openstack-ironic | 15:09 | |
| *** agordeev has joined #openstack-ironic | 15:09 | |
| rloo | vsaienko: sigh. | 15:09 |
| *** patrickeast has joined #openstack-ironic | 15:09 | |
| *** vdrok-afk has joined #openstack-ironic | 15:10 | |
| *** appu103_ has quit IRC | 15:14 | |
| *** appu103_ has joined #openstack-ironic | 15:15 | |
| openstackgerrit | Sam Betts proposed openstack/ironic: [DO NOT MERGE] Testing Cisco CI https://review.openstack.org/289312 | 15:30 |
| *** penick has quit IRC | 15:33 | |
| *** vdrok has quit IRC | 15:34 | |
| *** mjturek1 has joined #openstack-ironic | 15:35 | |
| *** ChrisAusten has joined #openstack-ironic | 15:36 | |
| *** praneshp has joined #openstack-ironic | 15:43 | |
| thiagop | jroll: people in the infra is very busy today so... have you ever faced a failure on the publish-to-pypi job? | 15:45 |
| sambetts | thiagop: what sort of failure? | 15:45 |
| sambetts | thiagop: I've seen problems when pypi has had a glitch or if the pypi repo isn't setup with the right permissions | 15:46 |
| thiagop | timeout building venv, don't know why (the logs don't say nothing more). I'm looking for a way to run it again | 15:46 |
| *** sacharya has joined #openstack-ironic | 15:46 | |
| jroll | thiagop: no, but I'd say ask infra or release teams | 15:46 |
| *** sacharya has quit IRC | 15:46 | |
| jroll | thiagop: we can't manually run them, if that answers your question | 15:46 |
| jroll | thiagop: send an email if they're busy | 15:46 |
| thiagop | jroll: I'll really have to push another tag? ¬¬ | 15:47 |
| thiagop | jroll: thanks for the tip | 15:47 |
| jroll | thiagop: no, don't push a new tag... | 15:48 |
| jroll | email infra and ask for help, they will help you | 15:48 |
| thiagop | jroll: roger roger | 15:48 |
| *** Goneri has quit IRC | 15:50 | |
| openstackgerrit | Merged openstack/bifrost: introduce install_dib varible https://review.openstack.org/318422 | 15:52 |
| openstackgerrit | Sam Betts proposed openstack/ironic: [DO NOT MERGE] Testing Cisco CI https://review.openstack.org/289312 | 15:54 |
| openstackgerrit | Merged openstack/bifrost: Make booleans in templates explicit https://review.openstack.org/321151 | 15:56 |
| *** mgoddard__ has quit IRC | 15:57 | |
| *** joprovost has quit IRC | 15:58 | |
| *** tesseract- has quit IRC | 15:59 | |
| *** mgoddard has joined #openstack-ironic | 15:59 | |
| *** hshiina has quit IRC | 16:00 | |
| *** rcernin has quit IRC | 16:01 | |
| *** e0ne has quit IRC | 16:06 | |
| dtantsur | folks, a pretty small spec for your consideration: https://review.openstack.org/#/c/305864/ (inspector boot management) | 16:08 |
| dtantsur | this blocks people who want to use inspector with virtual media | 16:08 |
| dtantsur | (e.g. HPE) | 16:08 |
| openstackgerrit | Merged openstack/bifrost: Fix Bug #1583539 - rpm part https://review.openstack.org/321311 | 16:08 |
| openstack | bug 1583539 in bifrost "Missing dependencies in documentation and scripts for a CentOS 7 install of bifrost" [Undecided,New] https://launchpad.net/bugs/1583539 | 16:08 |
| *** ayoung has quit IRC | 16:09 | |
| *** ayoung has joined #openstack-ironic | 16:09 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add description filed to ports - DB https://review.openstack.org/324644 | 16:12 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add description field to ports - DB https://review.openstack.org/324644 | 16:13 |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/323889 | 16:16 |
| *** mgoddard_ has joined #openstack-ironic | 16:19 | |
| lucasagomes | JayF, just want to give you an update before I go home. No dice on the coreos; rsyslog, sysklogd, syslog-ng are not included in the base image | 16:20 |
| lucasagomes | JayF, but that makes me think as well, do we need coreos at all now that we run things in a chroot? | 16:20 |
| lucasagomes | JayF, maybe we just use it because of the tooling to create the image, right? | 16:20 |
| *** amotoki has quit IRC | 16:20 | |
| dtantsur | see you tomorrow, g'night | 16:21 |
| mgould | good night dtantsur | 16:21 |
| rloo | bye dtantsur | 16:22 |
| *** mgoddard has quit IRC | 16:22 | |
| *** amotoki has joined #openstack-ironic | 16:22 | |
| *** dtantsur is now known as dtantsur|afk | 16:23 | |
| *** david-lyle has quit IRC | 16:24 | |
| lucasagomes | dtantsur|afk, bye bye | 16:24 |
| *** mgoddard_ has quit IRC | 16:24 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add nodes tagging support - objs https://review.openstack.org/233357 | 16:24 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add nodes tagging support - API https://review.openstack.org/250478 | 16:24 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add DB layer support for node tags filter https://review.openstack.org/272624 | 16:24 |
| *** praneshp has quit IRC | 16:26 | |
| *** sacharya has joined #openstack-ironic | 16:27 | |
| *** amotoki has quit IRC | 16:32 | |
| *** penick has joined #openstack-ironic | 16:33 | |
| *** mjturek1 has quit IRC | 16:34 | |
| *** Goneri has joined #openstack-ironic | 16:36 | |
| *** appu103_ has quit IRC | 16:36 | |
| *** appu103_ has joined #openstack-ironic | 16:36 | |
| jroll | dtantsur|afk: or anyone else, I think this is a dupe but can't find it, can you help? https://bugs.launchpad.net/nova/+bug/1575661 | 16:37 |
| openstack | Launchpad bug 1575661 in OpenStack Compute (nova) "can not deploy a partition image to Ironic node" [Undecided,Incomplete] | 16:37 |
| jroll | lucasagomes: ^^ | 16:37 |
| lucasagomes | jroll, looking | 16:38 |
| *** aparnav has joined #openstack-ironic | 16:39 | |
| *** appu103_ has quit IRC | 16:39 | |
| lucasagomes | jroll, maybe partially duplicated with https://bugs.launchpad.net/nova/+bug/1477490 ? | 16:40 |
| openstack | Launchpad bug 1477490 in OpenStack Compute (nova) "Ironic: Deleting while spawning can leave orphan ACTIVE nodes in Ironic" [Medium,Confirmed] | 16:40 |
| lucasagomes | jroll, would love to know which driver was used there as well | 16:40 |
| lucasagomes | + local boot, it required grub2 in the image. Not sure if we can do it with cirros | 16:40 |
| * lucasagomes adds a comment | 16:40 | |
| jroll | lucasagomes: oh yeah, does that bug leave instance uuid behind? | 16:41 |
| jroll | also | 16:41 |
| lucasagomes | jroll, yup | 16:41 |
| sambetts | vsaienko: I turns out the reason my CI isn't handling Depends-On is because of an out dated install of Zuul | 16:41 |
| jroll | lucasagomes: until they tell us the actual error I don't care about the driver :P | 16:41 |
| JayF | I need specs core review on https://review.openstack.org/#/c/313800/ -- already has +1s from Jim and ruby | 16:41 |
| jroll | sambetts: CD your CI :) | 16:41 |
| JayF | wrong link | 16:41 |
| jroll | JayF: specs core review on project-config? | 16:42 |
| jroll | heh | 16:42 |
| jroll | I was like why did we +1 | 16:42 |
| sambetts | jroll: apprently thats how your meant to run Zuul even though they have stable releases | 16:42 |
| lucasagomes | jroll, fair, just adding the comment so they when they do tell us the actual error they can also tell us the driver, which image they where deployed etc | 16:42 |
| jroll | lucasagomes: sure | 16:42 |
| JayF | I need specs core review on https://review.openstack.org/#/c/321153/ -- already has +2 from Jim | 16:42 |
| lucasagomes | better to get as many info as we can | 16:42 |
| sambetts | jroll: if you don't zuul doesn't work with the version of gerrit being run right now | 16:42 |
| jroll | sambetts: heh, fun | 16:43 |
| *** sdake has joined #openstack-ironic | 16:44 | |
| sambetts | jroll: yup :) | 16:44 |
| *** tesseract has joined #openstack-ironic | 16:44 | |
| lucasagomes | JayF, thought it was already merged :-/... anyway, +1 | 16:46 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add CLI tests to check JSON response body https://review.openstack.org/291239 | 16:47 |
| JayF | lucasagomes: the metrics spec was merged ... but there was a little bit of design work that started happening on the review, so we had to turn back the clock | 16:47 |
| * JayF assures you he dislikes reopening a spec the maximum amount permitted | 16:47 | |
| lucasagomes | JayF, oh I was talking about the project config one | 16:47 |
| JayF | ah, yeah | 16:48 |
| * lucasagomes looks at the other link | 16:48 | |
| JayF | that one has been dangling forever :( | 16:48 |
| lucasagomes | yup, I have one adding a job to virtualbmc that I've submitted on May 12th | 16:48 |
| lucasagomes | which is there yet :-/ | 16:48 |
| * lucasagomes goes ping -infra | 16:48 | |
| JayF | please don't let that merge before mine! lol | 16:48 |
| mgould | lucasagomes: https://review.openstack.org/#/c/240944/ has been open since November | 16:50 |
| sambetts | well after solving the problem with my CI, I'm gunna go while I'm on a high point | 16:50 |
| mgould | gnight sambetts | 16:50 |
| sambetts | night all | 16:50 |
| sambetts | o/ mgould | 16:50 |
| *** sambetts is now known as sambetts|afk | 16:50 | |
| *** tesseract has quit IRC | 16:51 | |
| lucasagomes | mgould, :-( | 16:52 |
| lucasagomes | JayF, marked by project-config patch as WIP will rebase tomorrow | 16:53 |
| lucasagomes | s/by/my/ | 16:54 |
| *** Romanenko_K has quit IRC | 16:56 | |
| jlvillal | vsaienko, ping? | 16:56 |
| *** kromanenko_ has quit IRC | 16:56 | |
| *** kromanenko__ has quit IRC | 16:56 | |
| *** fragatina has joined #openstack-ironic | 16:57 | |
| *** fragatina has quit IRC | 16:58 | |
| *** fragatina has joined #openstack-ironic | 16:59 | |
| *** wajdi has joined #openstack-ironic | 16:59 | |
| jlvillal | vsaienko, https://review.openstack.org/#/c/322656/ I'm going to update the comment line to make it more clear (in my opinion). Just an FYI | 16:59 |
| lucasagomes | calling it a day | 16:59 |
| lucasagomes | have a great evening all, talk to you all tomorrow | 17:00 |
| * lucasagomes run to catch his train back home | 17:00 | |
| *** lucasagomes is now known as lucas-afk | 17:00 | |
| jlvillal | lucas-afk, Ciao | 17:01 |
| *** PollyZ_ has quit IRC | 17:02 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add some missed test cases in node object tests https://review.openstack.org/268045 | 17:02 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Remove mock of not directly called ironic-lib methods https://review.openstack.org/289008 | 17:04 |
| *** joprovost has joined #openstack-ironic | 17:07 | |
| *** ayoung has quit IRC | 17:09 | |
| *** spandhe_ has joined #openstack-ironic | 17:11 | |
| *** spandhe has quit IRC | 17:13 | |
| *** spandhe_ is now known as spandhe | 17:13 | |
| openstackgerrit | Miles Gould proposed openstack/ironic-inspector: Remove iterated form of side effects to simplify code https://review.openstack.org/310613 | 17:15 |
| *** e0ne has joined #openstack-ironic | 17:16 | |
| *** mgould is now known as mgould|afk | 17:17 | |
| *** trown is now known as trown|lunch | 17:18 | |
| wajdi | good afternoon all. | 17:20 |
| thiagop | hey wajdi | 17:20 |
| wajdi | o/ | 17:20 |
| *** derekh has quit IRC | 17:21 | |
| *** aNupoisc has joined #openstack-ironic | 17:22 | |
| *** ifarkas has quit IRC | 17:22 | |
| *** e0ne has quit IRC | 17:25 | |
| *** mjturek1 has joined #openstack-ironic | 17:26 | |
| *** praneshp has joined #openstack-ironic | 17:27 | |
| *** e0ne has joined #openstack-ironic | 17:30 | |
| *** amotoki has joined #openstack-ironic | 17:32 | |
| *** joprovost has quit IRC | 17:34 | |
| *** joprovost has joined #openstack-ironic | 17:34 | |
| *** mjturek1 has quit IRC | 17:34 | |
| openstackgerrit | Yosef Hoffman proposed openstack/ironic-python-agent: lldp-timeout kernel parameter missing ipa- prefix https://review.openstack.org/324608 | 17:36 |
| *** derekh has joined #openstack-ironic | 17:37 | |
| aNupoisc | cinerama: TheJulia: gmmaha: after getting the stable/mitika I am getting same error. Here is pastebin for ironic-api.log http://pastebin.intel.com/view/5d836787 and console output http://pastebin.intel.com/view/532e81c2 | 17:37 |
| *** amotoki has quit IRC | 17:38 | |
| cinerama | hi aNupoisc one sec | 17:40 |
| aNupoisc | hi cinerama | 17:40 |
| *** electrofelix has quit IRC | 17:40 | |
| aNupoisc | yeah sure | 17:40 |
| cinerama | oh aNupoisc you are using that internal pastebin again, can you use the openstack one? | 17:40 |
| aNupoisc | cinerama: oops | 17:41 |
| aNupoisc | wait a sec | 17:41 |
| *** vmud213 has joined #openstack-ironic | 17:41 | |
| gmmaha | aNupoisc: Is this the same error as we saw yesterday? | 17:42 |
| aNupoisc | pastebin for ironic-api http://pastebin.com/4D2W1TuL and console ouput http://pastebin.com/sBrggREg | 17:43 |
| aNupoisc | cinerama TheJulia gmmaha | 17:43 |
| gmmaha | aNupoisc: i am not seeing hte msg where it says that the input to auth is str and it expects a dict | 17:43 |
| aNupoisc | gmmaha: yes. I mean i am getting machines can't be registerd with ironic error | 17:44 |
| gmmaha | is that the same error you are hitting? | 17:44 |
| aNupoisc | gmmaha: i am not getting that | 17:44 |
| gmmaha | aNupoisc: hmm | 17:44 |
| aNupoisc | gmmaha: i am getting the same error as alast time | 17:45 |
| gmmaha | aNupoisc: and last time is when? | 17:45 |
| *** derekh has quit IRC | 17:45 | |
| gmmaha | the last i remember was that the os_ironic module expected a dict as input to auth and was being given a string and it was complaining about that | 17:45 |
| aNupoisc | I am sourcing hacking/env-setup, defining BIFROST_INVENTORY_FILE. THis is the only things that i need to do before right? Can you please verify | 17:46 |
| gmmaha | aNupoisc: let me login | 17:46 |
| aNupoisc | gmmaha: yeah but when i tried I am hitting this one | 17:46 |
| aNupoisc | gmmaha: last time is with older codebase of bifrosdt | 17:46 |
| *** PollyZ__ has joined #openstack-ironic | 17:47 | |
| aNupoisc | gmmaha: yes sure | 17:47 |
| *** jjohnson2 has quit IRC | 17:55 | |
| cinerama | gmmaha, please let us know if you need help after you've logged in to confirm stuff...we might need a little more information too | 17:56 |
| gmmaha | cinerama: i am looking but not able to find anything obviously wrong with it | 17:56 |
| *** links has joined #openstack-ironic | 17:57 | |
| gmmaha | the ansible logs dont show much.. thought mysql was down, but the service is runnign and seems to be fine.. no issues there | 17:57 |
| cinerama | gmmaha, anything in the conductor logs | 17:57 |
| *** hshiina has joined #openstack-ironic | 17:57 | |
| *** PollyZ__ has quit IRC | 17:57 | |
| gmmaha | cinerama: hmm, the last conductor logs is a gzip and i dont see the service runnign at all in ps | 17:58 |
| *** vmud213 has quit IRC | 17:58 | |
| gmmaha | only ironic-api | 17:58 |
| * gmmaha scratches head | 17:58 | |
| *** vmud213 has joined #openstack-ironic | 17:58 | |
| cinerama | gmmaha, maybe just restart everything and give it another go? | 17:59 |
| gmmaha | cinerama: yeah, seems like a good idea.. i will go ahead and re-provision the VM as well.. might cleanup and kick start things properly | 17:59 |
| cinerama | gmmaha, ok sounds good...let us know what happens after that if you don't have much luck | 18:00 |
| aNupoisc | cinerama: gmmaha: i tried re provisioning. But yes we can restart everything | 18:00 |
| gmmaha | cinerama: will do.. | 18:01 |
| gmmaha | aNupoisc: aah.. sorry should have asked you before i went ahead and did that | 18:01 |
| gmmaha | anyways kicked it again.. lets see how it goes | 18:01 |
| aNupoisc | gmmaha: that's okay. We can try with everything fresh | 18:01 |
| *** e0ne has quit IRC | 18:02 | |
| *** ChubYann has joined #openstack-ironic | 18:05 | |
| *** hshiina has quit IRC | 18:06 | |
| JayF | TinyIPA + iPXE for most Ironic jobs just landed in project-config | 18:07 |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Correct firmware inventory issue on IBM M4 servers https://review.openstack.org/324744 | 18:07 |
| *** e0ne has joined #openstack-ironic | 18:08 | |
| *** SiRiuS__ has quit IRC | 18:08 | |
| *** sabeen has joined #openstack-ironic | 18:08 | |
| *** david-lyle has joined #openstack-ironic | 18:09 | |
| gmmaha | cinerama: aNupoisc: it worked on a VM reload with provision, but not unless i commented out this line https://github.com/openstack/bifrost/blob/master/playbooks/enroll-dynamic.yaml#L13 | 18:10 |
| gmmaha | this was the error. http://paste.openstack.org/show/507353/ | 18:10 |
| cinerama | gmmaha, so you commented out the inspection role there? | 18:14 |
| cinerama | gmmaha, just out of interest what version of ansible are you using | 18:14 |
| aNupoisc | cinerama: i guess it is 2.0.0 | 18:15 |
| gmmaha | cinerama: http://paste.openstack.org/show/507354/ | 18:15 |
| aNupoisc | i will just check | 18:15 |
| gmmaha | aNupoisc: ^^ | 18:15 |
| cinerama | oh interesting | 18:18 |
| cinerama | so if i'm right that error is a bit...misleading | 18:18 |
| cinerama | 1 sec | 18:18 |
| gmmaha | cinerama: thanks | 18:20 |
| *** ijw has joined #openstack-ironic | 18:21 | |
| cinerama | gmmaha, aNupoisc do you have /opt/stack/ansible/lib/ansible/modules/core/cloud/openstack/os_ironic_node.py | 18:22 |
| *** trown|lunch is now known as trown | 18:22 | |
| gmmaha | cinerama: have that file but in a different path (which is what i source when i run ansible) | 18:23 |
| gmmaha | http://paste.openstack.org/show/507356/ | 18:23 |
| gmmaha | i use the same method of installing ansible both in the host and in the guest.. so cant use the in /opt/stack ninside the vagrant VM.. | 18:24 |
| gmmaha | hence it sitting inside ~/bin | 18:24 |
| cinerama | gmmaha, okay the reason i ask is because the error you got in the pastebin can sometimes happen if a module that's referenced is not present | 18:24 |
| gmmaha | cinerama: aah.. any chance we are referencing it via the hardcoded path? | 18:25 |
| cinerama | gmmaha, if you're moving stuff you may need to specify "library = /path/to/libs:/path/to/more/libs" in ansible.cfg or ~/.ansible.cfg | 18:27 |
| thiagop | sambetts|afk: careful if you are going to update zuul/jenkins | 18:30 |
| thiagop | sambetts|afk: ran into some trouble today due to auto-update of puppet-openstackci. Ping me if you want details. | 18:30 |
| *** SiRiuS__ has joined #openstack-ironic | 18:32 | |
| *** mjturek1 has joined #openstack-ironic | 18:33 | |
| *** amotoki has joined #openstack-ironic | 18:34 | |
| *** garthb has joined #openstack-ironic | 18:34 | |
| *** aNupoisc has quit IRC | 18:34 | |
| *** david-lyle has quit IRC | 18:35 | |
| *** aNupoisc has joined #openstack-ironic | 18:37 | |
| *** vmud213 has quit IRC | 18:37 | |
| aNupoisc | gmmaha cinerama: ah okay | 18:37 |
| *** vmud213 has joined #openstack-ironic | 18:37 | |
| *** amotoki has quit IRC | 18:39 | |
| *** links has quit IRC | 18:39 | |
| *** mjturek1 has quit IRC | 18:43 | |
| *** harlowja has quit IRC | 18:44 | |
| gmmaha | cinerama: sorry got pulled into a meeting | 18:51 |
| gmmaha | let me see what i got in the cfg file | 18:51 |
| aNupoisc | gmmaha cinerama: i guess it is talking library = /usr/share/my_modules/ | 18:52 |
| aNupoisc | this is what i see in ansible.cfg | 18:52 |
| gmmaha | aNupoisc: so it will be /usr/share/my_modules:/home/vagrant/bin/ansible/modules/ | 18:52 |
| *** ChubYann has quit IRC | 18:53 | |
| gmmaha | or our version first and then the /usr/share/my_modules | 18:53 |
| *** ohamada has quit IRC | 18:54 | |
| aNupoisc | gmmaha: i mean shall we specify the absolute path? | 18:54 |
| gmmaha | cinerama: i added the path to ~/.ansible.cfg and still hit the same error caused by line #13 | 19:00 |
| cinerama | gmmaha, i'm just having lunch here, i'll be back in a little while | 19:02 |
| gmmaha | cinerama: thanks.. no rush.. for now we are good with the mild hack | 19:02 |
| gmmaha | :) | 19:02 |
| openstackgerrit | Merged openstack/bifrost: Install libssl-dev and libffi-dev https://review.openstack.org/318943 | 19:06 |
| *** ChubYann has joined #openstack-ironic | 19:06 | |
| *** e0ne has quit IRC | 19:09 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Add boot information into the inventory https://review.openstack.org/320448 | 19:11 |
| openstackgerrit | Merged openstack/ironic: Add support for dib based agent ramdisk in lib/ironic https://review.openstack.org/264579 | 19:16 |
| *** _vs has joined #openstack-ironic | 19:19 | |
| *** _vsaienko has joined #openstack-ironic | 19:19 | |
| *** vmud213 has quit IRC | 19:19 | |
| cinerama | gmmaha, so by the way, an easier way to do that might be to set inspect_nodes to false | 19:21 |
| cinerama | gmmaha, anyway after you did that were you able to successfully enroll a node? | 19:21 |
| gmmaha | cinerama: it is being set to false.. we arent setting ti at all.. so assumed the default will kick-in | 19:21 |
| gmmaha | cinerama: yes.. after commengint that out, we are able to enroll just fine.. for some reason during the first run i didnt ahve the conductor service running | 19:22 |
| JayF | rloo: I'll fix those issues in the metric piece and repush, thanks for the review! :) | 19:22 |
| gmmaha | cinerama: will be doing a full tear-down and re-build soon to see if this is a transient issue | 19:23 |
| cinerama | gmmaha, so do you have os_ironic_inspect.py in your library pat(s) somewhere? | 19:24 |
| *** david-lyle has joined #openstack-ironic | 19:24 | |
| cinerama | s/pat/path/ | 19:24 |
| rloo | JayF: yw | 19:25 |
| *** aNupoisc has quit IRC | 19:25 | |
| *** mjturek1 has joined #openstack-ironic | 19:28 | |
| *** Sukhdev has joined #openstack-ironic | 19:28 | |
| gmmaha | cinerama: yeah, they are there at /opt/stack/ansible/lib/ansible/modules/extras/cloud/openstack/os_ironic_inspect.py | 19:29 |
| gmmaha | hold the phone! | 19:29 |
| gmmaha | aha! | 19:30 |
| gmmaha | you are right.. its not in the ~/bin/ansible dir :) | 19:30 |
| gmmaha | and hence hte play failing.. now that makes sense | 19:30 |
| *** ijw has quit IRC | 19:30 | |
| * gmmaha goes to see why that mdoule wasnt installed when running the ansible install script | 19:30 | |
| openstackgerrit | Merged openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/323003 | 19:31 |
| *** causten_ has joined #openstack-ironic | 19:31 | |
| cinerama | gmmaha, did you run env-setup.sh, it may have needed to fetch that particular module | 19:32 |
| gmmaha | i was most certain i did.. but let me check again | 19:32 |
| *** ChrisAusten has quit IRC | 19:34 | |
| *** amotoki has joined #openstack-ironic | 19:34 | |
| *** sdake has quit IRC | 19:35 | |
| JayF | IPA gate is broken right now, checking with infra and going to update whiteboard. | 19:37 |
| JayF | I haven't looked at ironic yet, might be impacting there too | 19:37 |
| *** sdake has joined #openstack-ironic | 19:38 | |
| *** _vsaienko has quit IRC | 19:38 | |
| *** _vs has quit IRC | 19:38 | |
| *** amotoki has quit IRC | 19:39 | |
| *** _vs has joined #openstack-ironic | 19:39 | |
| *** _vsaienko has joined #openstack-ironic | 19:39 | |
| *** openstackstatus has quit IRC | 19:39 | |
| *** openstack has joined #openstack-ironic | 19:41 | |
| JayF | well I was obv going to check, I merged probably the biggest gate config change i've written | 19:42 |
| JayF | lol | 19:42 |
| jroll | heh | 19:42 |
| *** sdake has quit IRC | 19:43 | |
| gmmaha | cinerama: found the issue.. thanks for the tip about the library.. i will fix my scripts to try and aboid this again | 19:43 |
| rloo | JayF: I did wonder... | 19:43 |
| gmmaha | to give you a gist, i had copied the older version of env-setup.sh to avoid having to submodule the whole bifrost git repo inside mine. and this is the reason i didnt have the modules | 19:43 |
| JayF | It doesn't appear any ironic jobs have been kicked off since the change. I'm going to recheck something | 19:44 |
| gmmaha | i will probably mask the script to wget the raw file from github and just source it in shell.. safer bet | 19:44 |
| rloo | JayF: I also approved one or more changes to devstack recently | 19:44 |
| JayF | rloo: well the NOT_REGISTERED I don't think could be caused by devstack | 19:44 |
| JayF | rloo: only my changes :/ | 19:44 |
| rloo | JayF: OH | 19:44 |
| cinerama | gmmaha: that makes sense. but it's a little hard to support you in finding what's wrong when we aren't really sure what bits you've changed | 19:44 |
| rloo | JayF: so if a test isn't registered and the rest of the tests pass, doesn't that mean the patch passed :D (ok, wrong bright side of things) | 19:45 |
| JayF | heh | 19:45 |
| JayF | that'd be even worse | 19:45 |
| JayF | since all the IPA jobs have been switched around so all the tempest jobs on it have new names | 19:45 |
| gmmaha | cinerama: absolutely.. I apologize for that. bad coding on my part.. while the library mess was a side-effect of my ill doing, the other issue of conductor service not running is still a mystery.. i will try and recreate the VM to see if thats something thats just transient or something that persists | 19:46 |
| cinerama | gmmaha, check the conductor logs and/or try starting it on the command line, maybe with --debug too | 19:47 |
| *** _vsaienko has quit IRC | 19:47 | |
| *** _vs has quit IRC | 19:47 | |
| cinerama | just in case it's attempting to start & failing | 19:47 |
| gmmaha | cinerama: will do.. the VM on re-provision has the conductor serviceand its running without crash.. but will do the tear down and check again | 19:48 |
| gmmaha | will run it by you if this time i *really* find an issue.. | 19:49 |
| JayF | just did recheck on https://review.openstack.org/#/c/319183/ (Ironic), going to see if the new jobs work on Ironic, even if they're busted on IPA | 19:49 |
| gmmaha | sorry again about the red herring | 19:49 |
| JayF | It's running with the new job set b/c there are no more tinyipa jobs | 19:49 |
| *** mbound has quit IRC | 19:53 | |
| openstackgerrit | Merged openstack/pyghmi: Correct firmware inventory issue on IBM M4 servers https://review.openstack.org/324744 | 19:54 |
| *** joprovost has quit IRC | 19:56 | |
| *** david-lyle has quit IRC | 19:57 | |
| JayF | rloo: I suspect sometime soon (maybe next cycle or one after that?) we should formalize a "v2" of the IPA API (presuming you can even call what we have a v1 API), complete with versioning and actual design | 20:00 |
| rloo | JayF: is it too late to add versioning NOW? | 20:01 |
| openstackgerrit | Ganesh Maharaj Mahalingam proposed openstack/bifrost: Make ansible installation directory configurable https://review.openstack.org/324789 | 20:01 |
| openstackgerrit | Yosef Hoffman proposed openstack/ironic-python-agent: lldp-timeout kernel parameter missing ipa- prefix https://review.openstack.org/324608 | 20:01 |
| JayF | I honestly am not sure; but I do know we don't manage any of that stuff well now | 20:01 |
| rloo | JayF: we cannot wait for two cycles. i'm not even sure we can wait for next cycle. | 20:01 |
| cinerama | gmmaha, btw do you know about the venv installation option? that might solve some of your problems | 20:02 |
| JayF | Then hopefully someone more clever than I knows how to implement versioning on that after the fact :) | 20:02 |
| JayF | I'm much better at nuts and bolts than APIs :) | 20:02 |
| gmmaha | cinerama: yes, just saw that while i was reading the script and making this change ;) https://review.openstack.org/#/c/324789/ | 20:02 |
| rloo | JayF: Oh, we know how to implement microversioning after the fact :D | 20:02 |
| JayF | lol | 20:02 |
| gmmaha | guessnig my change is an overkill.. | 20:02 |
| rloo | JayF: i can add it to monday's agenda if that makes sense. or an email? | 20:03 |
| cinerama | gmmaha: my off-the-cuff review of your change is that there will need to be playbook changes as well to fully support it | 20:03 |
| JayF | I suspect this is more mailing list fodder | 20:03 |
| rloo | JayF: d*. I mean, darn right ;) | 20:03 |
| gmmaha | cinerama: aah.. did not realize that.. i can check the playbooks to see how invasive this would be.. | 20:04 |
| gmmaha | but do you tink this makes sense, or using a VENV would be the most suitable option | 20:04 |
| cinerama | gmmaha, so as the code stands now, if you use a venv, during env-setup.sh, the ansible git repository will get cloned into /opt/stack/ansible, but then it will be pip installed into a specified venv of your choice | 20:06 |
| *** causten__ has joined #openstack-ironic | 20:07 | |
| *** causten_ has quit IRC | 20:07 | |
| *** dprince has quit IRC | 20:08 | |
| cinerama | gmmaha, are you planning to run multiple instances of ironic at once? | 20:08 |
| gmmaha | cinerama: nope.. Just one. | 20:08 |
| gmmaha | think venv would be a better option than muddlign the code this bad then | 20:08 |
| *** Sukhdev has quit IRC | 20:09 | |
| *** Sukhdev has joined #openstack-ironic | 20:09 | |
| *** aNupoisc has joined #openstack-ironic | 20:10 | |
| cinerama | gmmaha, yeah if you just want to have multiple copies of stuff laying around give the venv a go | 20:10 |
| gmmaha | cinerama: yeah tesitng it now and i am sure this will work for us.. Will abandon the patch.. the venv is much cleaner than the patch | 20:10 |
| gmmaha | thanks again :) | 20:10 |
| cinerama | gmmaha, so, i still think that's a nice addition to have - but it's incomplete without the playbook changes | 20:11 |
| cinerama | gmmaha, so if you did want to propose it in full that'd be awesome | 20:11 |
| gmmaha | cinerama: aah.. will fix the patch then and push a new version up | 20:11 |
| gmmaha | cinerama: will do. :) | 20:11 |
| gmmaha | aNupoisc: i know you are out.. when you re back, we can talk about fixing our scripts to use venv | 20:12 |
| cinerama | gmmaha the changes should mostly be in the bifrost-ironic-install role | 20:12 |
| *** Sukhdev has quit IRC | 20:13 | |
| gmmaha | cinerama: that role is just adding all the other git projects (ironic, shade, etc) to /opt/stack which should be fine i guess.. | 20:15 |
| gmmaha | the place that will crash is the vagrant VM which assumes that running env-setup.sh will have ansible at /opt/stack when its calling the vagrant.yml | 20:15 |
| *** jralph has joined #openstack-ironic | 20:17 | |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to Network https://review.openstack.org/294559 | 20:19 |
| cinerama | gmmaha, right, that would need to change a bit as well | 20:20 |
| *** rcernin has joined #openstack-ironic | 20:20 | |
| cinerama | gmmaha, like set the ansible clone location in the Vagrantfile (where we ask users to edit a few other things) and then use a variable in the vagrant.yml | 20:21 |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to Network https://review.openstack.org/294559 | 20:24 |
| *** harlowja has joined #openstack-ironic | 20:24 | |
| aNupoisc | gmmaha: i will be back vy 2:30. right now in lab | 20:30 |
| *** ijw has joined #openstack-ironic | 20:32 | |
| *** piet_ has joined #openstack-ironic | 20:34 | |
| *** e0ne has joined #openstack-ironic | 20:34 | |
| *** amotoki has joined #openstack-ironic | 20:35 | |
| *** joprovost has joined #openstack-ironic | 20:37 | |
| jroll | JayF: howsit | 20:38 |
| JayF | Not 100% sure, still waiting for rechecks to finish and such | 20:39 |
| jroll | cool | 20:39 |
| JayF | I'm monitoring it though | 20:39 |
| jroll | awesome, ty | 20:39 |
| NobodyCam | dumb question: where are the ipa logs kept :p | 20:39 |
| *** amotoki has quit IRC | 20:40 | |
| JayF | NobodyCam: depends on the ramdisk. Whereever logs normally go for that OS | 20:40 |
| JayF | NobodyCam: in CoreOS; journalctl -u ironic-python-agent # will show you all the logs for the ipa unit | 20:40 |
| *** e0ne has quit IRC | 20:41 | |
| *** jcoufal has quit IRC | 20:43 | |
| NobodyCam | awesome thank you.. | 20:44 |
| *** mjturek1 has quit IRC | 20:44 | |
| *** e0ne has joined #openstack-ironic | 20:45 | |
| mrda | Morning Ironic | 20:46 |
| *** fragatina has quit IRC | 20:47 | |
| gmmaha | cinerama: right.. i think thats a do-able thing and i can read the env variable here as well.. that might do.. Let me update the test scripts as well and push a new patch | 20:51 |
| *** aNupoisc has quit IRC | 20:51 | |
| *** trown is now known as trown|outtypewww | 20:52 | |
| *** mbound has joined #openstack-ironic | 20:53 | |
| JayF | New Ironic jobs are working in Ironic-proper, using tinyipa and 384mb of ram | 20:56 |
| JayF | still pending to see if IPA jobs are succeeding | 20:56 |
| *** wajdi_ has joined #openstack-ironic | 20:57 | |
| *** mbound has quit IRC | 20:58 | |
| *** ayoung has joined #openstack-ironic | 20:59 | |
| NobodyCam | morning mrda | 21:01 |
| *** wajdi has quit IRC | 21:01 | |
| *** wajdi_ has quit IRC | 21:02 | |
| mrda | o/ | 21:03 |
| *** _vsaienko has joined #openstack-ironic | 21:04 | |
| *** _vs has joined #openstack-ironic | 21:04 | |
| *** baoli_ has quit IRC | 21:05 | |
| *** lindycoder has quit IRC | 21:08 | |
| *** _vsaienko has quit IRC | 21:08 | |
| *** _vs has quit IRC | 21:08 | |
| JayF | as you can see here: https://review.openstack.org/#/c/324608/ IPA jobs now working | 21:14 |
| *** SiRiuS__ has quit IRC | 21:14 | |
| JayF | I'll send something out to the ML, but Ironic jobs are now using tinyipa by default, and IPA is gating on coreos and tinyipa src jobs | 21:14 |
| JayF | \o/ | 21:14 |
| rloo | JayF: awesome! | 21:14 |
| rloo | JayF: and first cards to be 'Done' in trello :) | 21:15 |
| JayF | rloo: TIL you work on OSIC team too | 21:16 |
| *** aNupoisc has joined #openstack-ironic | 21:16 | |
| JayF | \o/ | 21:16 |
| *** adu has quit IRC | 21:16 | |
| rloo | JayF: what is TIL? | 21:16 |
| JayF | today I learned | 21:17 |
| rloo | Oh. I didn't know that. Are you sure? No one told me. yet. anyway? | 21:18 |
| JayF | Well you knew it was the first thing done in the trello? | 21:18 |
| JayF | Ohhhh, jroll's other trello for ironic priorities | 21:18 |
| rloo | JayF: ha ha. mystery solved. | 21:18 |
| JayF | 3 trello boards is about two too many to keep straight | 21:18 |
| rloo | JayF: sorry, I had no idea you were using it in other contexts. | 21:19 |
| JayF | oh it's fine, your timing is just impeccable re: conversation in another channel | 21:19 |
| JayF | honestly I don't feel like I did much re: getting jobs to run faster; just configuring things to make use of the good work others have done :D | 21:21 |
| devananda | JayF: fantastic! | 21:23 |
| *** lhcheng has joined #openstack-ironic | 21:23 | |
| *** ayoung has quit IRC | 21:25 | |
| JayF | Does anyone know if Yosef Hoffman is in IRC? | 21:25 |
| jroll | dtantsur|afk: devananda: anyone else that cares about releases, I just did all the stable things: https://review.openstack.org/#/q/project:openstack/releases+owner:%22Jim+Rollenhagen+%253Cjim%2540jimrollenhagen.com%253E%22+status:open | 21:26 |
| *** thiagop has quit IRC | 21:29 | |
| *** amotoki has joined #openstack-ironic | 21:36 | |
| *** lhcheng has left #openstack-ironic | 21:36 | |
| *** e0ne has quit IRC | 21:37 | |
| *** e0ne has joined #openstack-ironic | 21:38 | |
| *** rcernin_ has joined #openstack-ironic | 21:38 | |
| *** e0ne has quit IRC | 21:38 | |
| *** adu has joined #openstack-ironic | 21:41 | |
| *** amotoki has quit IRC | 21:42 | |
| *** PollyZ has joined #openstack-ironic | 21:43 | |
| *** blinky_ghost has quit IRC | 21:45 | |
| *** e0ne has joined #openstack-ironic | 21:48 | |
| openstackgerrit | Yosef Hoffman proposed openstack/ironic-python-agent: lldp-timeout kernel parameter missing ipa- prefix https://review.openstack.org/324608 | 21:49 |
| openstackgerrit | Merged openstack/python-ironicclient: Catch RetriableConnectionFailures from KAuth and retry https://review.openstack.org/323851 | 21:49 |
| JayF | jroll: ^ that patch from Yosef; I like what it's doing but I'm not sure what further we might need to do to deprecate the old config option | 21:49 |
| *** rbudden has quit IRC | 21:54 | |
| *** yohoffman has joined #openstack-ironic | 21:55 | |
| mrda | JayF: Good work on tinyipa and ipxe! :) | 21:56 |
| JayF | All I did was change some configs :) | 21:56 |
| mrda | That's still a thing | 21:58 |
| *** rbudden has joined #openstack-ironic | 22:03 | |
| *** ametts has quit IRC | 22:03 | |
| *** [1]cdearborn has quit IRC | 22:03 | |
| *** tangchen has quit IRC | 22:05 | |
| *** tangchen has joined #openstack-ironic | 22:06 | |
| *** rcernin_ has quit IRC | 22:06 | |
| *** causten_ has joined #openstack-ironic | 22:07 | |
| *** thrash is now known as thrash|g0ne | 22:10 | |
| *** causten__ has quit IRC | 22:11 | |
| *** ayoung has joined #openstack-ironic | 22:11 | |
| *** PollyZ has quit IRC | 22:14 | |
| *** sacharya has quit IRC | 22:17 | |
| openstackgerrit | Yosef Hoffman proposed openstack/ironic-python-agent: lldp-timeout kernel parameter missing ipa- prefix https://review.openstack.org/324608 | 22:18 |
| yohoffman | JayF: o/ | 22:19 |
| *** PollyZ has joined #openstack-ironic | 22:19 | |
| JayF | yohoffman: o/ welcome | 22:19 |
| *** openstackgerrit has quit IRC | 22:19 | |
| JayF | yohoffman: I asked jroll to look at your patch, to see what else we need to do to deprecate that old kernel param | 22:20 |
| JayF | yohoffman: this is a new thing we haven't done with IPA before, so I'm not sure what the right thing to do is, but your patch is definately the right start. | 22:20 |
| *** openstackgerrit has joined #openstack-ironic | 22:20 | |
| JayF | I presume we just have to keep legacy support in for two cycles and then pull it, but I'm uinsur | 22:20 |
| JayF | *unsure | 22:20 |
| yohoffman | JayF: But if we don't use the "deprecated" attribute, how will users know that it's been deprecated? | 22:21 |
| JayF | That's what I asked jroll for :) | 22:21 |
| JayF | even logging something being deprecated in IPA isn't that useful, because IPA logs aren't typically operator-visible | 22:21 |
| JayF | which is why this is stranger than usual | 22:21 |
| yohoffman | Is it terrible if we accept both options forever? | 22:22 |
| JayF | you know, that's exactly what I was just thinking | 22:22 |
| JayF | but someone might hate us for it in a few years so we should ask just to be sure :P | 22:22 |
| yohoffman | Well wouldn't want anyone to hate us over this... | 22:23 |
| JayF | lol | 22:24 |
| yohoffman | JayF: On a possibly related note... I was looking at https://bugs.launchpad.net/ironic-python-agent/+bug/1585663 . Is requesting to create parameters in this same file (ironic_python_agent/config.py)? | 22:29 |
| openstack | Launchpad bug 1585663 in ironic-python-agent "Add configuration options for DISK_WAIT_ATTEMPTS, DISK_WAIT_DELAY" [Low,Triaged] | 22:29 |
| JayF | I believe so, yeah. In order to make those bits configurable | 22:29 |
| JayF | that should be a pretty low hanging bug as well if you wanted to knock it out | 22:30 |
| yohoffman | Took a stab at it but seem to be missing a piece. | 22:32 |
| yohoffman | Added the options to that file, but tests fail because "NoSuchOptError: no such option in group DEFAULT: disk_wait_attempts" | 22:32 |
| JayF | if you want to push up the code I can take a look and see if I can help | 22:33 |
| JayF | Might I ask what your interest in Ironic/Openstack is? | 22:33 |
| yohoffman | I'm pretty new to OpenStack and looking around trying to see where I can get involved. | 22:33 |
| JayF | Awesome; more hands are always appreciated. Don't discount the value of reviewing code, and don't be shy about putting comments on too :) | 22:34 |
| JayF | if you'll push up what you have for bug 1585663 I'm willing to look at it and try to point you in a right direction | 22:34 |
| openstack | bug 1585663 in ironic-python-agent "Add configuration options for DISK_WAIT_ATTEMPTS, DISK_WAIT_DELAY" [Low,Triaged] https://launchpad.net/bugs/1585663 | 22:34 |
| yohoffman | Of course =) and thanks! | 22:35 |
| *** aNupoisc has quit IRC | 22:35 | |
| *** hshiina has joined #openstack-ironic | 22:35 | |
| *** causten_ has quit IRC | 22:36 | |
| *** aNupoisc has joined #openstack-ironic | 22:36 | |
| *** frontrunner has quit IRC | 22:38 | |
| openstackgerrit | Yosef Hoffman proposed openstack/ironic-python-agent: [WIP] Add configuration options for DISK_WAIT https://review.openstack.org/324909 | 22:38 |
| JayF | yohoffman: I think you have to import the option | 22:41 |
| JayF | let me find you some example code | 22:41 |
| yohoffman | Import it in the hardware.py file or in the tests? | 22:43 |
| *** PollyZ___ has joined #openstack-ironic | 22:44 | |
| JayF | I'm thinking you should completely replace references to _DISK_WAIT_* with the conf equivalents | 22:44 |
| JayF | and you'll likely also need to import those config options for a test, or mock them out | 22:45 |
| JayF | plus you'll wanna add a unit test to verify the behavior changes when the config changes | 22:45 |
| JayF | yohoffman: I put that on the review; I have something else I have to work out for the rest of the day, but I'll review the code again when you push more up | 22:46 |
| JayF | good luck! | 22:46 |
| yohoffman | Cool, I'll have a go at that. Thanks for the help and pleasure to "meet" you! | 22:46 |
| *** fragatina has joined #openstack-ironic | 22:46 | |
| JayF | no problem | 22:47 |
| *** adu has quit IRC | 22:52 | |
| *** aparnav has quit IRC | 22:54 | |
| *** aparnav has joined #openstack-ironic | 22:54 | |
| *** frontrunner has joined #openstack-ironic | 22:56 | |
| *** ayoung has quit IRC | 23:04 | |
| *** r-daneel has joined #openstack-ironic | 23:07 | |
| *** rcernin has quit IRC | 23:10 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Update metrics spec to reflect review comments https://review.openstack.org/321153 | 23:12 |
| JayF | jroll: rloo ^ updated to fix rendering issues as brought up in review, and fixed that nit regarding networks | 23:12 |
| *** PollyZ has quit IRC | 23:18 | |
| *** PollyZ___ has quit IRC | 23:18 | |
| *** david-lyle has joined #openstack-ironic | 23:20 | |
| *** sabeen has quit IRC | 23:22 | |
| *** adu has joined #openstack-ironic | 23:23 | |
| *** yohoffman has quit IRC | 23:25 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/324855 | 23:35 |
| *** amotoki has joined #openstack-ironic | 23:39 | |
| *** amotoki has quit IRC | 23:44 | |
| *** sabeen has joined #openstack-ironic | 23:49 | |
| *** vishwanathj has joined #openstack-ironic | 23:49 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Make agent driver always PXE before cleaning https://review.openstack.org/263842 | 23:49 |
| *** sdake has joined #openstack-ironic | 23:54 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Make agent driver always PXE before cleaning https://review.openstack.org/263842 | 23:55 |
| *** sdake_ has quit IRC | 23:56 | |
| *** sdake has quit IRC | 23:57 | |
| *** wajdi has joined #openstack-ironic | 23:58 | |
| *** Goneri has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!