*** xhku_ has joined #openstack-ironic | 00:09 | |
*** xhku has quit IRC | 00:09 | |
*** mat128 has quit IRC | 00:10 | |
*** mat128 has joined #openstack-ironic | 00:10 | |
*** amotoki has joined #openstack-ironic | 00:19 | |
*** rama_y has joined #openstack-ironic | 00:24 | |
*** _vsaienko has quit IRC | 00:28 | |
*** _vs has quit IRC | 00:28 | |
*** _vsaienko has joined #openstack-ironic | 00:31 | |
*** _vs has joined #openstack-ironic | 00:31 | |
*** amotoki has quit IRC | 00:32 | |
*** _vsaienko has quit IRC | 00:35 | |
*** _vs has quit IRC | 00:35 | |
*** amotoki has joined #openstack-ironic | 00:39 | |
*** _vsaienko has joined #openstack-ironic | 00:40 | |
*** _vs has joined #openstack-ironic | 00:40 | |
*** hoangcx has joined #openstack-ironic | 00:45 | |
*** YiboCai has joined #openstack-ironic | 00:46 | |
*** hoangcx has quit IRC | 00:47 | |
*** ijw has joined #openstack-ironic | 00:48 | |
*** hoangcx has joined #openstack-ironic | 00:52 | |
*** phuongnh has joined #openstack-ironic | 00:53 | |
*** _vs has quit IRC | 00:53 | |
*** _vsaienko has quit IRC | 00:53 | |
*** rloo has quit IRC | 00:53 | |
*** ijw has quit IRC | 00:54 | |
*** _vsaienko has joined #openstack-ironic | 00:56 | |
*** _vs has joined #openstack-ironic | 00:56 | |
*** hoangcx has quit IRC | 00:56 | |
*** hoangcx has joined #openstack-ironic | 00:57 | |
*** sacharya has joined #openstack-ironic | 01:12 | |
*** baoli has joined #openstack-ironic | 01:13 | |
*** sacharya has quit IRC | 01:17 | |
*** baoli has quit IRC | 01:18 | |
*** wajdi has joined #openstack-ironic | 01:24 | |
*** Sukhdev has joined #openstack-ironic | 01:29 | |
*** ijw has joined #openstack-ironic | 01:31 | |
*** mtanino has quit IRC | 01:49 | |
*** Goneri has joined #openstack-ironic | 01:49 | |
openstackgerrit | Ruby Loo proposed openstack/ironic-python-agent: Remove unused SUPPORTED_ROOT_DEVICE_HINTS https://review.openstack.org/323050 | 01:52 |
---|---|---|
*** wajdi has quit IRC | 01:53 | |
*** praneshp has joined #openstack-ironic | 01:53 | |
*** wajdi has joined #openstack-ironic | 01:53 | |
openstackgerrit | Yibo Cai proposed openstack/ironic: Support multi arch deployment https://review.openstack.org/322632 | 01:53 |
*** wajdi_ has joined #openstack-ironic | 01:58 | |
*** sacharya has joined #openstack-ironic | 01:59 | |
*** wajdi has quit IRC | 02:01 | |
*** _vsaienko has quit IRC | 02:04 | |
*** _vs has quit IRC | 02:05 | |
*** _vsaienko has joined #openstack-ironic | 02:11 | |
*** _vs has joined #openstack-ironic | 02:11 | |
*** _vs has quit IRC | 02:16 | |
*** _vsaienko has quit IRC | 02:16 | |
*** yuikotakadamori has joined #openstack-ironic | 02:17 | |
*** Sukhdev has quit IRC | 02:19 | |
*** _vsaienko has joined #openstack-ironic | 02:19 | |
*** _vs has joined #openstack-ironic | 02:19 | |
*** Sukhdev has joined #openstack-ironic | 02:28 | |
*** baoli has joined #openstack-ironic | 02:30 | |
*** Sukhdev has quit IRC | 02:32 | |
*** _vs has quit IRC | 02:33 | |
*** _vsaienko has quit IRC | 02:33 | |
*** _vsaienko has joined #openstack-ironic | 02:33 | |
*** _vs has joined #openstack-ironic | 02:33 | |
*** wajdi_ has quit IRC | 02:37 | |
*** fragatin_ has joined #openstack-ironic | 02:41 | |
*** fragatin_ has quit IRC | 02:44 | |
*** fragatin_ has joined #openstack-ironic | 02:45 | |
*** fragatina has quit IRC | 02:45 | |
*** baoli has quit IRC | 02:45 | |
*** sdake_ has joined #openstack-ironic | 02:46 | |
*** yuanying has quit IRC | 02:47 | |
*** fragatin_ has quit IRC | 02:49 | |
*** sdake has quit IRC | 02:50 | |
*** yuanying has joined #openstack-ironic | 02:52 | |
*** _vs has quit IRC | 02:52 | |
*** _vsaienko has quit IRC | 02:53 | |
*** Goneri has quit IRC | 02:54 | |
*** _vsaienko has joined #openstack-ironic | 02:56 | |
*** _vs has joined #openstack-ironic | 02:56 | |
*** fragatina has joined #openstack-ironic | 02:57 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/322986 | 03:00 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/322987 | 03:01 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements https://review.openstack.org/323066 | 03:01 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/323067 | 03:01 |
*** fragatina has quit IRC | 03:02 | |
*** Haomeng has joined #openstack-ironic | 03:03 | |
*** _vs has quit IRC | 03:04 | |
*** _vsaienko has quit IRC | 03:05 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/323002 | 03:06 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/323003 | 03:06 |
*** _vsaienko has joined #openstack-ironic | 03:06 | |
*** _vs has joined #openstack-ironic | 03:06 | |
*** sdake_ has quit IRC | 03:07 | |
*** _vsaienko has quit IRC | 03:17 | |
*** _vs has quit IRC | 03:18 | |
*** _vsaienko has joined #openstack-ironic | 03:18 | |
*** _vs has joined #openstack-ironic | 03:18 | |
*** _vsaienko has quit IRC | 03:24 | |
*** _vs has quit IRC | 03:24 | |
*** _vsaienko has joined #openstack-ironic | 03:28 | |
*** _vs has joined #openstack-ironic | 03:28 | |
openstackgerrit | Tan Lin proposed openstack/ironic: Add one use case for configdrive https://review.openstack.org/323099 | 03:30 |
*** _vsaienko has quit IRC | 03:35 | |
*** _vs has quit IRC | 03:35 | |
*** _vsaienko has joined #openstack-ironic | 03:38 | |
*** _vs has joined #openstack-ironic | 03:38 | |
*** flwang1 has joined #openstack-ironic | 03:38 | |
*** yuanying has quit IRC | 03:43 | |
*** _vsaienko has quit IRC | 03:44 | |
*** haomeng__ has joined #openstack-ironic | 03:44 | |
haomeng__ | flwang1, hi | 03:45 |
*** _vs has quit IRC | 03:46 | |
*** yuanying has joined #openstack-ironic | 03:47 | |
*** fragatina has joined #openstack-ironic | 03:48 | |
*** rcernin has joined #openstack-ironic | 03:49 | |
*** links has joined #openstack-ironic | 03:50 | |
*** _vsaienko has joined #openstack-ironic | 03:50 | |
*** _vs has joined #openstack-ironic | 03:51 | |
*** _vsaienko has quit IRC | 03:54 | |
*** appu103_ has joined #openstack-ironic | 03:54 | |
*** _vs has quit IRC | 03:55 | |
*** Sukhdev has joined #openstack-ironic | 03:55 | |
*** _vsaienko has joined #openstack-ironic | 03:58 | |
*** _vs has joined #openstack-ironic | 03:58 | |
*** Sukhdev has quit IRC | 03:59 | |
*** _vsaienko has quit IRC | 04:04 | |
*** _vs has quit IRC | 04:04 | |
*** _vsaienko has joined #openstack-ironic | 04:10 | |
*** _vs has joined #openstack-ironic | 04:10 | |
*** vmud213 has joined #openstack-ironic | 04:10 | |
*** _vsaienko has quit IRC | 04:17 | |
*** _vs has quit IRC | 04:17 | |
*** sacharya has quit IRC | 04:19 | |
*** sacharya has joined #openstack-ironic | 04:20 | |
*** sacharya has quit IRC | 04:21 | |
*** sacharya_ has joined #openstack-ironic | 04:21 | |
*** _vsaienko has joined #openstack-ironic | 04:23 | |
*** _vs has joined #openstack-ironic | 04:23 | |
*** _vsaienko has quit IRC | 04:29 | |
*** _vs has quit IRC | 04:29 | |
*** _vsaienko has joined #openstack-ironic | 04:33 | |
*** _vs has joined #openstack-ironic | 04:33 | |
*** rama_y has quit IRC | 04:38 | |
*** _vs has quit IRC | 04:42 | |
*** _vsaienko has quit IRC | 04:43 | |
*** _vsaienko has joined #openstack-ironic | 04:44 | |
*** _vs has joined #openstack-ironic | 04:44 | |
*** rcernin has quit IRC | 04:52 | |
*** _vsaienko has quit IRC | 04:52 | |
*** _vs has quit IRC | 04:53 | |
*** chlong has quit IRC | 04:54 | |
*** _vsaienko has joined #openstack-ironic | 04:56 | |
*** _vs has joined #openstack-ironic | 04:57 | |
*** baoli has joined #openstack-ironic | 04:57 | |
*** baoli has quit IRC | 05:02 | |
*** _vs has quit IRC | 05:04 | |
*** _vsaienko has quit IRC | 05:04 | |
*** _vsaienko has joined #openstack-ironic | 05:05 | |
*** _vs has joined #openstack-ironic | 05:05 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Clear target_power_state on conductor startup https://review.openstack.org/310090 | 05:06 |
*** ChubYann has quit IRC | 05:08 | |
*** sacharya_ has quit IRC | 05:09 | |
*** _vs has quit IRC | 05:10 | |
*** _vsaienko has quit IRC | 05:11 | |
*** chlong has joined #openstack-ironic | 05:12 | |
*** _vsaienko has joined #openstack-ironic | 05:15 | |
*** _vs has joined #openstack-ironic | 05:15 | |
*** _vs has quit IRC | 05:24 | |
*** _vsaienko has quit IRC | 05:24 | |
*** rama_y has joined #openstack-ironic | 05:27 | |
*** Sukhdev has joined #openstack-ironic | 05:28 | |
*** tyagi has joined #openstack-ironic | 05:32 | |
*** tyagi has quit IRC | 05:32 | |
*** vishwanathj has joined #openstack-ironic | 05:33 | |
*** ishant has joined #openstack-ironic | 05:33 | |
openstackgerrit | Yuiko Takada Mori proposed openstack/ironic-specs: Nova compatible serial console support https://review.openstack.org/319505 | 05:34 |
*** praneshp has quit IRC | 05:34 | |
*** ishant_ has joined #openstack-ironic | 05:35 | |
*** ishant_ has left #openstack-ironic | 05:35 | |
*** ishant_ has quit IRC | 05:35 | |
*** hoangcx has quit IRC | 05:55 | |
*** fragatina has quit IRC | 06:08 | |
*** mjura has joined #openstack-ironic | 06:10 | |
*** rama_y has quit IRC | 06:12 | |
*** mjura has quit IRC | 06:15 | |
*** mjura has joined #openstack-ironic | 06:15 | |
*** moshele has joined #openstack-ironic | 06:22 | |
*** parlos_ has joined #openstack-ironic | 06:50 | |
*** parlos has quit IRC | 06:51 | |
*** e0ne has joined #openstack-ironic | 06:56 | |
*** tesseract has joined #openstack-ironic | 06:57 | |
*** rcernin has joined #openstack-ironic | 07:02 | |
*** yolanda has quit IRC | 07:05 | |
*** yolanda has joined #openstack-ironic | 07:05 | |
openstackgerrit | Tan Lin proposed openstack/ironic: Add a doc about appending kernel parameters to boot instances https://review.openstack.org/320848 | 07:09 |
*** sacharya has joined #openstack-ironic | 07:09 | |
*** hoangcx has joined #openstack-ironic | 07:12 | |
*** sacharya has quit IRC | 07:14 | |
*** praneshp has joined #openstack-ironic | 07:14 | |
*** Sukhdev has quit IRC | 07:18 | |
*** Sukhdev has joined #openstack-ironic | 07:19 | |
*** parlos_ has quit IRC | 07:23 | |
*** haomeng__ has quit IRC | 07:28 | |
*** ijw has quit IRC | 07:32 | |
*** ifarkas has joined #openstack-ironic | 07:34 | |
*** sdake has joined #openstack-ironic | 07:34 | |
*** stendulker has joined #openstack-ironic | 07:35 | |
*** vishwanathj has quit IRC | 07:37 | |
*** praneshp has quit IRC | 07:49 | |
lucasagomes | morning all | 07:50 |
stendulker | morning lucasagomes :) | 07:52 |
*** yolanda has quit IRC | 07:54 | |
*** yolanda has joined #openstack-ironic | 07:55 | |
*** parlos has joined #openstack-ironic | 07:56 | |
*** yolanda has quit IRC | 07:56 | |
*** yolanda_ has joined #openstack-ironic | 07:56 | |
*** Sukhdev has quit IRC | 07:59 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-ironic | 08:01 | |
*** dtantsur|afk is now known as dtantsur | 08:01 | |
dtantsur | Morning Ironic | 08:01 |
*** hoangcx has quit IRC | 08:07 | |
stendulker | morning dtantsur :) | 08:08 |
vsaienko | morning dtantsur stendulker, lucasagomes and all Ironic'ers! | 08:09 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: move create-node.sh to create_vms func https://review.openstack.org/322850 | 08:09 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Refactor enroll-node code https://review.openstack.org/323190 | 08:09 |
stendulker | morning vsaienko :) | 08:09 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 08:10 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 08:10 |
dtantsur | hi stendulker, vsaienko | 08:10 |
*** sacharya has joined #openstack-ironic | 08:11 | |
*** ohamada has joined #openstack-ironic | 08:11 | |
*** fragatina has joined #openstack-ironic | 08:12 | |
*** vishwanathj has joined #openstack-ironic | 08:13 | |
*** vishwanathj has quit IRC | 08:15 | |
*** sacharya has quit IRC | 08:16 | |
*** vishwanathj has joined #openstack-ironic | 08:16 | |
*** hoangcx has joined #openstack-ironic | 08:25 | |
*** dmk0202 has joined #openstack-ironic | 08:26 | |
*** MattMan has quit IRC | 08:30 | |
*** MattMan has joined #openstack-ironic | 08:31 | |
*** parlos has quit IRC | 08:31 | |
*** vishwanathj is now known as vishwanathj_ | 08:32 | |
*** vishwanathj_ is now known as vishwanathj | 08:32 | |
*** sdake_ has joined #openstack-ironic | 08:32 | |
*** _vsaienko has joined #openstack-ironic | 08:33 | |
*** _vs has joined #openstack-ironic | 08:33 | |
openstackgerrit | Merged openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/323002 | 08:34 |
*** sdake has quit IRC | 08:35 | |
*** vdrok has joined #openstack-ironic | 08:37 | |
vdrok | morning ironic! | 08:39 |
dtantsur | morning vdrok | 08:39 |
vdrok | morning dtantsur | 08:39 |
*** _vsaienko has quit IRC | 08:43 | |
*** _vs has quit IRC | 08:43 | |
*** _vsaienko has joined #openstack-ironic | 08:43 | |
*** _vs has joined #openstack-ironic | 08:44 | |
vdrok | dtantsur, if you have a bit of time, very easy change - https://review.openstack.org/322958 | 08:44 |
dtantsur | will take a look a bit later, thanks | 08:45 |
*** _vsaienko has quit IRC | 08:45 | |
*** _vs has quit IRC | 08:45 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Refactor enroll-node code https://review.openstack.org/323190 | 08:45 |
vdrok | sure, thank you :) | 08:46 |
stendulker | morning vdrok :) | 08:51 |
*** milan has quit IRC | 08:51 | |
*** milan has joined #openstack-ironic | 08:52 | |
vdrok | morning stendulker ! | 08:55 |
openstackgerrit | Merged openstack/ironic: Remove support for 'hexraw' iPXE type https://review.openstack.org/322875 | 08:58 |
openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/322986 | 08:59 |
*** kromanenko__ has joined #openstack-ironic | 09:00 | |
*** Romanenko_K has joined #openstack-ironic | 09:00 | |
*** kromanenko_ has joined #openstack-ironic | 09:00 | |
*** electrofelix has joined #openstack-ironic | 09:01 | |
*** ijw has joined #openstack-ironic | 09:02 | |
*** derekh has joined #openstack-ironic | 09:03 | |
*** ijw has quit IRC | 09:09 | |
openstackgerrit | Imre Farkas proposed openstack/ironic: [WIP] DRAC RAID configuration https://review.openstack.org/176319 | 09:10 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Revert "iscsi: wipe the disk before deployment" https://review.openstack.org/322932 | 09:20 |
*** daemontool has joined #openstack-ironic | 09:22 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Gracefully degrade start_iscsi_target for Mitaka ramdisk https://review.openstack.org/319183 | 09:24 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Revert "iscsi: wipe the disk before deployment" https://review.openstack.org/322932 | 09:27 |
*** chlong has quit IRC | 09:29 | |
sambetts|afk | Morning all | 09:36 |
*** sambetts|afk is now known as sambetts | 09:36 | |
dtantsur | hey sambetts | 09:38 |
milan | morning Ironic! | 09:38 |
milan | sambetts, dtantsur o/ :) | 09:38 |
dtantsur | morning milan :) | 09:38 |
sambetts | Morning milan, dtantsur, did much happen yesterday while i was on holiday? | 09:38 |
dtantsur | next to nothing :) one Monday gate breakage that was quickly fixed | 09:39 |
milan | sambetts, not much with milan --- just created and destroyed a vm for grenade stuff ;) | 09:39 |
dtantsur | I'm still fighting with my new laptop :( | 09:39 |
sambetts | :( whats the problem? | 09:39 |
dtantsur | sambetts, it's Thinkpad T460s and it's not fully supported by *any* released Linux | 09:40 |
milan | :-/ | 09:40 |
dtantsur | and has serious bugs with betas (e.g. suspend does not work for me on F24) | 09:40 |
sambetts | :( oh man that sucks ... I thought the thinkpads were normally really good with linux support | 09:41 |
dtantsur | a lot of fun..... I miss my good old T530 whose only downside was its weight :) | 09:41 |
dtantsur | sambetts, intel did not do great job this time pushing drivers upstream | 09:41 |
sambetts | ah :( | 09:41 |
sambetts | I was thinking about getting myself a ThinkPad X1 for personal use, I like how thin it is | 09:42 |
dtantsur | if you use linux, don't take gen 4 (this year) for now | 09:43 |
dtantsur | the previous generation seems fine | 09:43 |
milan | dtantsur, you're in the office? | 09:43 |
sambetts | they using the same chips as in your new one? | 09:43 |
* milan hesitates to go or not to | 09:43 | |
dtantsur | milan, no.. I planned on, but got distracted by $errands | 09:43 |
dtantsur | sambetts, yep | 09:44 |
milan | dtantsur, ok | 09:44 |
openstackgerrit | Merged openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/322987 | 09:45 |
*** chlong has joined #openstack-ironic | 09:47 | |
*** phuongnh has quit IRC | 09:47 | |
*** phuongnh has joined #openstack-ironic | 09:49 | |
openstackgerrit | Merged openstack/ironic: Use messaging notifications transport instead of default https://review.openstack.org/322639 | 09:49 |
milan | sambetts, I was wondering about this patch https://review.openstack.org/#/c/308379/ before the rebase you wanted to merge the processing and reapply functions | 09:49 |
milan | sambetts, should we make such a refactor a separate patch? | 09:50 |
sambetts | milan: right, I think there is a lot of shared logic there, and IMO passing data in via the unprocessed API should be more or less the same as the ramdisk sending some data to inspector (with some minor changes of course, e.g. no power off at the end) | 09:51 |
sambetts | milan: it may be quite a large refactor of the processing logic, so I wouldn't be too concerned about making it a separate patch I guess | 09:54 |
milan | sambetts, +1 | 09:54 |
milan | we could merge this change as it is now before it needs rebasing again? | 09:54 |
sambetts | milan: I also have some ideas about how to prevent us having to pass through things like power_off=False everywhere, e.g. checking Ironic to see if its already off | 09:54 |
sambetts | milan: Sure, give me a couple of minutes to finish up something and I'll go through it and see if it can be merged | 09:56 |
milan | right, atm it's rather about stating an intention: in reapply I intend not to power off at all (no matter the state) in process I intend power off (if it is off, no need to do anything) | 09:56 |
milan | sambetts, sure | 09:56 |
milan | thx | 09:56 |
vsaienko | morning sambetts, milan | 09:56 |
sambetts | milan: yeah good point | 09:56 |
sambetts | vsaienko: Hi! | 09:56 |
milan | morning vsaienko :) | 09:56 |
*** sdake_ is now known as sdake | 09:57 | |
vsaienko | sambetts where I can check cisco CI status for https://review.openstack.org/#/c/323190 | 09:57 |
sambetts | vsaienko: It should have left a comment ... :/ | 09:58 |
vdrok | good morning milan and sambetts | 09:58 |
sambetts | vsaienko: All our CI stats are here: http://192.133.158.2:8080/view/openstack%20ironic/ | 09:58 |
sambetts | vsaienko: it seems like it still running through devstack on the Cisco CI | 09:59 |
sambetts | http://192.133.158.2:8080/view/openstack%20ironic/job/tempest-dsvm-ironic-pxe_ucs/1488/console | 09:59 |
sambetts | for that ^ job anyway | 09:59 |
vsaienko | sambetts, that was exactly I looking for, thanks | 10:00 |
milan | morning vdrok :) | 10:00 |
sergek | morning vdrok, milan, vsaienko and all :) | 10:01 |
vdrok | o/ | 10:01 |
*** _vsaienko has joined #openstack-ironic | 10:03 | |
*** _vs has joined #openstack-ironic | 10:03 | |
vsaienko | sergek o/ | 10:04 |
*** mgoddard has joined #openstack-ironic | 10:04 | |
*** _vsaienko has quit IRC | 10:04 | |
*** ijw has joined #openstack-ironic | 10:05 | |
*** _vs has quit IRC | 10:05 | |
*** _vs has joined #openstack-ironic | 10:06 | |
*** YiboCai has quit IRC | 10:07 | |
ishant | sambetts, Hi | 10:08 |
sambetts | Morning ishant | 10:09 |
ishant | Morning sambetts | 10:09 |
ishant | sambetts, I was going through https://review.openstack.org/#/c/277853 . I want to know how you will make macs_for_instance a no op . | 10:10 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 10:10 |
*** ijw has quit IRC | 10:10 | |
*** pcaruana has joined #openstack-ironic | 10:11 | |
ishant | sambetts, A non admin tenant wont be able to update the mac of the port, so in non admin user context how ironic will change the mac of port? | 10:12 |
*** sacharya has joined #openstack-ironic | 10:12 | |
ishant | or am I missing something here | 10:12 |
dtantsur | ishant, what's the use case for non-admin to change a MAC? | 10:12 |
sambetts | ishant: We would simply not implement macs_for_instance in the Ironic virt driver, the default implmentation for that returns None | 10:12 |
*** _vs has quit IRC | 10:13 | |
sambetts | ishant: The Ironic service user is an admin user, and can change things like the mac address of a port | 10:13 |
dtantsur | ah, this spec. got it. | 10:13 |
sambetts | dtantsur: you say it like its a bad thing ;) | 10:13 |
ishant | sambetts, ohh so ironic wont work on user context? It always works on admin context? | 10:14 |
dtantsur | ;) | 10:14 |
dtantsur | yes. ironic is admin-only | 10:14 |
dtantsur | nova is our non-privileged frontend | 10:14 |
ishant | dtantsur, sambetts , thanks . I thought the user context gets passed to ironic from nova | 10:15 |
*** chlong has quit IRC | 10:15 | |
ishant | and then ironic uses that context | 10:15 |
*** _vsaienko has joined #openstack-ironic | 10:15 | |
dtantsur | no, nova has ironic admin credentials configured in nova.conf | 10:15 |
*** _vs has joined #openstack-ironic | 10:15 | |
sambetts | Ironic has its own user for interservice communication and that should be an admin level user | 10:16 |
*** sacharya has quit IRC | 10:16 | |
sambetts | also what dtantsur said :) | 10:16 |
ishant | sambetts, dtantsur : thanks :) didnt know this | 10:17 |
dtantsur | sambetts, offtopic: do you know a good book to learn all this modern networking stuff (vlans, sdn and other buzzwords ;) | 10:17 |
* dtantsur is unacceptably clueless | 10:18 | |
*** mgould|afk is now known as mgould | 10:18 | |
vdrok | I'd read that too :) | 10:19 |
sambetts | dtantsur: https://www.packtpub.com/virtualization-and-cloud/learning-openstack-networking-neutron <- I have this on my desk at all times, it really helped me understand neutron better | 10:21 |
dtantsur | oh, this is one we got on the vancouver summit, right? | 10:22 |
* dtantsur has it somewhere then | 10:22 | |
sambetts | Right, I picked it up in paris, and I tried to get as many people to get it in vancouver as possible because it was super useful | 10:23 |
dtantsur | cool | 10:24 |
dtantsur | I see the same author has something even newer: https://www.safaribooksonline.com/library/view/openstack-networking-essentials/9781785283277/ | 10:24 |
sambetts | ooooo | 10:24 |
sambetts | I wondered if there was a newer one, I've been looking out for it at the last 2 summit s | 10:25 |
* dtantsur adds both to his read list | 10:25 | |
*** _vs has quit IRC | 10:25 | |
*** _vsaienko has quit IRC | 10:25 | |
dtantsur | sambetts, also there is the 2nd edition of your book from autumn 2015 | 10:25 |
dtantsur | https://www.safaribooksonline.com/library/view/learning-openstack-networking/9781785287725/ | 10:26 |
*** yolanda has joined #openstack-ironic | 10:27 | |
*** yolanda_ has quit IRC | 10:27 | |
*** chlong has joined #openstack-ironic | 10:27 | |
* sambetts should update to one or other of the above | 10:28 | |
*** _vsaienko has joined #openstack-ironic | 10:29 | |
*** _vs has joined #openstack-ironic | 10:29 | |
* dtantsur got a one year safari subscription from RH, so he's like "READ ALL DA BOOKZZ!!1" | 10:32 | |
sambetts | LOL! I think I can get safari too, I need to look into that | 10:32 |
ishant | sambetts, I like the idea of interface-attach and iterface-detach in nova ironic driver. | 10:32 |
sambetts | ishant: Thank you :) it'd really help with allow more customisation on the ironic networking side | 10:33 |
sambetts | at a driver level | 10:34 |
ishant | but you should have specified this in VLAN-aware-baremetal spec also, because making macs_for_instance no-op seems incomplete | 10:34 |
sambetts | ishant: I'm not sure what you mean, they can both work independantly | 10:34 |
ishant | sambetts, yes driver should have control over its nodes mac's | 10:34 |
*** _vsaienko has quit IRC | 10:35 | |
*** _vsaienko has joined #openstack-ironic | 10:36 | |
*** _vs has quit IRC | 10:37 | |
*** _vs has joined #openstack-ironic | 10:37 | |
*** _vs has quit IRC | 10:39 | |
*** _vsaienko has quit IRC | 10:39 | |
ishant | sambetts, if you make macs_for_instance a no-op then how will ironic know about the neutron port which is created with neutron generated mac | 10:41 |
ishant | I thought interface-attach will pass the info to ironic | 10:41 |
sambetts | ishant: making macs_for_instance a no-op doesn't change the fact we do plug_vifs | 10:42 |
sambetts | macs_for_instance doesn't do any Ironic port to neutron port mapping, it simply returns a list of mac addresses, plug_vifs does that mapping logic | 10:43 |
sambetts | and in the VLAN aware BM spec that is modified very slightly, according to the spec | 10:43 |
*** _vsaienko has joined #openstack-ironic | 10:43 | |
*** _vs has joined #openstack-ironic | 10:43 | |
sambetts | the interface attach / detach spec, independant of the VLAN aware BM spec, is designed to moved the plug_vifs logic out of the nova driver, so that it can be customised by the Ironic network drivers | 10:44 |
ishant | sambetts, okay got you , thanks | 10:46 |
sambetts | :D | 10:46 |
sambetts | ishant: thanks for reviewing the specs | 10:46 |
sambetts | the more eyes the better | 10:47 |
ishant | sambetts, so if I want to use virtual mac (neutron generated) I need the interface-attach and detach api's which passes vif details to then ironic driver and then driver can create its own ironic ports | 10:50 |
ishant | sambetts, am i correct in my understanding :D | 10:50 |
ishant | sambetts: brb | 10:56 |
*** stendulker has quit IRC | 10:58 | |
jroll | dtantsur: https://review.openstack.org/#/c/321491/2/doc/source/deploy/install-guide.rst@692 <- should we remove that or? maybe someone from tripleo can follow up with another patch since I don't know much about it? :) | 10:58 |
sambetts | ishant: Right, thats one way to do it, using that spec, because there is a direct mapping 1-1 of Ironic port to neutron port so you have to create a new Ironic port per virtual port (thats actually very similar to how we've implemented it for our card), once VLAN aware BMs merges, the model changes slightly because you can then assign multiple neutron ports to 1 ironic port, so the Ironic port | 10:59 |
sambetts | can represent the physical nic on your flex card with its physical mac address, and then you can just map multiple neutron ports to that Ironic port, then at configure tenant networking time you can process that however your driver wants, e.g. creating new virtual nics or creating a neutron trunk or both, once both specs are merged all the doors open for us | 10:59 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Remove links to github for OpenStack things https://review.openstack.org/321491 | 11:00 |
*** ishant has quit IRC | 11:01 | |
dtantsur | jroll, diskimage-builder should be installed via pip imo. I never tried what's written there. | 11:02 |
dtantsur | also morning jroll | 11:02 |
jroll | morning :) | 11:03 |
sambetts | Morning jroll, dtantsur I've always installed DIB from pip too | 11:03 |
jroll | well, this is about system deps right? | 11:04 |
dtantsur | jroll, I can only say that I never did that and it worked for me | 11:04 |
dtantsur | "This script should die" (c) https://github.com/openstack/tripleo-incubator/blob/master/scripts/install-dependencies | 11:05 |
jroll | dtantsur: and last time I ran DIB I need to install a bunch of things :P | 11:05 |
dtantsur | also this script does A LOT of things unrelated to DIB, like generation SSH keys and configuring libvirt | 11:06 |
jroll | yeah | 11:06 |
dtantsur | and would advise against using it | 11:06 |
dtantsur | s/and/I/ | 11:06 |
jroll | I'm fine with just deleting that note | 11:06 |
dtantsur | yeah, lets convert it to pip install for greater good | 11:06 |
*** ijw has joined #openstack-ironic | 11:06 | |
jroll | it already is :P | 11:07 |
jroll | that's just a note about deps | 11:07 |
dtantsur | ok :) | 11:07 |
*** phuongnh has quit IRC | 11:07 | |
jroll | http://docs.openstack.org/developer/ironic/deploy/install-guide.html#image-requirements | 11:07 |
*** dmk0202 has quit IRC | 11:07 | |
* mgould feels the correct place to document dependencies for DIB is in DIB itself | 11:07 | |
dtantsur | mgould++ | 11:07 |
dtantsur | jroll, an unrelated question: what can cause the 1st cleaning to fail with "Block device /dev/sda is frozen and cannot be erased"? | 11:08 |
sambetts | Is there a python-disk-image-builder system package we can point people at for those without pip/I guess it should also install all the right system deps too | 11:08 |
sambetts | Hi mgould | 11:08 |
mgould | hi sambetts | 11:08 |
jroll | dtantsur: a locked SSD | 11:08 |
dtantsur | sambetts, it's called diskimage-builder on Fedora | 11:09 |
dtantsur | jroll, do you know how to unlock such? | 11:09 |
mgould | sambetts: looks like there's a diskimage-builder RPM package | 11:09 |
jroll | dtantsur: not off the top of my head, um | 11:10 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Remove links to github for OpenStack things https://review.openstack.org/321491 | 11:10 |
*** vishwanathj has quit IRC | 11:10 | |
*** vishwanathj has joined #openstack-ironic | 11:10 | |
sambetts | mgould, dtantsur: I guess that makes sense if RDO uses it :-P | 11:10 |
*** _vs has quit IRC | 11:10 | |
*** _vsaienko has quit IRC | 11:10 | |
jroll | I'm not documenting how to install DIB without pip in this patch, y'all can put up another if you wish | 11:10 |
sambetts | :-P | 11:11 |
sambetts | just my playing devils advocate again heh | 11:11 |
*** ijw has quit IRC | 11:12 | |
jroll | dtantsur: something like https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/hardware.py#L715 | 11:14 |
jroll | but with the right password, I guess, if this isn't working | 11:14 |
dtantsur | meh.. | 11:14 |
dtantsur | thanks jroll | 11:15 |
jroll | np | 11:15 |
jroll | might be able to do it in bios somehow | 11:15 |
dtantsur | yeah.. I've dumped this conversation to the guy | 11:15 |
*** vishwanathj has quit IRC | 11:16 | |
* dtantsur thinks their QE folk should work upstream more actively | 11:16 | |
*** vishwanathj has joined #openstack-ironic | 11:16 | |
milan | morning mgould, jroll | 11:16 |
*** hoangcx has quit IRC | 11:17 | |
mgould | morning milan | 11:18 |
*** divya has joined #openstack-ironic | 11:23 | |
divya | ConnectFailure: Unable to establish connection to http://172.26.24.110:6385/v1/nodes/detail | 11:23 |
divya | nova-comp not starting | 11:23 |
divya | what it could be | 11:23 |
dtantsur | jroll, seems like this affects mitaka too: https://review.openstack.org/#/c/323301/ | 11:34 |
dtantsur | divya, is ironic api even running? | 11:34 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Refactor enroll-node code https://review.openstack.org/323190 | 11:35 |
jroll | dtantsur: waiting on CI but will +A when done | 11:35 |
mat128 | dtantsur, jroll: a typical BIOS will lock ATA devices on boot | 11:36 |
dtantsur | aha | 11:36 |
mat128 | typical workarounds: go to sleep and come back, often unlocks the device (unfeasible in an automation/datacenter context) | 11:36 |
mat128 | look for a BIOS setting to disable that automatic lock on boot | 11:36 |
dtantsur | yeah, not something you do on a server. | 11:36 |
dtantsur | mat128, thanks! | 11:36 |
mat128 | or c) work with your provider to get that option | 11:36 |
jroll | mat128: nice, thanks | 11:36 |
mat128 | for some reason this is not the kind of option they build into every bios | 11:37 |
mat128 | now, if you wan to fall back to doing shred instead of ATA secure erase, there's this work in progress by Julia: https://review.openstack.org/#/c/302819/ | 11:38 |
mat128 | it's already in IPA, just missing the Ironic config part | 11:38 |
mat128 | so with a custom driver you can already use it | 11:38 |
divya | Timed out waiting for a reply to message ID--error observed in ironic-api | 11:39 |
divya | dtantsur->nova compute, conductor not starting | 11:39 |
divya | what could be the issue | 11:39 |
divya | ir-api is started dtantsur | 11:41 |
mat128 | divya: ir-api talks to ir-cond over bus, so I would look at that | 11:41 |
mat128 | divya: if nova-conductor is not starting, I would look at it's logs | 11:42 |
dtantsur | yeah, the conductor should be started | 11:42 |
*** dmk0202 has joined #openstack-ironic | 11:42 | |
*** vishwanathj has quit IRC | 11:46 | |
divya | mat128-n-cond logs placed in http://paste.openstack.org/show/506550/ | 11:46 |
*** baoli has joined #openstack-ironic | 11:47 | |
mat128 | n-cond couldn't reach ir-api | 11:47 |
mat128 | divya: shouldn't prevent n-cond from starting though | 11:47 |
mat128 | divya: from the box running n-cond, can you curl http://172.26.24.110:6385/ | 11:47 |
divya | curl http://172.26.24.110:6385/ {"default_version": {"status": "CURRENT", "min_version": "1.1", "version": "1.18", "id": "v1", "links": [{"href": "None/v1/", "rel": "self"}]}, "versions": [{"status": "CURRENT", "min_version": "1.1", "version": "1.18", "id": "v1", "links": [{"href": "None/v1/", "rel": "self"}]}], "name": "OpenStack Ironic API", "description": "Ironic is an OpenStack project which aims to provision baremetal | 11:48 |
divya | this is the output | 11:48 |
divya | ir-api logs are copied to http://paste.openstack.org/show/506551/ | 11:49 |
mat128 | divya: your ir-api logs even show a successful /v1/nodes call | 11:51 |
mat128 | divya: I think your n-cond tracebacks might be outdated | 11:51 |
mat128 | divya: is ir-cond running? | 11:51 |
*** baoli has quit IRC | 11:52 | |
divya | ir-cond is running mat128 | 11:53 |
divya | i am able to give ironic node-list | 11:53 |
divya | ironic node-list +--------------------------------------+-------+---------------+-------------+--------------------+-------------+ | UUID | Name | Instance UUID | Power State | Provisioning State | Maintenance | +--------------------------------------+-------+---------------+-------------+--------------------+-------------+ | ea0c712b-ae13-425e-b81d-0f2e0ef1e49c | test1 | None | Non | 11:53 |
sambetts | that means that ironic api is running | 11:53 |
sambetts | not the conductor | 11:53 |
divya | what command for ironic conductor | 11:53 |
mat128 | changing a state would force it | 11:54 |
mat128 | ironic node-set-power-state test1 on | 11:54 |
divya | ironic node-list +--------------------------------------+-------+---------------+-------------+--------------------+-------------+ | UUID | Name | Instance UUID | Power State | Provisioning State | Maintenance | +--------------------------------------+-------+---------------+-------------+--------------------+-------------+ | ea0c712b-ae13-425e-b81d-0f2e0ef1e49c | test1 | None | powe | 11:55 |
divya | yea | 11:55 |
divya | i changed the power state to off | 11:55 |
divya | i was changed | 11:55 |
mat128 | if it worked, that means ir-api -> ir-cond seems fine | 11:55 |
mat128 | because the conductor is doing the work in that case | 11:55 |
divya | ok.. | 11:55 |
mat128 | so besides logs, what's your original concern? | 11:55 |
divya | but why is my n-comp not starting | 11:55 |
mat128 | I believe it is starting but at some point it wasnt able to reach ir-api | 11:56 |
*** thrash|g0ne is now known as thrash | 11:56 | |
mat128 | do you see your node when you do `nova hypervisor-list` | 11:56 |
divya | no | 11:57 |
vsaienko | morning mat128, jroll, divya | 11:57 |
divya | nova hypervisor-list +----+--------------------------------------+-------+---------+ | ID | Hypervisor hostname | State | Status | +----+--------------------------------------+-------+---------+ | 22 | ea0c712b-ae13-425e-b81d-0f2e0ef1e49c | up | enabled | +----+--------------------------------------+-------+---------+ | 11:57 |
vsaienko | jroll what do you think about https://review.openstack.org/#/c/322932/ | 11:57 |
divya | yes i see that | 11:57 |
mat128 | o/ Vasyl | 11:57 |
divya | good morning vsaienko | 11:57 |
*** baoli_ has joined #openstack-ironic | 11:58 | |
mat128 | divya: that would be a yes ;) | 11:58 |
mat128 | divya: as far as I'm concerned, I don't see any problem in your stack | 11:59 |
mat128 | divya: your n-cond logs could have been when ir-api was not up yet | 11:59 |
mat128 | divya: communication is clearly working now because nodes are listed in your hypervisor-list | 11:59 |
*** dprince has joined #openstack-ironic | 12:00 | |
*** mat128 is now known as mat128|brb | 12:00 | |
nicodemos | good morning, ironic | 12:02 |
*** vishwanathj has joined #openstack-ironic | 12:02 | |
vdrok | morning jroll mat128|brb nicodemos | 12:02 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Add a log extension https://review.openstack.org/248832 | 12:07 |
*** _vsaienko has joined #openstack-ironic | 12:07 | |
*** lucasagomes is now known as lucas-hungry | 12:07 | |
* lucas-hungry brb lunch | 12:07 | |
*** _vs has joined #openstack-ironic | 12:07 | |
lucas-hungry | jroll, morning, btw you and/or JayF have some time: https://bugs.launchpad.net/ironic/+bug/1587143 | 12:08 |
openstack | Launchpad bug 1587143 in Ironic "[RFE] Collect logs from IPA on deploy failure" [Wishlist,New] - Assigned to Lucas Alvares Gomes (lucasagomes) | 12:08 |
*** mgoddard has quit IRC | 12:08 | |
*** ijw has joined #openstack-ironic | 12:08 | |
*** mgoddard has joined #openstack-ironic | 12:08 | |
openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal power state commands https://review.openstack.org/172517 | 12:11 |
openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Add maintenance mode commands https://review.openstack.org/216727 | 12:11 |
openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal port create https://review.openstack.org/172461 | 12:11 |
openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Bring OSC plugin inline with approved spec https://review.openstack.org/284160 | 12:11 |
openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Add provision state commands https://review.openstack.org/206119 | 12:11 |
*** sacharya has joined #openstack-ironic | 12:13 | |
*** ijw has quit IRC | 12:13 | |
*** mat128|brb is now known as mat128 | 12:15 | |
*** sacharya has quit IRC | 12:17 | |
*** xavierr_afk is now known as xavierr | 12:19 | |
xavierr | good morning guys :) | 12:20 |
mat128 | morning vdrok xavierr | 12:23 |
vdrok | morning xavierr | 12:25 |
*** sdake has quit IRC | 12:27 | |
*** trown|outtypewww is now known as trown | 12:27 | |
xavierr | hey mat128 vdrok o/ | 12:32 |
*** appu103_ has quit IRC | 12:33 | |
*** dtantsur is now known as dtantsur|bbl | 12:35 | |
*** rbudden has joined #openstack-ironic | 12:36 | |
*** watanabe_isao has joined #openstack-ironic | 12:36 | |
TheJulia | Good morning | 12:44 |
*** parlos has joined #openstack-ironic | 12:46 | |
*** parlos_ has joined #openstack-ironic | 12:46 | |
mat128 | morning TheJulia | 12:49 |
*** parlos has quit IRC | 12:50 | |
*** parlos_ has quit IRC | 12:50 | |
*** ppiela has joined #openstack-ironic | 12:53 | |
openstackgerrit | Hugo Nicodemos proposed openstack/python-oneviewclient: Adding dynamic allocation operations https://review.openstack.org/258699 | 12:54 |
*** absubram has joined #openstack-ironic | 12:54 | |
*** absubram_ has joined #openstack-ironic | 12:55 | |
vdrok | TheJulia, good morning | 12:58 |
*** lucas-hungry is now known as lucasagomes | 12:59 | |
*** absubram has quit IRC | 12:59 | |
*** absubram_ is now known as absubram | 12:59 | |
*** Goneri has joined #openstack-ironic | 13:02 | |
lucasagomes | mat128, thanks for the review on the RFE, replied to your comment | 13:03 |
*** vmud213 has quit IRC | 13:03 | |
mat128 | lucasagomes: oh great, thanks :) | 13:04 |
mat128 | I'd love to have that feature right now in production :) | 13:04 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Refactor enroll-node code https://review.openstack.org/323190 | 13:04 |
*** lindycoder has joined #openstack-ironic | 13:05 | |
*** fragatina has quit IRC | 13:09 | |
*** ijw has joined #openstack-ironic | 13:09 | |
lucasagomes | mat128, o/ | 13:10 |
*** fragatina has joined #openstack-ironic | 13:10 | |
*** wajdi has joined #openstack-ironic | 13:11 | |
*** fragatina has quit IRC | 13:11 | |
*** _vs has quit IRC | 13:11 | |
*** _vs has joined #openstack-ironic | 13:12 | |
*** rloo has joined #openstack-ironic | 13:12 | |
*** wajdi_ has joined #openstack-ironic | 13:13 | |
NobodyCam | Good Morning Ironicers Hope everyone had a good weekend | 13:15 |
*** ijw has quit IRC | 13:15 | |
*** fragatina has joined #openstack-ironic | 13:15 | |
*** _vsaienko has quit IRC | 13:15 | |
*** _vs has quit IRC | 13:15 | |
*** _vsaienko has joined #openstack-ironic | 13:16 | |
*** wajdi has quit IRC | 13:16 | |
*** _vs has joined #openstack-ironic | 13:16 | |
*** rbartal has joined #openstack-ironic | 13:21 | |
*** ishant has joined #openstack-ironic | 13:21 | |
rbartal | lucasagomes, ping | 13:21 |
lucasagomes | rbartal, hi there | 13:21 |
*** _vs has quit IRC | 13:21 | |
*** _vsaienko has quit IRC | 13:21 | |
*** ayoung has joined #openstack-ironic | 13:22 | |
*** _vsaienko has joined #openstack-ironic | 13:22 | |
*** _vs has joined #openstack-ironic | 13:22 | |
vdrok | morning NobodyCam ! | 13:24 |
NobodyCam | Good morning vdrok :) | 13:24 |
lucasagomes | NobodyCam, vdrok good morning :-) | 13:25 |
NobodyCam | morning lucasagomes :) | 13:25 |
vdrok | good afternoon Lucas :) | 13:25 |
*** Goneri has quit IRC | 13:26 | |
*** ishant has quit IRC | 13:26 | |
*** absubram has quit IRC | 13:33 | |
TheJulia | good morning vdrok | 13:33 |
TheJulia | good morning NobodyCam | 13:34 |
NobodyCam | morning TheJulia | 13:34 |
*** ChrisAusten has joined #openstack-ironic | 13:36 | |
rloo | morning vdrok, NobodyCam, TheJulia, lucasagomes | 13:38 |
vdrok | morning rloo | 13:38 |
lucasagomes | rloo, hi there! | 13:38 |
NobodyCam | good morning rloo | 13:38 |
*** watanabe_isao has quit IRC | 13:38 | |
*** electrofelix has quit IRC | 13:39 | |
*** mtanino has joined #openstack-ironic | 13:39 | |
*** yolanda_ has joined #openstack-ironic | 13:43 | |
*** rloo has quit IRC | 13:43 | |
*** Goneri has joined #openstack-ironic | 13:43 | |
*** rloo has joined #openstack-ironic | 13:43 | |
*** yolanda has quit IRC | 13:44 | |
*** mgoddard_ has joined #openstack-ironic | 13:45 | |
*** mgoddard has quit IRC | 13:45 | |
*** thiagop has joined #openstack-ironic | 13:46 | |
sambetts | o/ rloo TheJulia NobodyCam xavierr | 13:47 |
NobodyCam | morning sambetts :) | 13:47 |
thiagop | Good morning Ironic | 13:48 |
xavierr | hey sambetts \o\ | 13:48 |
NobodyCam | morning thiagop and xavierr | 13:48 |
thiagop | NobodyCam: o/ | 13:49 |
xavierr | morning NobodyCam and thiagop | 13:49 |
NobodyCam | :) | 13:49 |
*** KennethWilke has left #openstack-ironic | 13:49 | |
thiagop | TheJulia: after a few days out of the air to concentrated, the tests are now running on the new CI infra | 13:49 |
thiagop | to concentrate* | 13:50 |
*** electrofelix has joined #openstack-ironic | 13:51 | |
*** mtanino has quit IRC | 13:52 | |
milan | morning NobodyCam rloo TheJulia thiagop xavierr and all I've forgotten to mention :) | 13:53 |
thiagop | morning milan | 13:53 |
NobodyCam | morning milan :) | 13:54 |
rloo | good afternoon milan, sambetts, xavierr thiagop | 13:55 |
*** mgoddard__ has joined #openstack-ironic | 13:55 | |
*** ametts has joined #openstack-ironic | 13:56 | |
xavierr | good afternoon rloo :) | 13:56 |
*** ohamada has quit IRC | 13:58 | |
*** mgoddard_ has quit IRC | 13:58 | |
*** amotoki has quit IRC | 14:08 | |
*** _vsaienko has quit IRC | 14:08 | |
*** _vs has quit IRC | 14:09 | |
*** ijw has joined #openstack-ironic | 14:11 | |
*** wajdi_ has quit IRC | 14:12 | |
*** d0ugal has quit IRC | 14:15 | |
*** _vsaienko has joined #openstack-ironic | 14:16 | |
*** _vs has joined #openstack-ironic | 14:16 | |
*** d0ugal has joined #openstack-ironic | 14:16 | |
*** amotoki has joined #openstack-ironic | 14:17 | |
*** ijw has quit IRC | 14:17 | |
TheJulia | thiagop: \o/ | 14:18 |
TheJulia | good morning rloo sambetts xavierr thiagop | 14:19 |
*** lucasagomes is now known as lucas-afk | 14:19 | |
*** piet has joined #openstack-ironic | 14:22 | |
xavierr | morning TheJulia :) | 14:22 |
*** parlos has joined #openstack-ironic | 14:31 | |
mgould | morning TheJulia | 14:32 |
xavierr | hey mgould o/ | 14:32 |
*** ohamada has joined #openstack-ironic | 14:33 | |
*** ishant has joined #openstack-ironic | 14:34 | |
*** vishwanathj has quit IRC | 14:38 | |
*** amotoki has quit IRC | 14:38 | |
*** mgoddard__ has quit IRC | 14:39 | |
*** mgoddard has joined #openstack-ironic | 14:39 | |
*** joprovost has joined #openstack-ironic | 14:40 | |
mgould | xavierr: hi! | 14:40 |
vsaienko | sambetts: does cisco-ci supports depends-on to devstack patches? | 14:42 |
sambetts | vsaienko: it should do yes | 14:42 |
*** _vsaienko has quit IRC | 14:43 | |
vsaienko | sambetts: http://192.133.158.2:8080/view/openstack%20ironic/job/tempest-dsvm-ironic-pxe_ucs/1495/consoleFull | 14:43 |
*** _vs has quit IRC | 14:43 | |
*** _vsaienko has joined #openstack-ironic | 14:45 | |
*** _vs has joined #openstack-ironic | 14:45 | |
sambetts | vsaienko: the error I'm seeing there has nothing to do with the patch that triggered the change or the one it depends on :/ | 14:46 |
sambetts | oh actually hold on :/ | 14:46 |
vsaienko | sambetts: [localhost] out: 2016-05-31 14:24:45.388 | +++/opt/stack/ironic/devstack/lib/ironic:enroll_nodes:952 [m iniget_subsections /opt/stack/data/ironic/ironic_hw_nodes 'node-0\properties' [localhost] out: 2016-05-31 14:24:45.389 | /opt/stack/ironic/devstack/lib/ironic: line 952: iniget_subsections: command not found | 14:47 |
sambetts | yeah, I just spotted that as I walked up the logs | 14:47 |
*** rbartal has quit IRC | 14:47 | |
vsaienko | sambetts: I can prepare temp patch to ironic that will add this function to ironic tree to tests that change | 14:48 |
sambetts | vsaienko: something in my gut tells me that Depends-On on openstack/devstack might be a special edge case | 14:50 |
*** ayoung has quit IRC | 14:51 | |
sambetts | vsaienko: looking into it now, thanks for bringing it to my attention | 14:51 |
*** wajdi has joined #openstack-ironic | 14:52 | |
vsaienko | sambetts: thanks for help | 14:52 |
*** mtanino has joined #openstack-ironic | 14:55 | |
*** _vs has quit IRC | 14:55 | |
*** _vsaienko has quit IRC | 14:55 | |
*** vishwanathj has joined #openstack-ironic | 14:58 | |
*** rama_y has joined #openstack-ironic | 14:59 | |
*** _vsaienko has joined #openstack-ironic | 15:04 | |
*** _vs has joined #openstack-ironic | 15:04 | |
thiagop | krtaylor: around? | 15:05 |
*** _vsaienko has quit IRC | 15:06 | |
*** _vs has quit IRC | 15:06 | |
*** rcernin has quit IRC | 15:07 | |
*** daemontool_ has joined #openstack-ironic | 15:09 | |
*** mjura has quit IRC | 15:09 | |
krtaylor | thiagop, yes, sup? | 15:09 |
thiagop | krtaylor: do we have the requirements for third party ci documented (specially that thing about 45 patches/day) somewhere? | 15:10 |
*** daemontool has quit IRC | 15:10 | |
*** daemontool_ has quit IRC | 15:11 | |
*** daemontool__ has joined #openstack-ironic | 15:11 | |
*** moshele has quit IRC | 15:12 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Update the api-ref documentation for Ports https://review.openstack.org/313187 | 15:12 |
*** ijw has joined #openstack-ironic | 15:13 | |
*** links has quit IRC | 15:15 | |
krtaylor | thiagop, we have the spec, but it does not document test sizing info, that would change over time so I'd think we wouldn't want to | 15:18 |
*** ijw has quit IRC | 15:18 | |
thiagop | krtaylor: I'd rather have it at the spec and update it when the community agree to change the requirement that have it only in "our minds" | 15:20 |
thiagop | krtaylor: but, thanks for the info anyway ;) | 15:21 |
krtaylor | thiagop, that works for me, I'll add it to the list of stuff to document | 15:21 |
*** rbudden has quit IRC | 15:23 | |
JayF | lucas-afk: curious if 1587143 might need a spec given the back and forth already in the bug. I'm with mat128 that we should not pass additional credentials to the agent. | 15:23 |
*** jjohnson2 has joined #openstack-ironic | 15:24 | |
krtaylor | thiagop, you have seen this? https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/third-party-ci.html | 15:24 |
*** dprince has quit IRC | 15:25 | |
vsaienko | lucasagomes, dtantsur: please have look once you have a time: https://review.openstack.org/#/c/299318/. The patch adds a nv job to ironic-staging-drivers | 15:27 |
thiagop | krtaylor: I myself don't, but some people on my team did some time ago. I'm kinda taking it over. | 15:27 |
krtaylor | thiagop, gotcha, that would be a good thing to read, and the pointers to the infra docs | 15:28 |
thiagop | krtaylor: will do. Today is documentation day. I'm writing how to reproduce the CI environment in cause of flodding #kidding | 15:29 |
thiagop | in case* | 15:29 |
*** openstackgerrit has quit IRC | 15:33 | |
*** openstackgerrit has joined #openstack-ironic | 15:33 | |
JayF | jroll: wdyt about backporting jcook and my changes for nova driver into nova stable? I think it's probably justified and a significant enough fix | 15:37 |
jroll | JayF: sure? I don't have any say in what gets backported to nova :) | 15:37 |
JayF | jroll: mikal suggested when he landed mine that we do a stable backport; and my fix depends on jcook's fix really | 15:37 |
jroll | JayF: sure | 15:37 |
*** sacharya has joined #openstack-ironic | 15:38 | |
jroll | JayF: just saying, I don't think I'm the right person to ask about nova backports :) | 15:38 |
JayF | For something like that, would I cherry pick both patches and squash them into one, or would it be two stable fixes? | 15:38 |
jroll | if there's a bug in stable we should fix it, in general :) | 15:38 |
JayF | Just generally asking your opinion on if it's buggy enough to backport | 15:38 |
JayF | not an analysis on nova backport policies | 15:38 |
JayF | lol | 15:38 |
jroll | if the first is broken without the second, it's squashed, otherwise two, afaik | 15:38 |
jroll | well | 15:38 |
*** Sukhdev has joined #openstack-ironic | 15:38 | |
JayF | first is a good fix without the second, but incomplete | 15:38 |
jroll | I tend to backport small bug fixes anyway | 15:39 |
jroll | I'd just upload both and ask mriedem if he thinks you should squash | 15:39 |
*** vishwanathj has quit IRC | 15:40 | |
sambetts | o/ JayF | 15:41 |
JayF | o/ | 15:41 |
*** dmk0202 has quit IRC | 15:42 | |
*** ifarkas has quit IRC | 15:43 | |
*** Sukhdev has quit IRC | 15:44 | |
*** sabeen has joined #openstack-ironic | 15:44 | |
mat128 | hey everyone :) | 15:45 |
TheJulia | krtaylor: thiagop: my $0.02 is documentation of sizing context is vital, and I think as long as any update details, in essence states "Hey, this may change at a later point in time, it is based on this xyz things as of blah", we will be best off in the long run. | 15:45 |
*** ayoung has joined #openstack-ironic | 15:45 | |
mat128 | JayF: lucas-afk confirmed Ironic would do the upload to swift so that alleviates my concern | 15:45 |
krtaylor | TheJulia, noted, and actually said that way, I agree completely | 15:50 |
*** lucas-afk is now known as lucasagomes | 15:52 | |
lucasagomes | JayF, hi there, I can make a spec for it if needed. But about the credentials specifically, as mat128 pointed out, I won't be passing it to IPA at all | 15:53 |
*** tesseract has quit IRC | 15:54 | |
JayF | Yeah, I'll take a closer look and see | 15:54 |
lucasagomes | JayF, I will get the logs gzipped and base64 enconded as a string from the result of the collect_logs_command() via the IPA API, and then Ironic itself will upload it to swift | 15:54 |
JayF | if I like it and don't think it needs a spec, what's the process? | 15:54 |
JayF | rfc -> rfc-approved? | 15:54 |
* JayF doesn't like an "approval process" that doesn't have voting :( | 15:54 | |
mat128 | lucasagomes: I'm personally fine without a spec, but I think integration tests that work against all images is required | 15:54 |
lucasagomes | JayF, yeah, if cores are OK with it we just change the tag | 15:54 |
mat128 | JayF: maybe we can introduce lightweight specs ;) | 15:55 |
lucasagomes | mat128, yeah, so I'm now testing with DIB, I've tested with coreos and tinyipa (the ipa code is here: https://review.openstack.org/#/c/248832/) | 15:55 |
*** _vsaienko has joined #openstack-ironic | 15:55 | |
*** _vs has joined #openstack-ironic | 15:55 | |
mat128 | yeah I've seen the usage of dmesg + getting /var/log | 15:56 |
mat128 | what would be a good integration test for this? Is there anything that can be asserted? | 15:56 |
mat128 | (logs contained "Linux" somewhere? lol) | 15:56 |
lucasagomes | mat128, yeah, for systems with systemd is easier because we just get the journal log which contains everything | 15:57 |
lucasagomes | mat128, for systems withtout it I'm gzipping the whole /var/log + dmesg, we could extend it as needed | 15:57 |
JayF | lucasagomes: if only we had some kind of voting and approval system to see if cores are OK with something :( | 15:57 |
*** vishwanathj has joined #openstack-ironic | 15:57 | |
lucasagomes | JayF, I iwll put a spec up | 15:58 |
lucasagomes | JayF, I think that will be better for getting a consensus | 15:58 |
JayF | I don't want you to have to write a spec just because I dislike the system though | 15:58 |
JayF | I just never like ambiguity. | 15:58 |
JayF | Which means I'll just never approve a non-spec'd RFE bug | 15:58 |
JayF | since there's no reliable process to know when it's the right thing | 15:58 |
mat128 | JayF: wasnt there a document on how we do RFE different than blueprint that explained it? | 15:58 |
lucasagomes | JayF, yeah, I hear that :-/ in fairness the RFE already looks like a spec | 15:58 |
*** vmud213 has joined #openstack-ironic | 15:58 | |
lucasagomes | so it won't be much work | 15:58 |
lucasagomes | mat128, yeah, 1 sec | 15:58 |
lucasagomes | mat128, http://docs.openstack.org/developer/ironic/dev/code-contribution-guide.html | 15:59 |
jroll | JayF: if you aren't comfortable with 2-3 cores saying "yeah, spec-less is fine here" then ask for a spec | 15:59 |
*** pcaruana has quit IRC | 15:59 | |
mat128 | JayF: easy way out: you change to rfe-approved and add a comment with "x, y and z were fine with this" | 16:00 |
*** dprince has joined #openstack-ironic | 16:01 | |
vmud213 | TheJulia: Would you mind taking a look at https://review.openstack.org/#/c/302143/ ? It's already a +2 and pending for long time | 16:04 |
*** _vsaienko has quit IRC | 16:06 | |
*** _vs has quit IRC | 16:06 | |
*** rbudden has joined #openstack-ironic | 16:10 | |
*** fragatina has quit IRC | 16:10 | |
*** moshele has joined #openstack-ironic | 16:11 | |
*** ijw has joined #openstack-ironic | 16:14 | |
*** _vsaienko has joined #openstack-ironic | 16:15 | |
*** _vs has joined #openstack-ironic | 16:15 | |
JayF | Backports for Ironic driver: https://review.openstack.org/323477 Unavailable hosts have no resources for use + https://review.openstack.org/323196 Ironic nodes with instance_uuid are not available are up for review against the stable/mitaka branch | 16:17 |
JayF | jcook: ^ your change backport, fyi | 16:17 |
rloo | JayF: I understand what you mean about specless rfe's. what jroll says though. even after that, i may still ask questions as you've noticed :) | 16:17 |
rloo | JayF: as long as the info is somewhere, in spec or in the bug/RFE itself, with enough detail to know what/how/etc. | 16:17 |
lucasagomes | rloo, can you take a look at that one specifically when you have time and say whether it needs a spec or not? | 16:18 |
* lucasagomes is really fine writing a spec for it if needed | 16:18 | |
rloo | lucasagomes: darn, i shouldn't have said anything. give me a few minutes. | 16:19 |
JayF | dtantsur|bbl: apparently I just pushed ^ one of those patches over your cherry pick. Didn't mean to stomp on you but both changes are needed anyway, so GMTA? | 16:19 |
lucasagomes | rloo, hah no worries, only when you get some time really | 16:19 |
*** fellypefca has joined #openstack-ironic | 16:19 | |
JayF | lucasagomes: honestly, adding an extension to IPA should be treated like a new API being added | 16:19 |
JayF | lucasagomes: which is something we call out as needing a spec typically | 16:19 |
lucasagomes | JayF, yeah it does add a new command | 16:19 |
lucasagomes | JayF, cool, fair enough I will put a spec up | 16:19 |
JayF | so we need to have a spec | 16:19 |
JayF | Thanks! | 16:20 |
lucasagomes | ty for looking into it | 16:20 |
*** ijw has quit IRC | 16:20 | |
rloo | JayF, lucasagomes: OH, yes, if it affects the API, we should have a spec. | 16:20 |
lucasagomes | rloo, API for IPA only | 16:21 |
lucasagomes | but still | 16:21 |
*** sabeen has quit IRC | 16:21 | |
JayF | I think it's easy to forget that adding a command/extension to IPA is an API change | 16:21 |
rloo | lucasagomes: Just skimmed the rfe, a spec would be good. | 16:21 |
lucasagomes | rloo, ty will do it | 16:21 |
rloo | thx lucasagomes | 16:23 |
TheJulia | vmud213: it is on my list, thank you | 16:23 |
vmud213 | TheJulia:nice.Thanks. | 16:24 |
*** ayoung has quit IRC | 16:25 | |
JayF | alineb: around? If so can you look at 321153 and read dtantsur|bbl's comment? I think he has the best idea of any of us re: naming but I didn't want to update the spec until you had looked at that too | 16:29 |
alineb | JayF: yes, i forgot it was possible, but i'm totally ok with it | 16:31 |
JayF | Awesome! I'll update the spec to reflect that | 16:32 |
JayF | rloo: ^ that should make you happy re: metrics and naming. Thanks for pushing to reopen that b/c it will be much better than what we had before | 16:32 |
JayF | I will push an updated spec with that design today | 16:33 |
*** _vsaienko has quit IRC | 16:33 | |
*** _vs has quit IRC | 16:34 | |
*** derekh has quit IRC | 16:34 | |
*** garthb has joined #openstack-ironic | 16:35 | |
devananda | morning, all | 16:36 |
*** morgabra has quit IRC | 16:36 | |
*** morgabra has joined #openstack-ironic | 16:36 | |
devananda | krtaylor, thiagop: generally, a spec doesn't get updated after it is approved / completed. it's meant more as a point-in-time reference than a living document. SO. If you add words about how many patches/day we expect, I would suggest leaving it open-ended and _not_ updating it | 16:38 |
*** e0ne has quit IRC | 16:38 | |
*** sabeen has joined #openstack-ironic | 16:41 | |
*** ayoung has joined #openstack-ironic | 16:42 | |
*** fellypefca has quit IRC | 16:45 | |
*** dprince has quit IRC | 16:51 | |
*** dprince has joined #openstack-ironic | 16:51 | |
NobodyCam | morning JayF devananda mat128 jroll | 16:51 |
krtaylor | devananda, agreed, thats what TheJulia was referring to, basically making the documentation say about sizing for test systems "as of this release there were xyz..." | 16:51 |
jroll | hey NobodyCam | 16:52 |
NobodyCam | :) | 16:52 |
*** trown is now known as trown|lunch | 16:52 | |
*** spandhe has joined #openstack-ironic | 16:53 | |
JayF | gm | 16:55 |
*** links has joined #openstack-ironic | 17:00 | |
*** clenimar has joined #openstack-ironic | 17:00 | |
*** alex_xu has quit IRC | 17:00 | |
*** rcernin has joined #openstack-ironic | 17:01 | |
*** kromanenko_ has quit IRC | 17:02 | |
*** kromanenko__ has quit IRC | 17:02 | |
*** Romanenko_K has quit IRC | 17:02 | |
thiagop | morning devananda, ok then :) | 17:02 |
*** alex_xu has joined #openstack-ironic | 17:02 | |
*** vishwanathj has quit IRC | 17:10 | |
*** links has quit IRC | 17:11 | |
vdrok | good night! | 17:12 |
*** vdrok has quit IRC | 17:12 | |
*** sabeen has quit IRC | 17:13 | |
*** dtantsur|bbl is now known as dtantsur | 17:17 | |
dtantsur | JayF, morning. no worries as soon as we do get it backported :) | 17:18 |
dtantsur | also morning NobodyCam, devananda, TheJulia, xavierr, rloo and everyone else | 17:18 |
JayF | gm | 17:19 |
NobodyCam | hey hey dtantsur Good morning... just doing the catch-up thing for being off yesterday | 17:19 |
*** parlos has quit IRC | 17:19 | |
JayF | dtantsur: I also backported my second half of fixing that kind of case | 17:20 |
sambetts | Night all, I'm heading off | 17:20 |
*** ijw has joined #openstack-ironic | 17:20 | |
NobodyCam | night sambetts | 17:20 |
sambetts | cya NobodyCam | 17:20 |
*** sambetts is now known as sambetts|afk | 17:20 | |
dtantsur | JayF, awesome! Actually I should have waited for you to appear, but I wanted to get this patch out of my head :) | 17:20 |
dtantsur | g'night sambetts|afk | 17:20 |
dtantsur | jroll, hey, we finally got broken by testing ssh internals in mitaka too :) please review https://review.openstack.org/#/c/323506/ | 17:22 |
dtantsur | devananda, ^^^ | 17:22 |
devananda | dtantsur: heh. looking | 17:22 |
*** penick has joined #openstack-ironic | 17:23 | |
rloo | hi dtantsur | 17:24 |
NobodyCam | ahh the logs droped off the bug on that | 17:24 |
*** ijw has quit IRC | 17:25 | |
devananda | dtantsur: curious,what changed upstream? | 17:27 |
dtantsur | devananda, we're checking how one 3rd party function calls another. now they're calling them with a bit different signature | 17:28 |
dtantsur | ok, the cherry-pick passes the gate. please review | 17:31 |
*** ijw has joined #openstack-ironic | 17:31 | |
*** mgould is now known as mgould|afk | 17:33 | |
jcook | JayF sweet! | 17:34 |
jcook | JayF I saw the PR for the add on fix Friday, but I was done at that point. Glad to see it got merged and backported. | 17:34 |
devananda | just added ironic here: https://bugs.launchpad.net/ironic/+bug/1586979 | 17:35 |
openstack | Launchpad bug 1586979 in OpenStack Compute (nova) "AMQP 2.0 prevents services from starting" [Undecided,New] | 17:35 |
devananda | because that has broken me locally and several folks downstream today | 17:35 |
dtantsur | devananda, I suggest it rather go to oslo.messaging | 17:36 |
dtantsur | at first glance it does not like something we can fix | 17:36 |
dtantsur | but thanks for tracking it! | 17:36 |
devananda | dtantsur: agreed | 17:37 |
dtantsur | devananda, btw http://docs.openstack.org/project-team-guide/stable-branches.html#review-guidelines says you can approve a backported provided by me (if you feel like ofc) | 17:38 |
*** e0ne has joined #openstack-ironic | 17:39 | |
devananda | https://bugs.launchpad.net/oslo.messaging/+bug/1586840 << oslo bug | 17:39 |
openstack | Launchpad bug 1586840 in oslo.messaging "amqp 2.0 breaks the kombu driver" [Undecided,In progress] - Assigned to Kirill Bespalov (k-besplv) | 17:39 |
*** jtomasek has quit IRC | 17:39 | |
*** ohamada has quit IRC | 17:39 | |
dtantsur | devananda, should we maybe mark our bug as duplicate? or do you want to keep it for tracking? | 17:39 |
devananda | depending on how long the oslo bug takes to fix, if we need to land a pin, it might be good | 17:40 |
devananda | *to track it | 17:40 |
dtantsur | got it | 17:40 |
*** ishant has quit IRC | 17:40 | |
dtantsur | devananda, (I know I ping you way too often, sorry :) I'm collecting feedback on the driver composition spec https://review.openstack.org/#/c/188370/ could you please at least take a look at API and upgrade parts which are totally new? | 17:41 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs: Collect system logs from IPA upon a deploy failure https://review.openstack.org/323511 | 17:41 |
lucasagomes | mat128, JayF ^ | 17:41 |
lucasagomes | when you guys have some time | 17:42 |
lucasagomes | with that, I will call it a day :-) | 17:43 |
devananda | I wonder why https://bugs.launchpad.net/oslo.messaging/+bug/1586840 hasn't broken the gates yet | 17:43 |
openstack | Launchpad bug 1586840 in oslo.messaging "amqp 2.0 breaks the kombu driver" [Undecided,In progress] - Assigned to Kirill Bespalov (k-besplv) | 17:43 |
lucasagomes | have a great evening all, see you all tomorrow | 17:43 |
devananda | lucasagomes: g'night! | 17:43 |
devananda | dtantsur: ack, adding to my queue | 17:43 |
lucasagomes | devananda, oh morning! | 17:43 |
*** praneshp has joined #openstack-ironic | 17:43 | |
*** lucasagomes is now known as lucas-dinner | 17:43 | |
thiagop | see ya lucas-dinner | 17:43 |
dtantsur | devananda, because https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L60 and it's unlikely to get promoted | 17:43 |
dtantsur | devananda, remember, nova is in the constraints gate (unlike us) | 17:44 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs: Collect system logs from IPA upon a deploy failure https://review.openstack.org/323511 | 17:44 |
lucas-dinner | thiagop, see ya | 17:44 |
* lucas-dinner fixes a typo and go | 17:44 | |
dtantsur | g'night, lucas-dinner | 17:46 |
NobodyCam | night lucas-dinner | 17:47 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs: Collect system logs from IPA upon a deploy failure https://review.openstack.org/323511 | 17:48 |
* dtantsur likes the summary of ^^^ already. it's a big pain in the neck we can't do it now (compared to e.g. inspection) | 17:49 | |
dtantsur | jroll, btw I've stepped on your shoes a bit and proposed an ironicclient mitaka release (https://review.openstack.org/#/c/322891/). people were complaining we are not releasing an important bug fix | 17:51 |
*** vmud213 has quit IRC | 17:52 | |
*** penick_ has joined #openstack-ironic | 17:52 | |
*** praneshp has quit IRC | 17:52 | |
*** Sukhdev has joined #openstack-ironic | 17:52 | |
*** penick has quit IRC | 17:52 | |
*** penick_ is now known as penick | 17:52 | |
*** praneshp has joined #openstack-ironic | 17:53 | |
devananda | dtantsur: ah. I see. our requirements doesn't specify amqp, and oslo doesn't pin it: https://github.com/openstack/oslo.messaging/blob/master/requirements.txt#L36 | 17:54 |
devananda | so this will actually break in a bifrost deploy | 17:54 |
devananda | which is where it got reported downstream here | 17:54 |
dtantsur | devananda, correct. everything not using upper-constraints will probably get broken | 17:54 |
*** daemontool__ has quit IRC | 17:54 | |
TheJulia | :( | 17:55 |
*** daemontool has joined #openstack-ironic | 17:55 | |
* devananda wonders if bifrost' gate is broken now | 17:55 | |
dtantsur | TheJulia, does it make sense for bifrost to use upper-constraints? | 17:55 |
TheJulia | it actually does | 17:56 |
TheJulia | we just don't define the upper consstraints presently, if my memory is serving me on how it works | 17:58 |
devananda | dtantsur: nova's requirements.txt doesn't pin it either, so anyone downstream who is installing nova from source // relying on "pip install -r requirements.txt" is going to be broken | 17:58 |
devananda | ah, I see. the gate isn't breaking because of u-c, but downstream is | 17:59 |
dtantsur | yes. only people using upper-constraints (like our gates) are saved for now | 17:59 |
dtantsur | correct | 17:59 |
*** moshele has quit IRC | 18:01 | |
*** fragatina has joined #openstack-ironic | 18:03 | |
*** trown|lunch is now known as trown | 18:04 | |
*** ChubYann has joined #openstack-ironic | 18:08 | |
*** ChubYann has quit IRC | 18:09 | |
*** ChubYann has joined #openstack-ironic | 18:09 | |
*** sabeen has joined #openstack-ironic | 18:11 | |
*** electrofelix has quit IRC | 18:12 | |
openstackgerrit | Ganesh Maharaj Mahalingam proposed openstack/bifrost: Install dependencies inside vagrant VM https://review.openstack.org/323523 | 18:21 |
gmmaha | TheJulia: cinerama: trying to bring up the bifrost Vm with stable/mitaka and we found that the setup.sh fails with cryptography needing these packages | 18:21 |
gmmaha | think we have fixed it the right way. your thoughts on that would help. https://review.openstack.org/323523 | 18:21 |
*** moshele has joined #openstack-ironic | 18:24 | |
cinerama | hey gmmaha we have a couple patches open in master which address this | 18:25 |
gmmaha | cinerama: aaah.. :) sorry should have looked for open patches | 18:25 |
cinerama | gmmaha, it's all good, thanks for the contribution | 18:25 |
gmmaha | cinerama: wold you like me to merge my patch with the other one thats open.. i saw that they havent updated the packages for yum based distros | 18:26 |
cinerama | gmmaha, we approved the other patches (there are two) which duplicate your change | 18:27 |
cinerama | gmmaha, if you need that backported to the mitaka branch that is something that needs to be done | 18:27 |
TheJulia | gmmaha: it happens :) | 18:27 |
gmmaha | cinerama: yes definitely a backport would help.. is that a patch that i can push? | 18:27 |
cinerama | gmmaha, yes | 18:28 |
gmmaha | cinerama: cool.. will get them up asap.. | 18:28 |
* gmmaha goes to find the other two patches | 18:28 | |
gmmaha | TheJulia: :) | 18:28 |
*** baoli_ has quit IRC | 18:29 | |
openstackgerrit | Ruby Loo proposed openstack/ironic: NOTIFICATION_TRANSPORT should be global https://review.openstack.org/323526 | 18:30 |
gmmaha | cinerama: i found one patch that you have approved, https://review.openstack.org/318943 but didnt find another that touches the yum packages | 18:31 |
gmmaha | any chance you have them handy? I can keep hunting if you dont | 18:31 |
openstackgerrit | Merged openstack/bifrost: Document that ssh_public_key_path must be set https://review.openstack.org/312782 | 18:33 |
gmmaha | cinerama: nevermind.. found nit | 18:40 |
gmmaha | s/nit/it/ | 18:40 |
openstackgerrit | Merged openstack/ironic: Remove links to github for OpenStack things https://review.openstack.org/321491 | 18:44 |
*** fragatina has quit IRC | 18:51 | |
*** fragatina has joined #openstack-ironic | 18:52 | |
openstackgerrit | Merged openstack/ironic: Remove hard-coded keystone version from setup https://review.openstack.org/323026 | 18:56 |
openstackgerrit | Merged openstack/ironic: Use a single uuid parameter in api-ref https://review.openstack.org/322958 | 18:59 |
gmmaha | cinerama: TheJulia: https://review.openstack.org/323535 && https://review.openstack.org/323538 | 19:01 |
*** cinerama has left #openstack-ironic | 19:01 | |
jroll | dtantsur: ack on that stable patch + ironicclient release | 19:04 |
dtantsur | thnx | 19:04 |
TheJulia | gmmaha: Thank you! | 19:05 |
*** moshele has quit IRC | 19:05 | |
*** e0ne has quit IRC | 19:07 | |
*** thrash is now known as thrash|biab | 19:08 | |
*** PollyZ has joined #openstack-ironic | 19:10 | |
*** adu has joined #openstack-ironic | 19:12 | |
*** dtantsur is now known as dtantsur|afk | 19:19 | |
*** e0ne has joined #openstack-ironic | 19:20 | |
*** e0ne has quit IRC | 19:28 | |
*** _vsaienko has joined #openstack-ironic | 19:35 | |
*** _vs has joined #openstack-ironic | 19:35 | |
*** rama_y has quit IRC | 19:38 | |
*** daemontool has quit IRC | 19:38 | |
*** sdake has joined #openstack-ironic | 19:45 | |
*** adu has quit IRC | 19:46 | |
*** piet has quit IRC | 19:47 | |
*** rbudden has quit IRC | 19:48 | |
*** piet has joined #openstack-ironic | 19:48 | |
*** sdake_ has joined #openstack-ironic | 19:49 | |
*** thrash|biab is now known as thrash | 19:50 | |
*** sdake has quit IRC | 19:51 | |
*** moshele has joined #openstack-ironic | 19:51 | |
*** ayoung has quit IRC | 19:54 | |
*** Goneri has quit IRC | 19:55 | |
*** ayoung has joined #openstack-ironic | 19:55 | |
*** lucas-dinner has quit IRC | 19:59 | |
*** rama_y has joined #openstack-ironic | 20:01 | |
TheJulia | rloo: you around? | 20:02 |
rloo | in meeting, what's up? | 20:02 |
*** moshele has quit IRC | 20:03 | |
TheJulia | looking at manual cleaning, with an agent driver, and see https://github.com/openstack/ironic/blame/master/ironic/conductor/manager.py#L915 in the path. Looks like witht he agent driver, 905 is always going to be met if there is not a failure with the driver, which makes me think that we never get to 917 properly | 20:03 |
*** lucasagomes has joined #openstack-ironic | 20:05 | |
*** jcoufal has joined #openstack-ironic | 20:05 | |
*** dmk0202 has joined #openstack-ironic | 20:05 | |
rloo | TheJulia: sorry, I have to listen | 20:05 |
TheJulia | :) | 20:05 |
TheJulia | ok | 20:05 |
*** rbudden has joined #openstack-ironic | 20:08 | |
*** garthb_ has joined #openstack-ironic | 20:09 | |
*** garthb has quit IRC | 20:10 | |
*** rbudden has quit IRC | 20:10 | |
TheJulia | rloo: I have someone in an environment making a minor tweak and retrying | 20:13 |
*** daemontool has joined #openstack-ironic | 20:15 | |
*** clenimar has quit IRC | 20:24 | |
*** ayoung has quit IRC | 20:27 | |
*** dmk0202 has quit IRC | 20:28 | |
*** daemontool_ has joined #openstack-ironic | 20:31 | |
*** clenimar has joined #openstack-ironic | 20:32 | |
*** daemontool has quit IRC | 20:33 | |
*** fragatina has quit IRC | 20:34 | |
*** dprince has quit IRC | 20:35 | |
*** baoli has joined #openstack-ironic | 20:35 | |
*** dprince has joined #openstack-ironic | 20:37 | |
*** _vsaienko has quit IRC | 20:41 | |
*** _vs has quit IRC | 20:41 | |
nicodemos | good night, everyone | 20:44 |
*** PollyZ__ has joined #openstack-ironic | 20:44 | |
*** nicodemos is now known as nico-afk | 20:44 | |
*** dmk0202 has joined #openstack-ironic | 20:44 | |
*** PollyZ has quit IRC | 20:45 | |
*** PollyZ__ is now known as PollyZ | 20:45 | |
gmmaha | TheJulia: the bifrost patches seem to be failing for the pycrypto package.. it fails even for the patches that fix the issue.. | 20:47 |
gmmaha | i saw this in the logs, mkdir: cannot create directory ‘/opt/stack’: Permission denied | 20:47 |
*** moshele has joined #openstack-ironic | 20:49 | |
*** Sukhdev has quit IRC | 20:50 | |
*** xavierr is now known as xavierr_afk | 20:50 | |
*** Sukhdev has joined #openstack-ironic | 20:51 | |
*** dprince has quit IRC | 20:53 | |
*** dprince has joined #openstack-ironic | 20:53 | |
* TheJulia sighs | 20:53 | |
gmmaha | TheJulia: sorry, wish i could be of more help against just pointing out the obvious | 20:54 |
gmmaha | dont know enough about these tests to jump in a fix them.. atleast not right away | 20:54 |
TheJulia | gmmaha: apparently while I was out on medical leave, pycrypto disappeared from the base images. That fix got merged by putting it in requirements.txt, although appraently spawned an argument of how installation of ansible is occuring :\ | 20:54 |
gmmaha | ohhhh | 20:56 |
*** wajdi_ has joined #openstack-ironic | 20:57 | |
*** joprovost has quit IRC | 20:57 | |
*** jcoufal has quit IRC | 20:57 | |
gmmaha | TheJulia: the ansible package itself? and guessing folks rather use the ppa piece? | 20:58 |
*** trown is now known as trown|outtypewww | 20:58 | |
TheJulia | some folks would rather the requirement be managed as part of requirements.txt it's self, the problem is forcing that could result in some very bad things occuring | 20:59 |
*** wajdi has quit IRC | 21:00 | |
TheJulia | OSA uses 1.9.4 afaik... installed from their git repo, although apparently the direct invokation of "Hey, we need pycrypto" is missing in the entire path somewhere, at least that was my takeaway from the whirlwind that was last week. Truthfully it is a post-it note on my wall | 21:00 |
*** wajdi_ has quit IRC | 21:01 | |
gmmaha | TheJulia: :) | 21:01 |
*** daemontool_ has quit IRC | 21:02 | |
*** rama_y has quit IRC | 21:03 | |
*** mjturek1 has quit IRC | 21:10 | |
*** garthb_ has quit IRC | 21:12 | |
*** garthb_ has joined #openstack-ironic | 21:14 | |
*** jjohnson2 has quit IRC | 21:19 | |
PollyZ | when I was running “git review -s”, it gives me this error “raceback (most recent call last): | 21:19 |
PollyZ | File "/usr/bin/git-review", line 10, in <module> | 21:19 |
PollyZ | sys.exit(main()) | 21:19 |
PollyZ | File "/usr/lib/python2.7/dist-packages/git_review/cmd.py", line 1157, in main | 21:19 |
PollyZ | raise no_git_dir | 21:19 |
PollyZ | git_review.cmd.GitDirectoriesException: Cannot determine where .git directory is. | 21:19 |
PollyZ | The following command failed with exit code 128 | 21:19 |
PollyZ | "git rev-parse --show-toplevel --git-dir" | 21:19 |
PollyZ | fatal: Not a git repository (or any of the parent directories): .git | 21:19 |
PollyZ | ” | 21:19 |
PollyZ | how can i fix it? thanks | 21:19 |
mat128 | you're not in a git repo | 21:21 |
mat128 | PollyZ: try `git status` | 21:21 |
openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Update metrics spec to reflect review comments https://review.openstack.org/321153 | 21:22 |
*** causten_ has joined #openstack-ironic | 21:22 | |
PollyZ | i went to the ironic folder, and run the “git review -s”, it told me “Fetching gerrit fatal: Project not found: openstack/project fatal: Could not read from remote repository. ” | 21:23 |
*** dprince has quit IRC | 21:23 | |
JayF | PollyZ: what is the output of `git status` on that dir? | 21:23 |
*** ayoung has joined #openstack-ironic | 21:23 | |
JayF | Also, everyone say hi to PollyZ, she's interning with Rackspace this summer on the OnMetal team | 21:23 |
*** clenimar has quit IRC | 21:24 | |
PollyZ | “On branch TOPIC-BRANCH | 21:25 |
PollyZ | Untracked files: | 21:25 |
PollyZ | (use "git add <file>..." to include in what will be committed) | 21:25 |
PollyZ | etc/ironic/ironic.conf.local | 21:25 |
PollyZ | nothing added to commit but untracked files present (use "git add" to track) | 21:25 |
PollyZ | ” | 21:25 |
JayF | when you need to paste stuff in that's more than 2 lines, you can go to gist.github.com and paste it in there, submit it, and send a link into here | 21:25 |
PollyZ | thx Jay | 21:25 |
JayF | PollyZ: what about `git remote -v` on that dir? | 21:25 |
*** ChrisAusten has quit IRC | 21:26 | |
PollyZ | https://gist.github.com/PollyZ/f74c1ccfbdc9746a464e6b5df9fc0bb7 | 21:27 |
PollyZ | like this? | 21:27 |
JayF | and from that dir, you ran git review -s and it gave that error? | 21:28 |
jroll | gerrit ssh://PollyZhou@review.openstack.org:29418/openstack/project.git | 21:28 |
*** praneshp has quit IRC | 21:28 | |
jroll | openstack/project is the problem there | 21:28 |
JayF | oh good eye | 21:29 |
JayF | PollyZ: git remote rm gerrit # should remove the bad remote; then try git review -s again | 21:29 |
gmmaha | TheJulia: anyway i can help, please do let me know.. | 21:31 |
PollyZ | i entered “yes”, it gave me this https://gist.github.com/PollyZ/f74c1ccfbdc9746a464e6b5df9fc0bb7 | 21:32 |
JayF | Ah, good, that's a better error | 21:33 |
JayF | I think you missed a step in the docs, let me find it | 21:33 |
*** piet has quit IRC | 21:33 | |
*** rama_y has joined #openstack-ironic | 21:33 | |
*** piet has joined #openstack-ironic | 21:34 | |
PollyZ | At first I entered “openstack/project”, later on, I caught that, so I modified to “/ironic”, but it didnt work | 21:34 |
JayF | PollyZ: http://docs.openstack.org/infra/manual/developers.html#account-setup at the very end of this section, just before "installing git-review", there's a paragraph starting "You'll also want to upload an SSH key to Gerrit" | 21:35 |
JayF | that step was either missed or done incorrectly | 21:35 |
JayF | because gerrit doesn't know who you are. | 21:35 |
PollyZ | Actually I have done that, and it seems to be right. https://gist.github.com/PollyZ/f74c1ccfbdc9746a464e6b5df9fc0bb7 | 21:37 |
*** dmk0202 has quit IRC | 21:38 | |
JayF | No, for that part, you actually will have to login to review.openstack.org with the account you created | 21:39 |
JayF | and upload a copy of your public key | 21:39 |
JayF | it's not a local config that's missing | 21:39 |
PollyZ | you mean the ssh key right | 21:40 |
JayF | yes, your ssh public key | 21:40 |
mat128 | gerrit normally uses a "gerrit" remote in your repo | 21:40 |
jroll | here, specifically: https://review.openstack.org/#/settings/ssh-keys | 21:40 |
mat128 | that it sets up when you do git review -s | 21:40 |
JayF | mat128: if you see two gists ago, we're to git-review throwing ssh auth errors | 21:40 |
PollyZ | did that. | 21:40 |
mat128 | oops | 21:41 |
mat128 | JayF: lost track, my bad | 21:41 |
mat128 | Ahh | 21:41 |
mat128 | JayF, PollyZ: editing the same gist | 21:41 |
mat128 | confusing :) | 21:41 |
JayF | I didn't even notice, I just click, read, close | 21:42 |
*** rama_y has quit IRC | 21:42 | |
rloo | TheJulia: sorry, meetings are over. | 21:42 |
mat128 | alright, signing off | 21:42 |
rloo | TheJulia: yes that is right, it doesn't get to 917. | 21:43 |
JayF | PollyZ: if you're continuing to have trouble, I can jump on a vidyo and pair to help you get this setup for about 30m or so. | 21:43 |
*** rbudden has joined #openstack-ironic | 21:43 | |
TheJulia | rloo: so.... sounds like we have a bug then :( | 21:44 |
rloo | TheJulia: should it? the agent always has to prepare cleaning which is async. | 21:44 |
rloo | TheJulia: what is the bug? | 21:44 |
PollyZ | Yes please Jay! | 21:44 |
PollyZ | how to tag someone btw? | 21:44 |
rloo | TheJulia: if i recall correctly the agent heartbeats, and there is code that ends up calling the conductor's _do_next_clean_step. | 21:44 |
rloo | TheJulia: do you want me to check? | 21:45 |
jroll | PollyZ: in irc? just say their nickname like I just did to you :) | 21:45 |
* jroll says JayF for another example | 21:45 | |
PollyZ | jroll: like that | 21:45 |
jroll | yep | 21:45 |
PollyZ | cool. Thanks | 21:45 |
jroll | np | 21:46 |
jroll | your client should also be able to tab complete names so you don't need to type them all out | 21:46 |
TheJulia | rloo: Oddly I didn't see that, but I also found out that my user might have some unknown code, they pieced together their isntall quite a bit by hand :( If you can quickly verify that heartbeat does the right thing, then It might be a good starting point for backtracking if their install from current known-working source code goes sideways | 21:47 |
*** piet has quit IRC | 21:47 | |
*** baoli has quit IRC | 21:47 | |
*** krtaylor has quit IRC | 21:47 | |
rloo | TheJulia: looking ... | 21:47 |
*** baoli has joined #openstack-ironic | 21:48 | |
rloo | TheJulia: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_base_vendor.py#L494 | 21:49 |
TheJulia | thanks | 21:49 |
TheJulia | I just found that myself, so it must have been some vendor package issue or something. ERRINSUFFICENTINFO at the moment | 21:50 |
rloo | TheJulia: phew :) | 21:50 |
PollyZ | JayF: I am on my vidyo room now | 21:50 |
rloo | TheJulia: I mean, phew, the problem seems to be at your end :) | 21:50 |
TheJulia | lol | 21:51 |
*** fragatina has joined #openstack-ironic | 21:52 | |
TheJulia | Yeah, I have an open ticket to get access to the system in question to investigate further since, something is not right | 21:52 |
*** rbudden has quit IRC | 21:55 | |
NobodyCam | TheJulia: I expect ticket will be worked on until tomorrow thou :( | 22:01 |
TheJulia | NobodyCam: I'm going to call it a night then | 22:02 |
openstackgerrit | Merged openstack/ironic-python-agent: Consolidate IPA configuration into a config module https://review.openstack.org/320867 | 22:03 |
NobodyCam | ++ | 22:03 |
*** moshele has quit IRC | 22:07 | |
*** baoli has quit IRC | 22:08 | |
*** baoli_ has joined #openstack-ironic | 22:08 | |
*** sdake has joined #openstack-ironic | 22:10 | |
*** sdake_ has quit IRC | 22:12 | |
*** ametts has quit IRC | 22:14 | |
*** clenimar has joined #openstack-ironic | 22:22 | |
*** rama_y has joined #openstack-ironic | 22:23 | |
*** praneshp has joined #openstack-ironic | 22:30 | |
*** baoli_ has quit IRC | 22:40 | |
*** lindycoder has quit IRC | 22:43 | |
*** causten_ has quit IRC | 22:49 | |
*** PollyZ has quit IRC | 23:08 | |
*** piet has joined #openstack-ironic | 23:11 | |
JayF | jroll: https://review.openstack.org/#/c/318188/ has 4x+2 2x+1 but I figure it's the sort of think you should land, if you get a moment in the morning | 23:11 |
JayF | can someone with stable perms land this? been outstanding a long time https://review.openstack.org/#/c/311373/ (IPA requirements bump) | 23:22 |
*** clenimar has quit IRC | 23:27 | |
*** piet has quit IRC | 23:27 | |
JayF | lucasagomes: around? | 23:29 |
mrda | JayF: wrong time of day :) | 23:29 |
JayF | that's what I thought, but usually the /nick would be -afk or something | 23:30 |
JayF | lucasagomes: just -1'd a bugfix you probably care about it; I can talk about it a little in the morning if that's the absolute last resort (re: https://review.openstack.org/#/c/302143) | 23:34 |
JayF | There are a very large amount of outstanding, 1x+2 patches to devstack/lib/ironic -- I've reviewed almost all of them personally too, and they seem to be important fixes for one of our 3rd party CI folks if someone wants to review and potentially approve them | 23:39 |
*** sabeen has quit IRC | 23:39 | |
openstackgerrit | Merged openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/323067 | 23:52 |
*** chlong has quit IRC | 23:53 | |
*** wajdi has joined #openstack-ironic | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!