| *** Sukhdev has joined #openstack-ironic | 00:13 | |
| *** sdake has quit IRC | 00:22 | |
| *** Sukhdev has quit IRC | 00:26 | |
| *** Sukhdev has joined #openstack-ironic | 00:28 | |
| *** rbrud has joined #openstack-ironic | 00:32 | |
| *** sdake has joined #openstack-ironic | 00:32 | |
| *** hoangcx has joined #openstack-ironic | 00:33 | |
| *** ijw has joined #openstack-ironic | 00:35 | |
| *** ijw has quit IRC | 00:46 | |
| *** ijw has joined #openstack-ironic | 01:01 | |
| *** ijw has quit IRC | 01:21 | |
| *** baoli_ has quit IRC | 01:22 | |
| *** baoli has joined #openstack-ironic | 01:23 | |
| *** baoli has quit IRC | 01:28 | |
| *** amotoki has joined #openstack-ironic | 01:30 | |
| openstackgerrit | Xu Ao proposed openstack/ironic: Make ironic console supporting the nokeepalive mode https://review.openstack.org/372830 | 01:31 |
|---|---|---|
| *** Sukhdev has quit IRC | 01:33 | |
| *** ijw has joined #openstack-ironic | 01:34 | |
| *** ijw has quit IRC | 01:40 | |
| *** ijw has joined #openstack-ironic | 01:48 | |
| *** mtanino has joined #openstack-ironic | 01:59 | |
| *** ijw has quit IRC | 02:05 | |
| *** tiendc has joined #openstack-ironic | 02:23 | |
| *** baoli has joined #openstack-ironic | 02:29 | |
| *** early has quit IRC | 02:29 | |
| *** stevemar has quit IRC | 02:30 | |
| *** stevemar has joined #openstack-ironic | 02:31 | |
| *** baoli has quit IRC | 02:33 | |
| *** sdake has quit IRC | 02:35 | |
| *** early has joined #openstack-ironic | 02:35 | |
| *** tuanla has joined #openstack-ironic | 02:36 | |
| *** amotoki has quit IRC | 02:41 | |
| openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Support i18n for baremetal chassis cmds https://review.openstack.org/375972 | 02:52 |
| openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Support i18n for baremetal port cmds https://review.openstack.org/375971 | 02:52 |
| openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Support i18n for baremetal node cmds https://review.openstack.org/357639 | 02:52 |
| *** yuanying has quit IRC | 02:52 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Using sys.exit(main()) instead of main() https://review.openstack.org/376088 | 02:53 |
| *** causten has quit IRC | 03:02 | |
| *** causten has joined #openstack-ironic | 03:02 | |
| *** david-lyle has quit IRC | 03:04 | |
| *** ijw has joined #openstack-ironic | 03:06 | |
| *** causten has quit IRC | 03:07 | |
| *** ijw has quit IRC | 03:11 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic-python-agent: Using sys.exit(main()) instead of main() https://review.openstack.org/376094 | 03:12 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Using sys.exit(main()) instead of main() https://review.openstack.org/376088 | 03:26 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic-python-agent: Using sys.exit(main()) instead of main() https://review.openstack.org/376094 | 03:27 |
| *** amotoki has joined #openstack-ironic | 03:30 | |
| *** Sukhdev has joined #openstack-ironic | 03:38 | |
| *** vikrant has joined #openstack-ironic | 03:41 | |
| *** mtanino has quit IRC | 03:41 | |
| *** yuanying has joined #openstack-ironic | 03:47 | |
| *** amotoki has quit IRC | 03:47 | |
| *** yuanying has quit IRC | 03:49 | |
| *** rbudden has quit IRC | 03:52 | |
| *** yuanying has joined #openstack-ironic | 03:55 | |
| *** appu103_ has joined #openstack-ironic | 04:00 | |
| *** appu103_ is now known as aparna | 04:00 | |
| *** hj-hpe has joined #openstack-ironic | 04:02 | |
| *** ijw has joined #openstack-ironic | 04:07 | |
| openstackgerrit | Cao Xuan Hoang proposed openstack/ironic-inspector: Use assertEqual() instead of assertDictEqual() https://review.openstack.org/376110 | 04:10 |
| *** ijw has quit IRC | 04:13 | |
| *** baoli has joined #openstack-ironic | 04:17 | |
| *** baoli has quit IRC | 04:21 | |
| *** moshele has joined #openstack-ironic | 04:36 | |
| *** causten has joined #openstack-ironic | 05:03 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off https://review.openstack.org/216730 | 05:07 |
| *** causten has quit IRC | 05:08 | |
| *** ijw has joined #openstack-ironic | 05:09 | |
| *** yee3793 has joined #openstack-ironic | 05:11 | |
| *** xuao has joined #openstack-ironic | 05:12 | |
| *** francoblanco_ has joined #openstack-ironic | 05:13 | |
| *** NobodyCam_ has joined #openstack-ironic | 05:13 | |
| *** kozhukalov_ has joined #openstack-ironic | 05:13 | |
| *** betherly_ has joined #openstack-ironic | 05:13 | |
| *** romcheg_ has joined #openstack-ironic | 05:13 | |
| *** crinkle_ has joined #openstack-ironic | 05:13 | |
| *** serverascode_ has joined #openstack-ironic | 05:15 | |
| *** tiendc has quit IRC | 05:15 | |
| *** hwoarang_ has joined #openstack-ironic | 05:15 | |
| *** ijw has quit IRC | 05:15 | |
| *** zer0c00l has joined #openstack-ironic | 05:16 | |
| *** Guest66676 has joined #openstack-ironic | 05:16 | |
| *** igordcar1 has joined #openstack-ironic | 05:16 | |
| *** pleia2_ has joined #openstack-ironic | 05:16 | |
| *** lynxman_ has joined #openstack-ironic | 05:17 | |
| *** xilet has joined #openstack-ironic | 05:17 | |
| *** stevebak` has joined #openstack-ironic | 05:19 | |
| *** yhvh- has joined #openstack-ironic | 05:19 | |
| *** ionutbalutoiu_ has joined #openstack-ironic | 05:19 | |
| *** another_larsks has joined #openstack-ironic | 05:19 | |
| *** jlvillal_ has joined #openstack-ironic | 05:19 | |
| *** alaski_ has joined #openstack-ironic | 05:19 | |
| *** sirushti_ has joined #openstack-ironic | 05:20 | |
| *** hecliunyx_ has joined #openstack-ironic | 05:20 | |
| *** hwoarang has quit IRC | 05:21 | |
| *** kun_huang has quit IRC | 05:21 | |
| *** betherly has quit IRC | 05:21 | |
| *** manjeets has quit IRC | 05:21 | |
| *** igordcard has quit IRC | 05:21 | |
| *** serverascode has quit IRC | 05:21 | |
| *** kozhukalov has quit IRC | 05:21 | |
| *** romcheg has quit IRC | 05:21 | |
| *** lekha has quit IRC | 05:21 | |
| *** lynxman has quit IRC | 05:21 | |
| *** NobodyCam has quit IRC | 05:21 | |
| *** jlvillal has quit IRC | 05:21 | |
| *** tangchen has quit IRC | 05:21 | |
| *** sirushti has quit IRC | 05:21 | |
| *** xilet_ has quit IRC | 05:21 | |
| *** yee379 has quit IRC | 05:21 | |
| *** zer0c00l_ has quit IRC | 05:21 | |
| *** larsks has quit IRC | 05:21 | |
| *** caiobo has quit IRC | 05:21 | |
| *** vsaienk0 has quit IRC | 05:21 | |
| *** kragniz has quit IRC | 05:21 | |
| *** hogepodge has quit IRC | 05:21 | |
| *** stevebaker has quit IRC | 05:21 | |
| *** hecliunyx has quit IRC | 05:21 | |
| *** devananda|bbl has quit IRC | 05:21 | |
| *** crinkle has quit IRC | 05:21 | |
| *** francoblanco has quit IRC | 05:21 | |
| *** yuriyz has quit IRC | 05:21 | |
| *** pleia2 has quit IRC | 05:21 | |
| *** alaski has quit IRC | 05:21 | |
| *** Guest66666 has quit IRC | 05:21 | |
| *** yhvh has quit IRC | 05:21 | |
| *** ionutbalutoiu has quit IRC | 05:21 | |
| *** SpamapS has quit IRC | 05:21 | |
| *** logan- has quit IRC | 05:21 | |
| *** ionutbalutoiu_ is now known as ionutbalutoiu | 05:21 | |
| *** lynxman_ is now known as lynxman | 05:21 | |
| *** manjeets has joined #openstack-ironic | 05:21 | |
| *** romcheg_ is now known as romcheg | 05:21 | |
| *** sirushti_ is now known as sirushti | 05:21 | |
| *** logan- has joined #openstack-ironic | 05:21 | |
| *** tiendc has joined #openstack-ironic | 05:21 | |
| *** kozhukalov_ is now known as kozhukalov | 05:22 | |
| *** NobodyCam_ is now known as NobodyCam | 05:22 | |
| *** francoblanco_ is now known as francoblanco | 05:22 | |
| *** betherly_ is now known as betherly | 05:23 | |
| *** jlvillal_ is now known as jlvillal | 05:23 | |
| *** jlvillal is now known as Guest97128 | 05:23 | |
| *** lekha has joined #openstack-ironic | 05:26 | |
| *** SpamapS has joined #openstack-ironic | 05:27 | |
| *** caiobo has joined #openstack-ironic | 05:28 | |
| *** devananda|bbl has joined #openstack-ironic | 05:28 | |
| *** tangchen has joined #openstack-ironic | 05:28 | |
| *** kun_huang has joined #openstack-ironic | 05:28 | |
| *** yuriyz has joined #openstack-ironic | 05:28 | |
| *** vsaienk0 has joined #openstack-ironic | 05:28 | |
| *** kragniz has joined #openstack-ironic | 05:29 | |
| *** rcernin has joined #openstack-ironic | 05:30 | |
| *** serverascode_ is now known as serverascode | 05:31 | |
| *** rbartal has joined #openstack-ironic | 05:34 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: https://review.openstack.org/376088 | 05:35 |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic management I/F for Inject NMI https://review.openstack.org/348191 | 05:39 |
| *** yonglihe has joined #openstack-ironic | 05:39 | |
| *** amotoki has joined #openstack-ironic | 05:50 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/python-ironicclient: TrivialFix Remove white space between print and () https://review.openstack.org/376185 | 05:50 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: TrivialFix Using sys.exit(main()) instead of main() https://review.openstack.org/376088 | 05:54 |
| *** vikrant has quit IRC | 06:06 | |
| *** mjura has joined #openstack-ironic | 06:07 | |
| *** ccamacho has joined #openstack-ironic | 06:08 | |
| *** tangchen has quit IRC | 06:09 | |
| *** ijw has joined #openstack-ironic | 06:12 | |
| *** makowals has joined #openstack-ironic | 06:14 | |
| *** mjura has quit IRC | 06:14 | |
| *** ijw has quit IRC | 06:18 | |
| *** vikrant has joined #openstack-ironic | 06:22 | |
| *** tangchen has joined #openstack-ironic | 06:26 | |
| *** mjura has joined #openstack-ironic | 06:27 | |
| *** Sukhdev has quit IRC | 06:35 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: Remove unused method is_valid_ipv6_cidr https://review.openstack.org/376214 | 06:43 |
| *** pcaruana has joined #openstack-ironic | 06:45 | |
| *** oz0ne has joined #openstack-ironic | 06:48 | |
| *** oz0ne has left #openstack-ironic | 06:52 | |
| openstackgerrit | Moshe Levi proposed openstack/ironic-python-agent: WIP Infiniband Support https://review.openstack.org/334816 | 06:56 |
| *** tangchen has quit IRC | 06:56 | |
| openstackgerrit | Moshe Levi proposed openstack/ironic-python-agent: Infiniband Support for IPA https://review.openstack.org/334816 | 06:57 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Remove import Object CONF https://review.openstack.org/376229 | 07:05 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url https://review.openstack.org/352289 | 07:06 |
| *** ijw has joined #openstack-ironic | 07:14 | |
| *** makowals has quit IRC | 07:14 | |
| *** ijw has quit IRC | 07:19 | |
| *** jtomasek has quit IRC | 07:22 | |
| *** vikrant has quit IRC | 07:27 | |
| *** jaosorior has joined #openstack-ironic | 07:29 | |
| *** vikrant has joined #openstack-ironic | 07:31 | |
| *** vikrant is now known as vikrant|lunch | 07:31 | |
| *** tiendc_ has joined #openstack-ironic | 07:32 | |
| *** tiendc has quit IRC | 07:33 | |
| *** jpich has joined #openstack-ironic | 07:42 | |
| dtantsur | Morning Ironic | 07:49 |
| *** vsaienk0 has quit IRC | 07:55 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:01 | |
| *** athomas has joined #openstack-ironic | 08:01 | |
| *** ohamada has joined #openstack-ironic | 08:06 | |
| tuanla | Morning Ironic | 08:07 |
| *** phuongnh has joined #openstack-ironic | 08:07 | |
| *** ijw has joined #openstack-ironic | 08:10 | |
| *** ijw has quit IRC | 08:18 | |
| tuanla | hi dtantsur, Did you seen https://bugs.launchpad.net/ironic/+bug/1626453 | 08:19 |
| openstack | Launchpad bug 1626453 in Ironic "Ironic fails to force boot device on ipmitool driver." [Undecided,New] | 08:19 |
| dtantsur | tuanla, hi, I haven't seen it. I think we don't set "persistent" for reasons mentioned in https://github.com/openstack/ironic/blob/57838f01da722cd897bb748f7ef19b9a17233ecf/ironic/drivers/modules/ipminative.py#L460-L461 | 08:22 |
| tuanla | Hi dtantsur, I test my hardware server but it have some proplem relevant ipmitool. | 08:23 |
| *** ijw has joined #openstack-ironic | 08:23 | |
| tuanla | dtantsur, when ipmitool set force pxe it will remove orther option for booting such as disk, CD | 08:24 |
| tuanla | dtantsur: there for after that set "disk' boot will don't work. | 08:25 |
| patchbot | Error: No closing quotation | 08:25 |
| dtantsur | tuanla, I don't think we can fix ipmitool behavior | 08:25 |
| dtantsur | unless you know an ipmitool command that fixes it, of course | 08:25 |
| tuanla | Yes dtantsur. I wonder is your lab have similar bugs in ipmitool. | 08:28 |
| dtantsur | never noticed anything like that, sorry | 08:28 |
| tuanla | dtantsur, If you have freetime please take a look http://paste.openstack.org/show/582913/ for more detail | 08:30 |
| vdrok | good morning dtantsur and tuanla | 08:31 |
| tuanla | good morning vdrok | 08:31 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url https://review.openstack.org/352289 | 08:31 |
| *** vikrant|lunch has quit IRC | 08:31 | |
| pas-ha | morning Ironic | 08:32 |
| tuanla | morning pas-ha | 08:32 |
| *** vikrant has joined #openstack-ironic | 08:32 | |
| *** vaggarwa_ has joined #openstack-ironic | 08:33 | |
| pas-ha | congrats (or condolences :) ) to jroll re re-election | 08:33 |
| dtantsur | morning vdrok, pas-ha | 08:34 |
| dtantsur | lol | 08:34 |
| dtantsur | yeah, congrats jroll, keep rocking :) | 08:34 |
| *** ijw has quit IRC | 08:37 | |
| *** tangchen has joined #openstack-ironic | 08:37 | |
| *** rbartal has quit IRC | 08:39 | |
| *** Romanenko_K has joined #openstack-ironic | 08:39 | |
| vdrok | morning pas-ha | 08:41 |
| *** e0ne has joined #openstack-ironic | 08:41 | |
| *** derekh has joined #openstack-ironic | 08:43 | |
| *** vaggarwa_ has quit IRC | 08:44 | |
| openstackgerrit | Markos Chandras proposed openstack/bifrost: Allow users to change the default provisioning timeout value https://review.openstack.org/376303 | 08:48 |
| dtantsur | devananda|bbl, FYI https://bugs.launchpad.net/ironic/+bug/1625804 | 08:48 |
| openstack | Launchpad bug 1625804 in Ironic "Cannot generate policy file" [Low,Triaged] | 08:48 |
| *** e0ne has quit IRC | 08:51 | |
| *** rbartal has joined #openstack-ironic | 08:52 | |
| *** baoli has joined #openstack-ironic | 08:54 | |
| *** baoli has quit IRC | 08:59 | |
| *** e0ne has joined #openstack-ironic | 09:01 | |
| *** causten has joined #openstack-ironic | 09:04 | |
| sergek | morning dtantsur , vdrok , pas-ha , tuanla | 09:05 |
| tuanla | morning sergek | 09:06 |
| dtantsur | hey sergek | 09:07 |
| sergek | :) | 09:07 |
| *** causten has quit IRC | 09:09 | |
| *** sambetts_ is now known as sambetts | 09:11 | |
| sambetts | Morning all | 09:11 |
| dtantsur | morning sambetts | 09:12 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic: Test ML2 Neutron Events feature https://review.openstack.org/370016 | 09:14 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Force persisten options in node_set_boot_device https://review.openstack.org/376322 | 09:15 |
| tuanla | morning sambetts | 09:15 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Force persistent options in node_set_boot_device https://review.openstack.org/376322 | 09:17 |
| *** jistr has quit IRC | 09:20 | |
| vdrok | morning sergek and sambetts | 09:23 |
| sergek | morning sambetts | 09:25 |
| tangchen | morning guys :) | 09:30 |
| tangchen | dtantsur, vdrok, lucasagomes: Hi, I saw this in the bug list. Do we still need to improve cmds to support accepting multiple nodes ? If so, I think I can take it over and continue to work on it. | 09:31 |
| tangchen | dtantsur, vdrok, lucasagomes: Working on the osc plugin commands should be enough, I think. | 09:32 |
| tangchen | wait a min, pasting the link | 09:32 |
| tangchen | dtantsur, vdrok, lucasagomes: https://bugs.launchpad.net/python-ironicclient/+bug/1526490 | 09:34 |
| openstack | Launchpad bug 1526490 in python-ironicclient "[RFE] support multi nodes operations" [Wishlist,In progress] - Assigned to Haomeng,Wang (whaom) | 09:34 |
| dtantsur | tangchen, yeah, this seems stuck, so feel free to overtake | 09:35 |
| tangchen | dtantsur: Thanks. Starting it. :) | 09:35 |
| *** rbartal has quit IRC | 09:42 | |
| *** ijw has joined #openstack-ironic | 09:43 | |
| *** ijw has quit IRC | 09:49 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/python-ironicclient: Trivialfix Unified formats of log_xxx function https://review.openstack.org/376343 | 09:52 |
| *** rbartal has joined #openstack-ironic | 09:53 | |
| *** daemontool has joined #openstack-ironic | 09:54 | |
| *** tiendc_ has quit IRC | 10:04 | |
| *** sdake has joined #openstack-ironic | 10:06 | |
| vdrok | morning tangchen | 10:07 |
| tangchen | morning vdrok. | 10:09 |
| openstackgerrit | Merged openstack/python-ironicclient: TrivialFix Remove white space between print and () https://review.openstack.org/376185 | 10:16 |
| *** aarefiev_ is now known as aarefiev | 10:17 | |
| aarefiev | morning all! | 10:18 |
| dtantsur | morning aarefiev | 10:18 |
| tangchen | aarefiev: morning. :) | 10:19 |
| tangchen | aarefiev: Would you please help to check if I made this patch correct ? https://review.openstack.org/#/c/349421/ and https://review.openstack.org/#/c/356233/ | 10:20 |
| patchbot | patch 349421 - ironic - Prepare for functional tests | 10:20 |
| patchbot | patch 356233 - openstack-infra/project-config - Enable ironic functional test | 10:20 |
| tangchen | aarefiev: I'm not sure how to test if it works. | 10:20 |
| tangchen | aarefiev: I just did "check experimental", but it didn't work, I think. | 10:21 |
| aarefiev | tangchen: yeah, we need to configure functional job first, so my question here is do we have community agreement what exactly these tests will cover? | 10:24 |
| aarefiev | job configuration will depend on this solution | 10:24 |
| tangchen | aarefiev: I have discussed the functional tests things with cores. It is just that Ironic is now mixing its unit and functional tests together. I'm spliting them. | 10:26 |
| tangchen | aarefiev: The functional tests will first cover api tests. | 10:26 |
| tangchen | aarefiev: It will cover more, but to keep things easy, I'd like to try to cover api tests first. | 10:27 |
| *** jtomasek has joined #openstack-ironic | 10:27 | |
| aarefiev | tangchen: ok, I have patch for just running functional tox env https://review.openstack.org/#/c/374785/ | 10:27 |
| patchbot | patch 374785 - openstack-infra/project-config - Add lightweight ironic functional job | 10:27 |
| aarefiev | I can update it | 10:28 |
| tangchen | aarefiev: Sorry, do you mean you can enable Ironic functional test in you patch ? Or would you please take a look at my patch first ? I'm not sure if my patch is needed. | 10:30 |
| *** xuao has quit IRC | 10:30 | |
| aarefiev | tangchen: aha, sorry missed project config one, you don't need devstack for now, all you need is run functional tox env | 10:33 |
| aarefiev | but I'd like to hear jroll opinions first, maybe dtantsur knows more about functional tests decision too | 10:34 |
| * dtantsur does not actually | 10:34 | |
| dtantsur | but I definitely prefer to not requiring devstack for functional testing | 10:34 |
| aarefiev | +1 | 10:35 |
| tangchen | dtantsur, aarefiev: Yes. Ironic functional is different from tempest tests. It faked a service itself. | 10:35 |
| mgould | morning Ironic, happy Monday | 10:35 |
| dtantsur | e.g. ironic-inspector functional tests can be run locally by 'tox -efunc' | 10:35 |
| dtantsur | morning mgould | 10:35 |
| tangchen | dtantsur, aarefiev: Yes, it is. | 10:36 |
| aarefiev | https://review.openstack.org/#/c/374785/ | 10:36 |
| patchbot | patch 374785 - openstack-infra/project-config - Add lightweight ironic functional job | 10:36 |
| aarefiev | this wip patch should help with this | 10:37 |
| tangchen | aarefiev: So, what I did in ( https://review.openstack.org/#/c/356233/) should be merged into your wip patch ? Or my patch is not needed ? | 10:39 |
| patchbot | patch 356233 - openstack-infra/project-config - Enable ironic functional test | 10:39 |
| aarefiev | tangchen: your patch configure devstack | 10:40 |
| aarefiev | and you don't need it | 10:40 |
| *** causten has joined #openstack-ironic | 10:41 | |
| tangchen | aarefiev: Oh...I see. Thanks. :) | 10:42 |
| aarefiev | looks like that job was configured for running tempest functional tests against devstack | 10:42 |
| *** baoli has joined #openstack-ironic | 10:43 | |
| aarefiev | I don't know what the plan was | 10:43 |
| tangchen | aarefiev: Sorry, there was no plan for the devstack. It was just that I didn't know how to make jenkins run Ironic functional test, and I tried this patch. :) | 10:44 |
| tangchen | aarefiev: Obviously it is incorrect. | 10:44 |
| tangchen | aarefiev: So please ignore it. :) | 10:44 |
| tangchen | aarefiev: What I want to do is make this patch run by jenkins. https://review.openstack.org/#/c/349421/ | 10:45 |
| patchbot | patch 349421 - ironic - Prepare for functional tests | 10:45 |
| *** ijw has joined #openstack-ironic | 10:45 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Add read and execute perms to httpboot and tftpboot folder https://review.openstack.org/374015 | 10:46 |
| vdrok | morning mgould | 10:47 |
| aarefiev | tangchen: sure, and my patch should help you, it runs new tox env you've created | 10:47 |
| tangchen | aarefiev: That's great. :) Thank you very much. | 10:48 |
| *** baoli has quit IRC | 10:48 | |
| *** tuanla has quit IRC | 10:50 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Add read and execute perms to httpboot and tftpboot folder https://review.openstack.org/374015 | 10:50 |
| *** ijw has quit IRC | 10:51 | |
| *** hoangcx has quit IRC | 10:54 | |
| *** milan has joined #openstack-ironic | 11:01 | |
| *** tangchen has quit IRC | 11:03 | |
| milan | morning Ironic, and a happy Monday! :) | 11:03 |
| dtantsur | hey milan | 11:04 |
| *** sdake has quit IRC | 11:04 | |
| *** sdake_ has joined #openstack-ironic | 11:04 | |
| milan | dtantsur, hey man! :) | 11:04 |
| mgould | morning milan | 11:04 |
| milan | morning mgould! :) | 11:04 |
| milan | sambetts, hey! How was the PTO? ;) | 11:05 |
| *** phuongnh has quit IRC | 11:06 | |
| sambetts | milan: Hey! it was good thank :D | 11:07 |
| *** vikrant has quit IRC | 11:07 | |
| milan | cool :) | 11:07 |
| milan | sambetts, so you good to review :P | 11:07 |
| sambetts | ;) yup | 11:08 |
| milan | nice! :D | 11:08 |
| dtantsur | sambetts, RUN | 11:08 |
| milan | :D | 11:09 |
| milan | lol | 11:09 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url https://review.openstack.org/352289 | 11:09 |
| sambetts | hahahaha | 11:10 |
| dtantsur | sambetts, so, re LLDP commands spec... the availability of the data is not directly tied to hooks | 11:12 |
| dtantsur | sambetts, and pci-devices uses IDs, it does not convert anything to names | 11:13 |
| dtantsur | (we could create a new hook doing that of course) | 11:13 |
| sambetts | yeah, so in my suggestion, the CLI wouldn't be aware of the hooks existance only that the data is present in the GET /v1/introspection/<Node ID>/data/readable output | 11:14 |
| vdrok | morning milan | 11:15 |
| milan | morning vdrok! :) | 11:15 |
| dtantsur | sambetts, well, 1. this will require uglier CLI design, 2. data does not depend on hooks | 11:16 |
| sambetts | dtantsur: the PCI-devices hook converts the vendor IDs to aliases | 11:16 |
| dtantsur | sambetts, could you point me where? I remember it only setting capabilities based on IDs? | 11:17 |
| sambetts | dtantsur: https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/plugins/pci_devices.py#L81 | 11:17 |
| sambetts | dtantsur: self.aliases | 11:17 |
| *** tangchen has joined #openstack-ironic | 11:17 | |
| dtantsur | sambetts, this is capabilities, not a ID->name conversion | 11:18 |
| dtantsur | also it will make CLI output configuration-dependent. e.g. if I call a capability for device 1234 my_device and you will call it has_my_device | 11:18 |
| sambetts | dtantsur: oh your right, my mistake I thought they were doing the id-> name conversion | 11:18 |
| dtantsur | a middle ground could be as follows: we get a new hook for TLV -> (name, value) transition. if the converted data is there, CLI uses it. if not, CLI converts only the standard data itself. | 11:21 |
| sambetts | we'd still need an API to get the processed data then? | 11:21 |
| *** vikrant has joined #openstack-ironic | 11:22 | |
| * milan thinks about tcpdump | uuencode > Swift the packets? | 11:23 | |
| lucasagomes | morning all | 11:23 |
| milan | morning lucasagomes! :) | 11:23 |
| sambetts | I suggest a GET /v1/introspection/<Node ID>/data/processed API, which returns a JSON output built by the processing hooks which can then be handled in the CLI | 11:23 |
| milan | so that one could query the raw LLDP | 11:23 |
| sambetts | on a openstack baremetal instrospection processed <uuid> | 11:24 |
| sambetts | CLI command | 11:24 |
| sambetts | that accepts --<field name> as filters | 11:24 |
| mgould | sambetts: is this about https://review.openstack.org/#/c/374381/ ? | 11:24 |
| patchbot | patch 374381 - ironic-inspector-specs - Display LLDP information via new commands | 11:24 |
| dtantsur | sambetts, how does it work for everything but LLDP? | 11:24 |
| dtantsur | do you suggest creating hooks just for the sake of this API? | 11:25 |
| dtantsur | mgould, yes | 11:25 |
| mgould | bfournie says he'll add an option to address this use-case in a later revision | 11:25 |
| vdrok | morning lucasagomes | 11:25 |
| mgould | [sorry if that was where the conversation started...] | 11:25 |
| dtantsur | sambetts, " openstack baremetal instrospection processed <uuid>" is so much worse than what Bob suggests, I'm not sure it's worth bothering about, sorry | 11:25 |
| vdrok | dtantsur: we can also set https://github.com/openstack-dev/devstack/blob/a2d18484195e700f28e0b7cca1a29e20d9d369b0/lib/nova#L86 right in the job https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L197 | 11:26 |
| vdrok | as devstack-gate patches merge pretty slowly :) | 11:26 |
| dtantsur | the good thing about this proposal now is that it ends up with a reasonable hierarchy of commands... if we again end up with essentially dumping a JSON, maybe it's not worth inventing API/CLI for it | 11:26 |
| * mgould notes that dtantsur mis-spelled the name of his suggested command, which is IMHO a further strike against it... | 11:26 | |
| dtantsur | vdrok, yeah, but it's a much bigger change.. but you're right, devstack-gate review is super slow usually | 11:27 |
| milan | sambetts, dtantsur GET /v1/introspection/<Node ID>/data that would give raw unprocessed data that optionally contains an unencoded tcpdump of that traffic? | 11:27 |
| milan | *lldp | 11:27 |
| sambetts | milan: thats what exists today | 11:27 |
| dtantsur | wait, wut? :) we already have raw LLDP data, we're talking about reporting it to a human being | 11:27 |
| milan | ah, me forgotten we have that | 11:28 |
| mgould | is there no existing tool that can do the decoding? | 11:28 |
| milan | sry | 11:28 |
| vdrok | dtantsur: why? if we want all of them to use configdrive, we just set this (one line) in the base devstack-virtual-ironic and that's it | 11:28 |
| dtantsur | mgould, milan, sorry guys, we started in private :) | 11:28 |
| mgould | np | 11:28 |
| dtantsur | vdrok, we have a few other jobs there, I think.. | 11:28 |
| sambetts | mgould: the format of the LLDP data we get from IPA isn't a raw raw packet, its processed into a dict in { T: LV } format which our processing hook on the inspector side handles | 11:30 |
| milan | so I think the cli might have an option to just dump the raw lldp data so that the user can just pipe it and process vendor specific data in a vendor specific way | 11:30 |
| * mgould wonders if there's a USian equivalent of the British expression "fag-ending", meaning "joining a conversation half-way through and missing early context" (like picking up a cigarette end and smoking it) | 11:30 | |
| milan | sorry if that's already suggested | 11:30 |
| dtantsur | fwiw https://github.com/syakesaba/scapy/blob/master/scapy-com/layers/lldp.py | 11:30 |
| vdrok | dtantsur: hm, right, inspector and grenade jobs are not using that :( | 11:30 |
| dtantsur | milan, we have such option essentially. we're thinking about making bfournie's proposal more generic. | 11:31 |
| mgould | sambetts: right, so the problem is that it's already semi-decoded, so OTS tools won't help | 11:31 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url https://review.openstack.org/352289 | 11:31 |
| milan | scapy is nice, but not sure it's actively developed :-/ | 11:31 |
| milan | dtantsur, ok, I got inspired by Dan Sneddon's comment | 11:32 |
| dtantsur | ah | 11:32 |
| *** seb31 has quit IRC | 11:32 | |
| sambetts | milan, mgould: My problem with the existing spec is that 1. we already have processing hooks to decode the data, so I'm not a fan of duplicating the processing client side, 2. we need to provide a way to extend the processing for vendor specifc TLVs, which we already do with the processing hooks server side, but not in the client | 11:32 |
| dtantsur | this ^^^ | 11:33 |
| dtantsur | my current idea is: we get a new hook for TLV -> (name, value) transition. if the converted data is there, CLI uses it. if not, CLI converts only the standard data itself. | 11:33 |
| dtantsur | this way we don't need API changes, and we can still have specific commands | 11:33 |
| milan | sambetts, but if we allow access to raw data then we needn't enforce custom hooks, or? | 11:33 |
| sambetts | dtantsur: but how does the CLI get that information? | 11:33 |
| dtantsur | sambetts, from stored data | 11:33 |
| dtantsur | just the TLV processing bit moves to a hook | 11:34 |
| sambetts | dtantsur: stored post-processed data? | 11:34 |
| dtantsur | yep | 11:34 |
| sambetts | we don't have an API to get processed data | 11:34 |
| sambetts | thats what I was suggesting | 11:34 |
| milan | sambetts, ++ | 11:34 |
| milan | we have a stale patch for that | 11:34 |
| milan | we could resurrect | 11:34 |
| dtantsur | ugh, wut? we don't have API for processed data? | 11:35 |
| milan | nope | 11:35 |
| sambetts | no, only raw | 11:35 |
| dtantsur | I think we have both | 11:35 |
| dtantsur | what /v1/introspection/UUID/data is then? | 11:35 |
| sambetts | dtantsur: stored raw data | 11:35 |
| sambetts | I thought | 11:35 |
| sambetts | now I'm confused | 11:35 |
| sambetts | :/ | 11:35 |
| dtantsur | I think it's the opposite: we don't provide access to raw data | 11:36 |
| dtantsur | see, data is stored after post hooks: https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/process.py#L272-L273 | 11:36 |
| milan | dtantsur, you're right, it's the processed data | 11:36 |
| milan | https://review.openstack.org/#/c/308379/ | 11:36 |
| patchbot | patch 308379 - ironic-inspector - Replace node data via uuid/data/unprocessed API | 11:36 |
| milan | the patch to introduce (write) access to raw data ^^ | 11:36 |
| dtantsur | right | 11:36 |
| milan | so I'm thinking the client might expose that for vendor specific processing | 11:37 |
| lucasagomes | folks is the grenade job (gate-grenade-dsvm-ironic) working ? | 11:37 |
| dtantsur | lucasagomes, no. totally forgot about it. it's not working. | 11:37 |
| sambetts | dtantsur: https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/process.py#L226 | 11:37 |
| lucasagomes | ack, will update the whiteboard | 11:37 |
| sambetts | dtantsur: we also store it there | 11:37 |
| mgould | morning lucasagomes | 11:37 |
| dtantsur | sambetts, yeah, we store 2 copies, I think we only expose the processed copy | 11:37 |
| milan | so the user would just pipe openstack baremetal introspection data <node_id>| jq '.lldp' | uuedecode | <vendor pipe> | 11:38 |
| lucasagomes | mgould, hi there | 11:38 |
| milan | and introspection interface might expose just the generic tlvs | 11:39 |
| bfournie | good morning team, catching up on the lldp discussion. sambetts thanks for reiewing. | 11:39 |
| dtantsur | milan, it's not impossible now, the problem is that it's ugly :) bfournie wants a ready to use end user tool | 11:39 |
| dtantsur | morning bfournie :) | 11:39 |
| milan | bfournie, morning! :) | 11:39 |
| sambetts | Hi bfournie | 11:39 |
| mgould | morning bfournie | 11:39 |
| milan | dtantsur, I'd love that too, this would be in order not to limit the user if they decide custom data hacking | 11:39 |
| milan | *processing | 11:40 |
| sambetts | I like the idea of pulling the information from the post-processed data instead of the raw data | 11:40 |
| milan | sambetts, doesn't that imply custom hooks to process the vendor-specific stuff? | 11:40 |
| milan | s/impy/enforce | 11:40 |
| sambetts | yeah, but we need that anyway, for adding vendor specific things to the local link connection information required by Ironic for the networking stuff | 11:41 |
| dtantsur | milan, it's exactly what we want to achieve: ability to hook into processing TLVs | 11:41 |
| sambetts | milan: I just don't want to do it twice | 11:41 |
| milan | sambetts, I see, I didn't realise Ironic should care, thanks for pointing out | 11:42 |
| dtantsur | bfournie, do you have the source of your existing too public? | 11:42 |
| bfournie | sambetts: just to clarify, you're suggesting processing and saving the data similar to what was done for chassis/port id? | 11:43 |
| *** sdake_ has quit IRC | 11:43 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/376135 | 11:43 |
| dtantsur | bfournie, processing yes, but not saving in ironic | 11:43 |
| dtantsur | bfournie, so essentially you take lldp data out of introspection_data, do something similar to https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/plugins/local_link_connection.py#L64-L77 and put it back under a new key. this is a new hook. | 11:44 |
| dtantsur | (at least that's how I understand it) | 11:44 |
| bfournie | dtantsur: I can make what I have public, it basically parses TLVs and integrates with cliff, its not integrated with ironic inspector client yet | 11:44 |
| dtantsur | bfournie, sure, just for our reference | 11:45 |
| sambetts | dtantsur: I would suggest we just improve the existing hook to handle all the standard LLDP | 11:45 |
| dtantsur | sambetts, this hook is not always needed, I guess.. though probably it won't hurt too, right? | 11:45 |
| dtantsur | sambetts, I'm just afraid of always enabling it, as it does overwrite some data | 11:45 |
| dtantsur | so I was thinking about lldp_parser hook enabled by default, and local_link_connection hook using its data | 11:46 |
| dtantsur | and then vendor_name_lldp_parse hooking between them to extend the parse data, etc | 11:46 |
| * dtantsur needs an etherpad as a mind map | 11:46 | |
| bfournie | dtantsur: yes I will upload it, give me a few minutes | 11:47 |
| sambetts | dtantsur: then a vendor needs to add 2 additional hooks? One to process the LLDP data and one to set their LLC data? | 11:47 |
| mgould | n00b question: are non-standard TLVs essentially opaque binary strings? | 11:47 |
| dtantsur | sambetts, gimme a second | 11:47 |
| mgould | [the V bit, at least] | 11:47 |
| *** ijw has joined #openstack-ironic | 11:47 | |
| *** jtomasek has quit IRC | 11:48 | |
| sambetts | mgould: the types are non-standard, so the value has no meaning without it | 11:48 |
| mgould | right, OK | 11:49 |
| * mgould was wondering if there was *any* generic processing we could do, but it sounds like "no" | 11:49 | |
| dtantsur | sambetts, mgould, bfournie, something like https://etherpad.openstack.org/p/ironic-inspector-lldp | 11:50 |
| sambetts | dtantsur: I'm into that idea you've suggested, with 2 hooks, because in my case, for Cisco switches I need additional things like the LLC but they aren't from Cisco specifc TLVs, so I'd only need to add a Cisco-LLC hook | 11:50 |
| *** dprince has joined #openstack-ironic | 11:51 | |
| *** ijw has quit IRC | 11:54 | |
| *** alaski_ is now known as alaski | 11:54 | |
| *** vikrant has quit IRC | 11:54 | |
| dtantsur | yeah, I like how it shapes now.. a bit more hooks, but hooks are cheap, I guess :) | 11:55 |
| sambetts | Yeah ++ | 11:55 |
| sambetts | I wonder if we can make the CLI side generic too, e.g. simply provide a nice way to display the processed data | 11:56 |
| TheJulia | Good morning | 11:56 |
| sambetts | Hey TheJulia | 11:56 |
| mgould | morning TheJulia | 11:57 |
| milan | morning TheJulia! :) | 11:57 |
| dtantsur | sambetts, I'm afraid that the generic way will also be less user-friendly.. | 11:57 |
| dtantsur | morning TheJulia | 11:57 |
| aarefiev | Does all ok with swift as storage here? | 11:57 |
| tangchen | morning milan :) | 11:58 |
| dtantsur | aarefiev, sorry? | 11:58 |
| aarefiev | I'm wondering about performance | 11:58 |
| milan | morning tangchen! :) | 11:58 |
| sambetts | aarefiev, dtantsur: Right now all of this requires a swift backend | 11:58 |
| dtantsur | correct | 11:58 |
| dtantsur | I'm not sure how it affects performance though | 11:58 |
| mgould | aarefiev: is this stage ever going to be a bottleneck? | 11:58 |
| aarefiev | Swift is ideal for storing unstructured data | 11:58 |
| bfournie | dtantsur, sambetts: I like the hooks, piece I'm missing is how do we retrieve the processed data for display? | 11:58 |
| milan | bfournie, ++ | 11:59 |
| dtantsur | bfournie, the same way as you do know: via get_data call | 11:59 |
| bfournie | still via ironic inspector? | 11:59 |
| tangchen | lucasagomes, milan: I have reported a bug for the i18n. Please help to check this series of patches. https://review.openstack.org/#/c/357639/ | 11:59 |
| bfournie | ok | 11:59 |
| patchbot | patch 357639 - python-ironicclient - Support i18n for baremetal node cmds | 11:59 |
| lucasagomes | tangchen, will add to the list of things to review | 11:59 |
| milan | tangchen, cool, I'll take a look :) | 11:59 |
| tangchen | lucasagomes, milan: Thank you for the help :) | 11:59 |
| aarefiev | If we need some filtering, how it will deal with it | 12:00 |
| dtantsur | aarefiev, all client-side | 12:00 |
| milan | dtantsur, so on the command line, what would you have to put to fetch a vendor-specific thing? | 12:00 |
| milan | or how the API would handle | 12:00 |
| aarefiev | not sure client side will have good performance on big envs | 12:00 |
| dtantsur | milan, get_data(uuid).get('lldp') :) | 12:01 |
| * milan just cannot imagine it right away | 12:01 | |
| milan | so the vendor specific things will work with the --fields switch too? | 12:02 |
| aarefiev | it would be nice to test it, because I saw problems with client-side filtering few times | 12:02 |
| mgould | another idiot question: does "hook" here mean ironic_inspector.plugins.base.ProcessingHook? | 12:03 |
| milan | mgould, yeah | 12:04 |
| mgould | cool | 12:04 |
| mgould | so the code to handle custom TLVs needs to be installed on the machine running the ironic-inspector service in this design | 12:05 |
| * TheJulia tries to wake up | 12:05 | |
| dtantsur | mgould, yes | 12:05 |
| mgould | TheJulia: https://etherpad.openstack.org/p/ironic-inspector-lldp | 12:05 |
| aarefiev | morning TheJulia | 12:05 |
| * mat128 invites TheJulia to the club | 12:05 | |
| dtantsur | mgould, correct | 12:05 |
| mat128 | TheJulia: freezy morning today | 12:05 |
| mgould | dtantsur: thanks | 12:05 |
| mgould | morning mat128 | 12:05 |
| *** vikrant has joined #openstack-ironic | 12:05 | |
| milan | morning mat128! :) | 12:05 |
| mgould | TheJulia: I'd offer you some coffee, but this pot's already getting cold | 12:05 |
| mat128 | morning everyone o/ | 12:06 |
| TheJulia | mat128: is the idea that people will be wrapping bash around all of the commands to make stuff go? | 12:07 |
| mat128 | TheJulia: what do you mean? | 12:08 |
| mat128 | TheJulia: the syntax? | 12:08 |
| mat128 | TheJulia: if thats what you mean, that should be the way to go since it makes sphinx use the right lexer, and gives us color highlighting | 12:08 |
| TheJulia | more so, I'm just wondering, for some reason in my mind, who is the expected user. A real person or a script | 12:08 |
| dtantsur | sambetts, I've pasted rough examples of the currently proposed commands.. they seem much more user-intuitive to me, than just dumping the whole structure | 12:09 |
| mat128 | TheJulia: I remember reading from the docs guidelines (cant find it atm) that it's intended for a user | 12:09 |
| *** causten has quit IRC | 12:09 | |
| mat128 | TheJulia: so it's fine "declaring" variables in text and using them in example | 12:09 |
| mat128 | TheJulia: they just have to be all caps | 12:09 |
| TheJulia | mat128: that makes sense then, I would just worry about a deployment with a thousand+ nodes | 12:11 |
| bfournie | dtantsur sambetts: I posted link to current prototype code on etherpad, I think a lot of this can be reused if we go with processing hook | 12:12 |
| TheJulia | mat128: and six connected network interfaces | 12:12 |
| jroll | morning y'all | 12:19 |
| jroll | busy in here today :) | 12:19 |
| sambetts | hey jroll | 12:19 |
| jroll | pas-ha: dtantsur: thanks :D | 12:19 |
| milan | morning jroll! :) | 12:19 |
| dtantsur | morning jroll | 12:19 |
| pas-ha | morning jroll | 12:19 |
| TheJulia | good morning jroll | 12:20 |
| lucasagomes | TheJulia, jroll pas-ha morning | 12:20 |
| jroll | \o morning sambetts, milan, dtantsur, pas-ha, TheJulia, lucasagomes, mat128, mgould, tangchen, aarefiev, and anyone I missed :) | 12:21 |
| jroll | (wow, it really is busy today) | 12:21 |
| milan | :) | 12:21 |
| dtantsur | hehe | 12:21 |
| aarefiev | morning jroll | 12:21 |
| milan | jroll, https://etherpad.openstack.org/p/ironic-inspector-lldp < bike shedding of the day ;) | 12:21 |
| jroll | do I want to know? heh | 12:22 |
| tangchen | jroll: morning :) | 12:22 |
| mgould | morning jroll | 12:22 |
| milan | jroll, dunno :D | 12:22 |
| mgould | jroll: where to decode Link Layer Discovery Protocol discovered during introspection and how to display it to users | 12:22 |
| jroll | yeah, seeing that | 12:23 |
| jroll | I would say "just use ironic node-show" but I suspect that isn't what you're looking for | 12:23 |
| TheJulia | mgould: I would almost wonder if it even matters to the user to visually see it as part of any process if things are properly populated to the port in ironic, the things that couldn't be identified I would worry about though | 12:24 |
| mgould | TheJulia: is performance a real concern here? AFAICT we're talking about querying Swift for data we already know | 12:24 |
| mgould | so "fetch it all and filter client-side" sounds fine to me, unless I'm missing something | 12:24 |
| mgould | [from a performance POV if not a UX one] | 12:24 |
| TheJulia | mgould: That is fine, although information overload would be the issue potentially | 12:25 |
| mgould | TheJulia: I think it's meant for debugging deployments, but will defer to bfournie | 12:25 |
| jroll | anecdotally, I use 'ironic node-list --detail | grep foo' all the time | 12:25 |
| jroll | and it's terrible :) | 12:25 |
| TheJulia | jroll: that is awful, you should at least use cut as well ;) | 12:25 |
| jroll | heh | 12:25 |
| dtantsur | TheJulia, jroll, one use case was to check that stuff is connected to correct ports, iirc | 12:26 |
| jroll | TheJulia: the hard part is counting which field is where | 12:26 |
| dtantsur | but yeah, bfournie knows better | 12:26 |
| jroll | dtantsur: hm, we do fine querying ironic for that | 12:26 |
| bfournie | yes, for debugging deployments, to ensure, for example that the vlans set for your deployment have been properly configured on the switch | 12:26 |
| TheJulia | dtantsur: that does make sense, and trying to align into the cleanest view possible makes a lot of sense then as well | 12:26 |
| TheJulia | I guess that would make sense if visible and if the separation of duties between server infrastructure and network infrastructure are such that there is no access to the TOR. | 12:27 |
| TheJulia | jroll: so, script, two variables, grep and wc ;) | 12:28 |
| jroll | TheJulia: lol, nice | 12:29 |
| jroll | it's one of those things you don't know you need until you need it | 12:29 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/376136 | 12:29 |
| * mgould votes for grep/jq/etc over adding a complex ad-hoc query language to the tool | 12:29 | |
| TheJulia | mgould: ++ | 12:29 |
| jroll | agree | 12:30 |
| TheJulia | I can't even begin to think of the number of times I've whipped up some little script to count the number of times xyz was the case | 12:30 |
| TheJulia | Then again, I had an image farm of like 200 million documents that I was responsible for... | 12:30 |
| *** ohamada has quit IRC | 12:32 | |
| *** ohamada has joined #openstack-ironic | 12:32 | |
| *** amotoki has quit IRC | 12:32 | |
| mat128 | TheJulia: http://aadrake.com/command-line-tools-can-be-235x-faster-than-your-hadoop-cluster.html | 12:32 |
| mat128 | ;) | 12:32 |
| dtantsur | lol | 12:32 |
| * mgould runs "grep -c '| grep' .bash_history", and is disappointed to only get 5 | 12:33 | |
| bfournie | btw, good a time as any, very quick introduction, I'm fairly new to Ironic/openstack and have been on the list for a couple months, but not actively participating. Seems to be a great group and looking forward to participating more. | 12:33 |
| mat128 | mgould: wc -l .bash_history | 12:33 |
| mat128 | maybe it's not long enough | 12:33 |
| mgould | yeah, only 1000 lines | 12:34 |
| mat128 | mgould: 4 here, 500 lines long | 12:34 |
| mat128 | interesting | 12:34 |
| TheJulia | mat128: not surprising in the slightest. It only gets bad when you start writing super optimized code to know how to be the nicest to your distributed filesystem to complete the task in under a few weeks | 12:34 |
| lucasagomes | mat128, nice I read that awhile ago | 12:35 |
| sambetts | dtantsur: sorry, I need to brb to a downstream thing | 12:35 |
| jroll | bfournie: we try <3 | 12:35 |
| dtantsur | sambetts, sure. maybe we should ask bfournie to move what we agreed on to the spec, and then continue bikeshedding on the CLI? | 12:35 |
| *** ccamacho is now known as ccamacho|lunch | 12:35 | |
| milan | mat128, reminds me of old perl one-liners for protein alignments | 12:36 |
| sambetts | dtantsur: makes sense to me | 12:36 |
| vdrok | good morning TheJulia mat128 and jroll | 12:36 |
| dtantsur | cool | 12:36 |
| mat128 | milan: woha | 12:36 |
| milan | bfournie, welcome! :) | 12:36 |
| mat128 | o/ vdrok | 12:36 |
| jroll | hey vdrok :) | 12:36 |
| bfournie | dtantsur, sambetts: sure, still grokking it but think I got most of it | 12:36 |
| dtantsur | bfournie, welcome again :) I think you can move the hook bits to the spec, and leave CLI itself alone for now, until we figure out if we can make it generic | 12:36 |
| dtantsur | lucasagomes, btw you can remove -2 from https://review.openstack.org/#/c/366742/ | 12:37 |
| patchbot | patch 366742 - ironic-python-agent - Add support for root device hints with operators | 12:37 |
| *** xavierr|afk is now known as xavierr | 12:37 | |
| lucasagomes | dtantsur, ++ ironic-lib was released already right ? | 12:38 |
| xavierr | good morning Ironic o/ | 12:38 |
| milan | morning xavierr! :) | 12:38 |
| *** daemontool has quit IRC | 12:38 | |
| lucasagomes | xavierr, hi there | 12:38 |
| bfournie | milan: thanks! dtantsur: will start pulling it into spec | 12:39 |
| dtantsur | lucasagomes, did it merge before the branching? anyway, you can Depend-On the global-requirements update | 12:39 |
| jroll | lucasagomes: not yet https://review.openstack.org/#/c/366757/ | 12:39 |
| patchbot | patch 366757 - releases - ironic-lib 2.2.0 | 12:39 |
| dtantsur | morning xavierr | 12:39 |
| * jroll bugs them | 12:39 | |
| dtantsur | jroll, does it include the whole disk fixes? | 12:39 |
| mgould | morning xavierr | 12:39 |
| dtantsur | if not, please do include them | 12:39 |
| lucasagomes | jroll, dtantsur ops, ok... yeah I need a new release for that (and bump the g-r version too) | 12:39 |
| jroll | dtantsur: yeah, lemme look | 12:40 |
| jroll | okay, updated | 12:40 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Local boot with partition images" https://review.openstack.org/369776 | 12:41 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Enrollment" and "Troubleshooting" sections https://review.openstack.org/369777 | 12:41 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import Install and configure sections https://review.openstack.org/367988 | 12:41 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Bare Metal service overview" https://review.openstack.org/366329 | 12:41 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Image requirements" https://review.openstack.org/369754 | 12:41 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "integration with other OpenStack components" https://review.openstack.org/369510 | 12:41 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Update developer's guide "Installation Guide" link https://review.openstack.org/374296 | 12:41 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Fix absolute links to install-guide.rst in developer docs https://review.openstack.org/374303 | 12:41 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Flavor creation" https://review.openstack.org/369761 | 12:41 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Setup the drivers for the Bare Metal service" https://review.openstack.org/374272 | 12:41 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Add link to new guide in old install guide https://review.openstack.org/369802 | 12:41 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Building or downloading a deploy ramdisk image" https://review.openstack.org/369800 | 12:41 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Trusted boot with partition image" https://review.openstack.org/369801 | 12:41 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import configdrive https://review.openstack.org/369798 | 12:41 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import "Appending kernel parameters to boot instances" https://review.openstack.org/369799 | 12:41 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [install-guide] Import HTTPS, standalone and root device hints https://review.openstack.org/369797 | 12:41 |
| mat128 | lucasagomes: fixed. ^ | 12:41 |
| lucasagomes | mat128, cheers | 12:41 |
| jroll | can we merge those yet :P | 12:43 |
| dtantsur | ugh | 12:43 |
| dtantsur | jroll, do you think https://bugs.launchpad.net/ironic/+bug/1618842 is rather an RFE? | 12:43 |
| openstack | Launchpad bug 1618842 in Ironic "serial over lan console do not support nokeepalive" [Medium,In progress] - Assigned to Xu Ao (xuao) | 12:43 |
| dtantsur | (easily approvable, but still) | 12:44 |
| *** daemontool has joined #openstack-ironic | 12:44 | |
| jroll | dtantsur: ¯\_(ツ)_/¯ | 12:44 |
| jroll | technically, yeah | 12:44 |
| dtantsur | lol | 12:44 |
| jroll | if it's trivial approval, the difference is only a tag :) | 12:44 |
| dtantsur | stats, stats matter!!1 | 12:44 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic: Test ML2 Neutron Events feature https://review.openstack.org/370016 | 12:44 |
| jroll | lol | 12:45 |
| dtantsur | vdrok, are you planning to implement the DEVSTACK_GATE_CONFIGDRIVE change or should I? | 12:47 |
| *** trown is now known as trown|brb | 12:47 | |
| vdrok | dtantsur: as you wish :) jroll thinks it's better to do that in project config tho | 12:48 |
| jroll | yeah, I don't like hard-coding things in d-s-g | 12:48 |
| dtantsur | that's why I mention DEVSTACK_GATE_CONFIGDRIVE, this is what we should set in the jobs | 12:48 |
| jroll | because then we do the two-step to make it configurable | 12:49 |
| jroll | ah :) | 12:49 |
| vdrok | oh, it is in d-s-g already, that's good | 12:49 |
| dtantsur | also our grenade job seems down, I haven't found time to look at it yet | 12:49 |
| dtantsur | jroll, ^^^ | 12:49 |
| jroll | oh fun! | 12:49 |
| jroll | I can take a look | 12:49 |
| vdrok | dtantsur: ok, doing that now | 12:49 |
| jroll | dtantsur: oh! | 12:49 |
| jroll | lol | 12:50 |
| jroll | we're on ocata now | 12:50 |
| jroll | I think I heard some noise about this, will dig in | 12:50 |
| dtantsur | ok :) | 12:50 |
| *** ijw has joined #openstack-ironic | 12:51 | |
| * mgould sees that jroll is once again our fearless leader: congrats! | 12:51 | |
| TheJulia | jroll: congrats | 12:52 |
| jroll | \o/ thanks | 12:53 |
| *** Goneri has joined #openstack-ironic | 12:53 | |
| lucasagomes | jroll, congrats! re grenade Apparently we can't ping one of the instances (I didn't dig much into it yet) | 12:53 |
| milan | congrats jroll! :) | 12:53 |
| jroll | lucasagomes: yeah, that's what nova had problems with last week | 12:54 |
| jroll | I will dig up the info | 12:54 |
| lucasagomes | cheers | 12:54 |
| *** ijw has quit IRC | 12:56 | |
| *** david-lyle has joined #openstack-ironic | 12:56 | |
| *** vishwanathj has joined #openstack-ironic | 12:57 | |
| *** trown|brb is now known as trown | 12:57 | |
| *** sdake has joined #openstack-ironic | 12:57 | |
| *** aparna has quit IRC | 13:00 | |
| vdrok | dtantsur: jroll https://review.openstack.org/376454 | 13:00 |
| patchbot | patch 376454 - openstack-infra/project-config - Use configdrive for ironic tempest jobs | 13:00 |
| vdrok | I don't think we need this for ironic-functional job, as we should not run tempest there I think | 13:00 |
| dtantsur | ++ | 13:01 |
| *** daemontool has quit IRC | 13:01 | |
| jroll | +1 | 13:02 |
| *** sdake_ has joined #openstack-ironic | 13:03 | |
| *** rloo has joined #openstack-ironic | 13:03 | |
| *** daemontool has joined #openstack-ironic | 13:03 | |
| vdrok | dtantsur: hm, or we don't need it for inspector-dsvm and inspector-dsvm-discovery? i see DEVSTACK_GATE_TEMPEST=0 there | 13:04 |
| dtantsur | vdrok, even though they don't use tempest, they do use nova | 13:04 |
| dtantsur | so yes, you need them there | 13:04 |
| vdrok | aha, ok | 13:05 |
| dtantsur | s/them/config drive/ | 13:05 |
| *** sdake has quit IRC | 13:06 | |
| vdrok | sorry, updated commit message to be more precise | 13:07 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Retry to connect to RPC if failed https://review.openstack.org/376462 | 13:07 |
| *** baoli has joined #openstack-ironic | 13:09 | |
| *** vikrant has quit IRC | 13:13 | |
| *** rbudden has joined #openstack-ironic | 13:16 | |
| *** jaosorior has quit IRC | 13:19 | |
| *** jaosorior has joined #openstack-ironic | 13:19 | |
| sambetts | TheJulia: with regards to TinyIPA missing from the docs, I think we just missed that out :/ | 13:21 |
| TheJulia | cinerama`: You might want to take a look at the discussion on https://review.openstack.org/#/c/376303/1 when you have a few minutes | 13:24 |
| patchbot | patch 376303 - bifrost - Allow users to change the default provisioning tim... | 13:24 |
| *** amotoki has joined #openstack-ironic | 13:26 | |
| *** wajdi has joined #openstack-ironic | 13:27 | |
| *** srobert has joined #openstack-ironic | 13:29 | |
| xavierr | congrats jroll :) | 13:34 |
| *** hshiina has joined #openstack-ironic | 13:34 | |
| jroll | thanks xavierr | 13:34 |
| xavierr | jroll: when ironic newton release will land? | 13:34 |
| jroll | xavierr: it went out last week, thursday morning iirc | 13:34 |
| xavierr | jroll: ok :) | 13:35 |
| lukl1 | dtantsur: hi | 13:35 |
| dtantsur | lukl1, hi | 13:35 |
| lukl1 | I added driver_info to: https://bugs.launchpad.net/ironic/+bug/1626453 | 13:36 |
| openstack | Launchpad bug 1626453 in Ironic "Ironic fails to force boot device on ipmitool driver." [Undecided,Incomplete] - Assigned to Tuan (tuanla) | 13:36 |
| *** Guest97128 is now known as jlvillal | 13:36 | |
| *** xuao has joined #openstack-ironic | 13:36 | |
| lukl1 | Internally I fixed the issue the way I described in the bug, though I'm not sure if that's the real root cause | 13:36 |
| lukl1 | Should I have anything extra defined in driver_info? | 13:37 |
| dtantsur | lukl1, hmm, I wonder if we should get rid of "pop" there | 13:37 |
| dtantsur | though not, it should work anyway | 13:38 |
| lukl1 | My though was the issue was coming from defaulting to persistent=False in the set_boot_device. | 13:38 |
| dtantsur | lukl1, maybe, why do you need it NOT persistent? when exactly does it happen? | 13:39 |
| lukl1 | I need to have it __persistent__, always boot from pxe, no matter how BIOS is configured | 13:40 |
| lukl1 | When I just added ipmi_force_boot_device it was working for 2 reboots and after that ironic node-get-boot-device stated that there is nothing configured | 13:41 |
| *** nicodemos has quit IRC | 13:41 | |
| *** nicodemos has joined #openstack-ironic | 13:41 | |
| dtantsur | lukl1, are you adding --persistent flag to node-set-boot-device? | 13:41 |
| lukl1 | yes | 13:42 |
| dtantsur | strange, I see 'is_next_boot_persistent': False in your driver_internal_info | 13:43 |
| *** tuan_luong has joined #openstack-ironic | 13:43 | |
| tuan_luong | Hello ironic | 13:43 |
| lukl1 | Hello :) | 13:44 |
| lukl1 | regarding is_next_boot_persistent, I noticed that, but didn't manage to find where this value comes from | 13:45 |
| lukl1 | https://github.com/openstack/ironic/search?utf8=%E2%9C%93&q=is_next_boot_persistent | 13:45 |
| lukl1 | It looks that only place where this is set is to set it to False | 13:45 |
| *** wajdi_ has joined #openstack-ironic | 13:46 | |
| tuan_luong | hello lukl1 | 13:47 |
| lukl1 | Hi :) | 13:47 |
| dtantsur | lukl1, yeah, but it should not be set to anything in your case... | 13:47 |
| *** wajdi__ has joined #openstack-ironic | 13:47 | |
| *** sdake has joined #openstack-ironic | 13:47 | |
| tuan_luong | Did you mean value default of function? | 13:48 |
| lukl1 | we're discussing is_next_boot_persistent flag in driver_internal_info structure | 13:49 |
| rloo | hi and morning all, tuan_luong, lukl1, dtantsur, jroll, xavierr, sambetts, TheJulia and everyone else working out there :) | 13:49 |
| dtantsur | morning rloo | 13:49 |
| lukl1 | hello | 13:49 |
| tuan_luong | Hello lukl1, dtantsur : I am a new member but want to join and and learning how to fix a bug. | 13:49 |
| dtantsur | lukl1, yeah, you're not supposed to see it, you should see "persistent_boot_device" there | 13:49 |
| *** wajdi has quit IRC | 13:49 | |
| rloo | jroll: wrt the newton release, maybe send out email about it? (I'm sure you're going to announce in meeting today) | 13:49 |
| jroll | morning rloo :) | 13:49 |
| dtantsur | tuan_luong, I'd recommend you start with low-hanging-fruit tag | 13:50 |
| jroll | rloo: there was an announcement email :) | 13:50 |
| jroll | rloo: I do have a draft going though, with some other things | 13:50 |
| *** ccamacho|lunch is now known as ccamacho | 13:50 | |
| rloo | jroll: the 'normal' announcement, right? was thinking about one from us, mentioning the ocata is open, blah blah. thank you all you wonderful contributers out there, etc. ha ha. | 13:50 |
| tuan_luong | dtantsur, yes, thank you, i will | 13:50 |
| *** sdake_ has quit IRC | 13:50 | |
| rloo | jroll: don't forget to thank your family too :D | 13:51 |
| dtantsur | lukl1, I'm trying to wrap my head around this code.. something does feel wrong there | 13:51 |
| *** sdake_ has joined #openstack-ironic | 13:51 | |
| *** wajdi_ has quit IRC | 13:51 | |
| jroll | rloo: heh, yeah, I could send that quickly - I was going to talk about doing a post-mortem and some other tasks for the next few weeks | 13:52 |
| rloo | jroll: ++ to the other stuff too. but separate emails, too much in one mail doesn't seem to be a good idea. | 13:52 |
| *** ijw has joined #openstack-ironic | 13:52 | |
| *** cinerama` is now known as cinerama | 13:52 | |
| jroll | sure | 13:52 |
| milan | morning rloo! :) | 13:53 |
| milan | and morning tuan_luong and luk11! :) | 13:54 |
| rloo | hi milan. Are congrats in order? or congrats-to-be when you become inspector core :) | 13:54 |
| tuan_luong | morning milan | 13:54 |
| *** sdake has quit IRC | 13:54 | |
| mariojv | morning ironic o/ | 13:54 |
| milan | rloo, :D thanks! I'm fine even with congrats twice! :D | 13:54 |
| rloo | milan: :) | 13:55 |
| *** wajdi__ has quit IRC | 13:55 | |
| rloo | morning mariojv. didn't get a chance to look at your notifications patch last week; on my list for today... | 13:55 |
| mariojv | yuriyz: let me know what you think of my proposal on https://review.openstack.org/#/c/321865/32 - i think this would be OK for searchlight and other use cases too without making the payload too large | 13:55 |
| patchbot | patch 321865 - ironic - Add power state change notifications | 13:55 |
| mariojv | thanks rloo | 13:55 |
| *** wajdi has joined #openstack-ironic | 13:55 | |
| dtantsur | lukl1, it does look like it's impossible to set a persistent boot device with the ipmitool driver... because we do https://github.com/openstack/ironic/blob/1c339b6806580719d20b303d4506c754b9c59e96/ironic/drivers/modules/ipmitool.py#L912 unconditionally, even before reboot | 13:56 |
| dtantsur | hmm, or not. ignore me | 13:57 |
| lukl1 | :) | 13:57 |
| lukl1 | I guess that what ipmi_force_boot_device is for, don't assume that persistent will work after reboot, because BMC forgets it | 13:58 |
| lukl1 | :) | 13:58 |
| *** ijw has quit IRC | 13:58 | |
| *** amotoki_ has joined #openstack-ironic | 13:58 | |
| *** xuao has quit IRC | 14:00 | |
| dtantsur | lukl1, I think we should remove is_next_boot_persistent here https://github.com/openstack/ironic/blob/1c339b6806580719d20b303d4506c754b9c59e96/ironic/drivers/utils.py#L223 | 14:00 |
| dtantsur | I've reproduced the issue locally, I think | 14:00 |
| *** amotoki has quit IRC | 14:01 | |
| lucasagomes | sambetts, not sure if you have seem this: https://bugzilla.redhat.com/show_bug.cgi?id=1379010 (it's about Cisco UCS M3 vs M4) | 14:01 |
| openstack | bugzilla.redhat.com bug 1379010 in openstack-ironic "Cisco UCS 200-M3 blades will not properly boot over the network from IPMI" [Unspecified,New] - Assigned to lmartins | 14:01 |
| dtantsur | lucasagomes, could you please try it? just add something like $ driver_internal_info.pop('is_next_boot_persistent', None) on that branch? | 14:02 |
| dtantsur | oops, that was for lukl1 | 14:02 |
| dtantsur | (autocomplete fail) | 14:02 |
| lucasagomes | :-D | 14:02 |
| *** amotoki_ has quit IRC | 14:05 | |
| dtantsur | "launchpad is temporary unavailable" well, of course... | 14:05 |
| *** rbrud has quit IRC | 14:07 | |
| *** rbrud has joined #openstack-ironic | 14:07 | |
| lucasagomes | it's been a while actually :-/ | 14:08 |
| *** sabeen1 has joined #openstack-ironic | 14:08 | |
| bfournie | sambetts: do you think its important that we parse LLDP-MED TLVs as part of plugin? They're not defined in IEEE 802.1AB-2009. Or maybe have plugin process what is defined in 802.1AB-2009, i.e. mandatary TLVs+basic management+802.1+802.3. Perhaps have another hook for LLDP-MED? | 14:08 |
| dtantsur | tuan_luong, do you still want to work on lukl1's bug? | 14:08 |
| tuan_luong | dtantsur, yes | 14:09 |
| tuan_luong | but i need more information | 14:09 |
| tuan_luong | because some time some work doesn't work because version of hardware | 14:10 |
| dtantsur | tuan_luong, I've mentioned the proposed fix on your patch | 14:10 |
| tuan_luong | dtantsur, Firstly, I want to join and find out what is root cause and fix that. But as you said i should choose another easier | 14:13 |
| dtantsur | tuan_luong, that's true. I suspect I know the cause for this particular issue, though - see in the comments. | 14:14 |
| *** absubram has joined #openstack-ironic | 14:14 | |
| dtantsur | it's fine if you want to pick something else, I'm just asking :) | 14:14 |
| *** moshele has quit IRC | 14:17 | |
| tuan_luong | dtantsur, I not sure i can and can't. Do you mind if i try to fix that. | 14:17 |
| dtantsur | tuan_luong, please do, thanks | 14:17 |
| tuan_luong | but maybe it take a time. I have some experiment in deploy ironic. and stuck in more than 2 months to deploy window | 14:18 |
| openstackgerrit | Markos Chandras proposed openstack/bifrost: Redirect mkisofs and genisoimage stderr/stdout to /dev/null https://review.openstack.org/376534 | 14:19 |
| tuan_luong | now it finish (actually with helping from my parners) | 14:19 |
| lukl1 | dtantsur: where do you want me to add this line? | 14:21 |
| lukl1 | It's already there in #199 line | 14:22 |
| *** rajinir has joined #openstack-ironic | 14:23 | |
| *** tuan_luong has quit IRC | 14:23 | |
| dtantsur | lukl1, yes, but line 199 should not be executed in your case. this field should be reset as soon as you request persistent boot via API. | 14:23 |
| rajinir | mjturek: Thank you. Is the cleanup a separate script like pre_test_hook? | 14:24 |
| lukl1 | So it should be before if statement? line 195? | 14:24 |
| *** beekneemech is now known as bnemec | 14:25 | |
| *** amotoki has joined #openstack-ironic | 14:25 | |
| mjturek | rajinir: yep! Invoked after you're done with the node | 14:25 |
| dtantsur | lukl1, after line 223 | 14:25 |
| sambetts | lucasagomes: I hadn't seen that, your suggestion is a good one with regards to using the UCSM driver, my only other suggestion would be to ensure the firmware on the blade is up to data | 14:32 |
| sambetts | date* | 14:32 |
| lucasagomes | sambetts, ++ | 14:32 |
| lucasagomes | yeah, lemme comment with it as well | 14:32 |
| *** mtanino has joined #openstack-ironic | 14:33 | |
| sambetts | bfournie: I personally don't need the LLDP-MED stuff, we can always stack hooks, e.g. processing-hooks=LLDP-basic, LLDP-med, LLDP-<vendor> etc | 14:33 |
| sambetts | bfournie: then people can enable the hooks they are interested it | 14:34 |
| sambetts | in* | 14:34 |
| *** moshele has joined #openstack-ironic | 14:34 | |
| bfournie | sambetts: thanks, yes I agree. That's how I am updating the spec | 14:34 |
| sambetts | bfournie: awesome :) | 14:35 |
| *** anush_ has joined #openstack-ironic | 14:35 | |
| lukl1 | dtantsur, just checked it and it's not solving the problem | 14:38 |
| dtantsur | hmm, that's bad.. anyway, it's worth fixing | 14:38 |
| lukl1 | :) | 14:38 |
| * dtantsur tries locally again | 14:39 | |
| *** moshele has quit IRC | 14:39 | |
| *** syed_ has joined #openstack-ironic | 14:41 | |
| mgould | is there an easy way to add something to a node's capabilities without having to copy-and-paste all existing values? | 14:41 |
| jroll | mgould: ironic node-update uuid add properites/capabilities/foo=bar? | 14:42 |
| lukl1 | after reboot is_next_boot_persistent=False showed up, next reboot node started from hdd and removed is_next_boot_persistent, ironic node-get-boot-device show pxe again, reboot=start from pxe and is_next_boot_persistent is false again | 14:43 |
| mgould | jroll: doesn't work for me :-( | 14:43 |
| lukl1 | I wonder where is_next_boot_persistent comes from | 14:43 |
| jroll | mgould: whaaat, do you get an error? | 14:43 |
| mgould | yep, one moment... | 14:44 |
| mgould | http://paste.openstack.org/show/582965/ | 14:45 |
| jroll | O_o | 14:45 |
| mgould | ah, it's treating it as a string rather than a nested dict | 14:45 |
| mariojv | i think morgabra_ may have had a fix for this a while back | 14:46 |
| dtantsur | jroll, that won't work | 14:47 |
| mariojv | this one? https://review.openstack.org/#/c/343911/ | 14:47 |
| patchbot | patch 343911 - ironic - Allow ~ in the PATCH path attribute. | 14:47 |
| dtantsur | jroll, this is why I'm so pushing for moving capabilities into a proper table | 14:47 |
| mgould | +1 | 14:47 |
| dtantsur | jroll, mgould, you have to copy all values and hope for better (i.e. no races) | 14:47 |
| jroll | dtantsur: ugh | 14:47 |
| mgould | dtantsur: aaaargh | 14:47 |
| mgould | but thanks :-/ | 14:48 |
| lukl1 | Out of curiosity, where is_next_boot_persistent is set? | 14:48 |
| lukl1 | I can't find any code in neither ironic or python agent | 14:48 |
| *** bnemec has quit IRC | 14:49 | |
| lucasagomes | lukl1, it should be in ironic (or ironic-lib) | 14:53 |
| lucasagomes | lukl1, ironic/drivers/utils.py | 14:54 |
| lukl1 | thanks | 14:54 |
| lukl1 | so, the only correct value for this field is False? | 14:55 |
| *** ijw has joined #openstack-ironic | 14:55 | |
| lukl1 | that's misleading :P | 14:55 |
| lucasagomes | lukl1, I'm not very familiar, but it could... It could be the case if set it's False if not it's True | 14:55 |
| lucasagomes | tho yeah | 14:55 |
| lucasagomes | totally misleading | 14:55 |
| lucasagomes | familiar with it* | 14:55 |
| dtantsur | lukl1, I've reproduced the problem, for some reason it changes to False after reboot | 14:56 |
| *** daemontool_ has joined #openstack-ironic | 14:57 | |
| *** hogepodge has joined #openstack-ironic | 14:58 | |
| * dtantsur has to jump on a meeting, sorry | 14:58 | |
| *** bnemec has joined #openstack-ironic | 14:58 | |
| rloo | heads up, i'm approving the install guide patches... | 14:58 |
| *** bdeschenes has joined #openstack-ironic | 14:58 | |
| jroll | \o/ | 14:58 |
| *** daemontool has quit IRC | 14:59 | |
| *** sdake_ has quit IRC | 14:59 | |
| *** ijw has quit IRC | 15:00 | |
| rloo | mat128: if you're around and have a few min to fix this nit that apparently bothers me: https://review.openstack.org/#/c/369802/15/doc/source/deploy/install-guide.rst | 15:03 |
| patchbot | patch 369802 - ironic - Add link to new guide in old install guide | 15:03 |
| *** mjura has quit IRC | 15:04 | |
| *** wajdi_ has joined #openstack-ironic | 15:06 | |
| *** jlvillal has quit IRC | 15:07 | |
| *** wajdi__ has joined #openstack-ironic | 15:07 | |
| *** jlvillal has joined #openstack-ironic | 15:08 | |
| openstackgerrit | Stenio Araujo proposed openstack/ironic: Fixing Typo https://review.openstack.org/376564 | 15:08 |
| *** wajdi has quit IRC | 15:09 | |
| vdrok | the gates seem to be broken with zuul cloner issue :( | 15:10 |
| vdrok | morning rloo | 15:10 |
| *** wajdi_ has quit IRC | 15:10 | |
| sambetts | vdrok: I also see that issue | 15:11 |
| vdrok | yep, and the solution seems to be 'wait for an hour and recheck' | 15:11 |
| sambetts | :( | 15:11 |
| xavierr | morning rloo :) | 15:13 |
| vdrok | morning xavierr | 15:13 |
| rloo | vdrok: hi. so sad about gate, and launchpad. anything else broken? maybe i should take the week off. ha ha. | 15:15 |
| vdrok | grenade too :) | 15:15 |
| *** sdake has joined #openstack-ironic | 15:15 | |
| *** rpioso has joined #openstack-ironic | 15:18 | |
| rajinir | mjturek: I have some parsing errors with the pre_test_hook script you posted. Are you running it in this shell? #!/bin/bash -xe | 15:21 |
| rajinir | mjturek: May be ${parameter} vs $parameter | 15:25 |
| openstackgerrit | boden proposed openstack/ironic: Replace retrying with tenacity https://review.openstack.org/376574 | 15:26 |
| *** openstackstatus has joined #openstack-ironic | 15:27 | |
| *** ChanServ sets mode: +v openstackstatus | 15:27 | |
| *** harlowja_at_home has joined #openstack-ironic | 15:36 | |
| *** ccamacho is now known as ccamacho|away | 15:36 | |
| *** athomas has quit IRC | 15:39 | |
| *** aNuposic has joined #openstack-ironic | 15:39 | |
| rloo | vdrok: do you know if grenade is 'working'? the gate is, but grenade tests failed. should i recheck or wait? | 15:41 |
| vdrok | rloo: nope, it's broken :( | 15:42 |
| rloo | vdrok: :( | 15:42 |
| -openstackstatus- NOTICE: Earlier job failures for "zuul-cloner: error: too few arguments" should now be solved, and can safely be rechecked | 15:42 | |
| vdrok | jroll said it might be issues that nova had recently | 15:42 |
| rloo | jroll, JayF, mat128: wrt the new install guide -- why are we running all the tests for that? shouldn't we block them since it is documentation? https://review.openstack.org/#/c/366329/ | 15:43 |
| patchbot | patch 366329 - ironic - [install-guide] Import "Bare Metal service overview" | 15:43 |
| *** hshiina has quit IRC | 15:44 | |
| vdrok | rloo: seems like install-guide/ is not added to project-config the same way as doc/ is | 15:44 |
| * rloo happy that we got our newton release out last week | 15:44 | |
| vdrok | i can fix that I think | 15:44 |
| rloo | vdrok: we should do that though, right? wanted to check to make sure it wasn't on purpose that we are testing. but am guessing it is cuz we forgot to add it to project-config. | 15:45 |
| rloo | vdrok: yes, please, do the patch :) | 15:45 |
| vdrok | yup, it should not be doing that | 15:45 |
| -openstackstatus- NOTICE: Launchpad SSO is offline, preventing login to https://review.openstack.org/, https://wiki.openstack.org/ and many other sites; no ETA has been provided by the LP admin team | 15:47 | |
| *** ChanServ changes topic to "Launchpad SSO is offline, preventing login to https://review.openstack.org/, https://wiki.openstack.org/ and many other sites; no ETA has been provided by the LP admin team" | 15:47 | |
| *** dprince has quit IRC | 15:47 | |
| jroll | vdrok: I started looking into it, the nova gates are happy now, and I can't find anything useful :( | 15:48 |
| vdrok | rloo: jroll https://review.openstack.org/376603 | 15:48 |
| patchbot | patch 376603 - openstack-infra/project-config - Skip slow ironic tests on install-guide changes | 15:48 |
| jcook | mariojv for notifications spec, why does the end notification have a new target_provision_state? | 15:52 |
| *** rcernin has quit IRC | 15:52 | |
| mjturek | rajinir: checking now | 15:53 |
| jcook | mariojv actually, I think it makes sense now that I think about it. You would potentially have a notification for an intermediate state (start and end) e.g. clean_wait. | 15:54 |
| jcook | is that right? | 15:54 |
| mjturek | rajinir: #! /bin/bash -x | 15:54 |
| * jroll grabbes lunch | 15:55 | |
| jroll | jcook: some states have a target, e.g. deploying has a target of active | 15:56 |
| jroll | yeah you got it | 15:56 |
| jcook | right on | 15:56 |
| *** ijw has joined #openstack-ironic | 15:57 | |
| *** Sukhdev has joined #openstack-ironic | 15:57 | |
| rloo | jroll: am going to block the install guide patches from landing, until vdrok's patch lands. no sense running all those tests on all those patches. | 16:00 |
| vdrok | The base one failed, the other ones won't merge anyway I think | 16:01 |
| vdrok | So they are blocked already :) | 16:01 |
| rloo | vdrok: right, unless someone does a 'recheck'. | 16:01 |
| vdrok | rloo: but it won't help anyway as grenade is down :) | 16:01 |
| *** hshiina has joined #openstack-ironic | 16:02 | |
| rloo | vdrok: i know. another reason i don't want someone to do a 'recheck'. waste of gate resources until grenade is fixed. | 16:02 |
| *** ijw has quit IRC | 16:02 | |
| vdrok | Yeah, that is true | 16:02 |
| *** lucasagomes is now known as lucas-hungry | 16:03 | |
| rloo | vdrok: normally, i wouldn't care so much but other projects may be trying to get their newton releases done, and i know how i felt in the past when the gate was s...l...ooooooooooooo...w | 16:03 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command https://review.openstack.org/251458 | 16:06 |
| *** dprince has joined #openstack-ironic | 16:07 | |
| *** Sukhdev_ has joined #openstack-ironic | 16:08 | |
| *** ChanServ changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic" | 16:09 | |
| -openstackstatus- NOTICE: Launchpad SSO logins are confirmed working correctly again | 16:09 | |
| *** Sukhdev has quit IRC | 16:09 | |
| *** Sukhdev_ is now known as Sukhdev | 16:09 | |
| jroll | rloo: okay | 16:12 |
| *** david-lyle has quit IRC | 16:13 | |
| *** rbartal has quit IRC | 16:13 | |
| *** david-lyle has joined #openstack-ironic | 16:14 | |
| *** anush_ has quit IRC | 16:14 | |
| *** daemontool_ has quit IRC | 16:17 | |
| *** makowals has joined #openstack-ironic | 16:18 | |
| *** sturivnyi_ has joined #openstack-ironic | 16:23 | |
| sturivnyi_ | Hi All=) I have bunch of patches for review https://review.openstack.org/#/q/status:open+project:openstack/python-ironicclient+branch:master+topic:%22functional+CLI+tests%22.Would be grateful if you could review them. Thanks) | 16:23 |
| *** amotoki has quit IRC | 16:24 | |
| *** e0ne has quit IRC | 16:25 | |
| rloo | jroll: is this a known bug (and maybe I've asked this in the past), the newton release notes include mitaka releases: http://docs.openstack.org/releasenotes/ironic/newton.html | 16:25 |
| jroll | rloo: I think so, not sure though | 16:26 |
| * jroll adds to todo list | 16:26 | |
| rloo | thx jroll | 16:29 |
| *** pcaruana has quit IRC | 16:29 | |
| *** jheroux has joined #openstack-ironic | 16:31 | |
| NobodyCam | Good Morning Ironic'ers | 16:33 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: DO NOT MERGE: testing xenial jobs https://review.openstack.org/376624 | 16:34 |
| *** krtaylor has joined #openstack-ironic | 16:34 | |
| *** harlowja_at_home has quit IRC | 16:39 | |
| mariojv | jcook: yeah, the target state is needed even for .end | 16:39 |
| *** ijw has joined #openstack-ironic | 16:43 | |
| *** jabroney has joined #openstack-ironic | 16:45 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic-inspector-specs: Display LLDP information via new commands https://review.openstack.org/374381 | 16:46 |
| dtantsur | morning NobodyCam | 16:46 |
| *** rbrud has quit IRC | 16:46 | |
| mat128 | rloo: because that commit is touching a .py file, the other commits only run docs jobs | 16:46 |
| *** rama_y has joined #openstack-ironic | 16:46 | |
| NobodyCam | Morning dtantsur :) | 16:47 |
| *** ohamada has quit IRC | 16:47 | |
| bfournie | dtantsur, sambetts, milan, mgould: ^^ lldp spec updated to use processing hooks, no changes to CLI (for now) | 16:47 |
| *** ijw has quit IRC | 16:48 | |
| milan | bfournie, cool, thx, I'm still reviewing the older version, sry about being a bit slow | 16:48 |
| jabroney | Any suggestions on how to trouble-shoot why ironic stopped handing out dhcp leases? Do nodes need to be in a specific state? | 16:48 |
| *** dprince has quit IRC | 16:50 | |
| bfournie | milan: thanks for reviewing, main changes from older version are under "Proposed change" and "Data model impact" | 16:51 |
| jlvillal | jabroney: Ironic doesn't hand out DHCP leases. That is the DHCP server which does that. | 16:52 |
| * jlvillal hopes he isn't confused about that. | 16:52 | |
| mat128 | jabroney, jlvillal: check neutron | 16:52 |
| jlvillal | mat128: +1 | 16:52 |
| *** derekh has quit IRC | 16:53 | |
| *** pleia2_ is now known as pleia2 | 16:53 | |
| milan | bfournie, I'll check new as well | 16:54 |
| mgould | bfournie: looking now, thanks | 16:55 |
| mgould | jlvillal: that's what DHCP servers are *for*, it would be a strange decision to run a DHCP server and duplicate that feature elsewhere :-) | 16:56 |
| *** lucas-hungry is now known as lucasagomes | 16:57 | |
| mgould | NobodyCam: morning | 16:57 |
| NobodyCam | morning mgould :) happy monday :p | 16:57 |
| jlvillal | mgould: Yep | 16:57 |
| jlvillal | Good morning all :) | 16:57 |
| NobodyCam | morning jlvillal :) | 16:57 |
| jlvillal | NobodyCam: :) | 16:57 |
| jabroney | jlvillal: Understood. I thought ironic played a part in the dhcp configuration. | 16:59 |
| *** anush_ has joined #openstack-ironic | 16:59 | |
| jroll | meeting tiiiiime | 16:59 |
| xavierr | o/ | 16:59 |
| xavierr | morning NobodyCam :) | 16:59 |
| *** ijw has joined #openstack-ironic | 16:59 | |
| NobodyCam | morning xavierr :) | 16:59 |
| xavierr | morning vdrok :) | 16:59 |
| mgould | morning xavierr | 16:59 |
| *** gabrielbezerra is now known as gabriel-bezerra | 17:00 | |
| xavierr | mgould, morning o/ | 17:00 |
| vdrok | morning jlvillal and NobodyCam | 17:01 |
| NobodyCam | morning vdrok :) | 17:01 |
| *** wajdi__ has quit IRC | 17:02 | |
| jabroney | * apologizes for interrupting developer-only circle-jerk * | 17:03 |
| *** wajdi has joined #openstack-ironic | 17:03 | |
| *** trown is now known as trown|lunch | 17:03 | |
| *** jabroney has left #openstack-ironic | 17:03 | |
| *** MattMan has quit IRC | 17:03 | |
| *** MattMan has joined #openstack-ironic | 17:03 | |
| * jlvillal is not sure why that person got upset | 17:03 | |
| *** ijw has quit IRC | 17:05 | |
| jroll | :( | 17:06 |
| *** dprince has joined #openstack-ironic | 17:07 | |
| jroll | jlvillal: I suspect they have had previous "devs only" bad experiences with another project | 17:07 |
| *** jaosorior has quit IRC | 17:07 | |
| * jlvillal thought we had attempted to answer their question. /me confused | 17:07 | |
| lucasagomes | haven't seem any other message from him... maybe he sent it to the wrong channel ? :D | 17:07 |
| lucasagomes | oh did we ? | 17:08 |
| lucasagomes | so ¯\_(ツ)_/¯ | 17:08 |
| jroll | don't worry about it, but feel free to bug me if they come back, I'm happy to talk to them | 17:08 |
| *** jpich has quit IRC | 17:10 | |
| *** devananda|bbl is now known as devananda | 17:11 | |
| *** Sukhdev has quit IRC | 17:17 | |
| aNuposic | Hi Folks, I am trying to update a bare metal node by adding an image_source and my image is uploaded in the glance service. Is it not possible to directly use that image? | 17:18 |
| *** cdearborn has joined #openstack-ironic | 17:18 | |
| devananda | aNuposic: you sould be able to simply use the glance image id -- if the ironic service is configured to use / connect to glance | 17:18 |
| mat128 | aNuposic: it should work, do you provide a full URL to the image? | 17:18 |
| lucasagomes | aNuposic, (I'm kinda afk due the upstream meeting)... Are you using ironic in standalone mode ? | 17:19 |
| aNuposic | mat128: devananda: I tried to directly use the image id as add instance_info/image_source=IMAGE_ID but it is throwing "No conductor service registered which supports driver agent_ipmitool" | 17:20 |
| aNuposic | lucasgomes: I am not using stand alone Ironic | 17:20 |
| aNuposic | lucasgomes: I mean I have a devstack setup with other services also running | 17:20 |
| devananda | aNuposic: that error isn't related to the image service or image id | 17:21 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Add link to new guide in old install guide https://review.openstack.org/369802 | 17:21 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Update developer's guide "Installation Guide" link https://review.openstack.org/374296 | 17:21 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Fix absolute links to install-guide.rst in developer docs https://review.openstack.org/374303 | 17:21 |
| devananda | aNuposic: but rather means that there is not an ironic-conductor service running, or the one running is not configured with the "agent_impitool" driver | 17:22 |
| devananda | aNuposic: can you run "ironic driver-list" ? | 17:22 |
| mat128 | aNuposic: try ironic driver-list to see which drivers | 17:22 |
| mat128 | ^this | 17:22 |
| aNuposic | mat128: devananda: lucasgomes: earlier i had mapped the image source to local image file using file: href but then it said only http is allowed so then thought to add in glance | 17:22 |
| devananda | that should return to you a list of all the configured drivers and which hosts are running them | 17:22 |
| aNuposic | mat128: devananda: okay let mme do that | 17:22 |
| aNuposic | mat128: devananda: oh yeah | 17:23 |
| aNuposic | i don't see any driver running | 17:23 |
| aNuposic | strange how come | 17:23 |
| mat128 | aNuposic: ironic-conductor must not be running | 17:23 |
| devananda | aNuposic: look at the status of the ironic-conductor service. my guess: it's not running | 17:23 |
| devananda | mat128: :) | 17:23 |
| aNuposic | mat128: devananda: yeah let me check that and bring the setup workign | 17:24 |
| aNuposic | i thought i have that running as there were before i went over the weekend :) | 17:24 |
| aNuposic | mat128: devananda: lucasgomes: Great it worked | 17:25 |
| aNuposic | yeah the conductor was not running | 17:25 |
| mat128 | aNuposic: glad we could help | 17:26 |
| *** rama_y has quit IRC | 17:26 | |
| *** rbrud has joined #openstack-ironic | 17:26 | |
| *** Romanenko_K has quit IRC | 17:27 | |
| aNuposic | was a quick fix. thnx :) | 17:27 |
| *** harlowja has joined #openstack-ironic | 17:30 | |
| *** e0ne has joined #openstack-ironic | 17:33 | |
| vdrok | is logstash.o.o working for anyone? It does not return me anything | 17:35 |
| * jroll steps away for a little while | 17:35 | |
| aNuposic | mat128: devananda: so after adding the image when i try to deploy the node using node-set-provision-state active I am getting "No VIFs found for the node when attempting to update DHCP boot options" These are the details of my node http://paste.openstack.org/show/582983/ | 17:36 |
| devananda | aNuposic: have you created any Port resources in ironic associated with that Node ? | 17:36 |
| mgould | vdrok: the homepage loads, but shows no data | 17:37 |
| aNuposic | is that it is not finding the network? | 17:37 |
| vdrok | mgould: yup, same for me :( | 17:37 |
| aNuposic | *thinking about port | 17:37 |
| vdrok | kind of hard to debug without it | 17:37 |
| mgould | :-( | 17:37 |
| milan | sambetts, quick opinion query ;) https://review.openstack.org/#/c/375045/3/specs/list-introspection-statuses.rst < infinity/never on the finished_before/_after filters, thx! | 17:37 |
| patchbot | patch 375045 - ironic-inspector-specs - Endpoint to list introspection statuses | 17:37 |
| devananda | aNuposic: also, this looks troubling... ipmi_address': u'127.0.0.1' | 17:37 |
| aNuposic | devananda: i guess not. Yes 127.0.0.1 is because this is devstack setup | 17:38 |
| devananda | aNuposic: are you trying to deploy a (separate) physical machine from your devstack? | 17:38 |
| *** hshiina has quit IRC | 17:39 | |
| aNuposic | devananda: No a VM only of bare metal flavor to say | 17:39 |
| mat128 | aNuposic: are you using Nova? | 17:40 |
| *** rama_y has joined #openstack-ironic | 17:40 | |
| aNuposic | devananda: Yeah i missed to create a network port in ironic and associalte that with the node. I guess i need to do that | 17:40 |
| aNuposic | mat128: What do you mean? I am running nova service but trying to deploy the bare metal node using ironic | 17:41 |
| aNuposic | mat128: with node-set-provision-state active | 17:41 |
| mat128 | aNuposic: Ironic manages bare metal nodes (aka "good old servers"), and an Ironic driver exists for Nova to provision bare metal *instances* | 17:41 |
| mat128 | aNuposic: Nova will take care of creating neutron ports (VIFs) and transitioning to the right provision states | 17:42 |
| mat128 | aNuposic: you can also use Ironic in a standalone mode, but you need to handle all of that | 17:42 |
| * milan gone | 17:42 | |
| mat128 | aNuposic: or use Bifrost | 17:42 |
| milan | Good night Ironic! :) | 17:42 |
| mat128 | o/ milan | 17:42 |
| mgould | good night everyone | 17:42 |
| *** mgould is now known as mgould|afk | 17:43 | |
| mat128 | o/ mgould|afk | 17:43 |
| *** milan is now known as milan|afk | 17:43 | |
| aNuposic | mat128: I am confused. So do i need to use nova to associate port with ironic node? or shall i use ironic port-create to attach port to the node i want? | 17:44 |
| dtantsur | yeah, good night | 17:44 |
| *** dtantsur is now known as dtantsur|afk | 17:44 | |
| aNuposic | mat128: I am not running stand alone | 17:44 |
| mat128 | aNuposic: your node needs ironic ports | 17:44 |
| mat128 | aNuposic: to describe the physical ports | 17:44 |
| mat128 | aNuposic: but Nova will create Neutron ports to describe virtual ports, and will put that in port_vifs | 17:45 |
| aNuposic | mat128: I see. So i will create a port with ironic port-create and that will create ironic port. But i am not clear whether this will be added in port_vifs. | 17:51 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add "agent_last_heartbeat_datetime" field to agent drivers https://review.openstack.org/376978 | 17:53 |
| *** e0ne has quit IRC | 17:54 | |
| openstackgerrit | Merged openstack/bifrost: Add new lines to improve pre-install yaml files https://review.openstack.org/375084 | 17:57 |
| lucasagomes | jroll, https://review.openstack.org/#/c/376990/ when you get some time | 17:58 |
| patchbot | patch 376990 - openstack-infra/project-config - Ironic: Replace *_ssh jobs with *_ipmitool | 17:58 |
| lucasagomes | devananda, dtantsur|afk ^ as well | 17:59 |
| * lucasagomes is not very comfortable working on project-config | 17:59 | |
| *** e0ne has joined #openstack-ironic | 18:00 | |
| mat128 | aNuposic: I have to go in a downstream meeting, Other folks here will help you :) | 18:01 |
| *** mat128 is now known as mat128|mtg | 18:01 | |
| *** srobert has quit IRC | 18:01 | |
| *** anush_ has quit IRC | 18:03 | |
| *** anush_ has joined #openstack-ironic | 18:08 | |
| * sambetts -> home | 18:09 | |
| sambetts | night all | 18:09 |
| *** sambetts is now known as sambetts|afk | 18:09 | |
| *** trown|lunch is now known as trown | 18:10 | |
| *** ijw has joined #openstack-ironic | 18:11 | |
| jlvillal | lucasagomes: I left a comment. Basically I'm wondering about changing the tests for stable/liberty & stable/newton. | 18:11 |
| lucasagomes | jlvillal, ouch ++ thanks | 18:12 |
| jlvillal | lucasagomes: I think we have to keep the old jobs and make them run for the stable releases and mark them for deprecation. | 18:13 |
| lucasagomes | yeah I agree | 18:13 |
| lucasagomes | very good point | 18:13 |
| *** ijw has quit IRC | 18:14 | |
| *** ijw has joined #openstack-ironic | 18:14 | |
| jlvillal | lucasagomes: Life would be easier if we didn't have to worry about those darn stable releases :) | 18:14 |
| *** ijw has quit IRC | 18:15 | |
| lucasagomes | jlvillal, https://twitter.com/pakalupapito/status/681801088333393920 | 18:15 |
| jlvillal | lucasagomes: :) | 18:16 |
| lucasagomes | :D | 18:16 |
| *** ijw has joined #openstack-ironic | 18:17 | |
| *** rook_ is now known as rook | 18:19 | |
| *** ijw has quit IRC | 18:21 | |
| *** ijw has joined #openstack-ironic | 18:24 | |
| wznoinsk | hi all | 18:26 |
| NobodyCam | morning wznoinsk | 18:27 |
| wznoinsk | I'm using pxe_ipmitool driver and just trying one node in boot:local mode, I see some weird issue when trying to launch that ironic node (mitaka), I've deployed with kolla mitaka, the issue I see is http://paste.openstack.org/show/582987/ | 18:29 |
| wznoinsk | so I'm wondering should that combination of driver and property work? | 18:29 |
| wznoinsk | the moment when I add kernel-id and ramdisk-id to the user image it works, I guess never hits that neutron port update code... just wanted to check here while I'm starting to dive into the code | 18:32 |
| mat128|mtg | vdrok, rloo: any traction on https://review.openstack.org/#/c/376603/ with -infra? | 18:32 |
| patchbot | patch 376603 - openstack-infra/project-config - Skip slow ironic tests on install-guide changes | 18:32 |
| rloo | mat128|mtg: need another infra person to review, +A. Do you have any connections there? | 18:33 |
| rloo | mat128|mtg: either that, or fixed grenade jobs... | 18:34 |
| *** e0ne has quit IRC | 18:36 | |
| lucasagomes | wznoinsk, I think currently pxe_ipmitool + whole disk image (when there's no {kernel, ramdisk}_id associated with the image) is broken... but I remember it was a dfiferent error | 18:37 |
| lucasagomes | what is also strange is that you get a Unauthorized error with a 404 code | 18:38 |
| lucasagomes | which makes lil sense... should be 401 | 18:39 |
| *** e0ne has joined #openstack-ironic | 18:39 | |
| wznoinsk | lucasagomes: I'll check my neutron dhcp commands, maybe they have a problem themselves | 18:40 |
| openstackgerrit | Merged openstack/ironic-inspector: Update reno for Newton https://review.openstack.org/374910 | 18:45 |
| *** dprince has quit IRC | 18:52 | |
| *** ccamacho|away has quit IRC | 18:52 | |
| *** anush_ has quit IRC | 18:52 | |
| *** anush__ has joined #openstack-ironic | 18:52 | |
| *** dprince has joined #openstack-ironic | 18:53 | |
| *** moshele has joined #openstack-ironic | 18:53 | |
| *** moshele has quit IRC | 19:02 | |
| *** ijw_ has joined #openstack-ironic | 19:07 | |
| *** ijw has quit IRC | 19:09 | |
| *** e0ne has quit IRC | 19:17 | |
| *** e0ne has joined #openstack-ironic | 19:22 | |
| *** trown is now known as trown|brb | 19:28 | |
| *** joprovost has joined #openstack-ironic | 19:29 | |
| *** moshele has joined #openstack-ironic | 19:29 | |
| *** trown|brb is now known as trown | 19:31 | |
| openstackgerrit | Stenio Araujo proposed openstack/python-oneviewclient: Adds attribute macType to a SPT and method has_physical_mac to Client https://review.openstack.org/377054 | 19:31 |
| *** sdake has quit IRC | 19:33 | |
| *** moshele has quit IRC | 19:35 | |
| *** crinkle_ is now known as crinkle | 19:45 | |
| *** anush__ has quit IRC | 19:45 | |
| *** sdake has joined #openstack-ironic | 19:46 | |
| *** anush_ has joined #openstack-ironic | 19:47 | |
| jcook | mariojv reviewed https://review.openstack.org/#/c/321865/32 | 19:50 |
| patchbot | patch 321865 - ironic - Add power state change notifications | 19:50 |
| *** e0ne has quit IRC | 19:54 | |
| *** anush_ has quit IRC | 19:55 | |
| *** anush_ has joined #openstack-ironic | 19:55 | |
| *** joprovost has quit IRC | 19:57 | |
| *** e0ne has joined #openstack-ironic | 19:59 | |
| *** e0ne has quit IRC | 20:05 | |
| openstackgerrit | Stenio Araujo proposed openstack/ironic: Adds another validation step when using dynamic allocation https://review.openstack.org/377073 | 20:08 |
| *** wajdi has quit IRC | 20:09 | |
| *** wajdi has joined #openstack-ironic | 20:09 | |
| *** e0ne has joined #openstack-ironic | 20:10 | |
| jlvillal | #success Have documented how to pronounce Bifrost :) | 20:19 |
| openstackstatus | jlvillal: Added success to Success page | 20:19 |
| jlvillal | heh | 20:19 |
| jroll | jlvillal: so what did we end up with? :P | 20:19 |
| mat128|mtg | jlvillal: where is it? | 20:19 |
| jroll | bye-frost? | 20:19 |
| *** mat128|mtg is now known as mat128 | 20:19 | |
| jlvillal | jroll: Yep | 20:19 |
| jroll | \o/ | 20:19 |
| jroll | I feel so vindicated \o/ | 20:19 |
| jlvillal | mat128: In the docs. First page | 20:19 |
| jlvillal | mat128: https://github.com/openstack/bifrost | 20:20 |
| mat128 | nice | 20:20 |
| jlvillal | I just wanted to try the success thingy :) | 20:20 |
| lucasagomes | hah cool | 20:20 |
| lucasagomes | folks, I'm calling it a day | 20:20 |
| lucasagomes | have a great evening all | 20:20 |
| jroll | ditto, might check back in later | 20:20 |
| mat128 | jlvillal: http://www.dictionary.com/browse/bifrost | 20:20 |
| jroll | g'night lucasagomes and everyone else \o | 20:20 |
| mat128 | pronounces it otherwise :( | 20:20 |
| lucasagomes | o/ see ya | 20:20 |
| *** lucasagomes is now known as lucas-afk | 20:21 | |
| mat128 | o/ lucasagomes | 20:21 |
| jlvillal | mat128: I won't let that bother me. | 20:21 |
| mat128 | Haha | 20:21 |
| jlvillal | mat128: I pronounce Firenza as Florence ;) | 20:21 |
| *** morgabra_ is now known as morgabra | 20:22 | |
| *** anush_ has quit IRC | 20:22 | |
| *** wajdi has quit IRC | 20:23 | |
| *** ijw_ has quit IRC | 20:24 | |
| jlvillal | lucas-afk: Adios | 20:24 |
| *** ijw has joined #openstack-ironic | 20:25 | |
| *** rama_y has quit IRC | 20:25 | |
| *** rama_y has joined #openstack-ironic | 20:26 | |
| *** ijw has quit IRC | 20:26 | |
| aNuposic | Hi Folks, for adding virtual network interface (VIF) to the bare metal node do we just need to update the bare metal node by adding extra/vif_port assigned to the neutron port? | 20:29 |
| aNuposic | When i am trying to deploy the bare metal node I am getting NO VIFs found for the node. But when i do ironic port-create with MAC address it says port already assigned | 20:31 |
| aNuposic | So looking for neutron port mapping to ironic port | 20:31 |
| *** trown is now known as trown|outtypewww | 20:38 | |
| *** krtaylor has quit IRC | 20:39 | |
| TheJulia | aNuposic: so, ports in ironic are informational interms of what exists with the hardware. To that end, the mapping for the vif port record is actually made on the ironic port, not in neutron anywhere beyond the actual port. In essence, you assign a port in neutron and then assign that port in Ironic to the VIF | 20:51 |
| *** baoli has quit IRC | 20:52 | |
| *** Goneri has quit IRC | 20:53 | |
| TheJulia | aNuposic: Sounds like your trying to deploy a node with networking, but without Nova? | 20:54 |
| *** absubram has quit IRC | 20:54 | |
| aNuposic | TheJulia: I see. So is that assigning done by adding extra/vif_port=NEUTRON_PORT_ID? | 20:54 |
| aNuposic | TheJulia: Yes without NOva | 20:55 |
| aNuposic | TheJulia: With node-set-provision-state active call | 20:55 |
| TheJulia | aNuposic: on the port, extra/vif_port_id | 20:55 |
| aNuposic | TheJulia: Aweesome. Then I will assign that and try to deploy | 20:57 |
| aNuposic | TheJulia: is there any pointer that describes a bit about VIFs? I would be curious to know about it more. I am seeing this first time | 20:58 |
| TheJulia | aNuposic: If memory serves it is essentially Nova's concept for a neutron port assignment. | 21:02 |
| mat128 | aNuposic: Virtual interface | 21:02 |
| mat128 | analog of PIF, Physical interface | 21:02 |
| openstackgerrit | Charlle Daniel proposed openstack/ironic: Ensures that OneView nodes are free for use by Ironic https://review.openstack.org/372762 | 21:03 |
| *** e0ne has quit IRC | 21:06 | |
| aNuposic | TheJulia: ah hmm. Will read on it. | 21:10 |
| aNuposic | mat128: are they some pointers? | 21:10 |
| mat128 | aNuposic: can't find any ATM, a quick search on Neutron VIF didnt find anything authoritative on the topic, but there are mentions of VIF in release notes and such | 21:12 |
| aNuposic | mat128: cool | 21:12 |
| aNuposic | mat128: thnx | 21:13 |
| *** catintheroof has joined #openstack-ironic | 21:15 | |
| aNuposic | mat128: TheJulia: adding extra/vif_port worked. Fab! | 21:17 |
| *** dprince has quit IRC | 21:18 | |
| openstackgerrit | Merged openstack/python-oneviewclient: Add URL of the resource in exception message https://review.openstack.org/373347 | 21:22 |
| *** krtaylor has joined #openstack-ironic | 21:24 | |
| *** jheroux has quit IRC | 21:25 | |
| *** causten has joined #openstack-ironic | 21:25 | |
| *** ijw has joined #openstack-ironic | 21:27 | |
| *** causten has quit IRC | 21:27 | |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Added validation for local_link_connection https://review.openstack.org/377103 | 21:29 |
| *** ijw has quit IRC | 21:31 | |
| *** ijw has joined #openstack-ironic | 21:31 | |
| openstackgerrit | Andre Aranha proposed openstack/ironic: Added validation for local_link_connection https://review.openstack.org/377106 | 21:32 |
| *** sdake has quit IRC | 21:33 | |
| *** bdeschenes has quit IRC | 21:48 | |
| *** vishwanathj has quit IRC | 21:51 | |
| *** aNuposic has quit IRC | 21:58 | |
| *** rbrud has quit IRC | 22:00 | |
| *** rama_y has quit IRC | 22:04 | |
| rloo | jlvillal: I thought bifrost was named after http://norse-mythology.org/cosmology/bifrost/ | 22:09 |
| jlvillal | rloo: Probably, I'm not sure. | 22:10 |
| cinerama | hands up if you pronounce linux "lee-nucks" | 22:10 |
| cinerama | :) | 22:10 |
| rloo | cinerama: ? what does that have to do with bifrost? | 22:12 |
| *** cdearborn has quit IRC | 22:13 | |
| *** rama_y has joined #openstack-ironic | 22:14 | |
| cinerama | just making a (attempt at humourous) point about generally accepted pronunciation vs correct pronunciation | 22:14 |
| *** bdeschenes has joined #openstack-ironic | 22:14 | |
| rloo | cinerama: oh. well, it seems if there is a correct pronounciation, we ought to use that. Doesn't generally accepted come about, cuz people don't know what the correct one is, or cannot pronounce it themselves? | 22:16 |
| *** ijw has quit IRC | 22:16 | |
| jlvillal | rloo: We use Marvel movies as our canonical source for pronunciation :) https://www.youtube.com/watch?v=3nF84V7ysE4&feature=youtu.be&t=1m40s | 22:18 |
| cinerama | on this one i'm more in the descriptivist camp | 22:18 |
| rloo | jlvillal: Oh, in that case ... :) | 22:18 |
| cinerama | ftr i thought the pronunciation guide was a silly fun patch :) | 22:19 |
| rloo | cinerama: it doesn't really matter to me except that i looked it up when the name was originally proposed, so i have been calling it beef roast. but i noticed i was in the minority so i have stopped vocalizing it. | 22:19 |
| jlvillal | I was motivated as I went to a presentation locally and the person pronounced it: bee-frost | 22:19 |
| rloo | cinerama: not being of English descent, I have a preference to use/try to use pronunciations from the 'natives'. Anyway, gonna go away now :) | 22:21 |
| jlvillal | rloo: Good night! | 22:21 |
| *** ijw has joined #openstack-ironic | 22:22 | |
| cinerama | no disrespect on my part towards other cultures is intended. i approved the patch since it is TheJulia's brainchild and she approved that pronunciation | 22:23 |
| *** rpioso has quit IRC | 22:26 | |
| mrda | Morning Ironic | 22:29 |
| *** ijw has quit IRC | 22:29 | |
| TheJulia | cinerama: FWIW, I saw the revision the same way... although I don't use movies. | 22:31 |
| * TheJulia now wants a beef roast.... | 22:34 | |
| mrda | yum :) | 22:41 |
| *** aNuposic has joined #openstack-ironic | 22:48 | |
| *** lucas-afk has quit IRC | 23:02 | |
| NobodyCam | morning mrda | 23:02 |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui: Added support for editing Ironic nodes https://review.openstack.org/376052 | 23:03 |
| mrda | hey NobodyCam | 23:04 |
| *** catintheroof has quit IRC | 23:04 | |
| NobodyCam | :) | 23:05 |
| *** lucasagomes has joined #openstack-ironic | 23:05 | |
| jroll | cinerama: I appreciated your reference, but I prefer linoox | 23:15 |
| jroll | (not really, I just like typing it) :D | 23:16 |
| jroll | morning mrda | 23:16 |
| mrda | hey jroll | 23:16 |
| *** adu has joined #openstack-ironic | 23:27 | |
| *** ijw has joined #openstack-ironic | 23:28 | |
| *** sabeen1 has quit IRC | 23:28 | |
| *** rama_y has quit IRC | 23:31 | |
| *** sabeen has joined #openstack-ironic | 23:31 | |
| *** ijw has quit IRC | 23:32 | |
| *** rloo has quit IRC | 23:44 | |
| *** sdake has joined #openstack-ironic | 23:44 | |
| *** ijw has joined #openstack-ironic | 23:48 | |
| *** patchbot_ has joined #openstack-ironic | 23:50 | |
| *** causten has joined #openstack-ironic | 23:51 | |
| *** patchbot has quit IRC | 23:52 | |
| *** ijw has quit IRC | 23:53 | |
| *** vishwanathj has joined #openstack-ironic | 23:53 | |
| *** Sukhdev has joined #openstack-ironic | 23:54 | |
| *** causten has quit IRC | 23:55 | |
| *** ijw has joined #openstack-ironic | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!