| *** mrda has quit IRC | 00:01 | |
| *** mrda has joined #openstack-ironic | 00:03 | |
| *** iron1 has quit IRC | 00:03 | |
| *** eguz has quit IRC | 00:10 | |
| *** newell_ has quit IRC | 00:13 | |
| JoshNang | Shrews: are you still track that memory issue? | 00:17 |
|---|---|---|
| JoshNang | *tracking | 00:17 |
| *** ChanServ changes topic to "All of the project infrastructure hosts are being restarted for security updates." | 00:21 | |
| *** openstackgerrit has quit IRC | 00:24 | |
| *** nosnos has joined #openstack-ironic | 00:29 | |
| *** blamar has quit IRC | 00:34 | |
| *** blamar has joined #openstack-ironic | 00:34 | |
| Shrews | JoshNang: https://bugs.launchpad.net/ironic/+bug/1300420 | 00:40 |
| Shrews | JoshNang: I see your 73121 review hit that bug. Please recheck with that bug number to help us track it | 00:42 |
| JoshNang | Shrews: thanks! k i'll do that | 00:43 |
| *** rloo has quit IRC | 01:07 | |
| *** blamar has quit IRC | 01:07 | |
| *** rloo has joined #openstack-ironic | 01:07 | |
| *** blamar has joined #openstack-ironic | 01:11 | |
| *** nosnos has quit IRC | 01:16 | |
| *** blamar has quit IRC | 01:17 | |
| *** matsuhashi has joined #openstack-ironic | 01:28 | |
| *** ChanServ changes topic to "OpenStack Bare Metal Provisioning | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic | Status: https://etherpad.openstack.org/p/IronicWhiteBoard" | 01:32 | |
| *** openstack has joined #openstack-ironic | 01:48 | |
| *** rloo has quit IRC | 01:57 | |
| *** rloo has joined #openstack-ironic | 01:57 | |
| *** rloo has quit IRC | 01:58 | |
| *** rloo has joined #openstack-ironic | 01:58 | |
| *** rloo has quit IRC | 02:02 | |
| *** Shrews has quit IRC | 02:30 | |
| *** harlowja is now known as harlowja_away | 02:30 | |
| *** Shrews has joined #openstack-ironic | 02:31 | |
| *** harlowja_away is now known as harlowja | 02:40 | |
| *** killer_prince has quit IRC | 02:54 | |
| *** coolsvap has joined #openstack-ironic | 03:05 | |
| *** dwalleck has joined #openstack-ironic | 03:25 | |
| *** matsuhashi has quit IRC | 03:29 | |
| *** blamar has joined #openstack-ironic | 03:30 | |
| *** blamar has quit IRC | 03:35 | |
| *** blamar has joined #openstack-ironic | 03:35 | |
| *** lazy_prince has joined #openstack-ironic | 03:41 | |
| *** lazy_prince is now known as killer_prince | 03:41 | |
| *** blamar has quit IRC | 03:45 | |
| *** lazy_prince has joined #openstack-ironic | 03:58 | |
| *** killer_prince has quit IRC | 03:59 | |
| *** lazy_prince is now known as killer_prince | 03:59 | |
| *** eghobo has joined #openstack-ironic | 04:01 | |
| *** matsuhashi has joined #openstack-ironic | 04:19 | |
| *** sabah has joined #openstack-ironic | 04:21 | |
| *** rameshg87 has joined #openstack-ironic | 04:26 | |
| *** dwalleck_ has joined #openstack-ironic | 04:33 | |
| *** dwalleck has quit IRC | 04:36 | |
| *** Mikhail_D_ltp has joined #openstack-ironic | 04:37 | |
| *** radsy has quit IRC | 04:41 | |
| *** pradipta_away is now known as pradipta | 04:50 | |
| *** Mikhail_D_ltp has quit IRC | 05:00 | |
| *** harlowja is now known as harlowja_away | 05:03 | |
| *** killer_prince has quit IRC | 05:05 | |
| *** killer_prince has joined #openstack-ironic | 05:17 | |
| *** max_lobur has joined #openstack-ironic | 05:18 | |
| *** saju_m has joined #openstack-ironic | 05:28 | |
| *** sabah has quit IRC | 05:33 | |
| *** dwalleck_ has quit IRC | 05:41 | |
| *** max_lobur has quit IRC | 05:41 | |
| *** vkozhukalov has joined #openstack-ironic | 05:54 | |
| *** sabah has joined #openstack-ironic | 05:59 | |
| openstackgerrit | Jenkins proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/83956 | 06:07 |
| *** vkozhukalov has quit IRC | 06:09 | |
| *** eghobo has quit IRC | 06:20 | |
| dtantsur | morning Ironic | 06:25 |
| *** sabah has quit IRC | 06:38 | |
| romcheg | Morning dtantsur | 06:40 |
| *** max_lobur has joined #openstack-ironic | 06:45 | |
| *** ifarkas has joined #openstack-ironic | 06:46 | |
| *** ifarkas has quit IRC | 06:51 | |
| *** ifarkas has joined #openstack-ironic | 06:54 | |
| *** max_lobur1 has joined #openstack-ironic | 06:56 | |
| *** max_lobur has quit IRC | 07:00 | |
| GheRivero | morning all | 07:15 |
| *** saju_m has quit IRC | 07:17 | |
| *** mrda is now known as mrda_away | 07:25 | |
| *** saju_m has joined #openstack-ironic | 07:25 | |
| *** ndipanov_gone is now known as ndipanov | 07:27 | |
| *** yuriyz has joined #openstack-ironic | 07:29 | |
| *** saju_m has quit IRC | 07:32 | |
| GheRivero | romcheg: ping | 07:34 |
| romcheg | GheRivero: pong | 07:34 |
| romcheg | Morning | 07:34 |
| yuriyz | morning GheRivero romcheg | 07:34 |
| romcheg | AFAIR we have to decide who will be the responsible guy for updating the common code | 07:35 |
| GheRivero | so, about the oslo liaison, what do you prefer? A duel, a mud fight? | 07:35 |
| GheRivero | :) | 07:35 |
| romcheg | GheRivero: Russian style — who drinks more vodka :D | 07:35 |
| GheRivero | no, please. Just thinking about it, give me hangover | 07:36 |
| romcheg | I can only suggest a dice roll really | 07:37 |
| *** Mikhail_D_ltp has joined #openstack-ironic | 07:37 | |
| GheRivero | sure. no problem. @all Any innocent hand around? | 07:38 |
| romcheg | dtantsur: around? | 07:39 |
| Mikhail_D_ltp | Morning all! :) | 07:39 |
| GheRivero | yuriyz: around? | 07:39 |
| yuriyz | here | 07:39 |
| GheRivero | we have a volunteer! | 07:40 |
| *** vkozhukalov has joined #openstack-ironic | 07:40 | |
| romcheg | yuriyz: Can you please forget that we work in one team and throw a coin/dice? | 07:40 |
| GheRivero | :) | 07:40 |
| dtantsur | romcheg, 5 minutes, please | 07:41 |
| yuriyz | romcheg ok | 07:41 |
| romcheg | yuriyz: Then you tell us who's the winner: /me or GheRivero | 07:41 |
| romcheg | Well, not the winner | 07:42 |
| romcheg | The chosen one :) | 07:42 |
| yuriyz | are you trust me for dice roll? | 07:43 |
| romcheg | yuriyz: You promice do be fair? | 07:43 |
| yuriyz | No! | 07:44 |
| yuriyz | hehe | 07:44 |
| yuriyz | a man from another team needed I think :) | 07:45 |
| romcheg | OK, let's ask dtantsur to be a referee then | 07:46 |
| GheRivero | :) | 07:46 |
| dtantsur | I can throw a coin, ok :) | 07:47 |
| dtantsur | select your sides | 07:47 |
| romcheg | No, you select, that would be more fair | 07:48 |
| *** dutsmoc has joined #openstack-ironic | 07:48 | |
| dtantsur | ehhhh... I know next to nothing about you both :) do you want me to chose what nick I like more? | 07:49 |
| romcheg | You can just throw a coin :) | 07:49 |
| GheRivero | I though this was going to be simple and quick :) Maybe the vodka is not such a bad idea | 07:51 |
| dtantsur | Ok, here I got 50 Czech korunas; city is GheRivero, lion is romcheg :) | 07:51 |
| dtantsur | winner is GheRivero | 07:52 |
| dtantsur | you can have 2nd round with vodka of course :) | 07:52 |
| romcheg | After the summit :-P | 07:52 |
| *** matsuhashi has quit IRC | 07:53 | |
| GheRivero | or during the summit. There are enough parties for a 3rd, 4th.... round | 07:53 |
| *** matsuhashi has joined #openstack-ironic | 07:54 | |
| romcheg | I'm thinking about that bug with mkfs and unsupported filesystems | 07:55 |
| romcheg | I will make a patch to oslo to support passing environment variables | 07:56 |
| romcheg | Then it would be possible to rely on the output of the mkfs utility | 07:56 |
| romcheg | It would also be useful potentially for our ipmitool stuff, because we rely on the process output much. | 07:57 |
| *** saju_m has joined #openstack-ironic | 07:58 | |
| *** jistr has joined #openstack-ironic | 07:59 | |
| dtantsur | makes sense | 08:00 |
| *** pbrooko has joined #openstack-ironic | 08:01 | |
| *** derekh has joined #openstack-ironic | 08:02 | |
| *** saju_m has quit IRC | 08:03 | |
| *** romcheg has quit IRC | 08:13 | |
| *** romcheg has joined #openstack-ironic | 08:14 | |
| *** saju_m has joined #openstack-ironic | 08:15 | |
| *** romcheg has quit IRC | 08:19 | |
| *** comstud has quit IRC | 08:20 | |
| *** dutsmoc is now known as comstud | 08:20 | |
| *** lucasagomes has joined #openstack-ironic | 08:31 | |
| *** romcheg has joined #openstack-ironic | 08:34 | |
| *** comstud has quit IRC | 08:48 | |
| *** comstud has joined #openstack-ironic | 08:48 | |
| *** athomas has joined #openstack-ironic | 08:49 | |
| openstackgerrit | Vladimir Kozhukalov proposed a change to openstack/ironic-python-agent: Added lvm partitioner https://review.openstack.org/85672 | 08:56 |
| *** comstud has quit IRC | 08:57 | |
| *** comstud has joined #openstack-ironic | 09:08 | |
| *** sabah has joined #openstack-ironic | 09:09 | |
| openstackgerrit | Sandhya Balakrishnan proposed a change to openstack/ironic: Ironic User Guide https://review.openstack.org/83343 | 09:24 |
| *** comstud has quit IRC | 09:28 | |
| *** comstud has joined #openstack-ironic | 09:28 | |
| *** overlayer has joined #openstack-ironic | 09:30 | |
| *** martyntaylor has joined #openstack-ironic | 09:36 | |
| *** comstud has quit IRC | 09:39 | |
| *** saju_m has quit IRC | 09:39 | |
| *** matsuhashi has quit IRC | 09:39 | |
| *** coolsvap has quit IRC | 09:39 | |
| *** Shrews has quit IRC | 09:39 | |
| *** mgagne has quit IRC | 09:39 | |
| *** harlowja_away has quit IRC | 09:39 | |
| *** russellb has quit IRC | 09:39 | |
| *** zul has quit IRC | 09:39 | |
| *** dshulyak has quit IRC | 09:39 | |
| *** adam_g has quit IRC | 09:39 | |
| *** wendar_ has quit IRC | 09:39 | |
| *** stevehuang has quit IRC | 09:39 | |
| *** russell_h has quit IRC | 09:39 | |
| *** rainya- has quit IRC | 09:39 | |
| *** dguerri_ has quit IRC | 09:39 | |
| *** sseago_ has quit IRC | 09:39 | |
| *** jroll has quit IRC | 09:39 | |
| *** EmilienM has quit IRC | 09:39 | |
| *** dhellmann has quit IRC | 09:39 | |
| *** dtantsur has quit IRC | 09:39 | |
| *** adam_g has joined #openstack-ironic | 09:39 | |
| *** adam_g has quit IRC | 09:39 | |
| *** adam_g has joined #openstack-ironic | 09:39 | |
| *** dtantsur has joined #openstack-ironic | 09:39 | |
| *** coolsvap has joined #openstack-ironic | 09:39 | |
| *** coolsvap has quit IRC | 09:39 | |
| *** coolsvap has joined #openstack-ironic | 09:39 | |
| *** matsuhashi has joined #openstack-ironic | 09:39 | |
| *** EmilienM has joined #openstack-ironic | 09:39 | |
| *** saju_m has joined #openstack-ironic | 09:39 | |
| *** coolsvap has quit IRC | 09:39 | |
| *** romcheg has quit IRC | 09:39 | |
| *** openstackgerrit has quit IRC | 09:39 | |
| *** lynxman has quit IRC | 09:39 | |
| *** zul has joined #openstack-ironic | 09:39 | |
| *** openstack has joined #openstack-ironic | 09:49 | |
| *** matsuhashi has quit IRC | 09:52 | |
| *** mikal has joined #openstack-ironic | 09:52 | |
| *** matsuhashi has joined #openstack-ironic | 09:53 | |
| *** athomas has quit IRC | 09:56 | |
| *** coolsvap has quit IRC | 09:56 | |
| *** coolsvap has joined #openstack-ironic | 09:56 | |
| *** hemna has quit IRC | 09:56 | |
| *** hemna has joined #openstack-ironic | 09:56 | |
| *** sabah has quit IRC | 09:57 | |
| *** sabah has joined #openstack-ironic | 09:57 | |
| *** mkerrin has quit IRC | 09:57 | |
| *** mkerrin has joined #openstack-ironic | 09:57 | |
| *** ndipanov has quit IRC | 09:57 | |
| *** ndipanov has joined #openstack-ironic | 09:57 | |
| *** dtantsur has quit IRC | 09:57 | |
| *** dtantsur has joined #openstack-ironic | 09:57 | |
| *** matty_dubs|gone has quit IRC | 09:58 | |
| *** matty_dubs|gone has joined #openstack-ironic | 09:58 | |
| *** stevehuang has joined #openstack-ironic | 10:03 | |
| *** overlayer has quit IRC | 10:03 | |
| *** wendar has quit IRC | 10:03 | |
| *** adam_g has quit IRC | 10:03 | |
| *** mrda_away has quit IRC | 10:03 | |
| *** bigjools has quit IRC | 10:03 | |
| *** zigo has quit IRC | 10:03 | |
| *** yongli has quit IRC | 10:03 | |
| *** john-n-seattle has quit IRC | 10:03 | |
| *** JoshNang has quit IRC | 10:03 | |
| *** jrist has quit IRC | 10:03 | |
| *** adam_g` has joined #openstack-ironic | 10:03 | |
| *** athomas has joined #openstack-ironic | 10:03 | |
| *** bigjools has joined #openstack-ironic | 10:03 | |
| *** JoshNang has joined #openstack-ironic | 10:03 | |
| *** mrda_away has joined #openstack-ironic | 10:03 | |
| *** bigjools has quit IRC | 10:03 | |
| *** bigjools has joined #openstack-ironic | 10:03 | |
| *** jrist has joined #openstack-ironic | 10:03 | |
| *** yongli has joined #openstack-ironic | 10:03 | |
| *** wendar has joined #openstack-ironic | 10:04 | |
| *** zigo has joined #openstack-ironic | 10:04 | |
| *** russell_h has quit IRC | 10:13 | |
| *** notq has quit IRC | 10:13 | |
| *** bigjools has quit IRC | 10:13 | |
| *** bigjools has joined #openstack-ironic | 10:13 | |
| *** bigjools has joined #openstack-ironic | 10:13 | |
| *** wendar has quit IRC | 10:14 | |
| *** amar has joined #openstack-ironic | 10:14 | |
| *** amar is now known as Guest57201 | 10:14 | |
| *** wendar has joined #openstack-ironic | 10:14 | |
| *** russellb has quit IRC | 10:15 | |
| *** rainya has quit IRC | 10:15 | |
| *** Guest57201 has quit IRC | 10:15 | |
| *** adam_g` has quit IRC | 10:15 | |
| *** JayF has quit IRC | 10:15 | |
| *** EmilienM has quit IRC | 10:15 | |
| *** jroll has quit IRC | 10:15 | |
| *** comstud has quit IRC | 10:15 | |
| *** mgagne has quit IRC | 10:16 | |
| *** adam_g has joined #openstack-ironic | 10:20 | |
| *** adam_g has quit IRC | 10:20 | |
| *** adam_g has joined #openstack-ironic | 10:20 | |
| *** lucasagomes has joined #openstack-ironic | 10:21 | |
| *** JayF has joined #openstack-ironic | 10:21 | |
| *** JayF has quit IRC | 10:21 | |
| *** JayF has joined #openstack-ironic | 10:21 | |
| *** EmilienM has joined #openstack-ironic | 10:22 | |
| *** Shrews has quit IRC | 10:26 | |
| *** killer_prince has quit IRC | 10:26 | |
| *** eloib_ has quit IRC | 10:26 | |
| *** killer_prince has joined #openstack-ironic | 10:27 | |
| *** Shrews has joined #openstack-ironic | 10:28 | |
| *** eloib_ has joined #openstack-ironic | 10:28 | |
| *** saju_m has quit IRC | 10:34 | |
| *** saju_m has joined #openstack-ironic | 10:35 | |
| *** coolsvap has quit IRC | 10:36 | |
| *** stevehuang has quit IRC | 10:39 | |
| *** ndipanov has quit IRC | 10:39 | |
| *** sabah has quit IRC | 10:39 | |
| *** antonym has quit IRC | 10:39 | |
| *** pbrooko has quit IRC | 10:39 | |
| *** Mikhail_D_ltp has quit IRC | 10:39 | |
| *** max_lobur1 has quit IRC | 10:39 | |
| *** ifarkas has quit IRC | 10:39 | |
| *** GheRivero has quit IRC | 10:39 | |
| *** lsmola_ has quit IRC | 10:39 | |
| *** datajerk1 has quit IRC | 10:39 | |
| *** ewindisch has quit IRC | 10:39 | |
| *** GheRivero has joined #openstack-ironic | 10:40 | |
| *** datajerk has joined #openstack-ironic | 10:40 | |
| *** ndipanov has joined #openstack-ironic | 10:40 | |
| *** stevehuang has joined #openstack-ironic | 10:40 | |
| *** sabah has joined #openstack-ironic | 10:40 | |
| *** max_lobur has joined #openstack-ironic | 10:40 | |
| *** lsmola has joined #openstack-ironic | 10:40 | |
| *** Mikhail_D_ltp has joined #openstack-ironic | 10:40 | |
| *** antonym has joined #openstack-ironic | 10:40 | |
| *** ifarkas has joined #openstack-ironic | 10:40 | |
| *** ndipanov has quit IRC | 10:40 | |
| *** ndipanov has joined #openstack-ironic | 10:40 | |
| *** ifarkas has quit IRC | 10:40 | |
| *** ifarkas has joined #openstack-ironic | 10:40 | |
| *** lsmola has quit IRC | 10:40 | |
| *** lsmola has joined #openstack-ironic | 10:40 | |
| *** ewindisch has joined #openstack-ironic | 10:44 | |
| *** Mikhail_D_ltp has left #openstack-ironic | 10:46 | |
| *** Mikhail_D_ltp has joined #openstack-ironic | 10:46 | |
| *** matsuhashi has quit IRC | 10:53 | |
| *** matsuhashi has joined #openstack-ironic | 10:54 | |
| *** matsuhashi has quit IRC | 10:58 | |
| *** matsuhashi has joined #openstack-ironic | 10:59 | |
| *** Mikhail_D_wk has joined #openstack-ironic | 11:03 | |
| romcheg | lucasagomes: around? | 11:04 |
| lucasagomes | romcheg, hi there | 11:04 |
| lucasagomes | yes | 11:04 |
| romcheg | I've seen your patch with the whitelist | 11:04 |
| romcheg | It won't pass the gates until devstack installation script is changed | 11:05 |
| romcheg | I can do that for you, if you wish | 11:05 |
| lucasagomes | no? | 11:06 |
| lucasagomes | it doesn't use the pxe_ssh driver? | 11:06 |
| lucasagomes | does it uses fake driver? | 11:06 |
| romcheg | Yes, fake driver is used to test APIs | 11:07 |
| lucasagomes | ohhh | 11:07 |
| lucasagomes | :( | 11:07 |
| romcheg | I clearly remember that because I implemented that | 11:07 |
| lucasagomes | I see | 11:07 |
| romcheg | It's not a problem actually, we can configure ironic in devstack | 11:08 |
| lucasagomes | yeah | 11:08 |
| romcheg | Give me a moment | 11:08 |
| lucasagomes | right thanks for pointing that out | 11:08 |
| * lucasagomes thought it was using pxe_ssh driver | 11:08 | |
| lucasagomes | heh at least we know it works :D | 11:08 |
| romcheg | No, that is for integration tests | 11:08 |
| lucasagomes | yeah | 11:09 |
| lucasagomes | where I can fix that? we can leave the whitelist patch as WIP until it's fixed in the tempest tests | 11:10 |
| romcheg | lucasagomes: I'm setting drivers_whitelist fake,pxe_ipminative,pxe_ipmitool,pxe_ssh | 11:10 |
| lucasagomes | romcheg, ack! thank you for that | 11:10 |
| romcheg | Well, we might not want use pxe_ipminative and pxe_ipmitool in the gates | 11:11 |
| lucasagomes | yeah, if u want to be more fine grained you can enable only the fake driver | 11:11 |
| romcheg | ssh_pxe will be used for intergration testing | 11:12 |
| lucasagomes | ack | 11:12 |
| romcheg | We can also move that config to devstack-gate | 11:12 |
| lucasagomes | I could enable fake by default on the patch, but we have this default config should be production ready rule | 11:13 |
| lucasagomes | so | 11:13 |
| lucasagomes | :/ | 11:13 |
| lucasagomes | (I agree with that, but make things more difficult heh) | 11:14 |
| romcheg | For not let's leave ssh_pxe and fake by default in devstack | 11:15 |
| romcheg | That will be faster and will work for both functional and integration testing | 11:15 |
| romcheg | After that I will update devstack-gate to support configuring those options | 11:16 |
| lucasagomes | +1 | 11:16 |
| lucasagomes | romcheg, thanks for that! | 11:16 |
| *** sabah has quit IRC | 11:20 | |
| romcheg | lucasagomes: Commented on the patch | 11:22 |
| romcheg | lucasagomes: https://review.openstack.org/#/c/85996/ | 11:22 |
| *** Mikhail_D_wk has quit IRC | 11:23 | |
| lucasagomes | romcheg, ta much!! | 11:24 |
| lucasagomes | I marked it as WIP with a pending pointing to ur review | 11:24 |
| *** lucasagomes is now known as lucas-hungry | 11:25 | |
| lucas-hungry | brb | 11:25 |
| romcheg | I think we need to re-check the rest of the ironic's settings in devstack. I suspect some of them might be outdated or deprecated | 11:36 |
| *** pradipta is now known as pradipta_away | 11:37 | |
| *** jroll has joined #openstack-ironic | 11:45 | |
| *** russellb has joined #openstack-ironic | 11:46 | |
| *** russell_h has joined #openstack-ironic | 11:46 | |
| *** mgagne has joined #openstack-ironic | 11:48 | |
| *** comstud has joined #openstack-ironic | 11:52 | |
| *** rameshg87 has left #openstack-ironic | 12:15 | |
| *** wendar has quit IRC | 12:17 | |
| *** bigjools has quit IRC | 12:17 | |
| *** zigo has quit IRC | 12:17 | |
| *** russell_h has quit IRC | 12:17 | |
| *** wendar has joined #openstack-ironic | 12:17 | |
| *** matsuhashi has quit IRC | 12:18 | |
| *** zigo has joined #openstack-ironic | 12:18 | |
| *** bigjools has joined #openstack-ironic | 12:18 | |
| *** comstud has quit IRC | 12:18 | |
| *** bigjools has quit IRC | 12:18 | |
| *** bigjools has joined #openstack-ironic | 12:18 | |
| *** russellb has quit IRC | 12:18 | |
| *** matsuhashi has joined #openstack-ironic | 12:18 | |
| *** adam_g has quit IRC | 12:18 | |
| *** jroll has quit IRC | 12:19 | |
| *** mgagne has quit IRC | 12:19 | |
| *** JayF has quit IRC | 12:19 | |
| *** matsuhashi has quit IRC | 12:19 | |
| *** matsuhashi has joined #openstack-ironic | 12:19 | |
| *** JayF has joined #openstack-ironic | 12:20 | |
| *** notq has joined #openstack-ironic | 12:20 | |
| *** comstud has joined #openstack-ironic | 12:20 | |
| *** mgagne has joined #openstack-ironic | 12:21 | |
| *** adam_g has joined #openstack-ironic | 12:21 | |
| *** adam_g has quit IRC | 12:21 | |
| *** adam_g has joined #openstack-ironic | 12:21 | |
| *** jroll has joined #openstack-ironic | 12:21 | |
| *** russell_h has joined #openstack-ironic | 12:21 | |
| *** russellb has joined #openstack-ironic | 12:21 | |
| *** lucas-hungry is now known as lucasagomes | 12:27 | |
| *** matsuhashi has quit IRC | 12:30 | |
| *** linggao has joined #openstack-ironic | 12:31 | |
| *** shausy has joined #openstack-ironic | 12:37 | |
| *** jroll has quit IRC | 12:38 | |
| *** jroll has joined #openstack-ironic | 12:39 | |
| *** blamar has joined #openstack-ironic | 12:40 | |
| *** jdob has joined #openstack-ironic | 12:45 | |
| *** urulama has joined #openstack-ironic | 12:46 | |
| *** jbjohnso_ has joined #openstack-ironic | 12:50 | |
| NobodyCam | good morning Ironic | 12:51 |
| linggao | good morning, NobodyCam. | 12:52 |
| lucasagomes | morning NobodyCam linggao | 12:52 |
| linggao | good morning lucasagomes. | 12:53 |
| NobodyCam | morning linggao lucasagomes | 12:54 |
| NobodyCam | :) | 12:54 |
| linggao | NobodyCam, got your coffee yet? | 12:55 |
| NobodyCam | maken it now :) | 12:55 |
| *** rloo has joined #openstack-ironic | 12:56 | |
| *** overlayer has joined #openstack-ironic | 12:56 | |
| linggao | lucasagomes, NobodayCam, if you guys are on the doses of coffee, can you help me think about console support now? | 12:57 |
| lucasagomes | linggao, hah sure :D | 12:58 |
| NobodyCam | :) I can try | 12:58 |
| linggao | First question, who is the consumer of the console? admin or tenants? | 12:58 |
| NobodyCam | I would say both | 12:59 |
| NobodyCam | admins for support and tentents as users | 12:59 |
| linggao | Is it safe to give tenants write acess of console? Or should they just view the console? | 13:00 |
| lucasagomes | yeah, although in our api right now only admin can request it | 13:00 |
| NobodyCam | thou currently we (ironic) only supports admin level rights | 13:00 |
| NobodyCam | :-p | 13:00 |
| lucasagomes | we need to proxy it through nova maybe | 13:00 |
| lucasagomes | yeah | 13:00 |
| NobodyCam | seems I need to do a quick walkies... brb | 13:01 |
| linggao | are you saying that all ironic commands only supports admin level rights? | 13:01 |
| romcheg | Morning NobodyCam, linggao | 13:02 |
| lucasagomes | linggao, yeah right now our api is admin-only | 13:02 |
| linggao | morning romcheg | 13:02 |
| linggao | lucasagomes, I feel better. | 13:02 |
| lucasagomes | linggao, so about read-only, I'd say, would be good to have a mechanism doing some plumbing and being possible to start it as read-only or writable | 13:03 |
| lucasagomes | but as right now we only have admins interacting with us | 13:03 |
| *** Mikhail_D_wk has joined #openstack-ironic | 13:03 | |
| lucasagomes | we probably only will need writeable console | 13:03 |
| linggao | ok, | 13:03 |
| *** urulama has left #openstack-ironic | 13:03 | |
| linggao | now the patch I inheritant from sjing, when the console is enabled, a shellinboxd will run for that node. | 13:04 |
| lucasagomes | right | 13:05 |
| linggao | node-get-console will return the url for the console, then the admin can place that url in a browser to to the console. | 13:06 |
| linggao | yesterday when talking to devananda and NobodyCam, they also would like to see console from command line. | 13:06 |
| lucasagomes | right | 13:07 |
| lucasagomes | it would be good | 13:07 |
| lucasagomes | but it doesn't need to come all in one patch | 13:07 |
| linggao | right. | 13:07 |
| lucasagomes | the idea is to have a minimal console support now for ironic | 13:07 |
| lucasagomes | so we can have the pair functionality with nova | 13:07 |
| lucasagomes | so, I would do it in layers | 13:08 |
| linggao | I am just thinking how to do that, can we use the shellinabox in command line? | 13:08 |
| lucasagomes | get the basic in, which is the simply case with shellinaboxd etc... and then we can start improving it later | 13:08 |
| lucasagomes | linggao, :/ idk as well | 13:08 |
| lucasagomes | I don't think so tho | 13:08 |
| linggao | we can do ssh -t <host> ipmitool ..., but it will not go through the ironic-api. | 13:10 |
| lucasagomes | yeah | 13:10 |
| lucasagomes | heh maybe using lynx ? :P | 13:10 |
| lucasagomes | but I wouldn't worry about it much right now | 13:11 |
| lucasagomes | just by having a way to access the console is a big step | 13:11 |
| lucasagomes | right now we basically have none | 13:11 |
| linggao | right. l'll check in the patch first. I'll make changes to processutil.execute. | 13:12 |
| lucasagomes | ack | 13:12 |
| linggao | or just use subprocess.Popen. | 13:12 |
| linggao | which one do you prefer? | 13:12 |
| NobodyCam | morning romcheg :) | 13:13 |
| lucasagomes | hmm, I don't know if I have a preference to be honest... If having background tasks fits into the processutil.execute model I would put it there | 13:13 |
| lucasagomes | but if you feel it doesn't Popen sounds reasonable to me | 13:13 |
| lucasagomes | cause idk, processutil.execute seems to be a wrapper to run a command and get all the info related to it, return code, stderr, stdout etc... | 13:14 |
| lucasagomes | idk if it will feel natural to use it for long running processes | 13:14 |
| * lucasagomes haven't looked at the processutil.execute code much | 13:14 | |
| linggao | I am thinking adding a background=True/False mode for processutils.execute, it is true, then DEVNULL stderr and stdout | 13:16 |
| lucasagomes | NobodyCam, https://review.openstack.org/#/c/85996/ by roman; https://review.openstack.org/#/c/84742/ (when u get a time) | 13:16 |
| lucasagomes | linggao, stdin as well | 13:17 |
| lucasagomes | sounds reasonable | 13:17 |
| linggao | yes | 13:17 |
| linggao | this should be a separate patch? | 13:18 |
| lucasagomes | :/ yeah, cause it's an oslo thing | 13:18 |
| lucasagomes | you might even want to do it in parallel... execute() or Popen() will have the same effect on the console patch | 13:18 |
| *** matty_dubs|gone is now known as matty_dubs | 13:18 | |
| lucasagomes | so you might just want to use Popen for now, while trying to teaching processutils.execute() in another patch for oslo | 13:19 |
| lucasagomes | teach execute() to run background tasks* | 13:19 |
| linggao | ok, souds like a plan. | 13:20 |
| lucasagomes | ack :) | 13:20 |
| linggao | thanks :) | 13:20 |
| NobodyCam | lucasagomes: on 85996.. should we have a comment on how to enable ipmi? pxe_ipmi or leave that to our docs? | 13:20 |
| lucasagomes | linggao, np, thank YOU for the work :) | 13:20 |
| lucasagomes | NobodyCam, hmm, I guess, probably people will look at the docs first instead of trying to find the comment in the code? | 13:21 |
| NobodyCam | lol /me is odd I dive in to code b4 manuals | 13:22 |
| lucasagomes | lol | 13:22 |
| lucasagomes | heh, we could have both as well | 13:22 |
| lucasagomes | won't do any harm | 13:22 |
| *** openstackgerrit has joined #openstack-ironic | 13:23 | |
| NobodyCam | it ok devstack is for test env's | 13:23 |
| *** openstackgerrit has quit IRC | 13:23 | |
| lucasagomes | yeah | 13:24 |
| *** openstackgerrit has joined #openstack-ironic | 13:24 | |
| NobodyCam | humm guess we'll need a tripleO patch to for whitelisting won't we ? | 13:24 |
| lucasagomes | NobodyCam, which driver you use there? | 13:25 |
| lucasagomes | I don't think so you are using pxe_ssh right? | 13:25 |
| lucasagomes | I left it as default | 13:25 |
| lucasagomes | pxe_ssh, pxe_ipmitool and pxe_ipminative | 13:25 |
| *** tatyana has joined #openstack-ironic | 13:26 | |
| NobodyCam | ahh :) | 13:27 |
| NobodyCam | brb | 13:36 |
| lucasagomes | romcheg, hey ping... do you have any updates on the migration script from nova? | 13:47 |
| romcheg | lucasagomes: souldn't it wait for the summit? | 13:47 |
| romcheg | I mean, the folks from Nova decided not to freeze their code | 13:48 |
| romcheg | AFAIR | 13:48 |
| rloo | hey, once ironic is integrated with openstack, the admin will still need to do some ironic commands, right? everything can't be done via nova? | 13:48 |
| lucasagomes | romcheg, ah right ok... I just wanted to check if it's still on the way, because I was looking at the requirements for graduation | 13:49 |
| lucasagomes | and the migration one of them | 13:49 |
| lucasagomes | is one* | 13:49 |
| NobodyCam | oh side quiestion. has any one thought about Tilera Support? | 13:49 |
| romcheg | There is a design track about replacing nova bm with Ironic | 13:49 |
| romcheg | we should discuss how to keep our migration path accurate | 13:50 |
| jroll | rloo: I think so, at least to add the nodes | 13:50 |
| jroll | good morning ironic! | 13:50 |
| romcheg | Morning jroll | 13:50 |
| NobodyCam | good morning jroll | 13:51 |
| rloo | jroll: thx. I thought so. (Just looking at an arch diagram and I think it needs user <-> ironic api. | 13:51 |
| rloo | jroll: i was just pulling up your diagram and I see you have the user there :-) | 13:51 |
| jroll | rloo: heh :) | 13:52 |
| jroll | theoretically, nova should be in there somewhere :P | 13:52 |
| rloo | jroll: nah, Ironic is the center of the universe. | 13:53 |
| jroll | \o/ | 13:53 |
| NobodyCam | :) | 13:53 |
| jroll | too much work to go through nova imo | 13:53 |
| * jroll has the ironic nodes api nearly memorized by now | 13:54 | |
| lucasagomes | romcheg, +1 :) | 13:55 |
| rloo | jroll: impressive! | 13:56 |
| * NobodyCam updates agenda for next meeting | 14:01 | |
| lucasagomes | morning rloo jrist | 14:01 |
| lucasagomes | jroll, | 14:01 |
| lucasagomes | jrist, sorry buddy! | 14:02 |
| *** Mikhail_D_wk has quit IRC | 14:02 | |
| *** dshulyak has joined #openstack-ironic | 14:03 | |
| jrist | I don't get the greeting? | 14:03 |
| jrist | sheesh! | 14:03 |
| NobodyCam | morning rloo and jrist :) | 14:03 |
| jrist | NobodyCam++ | 14:04 |
| lucasagomes | jrist, morning! | 14:04 |
| jrist | good morning ironic | 14:04 |
| rloo | morning NobodyCam, jrist, jroll, and afternoon lucasagome! | 14:04 |
| jrist | man, that feels good | 14:04 |
| jrist | afternoon, lucasagomes ! | 14:04 |
| *** Mikhail_D_wk has joined #openstack-ironic | 14:04 | |
| lucasagomes | :D | 14:04 |
| rloo | sorry, lucasagomes ^^ | 14:04 |
| NobodyCam | so the next stop on the list is Roswell New Mexico.. :) | 14:07 |
| rloo | NobodyCam, I forgot. You're on the road to Atlanta! | 14:08 |
| NobodyCam | yep.. :) | 14:09 |
| * NobodyCam must get a starbucks coffee cup from Roswell :) | 14:09 | |
| romcheg | NobodyCam: You encourage me to unpack my bike after the winter and give it a ride | 14:10 |
| NobodyCam | :) heheheh | 14:11 |
| NobodyCam | romcheg: http://www.amazon.com/Kamp-Rite-Bushtrekka-Bicycle-Trailer-Oversize/dp/B002KKBT9Y | 14:12 |
| romcheg | NobodyCam: Wow... | 14:12 |
| NobodyCam | hehehe | 14:13 |
| *** kobier has joined #openstack-ironic | 14:13 | |
| NobodyCam | more people then I would have thought have things like that | 14:13 |
| NobodyCam | we've quite a few on the road :) | 14:13 |
| *** jbjohnso_ has quit IRC | 14:13 | |
| romcheg | I usually chose forests and hills rather than roads so this thing won't survive :) | 14:14 |
| NobodyCam | ya | 14:14 |
| NobodyCam | not ment for that :-p | 14:14 |
| romcheg | Roads are good for motorcycles, but for bicycles they're too boring | 14:15 |
| NobodyCam | :-p | 14:15 |
| GheRivero | morning all | 14:19 |
| NobodyCam | oh wow somthing strange happened.. https://review.openstack.org/#/c/85529 note the jenkins test results.. 3,4,3 ?? strange | 14:20 |
| NobodyCam | morning GheRivero :) | 14:20 |
| rloo | Hi GheRivero. | 14:20 |
| *** shausy has quit IRC | 14:20 | |
| rloo | lucasagomes; wrt man page: https://review.openstack.org/#/c/52902/. Did you see my comment? | 14:20 |
| *** shausy has joined #openstack-ironic | 14:21 | |
| lucasagomes | rloo, man page? lemme see | 14:21 |
| lucasagomes | not really, I'm sorry | 14:21 |
| lucasagomes | rloo, oh nice, so it's common among other projects? | 14:21 |
| rloo | lucasagomes: no worries. Just want to know what you think. I'd like to get back to coding soon so want to try to get docs out of the way ;) | 14:22 |
| lucasagomes | rloo, look, I would +1 it, because I like man pages | 14:22 |
| rloo | lucasagomes: well, i wouldn't call them man pages, but that's what the others call them. | 14:22 |
| *** jgrimm has joined #openstack-ironic | 14:22 | |
| lucasagomes | if u wanna take my patch and revive him feel free to do so | 14:22 |
| lucasagomes | right | 14:22 |
| rloo | lucasagomes: basically just mentions 'ironic', 'ironic help', but doesn't list the actual subcommands or describe them. | 14:22 |
| lucasagomes | yeah... | 14:23 |
| lucasagomes | if we think about man pages, we could break each command/session into a diff file | 14:23 |
| rloo | lucasagomes: ok, i'll just start a new one cuz again, not really 'man'. I don't really understand why the other projects call it 'man', but anyway. | 14:23 |
| lucasagomes | like the git man page | 14:23 |
| lucasagomes | that would be it easier to update | 14:23 |
| lucasagomes | rloo, heh ack | 14:23 |
| rloo | lucasagomes: there is another review that describes each command. i was thinking it would be nice, but that we should have a script that generates the docn. otherwise, i don't see how we can easily maintain it. | 14:24 |
| lucasagomes | for ref:https://www.kernel.org/pub/software/scm/git/docs/ | 14:24 |
| lucasagomes | rloo, +1 yeah if we can automate it would be great | 14:25 |
| rloo | lucasagomes: this one: https://review.openstack.org/#/c/84346/. | 14:25 |
| lucasagomes | oh this commit rings me a bell | 14:25 |
| lucasagomes | I have to review it | 14:25 |
| rloo | lucasagomes: the git one is nice. maybe we can open a bug, as a wish-list? for that? is it a bug? | 14:26 |
| lucasagomes | rloo, as a wishlist it makes sense, I consider lack of docs a bug yes | 14:26 |
| rloo | ahh, that git link. it says "git - the stupid content tracker". i didn't know that! | 14:26 |
| lucasagomes | heh | 14:27 |
| lucasagomes | linus is the author so u can expect a couple of bad words in it :P | 14:27 |
| rloo | ha ha | 14:27 |
| lucasagomes | but yeah I like the architeture of it, one file per command | 14:28 |
| lucasagomes | so all the commands are super well described | 14:28 |
| lucasagomes | with examples and all | 14:28 |
| lucasagomes | that reminds me the april fools that someone suggested to migrate the svn code to git heh | 14:29 |
| lucasagomes | https://issues.apache.org/jira/browse/INFRA-7524 | 14:29 |
| lucasagomes | :P | 14:29 |
| rloo | :D | 14:29 |
| *** jbjohnso_ has joined #openstack-ironic | 14:29 | |
| rloo | lucasagomes: it would be nice to know how this is generated: http://docs.openstack.org/user-guide/content/keystoneclient_commands.html | 14:32 |
| lucasagomes | oh | 14:33 |
| lucasagomes | that's nice | 14:33 |
| lucasagomes | maybe worth asking some q at #openstack-doc | 14:33 |
| rloo | I *hope* it isn't manually generated. Yeah, I'll ask #openstack-doc. thx. | 14:34 |
| NobodyCam | lucasagomes: I lov that jira ticket | 14:35 |
| lucasagomes | NobodyCam, lol me too! that was the best april-fool 2k14 for me | 14:35 |
| NobodyCam | :) ++++ | 14:35 |
| *** lucasagomes_ has joined #openstack-ironic | 14:39 | |
| lucasagomes_ | errr my connection is dropping >.< happened yesterday as well | 14:40 |
| lucasagomes_ | urgh | 14:40 |
| *** Mikhail_D_ltp has quit IRC | 14:40 | |
| *** lucasagomes has quit IRC | 14:41 | |
| *** saju_m has quit IRC | 14:41 | |
| * NobodyCam notes he is getting much lag. | 14:41 | |
| *** mkerrin has quit IRC | 14:45 | |
| *** dhellmann has quit IRC | 14:45 | |
| *** vkozhukalov has quit IRC | 14:45 | |
| *** romcheg has quit IRC | 14:45 | |
| *** hemna has quit IRC | 14:45 | |
| *** lynxman has quit IRC | 14:45 | |
| *** sirushti has quit IRC | 14:45 | |
| *** martyntaylor has quit IRC | 14:45 | |
| *** yuriyz has quit IRC | 14:45 | |
| *** mordred has quit IRC | 14:45 | |
| *** dividehex has quit IRC | 14:45 | |
| *** agordeev has quit IRC | 14:45 | |
| *** SpamapS has quit IRC | 14:45 | |
| *** morgabra has quit IRC | 14:45 | |
| *** yuriyz has joined #openstack-ironic | 14:48 | |
| *** vkozhukalov has joined #openstack-ironic | 14:48 | |
| *** martyntaylor has joined #openstack-ironic | 14:49 | |
| *** hemna has joined #openstack-ironic | 14:49 | |
| *** sirushti has joined #openstack-ironic | 14:49 | |
| *** hemna has quit IRC | 14:49 | |
| *** hemna has joined #openstack-ironic | 14:49 | |
| *** agordeev has joined #openstack-ironic | 14:49 | |
| *** romcheg has joined #openstack-ironic | 14:49 | |
| *** dividehex has joined #openstack-ironic | 14:49 | |
| *** dhellmann has joined #openstack-ironic | 14:49 | |
| *** aedocw has joined #openstack-ironic | 14:50 | |
| *** SpamapS has joined #openstack-ironic | 14:50 | |
| *** morgabra has joined #openstack-ironic | 14:51 | |
| *** mordred has joined #openstack-ironic | 14:51 | |
| *** yuriyz has quit IRC | 14:53 | |
| *** shausy has quit IRC | 14:53 | |
| *** yuriyz has joined #openstack-ironic | 14:55 | |
| *** mkerrin has joined #openstack-ironic | 14:55 | |
| *** aedocw has quit IRC | 14:56 | |
| *** lucasagomes_ is now known as lucasagomes | 15:01 | |
| NobodyCam | So lucasagomes do you connect dirrectly to IRC | 15:02 |
| lucasagomes | NobodyCam, yes... but it's but internet connection that flicking :/ | 15:03 |
| *** russell_h has quit IRC | 15:03 | |
| *** russell_h has joined #openstack-ironic | 15:03 | |
| NobodyCam | I spun up a tiny instance on Hp's cloud where I run irrsi and just ssh to it | 15:04 |
| NobodyCam | i should be using tmux but I'm old school so I still using screen | 15:05 |
| lucasagomes | ahn, that's a good idea | 15:05 |
| lucasagomes | hmmmmmmmmmm | 15:05 |
| * lucasagomes thinks about doing the same | 15:06 | |
| NobodyCam | :) works great :) | 15:06 |
| lucasagomes | so I won't lose the logs hhe | 15:06 |
| lucasagomes | heh* | 15:06 |
| NobodyCam | hehehe :) | 15:06 |
| *** lynxman has joined #openstack-ironic | 15:08 | |
| *** lynxman has quit IRC | 15:08 | |
| *** lynxman has joined #openstack-ironic | 15:08 | |
| NobodyCam | ok we're going to start pack up to hit the road.. google says it's 2 hours, so I expect it will take us 2 and a 1/2 or three hours to dock in Roswell | 15:08 |
| NobodyCam | I'll be back soon.. If I am needed for anything while in the road gtalk it always on my phone | 15:09 |
| NobodyCam | s/gtalk it/gtalk is/ | 15:10 |
| openstackgerrit | Vladimir Kozhukalov proposed a change to openstack/ironic-python-agent: Added lvm partitioner https://review.openstack.org/85672 | 15:10 |
| romcheg | sfdisk/sgdisk or parted? | 15:10 |
| *** Mikhail_D_ltp has joined #openstack-ironic | 15:12 | |
| romcheg | AFAIR we didn't have time to discuss that yesterday | 15:14 |
| lucasagomes | romcheg, yeah | 15:17 |
| lucasagomes | romcheg, +1 for parted for me | 15:17 |
| lucasagomes | it just a more complete tool | 15:17 |
| lucasagomes | and the script mode works great | 15:17 |
| romcheg | +1 for parted as well | 15:17 |
| lucasagomes | other things like setting negative offsets in case we want to do things at the end of the disk is very useful as well | 15:18 |
| romcheg | Although it's more complex, I think using one tool instead of two tools will be easier at the end | 15:18 |
| lucasagomes | also that | 15:18 |
| lucasagomes | and idk hw much more complex is it | 15:18 |
| lucasagomes | if we only use for partitioning | 15:18 |
| lucasagomes | it does support fs operations, but we are not using it | 15:18 |
| romcheg | I also have concerns about non-stantard cluster sizes | 15:19 |
| romcheg | IFAIK parted supports that | 15:19 |
| romcheg | not sure about sgdisk/sfdisk | 15:20 |
| *** aedocw_ has joined #openstack-ironic | 15:20 | |
| *** vkozhukalov has quit IRC | 15:20 | |
| lucasagomes | I think it's more related to the fs itself no? | 15:21 |
| romcheg | I think it's about partitioning as well | 15:21 |
| romcheg | If you have, let's say, a WD Green (wchich is terrible for servers, btw) and the partitioning tool does not support non-standard cluster sizes, you have to calculate that yourself | 15:22 |
| lucasagomes | I see | 15:22 |
| lucasagomes | right, well if parted supports it great heh | 15:23 |
| romcheg | Or you can end up with much higher disk io | 15:23 |
| romcheg | when two partitions are on a single cluster | 15:24 |
| romcheg | So +1 for parted agan :) | 15:24 |
| lucasagomes | :D | 15:24 |
| *** jbjohnso_ has quit IRC | 15:30 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add ManagementInterface https://review.openstack.org/86063 | 15:32 |
| *** coolsvap has joined #openstack-ironic | 15:33 | |
| *** Mikhail_D_ltp has quit IRC | 15:43 | |
| *** jbjohnso_ has joined #openstack-ironic | 15:45 | |
| *** john-n-seattle1 has joined #openstack-ironic | 15:45 | |
| *** hemna_ has joined #openstack-ironic | 15:47 | |
| *** romcheg has quit IRC | 15:52 | |
| *** yuriyz has quit IRC | 15:53 | |
| *** jistr has quit IRC | 15:55 | |
| *** eghobo has joined #openstack-ironic | 15:57 | |
| *** matty_dubs is now known as matty_dubs|lunch | 15:57 | |
| JayF | linggao: lucasagomes: re: scrollback, I think there's a place for a writable, tenant-accessible console -- i.e. the 'rescue mode' type of scenario, where someone broke their OS and needs to fix it. | 16:00 |
| lucasagomes | JayF, right yeah, I think it's valid as well | 16:00 |
| *** morgabra has quit IRC | 16:00 | |
| *** morgabra has joined #openstack-ironic | 16:00 | |
| lucasagomes | we need some work on our api tho to accept request from non-admin users on some URIs | 16:00 |
| lucasagomes | or it could be implement by proxying through nova | 16:01 |
| JayF | I think proxying through nova is a better way | 16:01 |
| JayF | nova supports console for vms, right? | 16:01 |
| lucasagomes | JayF, yeah it does | 16:01 |
| JayF | I know Rackspace does | 16:01 |
| JayF | yeah, okay, so I'd say we should hook into that | 16:01 |
| lucasagomes | +1 | 16:01 |
| JayF | Ironic nodes should look the same as virtual nodes for stuff like that | 16:01 |
| lucasagomes | yeah makes sense, it's good because we keep the look-feel the same when managing vms or baremetal | 16:02 |
| JayF | exactly | 16:02 |
| JayF | in a perfect world, imo, we can get to a point where bare metal vs vm looks very similar to the tenant | 16:03 |
| lucasagomes | +1, I'm not very familiar with the console part of nova, but doesn't seem much difficult to only pass the start/stop/get console from nova to ironic via the nova ironic driver | 16:04 |
| *** blamar has quit IRC | 16:04 | |
| JayF | Yeah idk much about the plumbing there either, but it seems like from a UX perspective, that's what we'd want | 16:04 |
| rloo | but doesn't nova let 'normal' users get access to the console? | 16:05 |
| JayF | yes, and imo we should too | 16:06 |
| *** romcheg has joined #openstack-ironic | 16:06 | |
| JayF | how is a tenant on baremetal any different than a tenant on a vm? | 16:06 |
| rloo | I haven't been keeping track so don't know where the authentication points are wrt the console. | 16:06 |
| lucasagomes | rloo, if we proxy through the nova ironic driver we can get admin_context in the driver | 16:06 |
| lucasagomes | and request the ironic api | 16:06 |
| rloo | i think the problem if there is, is that ironic has a policy.json, but i don't know how much it is hooked into ironic. | 16:06 |
| lucasagomes | rloo, just like we do for "nova boot" | 16:07 |
| lucasagomes | to be able to deploy a machine with ironic | 16:07 |
| rloo | lucasagomes: ah, ok, then it'll work automagically :-) | 16:07 |
| lucasagomes | rloo, yup | 16:07 |
| rloo | I saw policy.json set to admin, but haven't played around with it to see if it'll let non-admins use ironic if I change the rules in policy.json. | 16:08 |
| lucasagomes | yeah, our api is admin-only right now | 16:08 |
| rloo | so policy.json isn't used? | 16:08 |
| lucasagomes | rloo, it is | 16:08 |
| lucasagomes | that's where we set the admin-only rule | 16:08 |
| rloo | lucasagomes. so by default, it is admin-only, but someone could change policy.json so that it isn't admin-only, right? | 16:09 |
| rloo | I'm only being picky cuz I had to try to document some of this ;) | 16:09 |
| lucasagomes | rloo, yeah, you don't even need to change the policy.json | 16:09 |
| lucasagomes | there's a config option as well to set authentication to none | 16:09 |
| lucasagomes | instead of keystone | 16:09 |
| lucasagomes | rloo, auth_strategy | 16:10 |
| rloo | lucasagomes. that's different. ie, might want keystone authentication (eg valid user), but might want to allow access to users with a different role than 'admin'. | 16:10 |
| lucasagomes | rloo, ahh, right yeah | 16:10 |
| lucasagomes | so then we gotta change the policy.json | 16:11 |
| rloo | i recall in nova though (haven't looked recently) that there were placed in code where it was hardcoded to be admin and wasn't controlled by policy.json. Was wondering if that is the case in ironic too. | 16:11 |
| rloo | anyway, one day when I feel like playing around with policy.json, i'll take a look. | 16:11 |
| lucasagomes | ic, not that I'm aware of anyway | 16:12 |
| rloo | btw lucasagomes. just looking at ironic CLI. We have 'ironic-url' and 'ironic-api-version'. | 16:13 |
| rloo | i notice glance has 'os-image-url', and cinder has 'os-volume-api-version'. | 16:14 |
| *** derekh has quit IRC | 16:14 | |
| lucasagomes | rloo, right, I think that commands came when I was coppying the base CLI files from ceilometer | 16:16 |
| lucasagomes | I thought at one point the cli was a bit standardized but not really heh | 16:17 |
| rloo | lucasagomes: ok, well, as long as ironic is standardized with some other project. ha ha. | 16:18 |
| lucasagomes | lol | 16:20 |
| lucasagomes | yeah | 16:20 |
| lucasagomes | I think they are oslofying the CLI stuff | 16:20 |
| *** krtaylor has quit IRC | 16:21 | |
| *** martyntaylor has left #openstack-ironic | 16:21 | |
| rloo | that'll make it easier in the future. | 16:22 |
| mgagne | lucasagomes: thanks god :D | 16:22 |
| lucasagomes | mgagne, lol +2! | 16:22 |
| rloo | lucasagomes: that reminds me, I wanted to ask you something else about client.py | 16:22 |
| rloo | https://github.com/openstack/python-ironicclient/blob/master/ironicclient/client.py#L51 | 16:22 |
| rloo | lucasagomes: why can the api_version be 1 or 2? | 16:23 |
| lucasagomes | rloo, ouch well it can't, we only have v1! | 16:23 |
| lucasagomes | and it's not even done | 16:23 |
| rloo | lucasagomes: ha ha. ok, I thought maybe you were planning for the future :-) | 16:24 |
| lucasagomes | mind filling a bug about it? or fixing that docstring? | 16:24 |
| lucasagomes | rloo, yeah hah well it's a good plumbing to have | 16:24 |
| lucasagomes | but we only support v1 right now (we only have v1) | 16:24 |
| lucasagomes | rloo, many parts of the cli was just copy-cat from other projects | 16:25 |
| rloo | lucasagomes: ok, i'll take care of it. ie, either file a bug or fix it, depends on how I feel ;) | 16:25 |
| lucasagomes | so it might have a bunch of not related docstrings there | 16:25 |
| lucasagomes | rloo, ta much! | 16:25 |
| linggao | JayF, rloo, lucasagomes, sorry I was in the meeting and missed the discussion on the console. | 16:26 |
| rloo | no worries linggao. | 16:26 |
| lucasagomes | linggao, no bothers :) | 16:26 |
| linggao | So JayF, how does a nova tenant get a console? | 16:27 |
| linggao | :) | 16:27 |
| linggao | I mean, what is the command do they use? | 16:27 |
| linggao | I seem to remember there was console logging that a tenant can view from Horizon, but never tried to run a console from command line. | 16:29 |
| rloo | linggao: there is a 'nova get-vnc-console' | 16:29 |
| *** blamar has joined #openstack-ironic | 16:29 | |
| rloo | linggao: also 'nova get-spice-console' | 16:29 |
| rloo | linggao: http://docs.openstack.org/user-guide/content/novaclient_commands.html. | 16:30 |
| lucasagomes | it might have a baremetal command somewhere to enable the console | 16:30 |
| lucasagomes | since nova bm supports console access | 16:30 |
| * lucasagomes idk where, never used it | 16:31 | |
| linggao | rloo, thanks for the link | 16:31 |
| rloo | linggao: yw, although it doesn't give much info, it is a start ;) The spice console is browser-based. | 16:32 |
| linggao | lucasagomes, the baremetal driver has functions on console, | 16:32 |
| *** blamar has quit IRC | 16:32 | |
| linggao | we just need to hook it with Ironic. | 16:32 |
| *** blamar has joined #openstack-ironic | 16:32 | |
| lucasagomes | linggao, right, actually we need to implement those functions on the Nova Ironic Driver | 16:33 |
| linggao | lucasagomes, yes. | 16:33 |
| lucasagomes | because we don't use the baremetal driver when deploying with ironic | 16:33 |
| linggao | understand | 16:33 |
| *** saju_m has joined #openstack-ironic | 16:34 | |
| linggao | But from security poin of view, is it dangerous to let the tenant have the write access to a console for baremetal node? | 16:34 |
| linggao | They could reconfigure the hardware, could they? | 16:36 |
| lucasagomes | I'm not very expert in security, but it sounds like we should put some thought on it yes | 16:38 |
| lucasagomes | I think that having a way to start the console as read-only and read-write would be a great plumbing | 16:39 |
| lucasagomes | then we can set the specific use cases depending on the user requesting to start the console | 16:39 |
| *** krtaylor has joined #openstack-ironic | 16:39 | |
| linggao | +1 | 16:40 |
| *** newell_ has joined #openstack-ironic | 16:40 | |
| linggao | We also need to support console-log command. | 16:41 |
| linggao | this one should be open to the tenant for sure. | 16:42 |
| rloo | linggao, i think that if it is configurable read/write/whatever, that lets admins specify how much access they want to give tenants. | 16:45 |
| *** vkozhukalov has joined #openstack-ironic | 16:46 | |
| *** dwalleck has joined #openstack-ironic | 16:47 | |
| openstackgerrit | Vladimir Kozhukalov proposed a change to openstack/ironic-python-agent: Added lvm partitioner https://review.openstack.org/85672 | 16:50 |
| *** jroll has quit IRC | 16:50 | |
| linggao | rloo, +1 | 16:51 |
| *** jroll has joined #openstack-ironic | 16:51 | |
| *** mdickson has quit IRC | 16:55 | |
| *** jgrimm has quit IRC | 16:56 | |
| linggao | lucasagomes, JayF, rloo, I read a little bit more, the nova get-vnc-console returns a vnc session url, You can then copy and paste this URL into the browser. | 16:56 |
| linggao | This is the way Ironic is doing for the first console patch. | 16:56 |
| lucasagomes | nice! although we are not using vnc | 16:57 |
| lucasagomes | but yeah, same workflow :D | 16:57 |
| linggao | yes. phew, feel better now. | 16:57 |
| *** mdickson has joined #openstack-ironic | 16:58 | |
| lucasagomes | :D | 17:01 |
| *** tatyana has quit IRC | 17:04 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add ManagementInterface https://review.openstack.org/86063 | 17:05 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: IPMITool to use the new ManagementInterface https://review.openstack.org/86092 | 17:06 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add ManagementInterface https://review.openstack.org/86063 | 17:08 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: IPMITool to use the new ManagementInterface https://review.openstack.org/86092 | 17:08 |
| openstackgerrit | Ruby Loo proposed a change to openstack/python-ironicclient: Fix docstring for client.get_client() https://review.openstack.org/86095 | 17:08 |
| *** jgrimm has joined #openstack-ironic | 17:09 | |
| *** matty_dubs|lunch is now known as matty_dubs | 17:10 | |
| *** jgrimm has quit IRC | 17:10 | |
| *** jgrimm has joined #openstack-ironic | 17:11 | |
| lucasagomes | aye, I'm done for the day | 17:13 |
| *** jdob_ has joined #openstack-ironic | 17:13 | |
| lucasagomes | have a g'night NobodyCam devananda rloo, linggao everyone... :D | 17:13 |
| devananda | g'night lucasagomes! | 17:13 |
| rloo | night lucasagomes! | 17:13 |
| linggao | good night, lucasagomes. | 17:14 |
| *** lucasagomes is now known as lucas-dinner | 17:14 | |
| devananda | also, g'morning everyone :) | 17:14 |
| *** athomas has quit IRC | 17:15 | |
| rloo | morning devananda :) | 17:15 |
| devananda | morning, rloo | 17:16 |
| *** harlowja has joined #openstack-ironic | 17:16 | |
| devananda | rloo: if you have any questions on your new core powers, dont hesitate to ask. | 17:17 |
| rloo | devananda: oh, am I already a core? Oh oh. better get those reviews out of the way ;) | 17:17 |
| matty_dubs | Congrats, rloo! Use your powers for good, not evil. | 17:18 |
| rloo | matty_dubs. thx! good, evil, aren't they all the same thing? :-) | 17:18 |
| harlowja | rloo u da bomb! | 17:18 |
| harlowja | ;) | 17:18 |
| harlowja | *a good bomb | 17:18 |
| matty_dubs | good == +2'ing my patches; bad == -1 (or -2!)'ing my patches ;) | 17:18 |
| rloo | matty_dubs. ha ha. | 17:19 |
| JayF | hmm. http://russellbryant.net/openstack-stats/ironic-reviewers-30.txt the rest of the folks with Core Reviewer on IPA have the **, but I don't. Something broken? | 17:19 |
| *** jdob_ has quit IRC | 17:25 | |
| devananda | JayF: what is your gerrit username? | 17:27 |
| JayF | JayF | 17:27 |
| devananda | JayF: reviewstats/projects/ironic.json is the reference for the generation of those stats | 17:27 |
| devananda | "jayf", | 17:27 |
| devananda | is there | 17:27 |
| JayF | I can look into it and submit a merge request if needed | 17:27 |
| JayF | do caps matter? | 17:27 |
| devananda | dunno | 17:27 |
| JayF | it's "JayF" | 17:27 |
| JayF | I'll look into it with the infra guys and figure it out | 17:28 |
| devananda | JayF: clark says caps matter | 17:28 |
| devananda | (i'm sitting next to him today) | 17:28 |
| JayF | ah, I'll get a fix in | 17:28 |
| devananda | thanks | 17:28 |
| devananda | slight off topic, i realized it's a bit confusing now | 17:28 |
| devananda | as the stats for both projects are comingled and it is not possible to see eg. how active jroll is in reviewing ironic itself | 17:28 |
| jroll | mmm | 17:29 |
| devananda | which is probably fine in the long term, as the projects converge | 17:29 |
| jroll | right, but makes it difficult for now | 17:29 |
| devananda | well, slightly confusing, but not difficult | 17:29 |
| *** coolsvap has quit IRC | 17:29 | |
| jroll | difficult to tell where the activity is, I mean | 17:30 |
| jroll | there's always stackalytics, too | 17:30 |
| JayF | devananda: http://stackalytics.com/report/contribution/ironic/30 could be useful? | 17:30 |
| JayF | stackalytics appears to break it up by repo if you choose | 17:30 |
| devananda | rjhttp://stackalytics.com/?release=icehouse&metric=marks&project_type=openstack&module=ironic&company=&user_id=jim-rollenhagen | 17:30 |
| JayF | vs http://stackalytics.com/report/contribution/ironic-python-agent/30 | 17:30 |
| devananda | jroll: ^ | 17:30 |
| devananda | right | 17:30 |
| jroll | right | 17:31 |
| * jroll needs to put his review hat on today | 17:31 | |
| JayF | I always try to hit reviews every morning. Today I'm upgrading lots of OpenSSL. | 17:31 |
| devananda | jroll: I used you as the example there since you've been most active in ironic reviews of the IPA team. also, thanks for that :) | 17:31 |
| jroll | heh, no problem :) | 17:31 |
| devananda | JayF: yay .... | 17:31 |
| harlowja | seems like everyone upgrading SSL today :-P | 17:34 |
| harlowja | the joy... | 17:34 |
| russell_h | need more opinions on this: https://review.openstack.org/#/c/84814/1/ironic/conductor/manager.py,unified | 17:44 |
| russell_h | the first comment | 17:44 |
| russell_h | if that happens, you're going to get a fair number of scary log messages | 17:44 |
| russell_h | but things are also very broken | 17:44 |
| russell_h | thoughts? | 17:45 |
| matty_dubs | Would that not happen in normal operation? I don't know enough about how this part of the code works yet. | 17:51 |
| notq | my company is setting up several of us for auth to work on openstack. i've been a sysadmin, and have written lots of scripts, but nothing oo. i can read it fine. any thoughts on how i can prepare to get up to speed so i can contribute to the project? my guess is start writing some oo python, watch some getting started with openstack development videos, and try to learn what i can from code reviews here, etc. | 17:56 |
| *** anteaya has quit IRC | 17:56 | |
| russell_h | matty_dubs: that should only happen if you have a node that uses driver "foo", but no conductors which support driver "foo" | 17:57 |
| russell_h | matty_dubs: in that case, you'll basically get one log message per broken node per periodic task per conductor | 17:57 |
| matty_dubs | No conductors in the pool, or just the one running the task? | 17:57 |
| russell_h | none in the pool | 17:57 |
| matty_dubs | Ah, so that is a pretty serious issue then. | 17:58 |
| russell_h | right | 17:58 |
| russell_h | I mean, IMO its definitely worth logging _something_ | 17:58 |
| russell_h | but without getting too fancy, the main options are to log a ton or to log nothing | 17:58 |
| rloo | notq: welcome! | 17:59 |
| notq | rloo: thx! | 17:59 |
| matty_dubs | I guess I'd argue that logging something excessively is better than not logging it. But I don't have a strong opinion / too much expertise to draw on. | 17:59 |
| matty_dubs | notq: Yes, welcome! Were you looking at Ironic in particular? | 17:59 |
| rloo | notq: you are new to openstack? | 17:59 |
| notq | i am relatively new to openstack, i've installed devstack with baremetal and got it to function. i've done some research, and have a basic understanding. | 18:00 |
| notq | my company very specifically wants ironic, that's the core of what we do. so yes, that's the focus for them thus focus for me :) | 18:00 |
| rloo | matty_dubs: (i haven't been following just saw your last comment.) the problem with excessive logging is that it is similar to not logging cuz people will ignore them and worse, they may miss other lines that are important. | 18:01 |
| matty_dubs | notq: Tangential to your main question, I was talking to some other developers (not specifically about Ironic), and we were lamenting that we had tons of developers but not tons of people with real-world admin experience. So you could be bringing some valuable experience/opinions! | 18:01 |
| notq | the other members of the team gaining access have experience with this sort of development, however i'll be trying to get them familiar with working with an open source project, and any preparations for them i can. | 18:01 |
| rloo | notq: you might try installing devstack with ironic then. | 18:02 |
| notq | rloo: i have done this successfully. | 18:02 |
| rloo | notq: great. reviews are a good way to learn the code. | 18:02 |
| rloo | notq: and you can work on a 'small' bug. | 18:03 |
| rloo | notq: low hanging fruit bugs are a good way to start. not sure if there are any. let me see. | 18:03 |
| notq | rloo: i was thinking of a small bug, or documentation as i get up to speed to help. | 18:03 |
| matty_dubs | Playing with it a bit, then poking at open reviews, can be a good way to get up to speed. Or tackling bugs -- see https://bugs.launchpad.net/ironic/ for some ideas | 18:03 |
| notq | rloo: thanks | 18:03 |
| rloo | notq: did you say DOCUMENTATION? :-) | 18:03 |
| *** aedocw_ has quit IRC | 18:04 | |
| matty_dubs | LOL, rloo and I have been looking at documentation recently: https://etherpad.openstack.org/p/IronicDocumentationTasks for some loose coordination | 18:04 |
| notq | thanks guys. | 18:04 |
| * devananda updates session list on our etherpad | 18:05 | |
| *** aedocw_ has joined #openstack-ironic | 18:05 | |
| notq | it feels intimidating, i think the only code i've ever released were nagios plugins. i'll take a look at the links provided. apperciate your thoughts greatly. | 18:06 |
| *** max_lobur has quit IRC | 18:06 | |
| devananda | rloo: re: documentation, have you talked with annegentle at all? | 18:07 |
| *** aedocw_ has quit IRC | 18:07 | |
| rloo | devananda: no, I haven't. should I? I noticed that one of their tools was updated for ironic. | 18:08 |
| devananda | rloo: matty_dubs: and allow me to draw your attention to http://summit.openstack.org/cfp/details/204 and http://summit.openstack.org/cfp/details/104 | 18:08 |
| devananda | rloo: yea. she's the PTL for openstack doc team | 18:08 |
| devananda | those two sessions look quite relevant for us | 18:08 |
| *** christopheraedo has joined #openstack-ironic | 18:08 | |
| matty_dubs | devananda: Ooh, yes they do. | 18:08 |
| matty_dubs | Maybe a n00b question, but do we vote on these? | 18:09 |
| devananda | matty_dubs: nope | 18:09 |
| rloo | devananda: ahh. | 18:09 |
| devananda | matty_dubs: users vote on talks. PTLs simply choose the design sessions | 18:09 |
| devananda | matty_dubs: but you can leave comments :) | 18:09 |
| matty_dubs | Ah, perfect. I will do that! | 18:10 |
| devananda | notq: try not to be intimidated by the release process for opensatck projects. being intimidated by that is my job ;) | 18:10 |
| notq | devananda: understood. :) | 18:11 |
| devananda | notq: another great way to help is, while you're poking at ironic with devstack, file bugs for things that don't work | 18:11 |
| rloo | devananda: I was hoping I could start some docn and then get back to coding, but I'll reach out to annegentle, and make sure we (one of us at least) are in the docn design session(s). | 18:11 |
| devananda | notq: are you testing with real hw, or just VMs, at this point? | 18:11 |
| notq | devananda: vms at the moment, but real hardware and a lot of it will be coming soon. | 18:12 |
| devananda | rloo: i dont want you to get sucked into docn land either! just being a point of contact between the teams is a great help | 18:12 |
| notq | devananda: i think i'll have several baremetal ivy bridge machines with as much ram as possible to deploy to. | 18:13 |
| rloo | devananda: ok! | 18:13 |
| devananda | notq: testing tripleo then? | 18:13 |
| devananda | notq: since ironic doesn't really care how much RAM is in the target node | 18:13 |
| notq | devananda: so far i've gotten as far as the devstack vm. tripleo is on the agenda. the key is, just deploying 4tb+ systems to run our database on. | 18:14 |
| * matty_dubs drools a little bit ;) | 18:15 | |
| notq | devananda: users won't end up ever having os access, so some of the security concerns from there aren't a worry. i haven't deployed tripleo yet. | 18:15 |
| notq | i think at the moment 6 of us are getting approval for openstack dev. how that comes in with their time, i don't know. but baremetal provisioning is the thing we want. that's the driver. | 18:17 |
| *** ironman_ has joined #openstack-ironic | 18:20 | |
| devananda | notq: ahh, gotcha! | 18:21 |
| devananda | notq: then yes, you probably don't want tripleo | 18:21 |
| devananda | notq: just use openstack w/ ironic to deploy your custom OS onto those beefy bare metal machines | 18:22 |
| ironman_ | Hi Deva, I have a question regarding force_node https://blueprints.launchpad.net/nova/+spec/baremetal-force-node. Is this feature ported to Ironic? | 18:22 |
| devananda | ironman_: hi! I dont know if anyone has tested that with the nova.virt.ironic driver, but I think it should still work | 18:23 |
| devananda | i don't think it needs any porting, actually | 18:23 |
| devananda | nova scheduler should be using the same host:node addressing with ironic, so the extension to force host:node should be the same ... or at least very, ,very similar | 18:24 |
| ironman_ | <deva> That's good to know. | 18:24 |
| ironman_ | <deva. Just in case it did not work, a bug report will be sufficient. Right? | 18:25 |
| devananda | ironman_: yep | 18:33 |
| *** overlayer has quit IRC | 18:40 | |
| *** mdickson has quit IRC | 18:40 | |
| matty_dubs | rloo: As I looked more at http://summit.openstack.org/cfp/details/204 I realized that it's all about the software they want to find/write to manage documentation, not about the actual process of writing and coordinating that documentation. | 18:43 |
| matty_dubs | Do you have the same opinion? This feels like the spec for writing a custom wiki app. | 18:43 |
| *** saju_m has quit IRC | 18:44 | |
| rloo | matty_dubs: just looked. yeah, it is about the s/w. there might be some info about the kinds of content they want to make avail. not sure. | 18:45 |
| rloo | matty_dubs: the other session seemed more important. | 18:45 |
| rloo | matty_dubs: http://summit.openstack.org/cfp/details/104 | 18:46 |
| *** mdickson has joined #openstack-ironic | 18:48 | |
| NobodyCam | hello from Roswell .. no UFO's soptted yet... | 18:53 |
| Shrews | NobodyCam: you never see them until it's too late | 18:53 |
| NobodyCam | ahhh :) | 18:53 |
| NobodyCam | heheheh | 18:53 |
| *** jistr has joined #openstack-ironic | 18:54 | |
| *** dkehn_ has joined #openstack-ironic | 18:56 | |
| Shrews | devananda, NobodyCam: fyi, it would be nice to see https://review.openstack.org/83105 merged soon-ish. I see a lot of reviews submitted after it that add to the old problem code, or conflict with my review. | 18:59 |
| * devananda looks | 18:59 | |
| *** derekh has joined #openstack-ironic | 19:00 | |
| russell_h | lucas-dinner: left some feedback on the ManagementInterface thing, but managed to mangle the comment formatting. My bad | 19:07 |
| NobodyCam | Shrews: are the imports ordered right in client_wrapper.py | 19:07 |
| Shrews | NobodyCam: no pep8 error. there should be check specifically for that | 19:08 |
| NobodyCam | ya.. hummmm | 19:10 |
| Shrews | i remember hitting it when i added the _ import in the wrong location | 19:10 |
| devananda | Shrews: do you have another patch refactoring the new class? | 19:11 |
| Shrews | devananda: sorry? | 19:12 |
| devananda | Shrews: instances of IronicClientWrapper aren't holding any state | 19:12 |
| devananda | there are no class- or object-level variables | 19:13 |
| devananda | Shrews: i agree with dtantsur's comment, and your reply -- just asking if there's a WIP for that | 19:13 |
| Shrews | devananda: oh, not yet. i see us adding that when we cache the result of _get_client() | 19:13 |
| devananda | right | 19:13 |
| openstackgerrit | Ruby Loo proposed a change to openstack/python-ironicclient: Add documentation for ironic CLI https://review.openstack.org/86127 | 19:14 |
| devananda | Shrews: let's do that :) | 19:14 |
| Shrews | devananda: obviously. i prefer doing it in a separate patch. are you requesting it in this one? | 19:15 |
| devananda | Shrews: nope. separate patch is good. | 19:15 |
| Shrews | goal was to mimic the current behavior as much as possible, then enhance after | 19:15 |
| devananda | Shrews: some times having a WIP (or a complete patch) is helpful when reviewing partial features like this | 19:16 |
| devananda | to see how change A plays out when change B is layered on top | 19:16 |
| devananda | as it is in my head, this is the right direction :) | 19:16 |
| devananda | Shrews: i assume you've tested doing deploys with this and such... i dont have time to do that today, but the change LGTM otherwise | 19:17 |
| Shrews | devananda: yes, tested a lot under devstack | 19:17 |
| Shrews | devananda:FYI, i see the driver itself eventually having a single IronicClientWrapper object (rather than create a new one each time), and then that object would cache the _get_client() result and reconnect when needed | 19:18 |
| Shrews | as a final goal | 19:18 |
| NobodyCam | Shrews: + + | 19:18 |
| Shrews | but i thought all of those changes at once would make for a more difficult review | 19:19 |
| NobodyCam | Shrews: that end goal would be good to put in the commit message. | 19:19 |
| *** hemna_ has quit IRC | 19:20 | |
| Shrews | NobodyCam: I can add that. Give me 1 sec... | 19:21 |
| devananda | Shrews: maybe... maybe not... | 19:22 |
| devananda | Shrews: right now, ironic requires an admin context, so taht could be cached on the nova side and used for all requests | 19:23 |
| devananda | Shrews: eventually we may need to pass the user's context through, which would mean not caching it -- or mutating the auth token for different requests. but taht's contingent upon other auth changes in ironic. just putting it out there to think about :) | 19:24 |
| JayF | agordeev++ That was a great comment on the LVM partitioner merge req for IPA, thanks. | 19:27 |
| JayF | agordeev: saw that earlier and was trying to think of a better way for parsing out LVM command output | 19:27 |
| Shrews | devananda: i'm not familiar with the potential auth changes you're referring to, so if you see an issue with this model based on those, let me know. | 19:28 |
| openstackgerrit | David Shrewsbury proposed a change to openstack/ironic: Encapsulate Ironic client retry logic https://review.openstack.org/83105 | 19:31 |
| Shrews | NobodyCam: there ya go | 19:32 |
| * NobodyCam looks again | 19:33 | |
| NobodyCam | :) ahh not blank line :-p | 19:34 |
| NobodyCam | s/not/no/ | 19:35 |
| Shrews | NobodyCam: oh, missed your nit. that was just a commit msg change | 19:35 |
| NobodyCam | hehe not with a repost for that | 19:35 |
| Shrews | NobodyCam: more changes are coming. i'll take care of that then | 19:35 |
| devananda | hmm, i'm late. /me runs to get food before TC meeting | 19:44 |
| *** max_lobur has joined #openstack-ironic | 19:46 | |
| *** anteaya has joined #openstack-ironic | 19:48 | |
| *** epim has joined #openstack-ironic | 19:52 | |
| *** dwalleck_ has joined #openstack-ironic | 19:54 | |
| Shrews | hrm, check-tripleo-ironic-seed-precise is supposed to be considered as voting now? failed on 83105, but not finding the actual error | 19:57 |
| *** dwalleck has quit IRC | 19:58 | |
| *** derekh has quit IRC | 20:02 | |
| *** dwalleck has joined #openstack-ironic | 20:03 | |
| *** hemna_ has joined #openstack-ironic | 20:03 | |
| *** jistr has quit IRC | 20:03 | |
| openstackgerrit | Vladimir Kozhukalov proposed a change to openstack/ironic-python-agent: Added lvm partitioner https://review.openstack.org/85672 | 20:04 |
| NobodyCam | Shrews: seed is working it undercloud that is broken | 20:04 |
| NobodyCam | Shrews: that looks like networking error http://logs.openstack.org/05/83105/10/check-tripleo/check-tripleo-ironic-seed-precise/a75cca0/console.html#_2014-04-08_19_47_14_306 | 20:05 |
| jbjohnso_ | devananda, anyway, thoughts on the general capability of that console management thingy I posted? | 20:05 |
| *** dwalleck_ has quit IRC | 20:06 | |
| Shrews | NobodyCam: yeah | 20:07 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Adding a reference driver for the agent https://review.openstack.org/84795 | 20:07 |
| linggao | matty_dubs ping | 20:18 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic-python-agent: Version node lookup payload https://review.openstack.org/86141 | 20:18 |
| rloo | NobodyCam: https://review.openstack.org/#/c/84742/1. If I +2, that gives a check mark, right? but it would still need to be approved? | 20:19 |
| NobodyCam | rloo: yes.. +2 is check then you have approved 0/1 | 20:20 |
| rloo | so we need 2 +2, then an approve? or one +2 is enough, then an approve? | 20:20 |
| rloo | or is the system smart enough so if I don't know what I'm doing, it'll tell me? | 20:20 |
| devananda | rloo: 2 +2 | 20:20 |
| NobodyCam | two +2 | 20:20 |
| JayF | the way I've been doing it in IPA is if it has a +2 already, and I'm the 2nd +2, I +2 and +A | 20:20 |
| devananda | JayF: ++ | 20:20 |
| NobodyCam | second +2'er can also +A | 20:21 |
| NobodyCam | \ | 20:21 |
| devananda | also as a rule, don't +2 or +A your own patches | 20:21 |
| JayF | Is there a reason why the second person to +2 wouldn't +A? To give someone else a chance to nack it? | 20:21 |
| rloo | JayF beat me to it. | 20:21 |
| devananda | yep | 20:21 |
| NobodyCam | I only ever vote on my own patches if someone else pushed a patch up for me then I only +1 it | 20:21 |
| rloo | no nacking allowed. | 20:21 |
| devananda | JayF: or for example, if another core had -1'd a previous pev | 20:21 |
| devananda | JayF: to give taht core time to come back and confirm their concern was addressed | 20:22 |
| JayF | Do core -1 stick around with new patchsets? Only -2s do, right? | 20:22 |
| devananda | JayF: correct. only -2's stick around | 20:22 |
| JayF | cool, that's what I thought | 20:22 |
| russell_h | is there a document on all this somewhere? | 20:22 |
| devananda | a -1 is an objection, ideally with feedback suggesting an improvement | 20:22 |
| russell_h | I feel like I've got it figured out now, but it'd be nice to have something to link people to | 20:22 |
| devananda | a -2 is an immovable block | 20:22 |
| devananda | other cores can not override your -2 | 20:23 |
| NobodyCam | I have overwritten -1 nits if they are called out as such | 20:23 |
| devananda | also a "0" doesn't count as a vote in the vote-counting systems | 20:23 |
| rloo | devananda: i think that's a bug in the vote-counting systems. | 20:23 |
| JayF | I wish there was a +0 or something independent of a 0, where you can indicate you read it and have no objections, but maybe don't have the confidence in that part of the code to give a +1 | 20:24 |
| devananda | russell_h: there's some stuff here that Nova wrote up: https://wiki.openstack.org/wiki/Nova/CoreTeam | 20:24 |
| NobodyCam | also https://wiki.openstack.org/wiki/ReviewChecklist | 20:24 |
| matty_dubs | linggao: pong | 20:24 |
| matty_dubs | (sorry, was afk) | 20:24 |
| devananda | JayF: then +1 with comment | 20:25 |
| JayF | OK, I'll start doing that then. | 20:25 |
| devananda | JayF: i often post eg. "code LGTM< but i haven't tested it" | 20:25 |
| devananda | it's not perfect | 20:25 |
| linggao | matty_dubs, np. I have seen a strange problem when testing console code, it drives me crazy today, just wonder if you have seen the same problem or not. | 20:25 |
| rloo | heh, harder to put "code LGTM but I don't really know how that code works" :-) | 20:26 |
| linggao | matty_dubs, when place the url on the broswer, it shows "File not found" | 20:26 |
| matty_dubs | linggao: No, that sounds odd. The shellinabox URL? | 20:26 |
| openstackgerrit | Russell Haering proposed a change to openstack/ironic: Factor hash ring management out of the conductor https://review.openstack.org/84814 | 20:26 |
| linggao | matty_dubs, yes | 20:27 |
| JayF | rloo: Yes, but loads better than a +1 without the indication you're at the edge of your knowledge | 20:27 |
| linggao | It only happens when I ironic created processes. | 20:27 |
| devananda | rloo: for "i dont realy now how it works" -- jsut don't vote | 20:27 |
| matty_dubs | linggao: I didn't hit that; weird. Is it actually loading shellinabox, and showing that inside? Or is it not getting to shellinaboxd? | 20:27 |
| devananda | rloo: or take teh time to investigate prior to voting ;) | 20:28 |
| linggao | matty_dubs, if I run the same exact process manually, then it works. | 20:28 |
| devananda | as the project grows, we all will have areas of the code that we know better than others | 20:28 |
| linggao | matty_dubs, I cannot tell. | 20:28 |
| rloo | JayF: I know what you mean. For code that I don't understand, I look sometimes to see how something might be done, but I don't vote or comment on it. Doesn't mtter to me if my looking is "counted" or not ;) | 20:28 |
| linggao | matty_dubs, I tried to use different ports. | 20:28 |
| linggao | matty_dubs, the stange thing is that it worked yesterday. | 20:29 |
| matty_dubs | Huh, that's really odd. | 20:29 |
| rloo | devananda: yeah, time... :-) | 20:29 |
| linggao | matty_dubs, I'll keep googling it. | 20:30 |
| matty_dubs | linggao: So is the process actually starting? Or does it not even get that far? | 20:30 |
| JayF | devananda: re: the reviewstats stuff, I put up a merge req here : https://review.openstack.org/#/c/86142/1 | 20:30 |
| NobodyCam | linggao: what dose the use look like | 20:30 |
| linggao | matty_dubs, the process got stared. I can see them using ps command. | 20:30 |
| NobodyCam | linggao: you can check port with nc | 20:31 |
| matty_dubs | linggao: You didn't end up with multiple sessions running, did you? | 20:31 |
| matty_dubs | That's the only thing I can think of. | 20:31 |
| matty_dubs | If the old one didn't get cleaned up properly or something. | 20:31 |
| linggao | matty_dubs, every time the shellinaboxd commnd is issued, it brings up 3 processes. | 20:32 |
| matty_dubs | Including when you start it yourself? | 20:33 |
| linggao | yes | 20:33 |
| NobodyCam | linggao: have you checked netstat to see what port it is on? | 20:33 |
| linggao | NobodyCam, I will. Do you know what options to use from top of your head? | 20:34 |
| NobodyCam | netstat -lnp is what I would use | 20:34 |
| openstackgerrit | A change was merged to openstack/python-ironicclient: node-list to show the maintenance field https://review.openstack.org/84742 | 20:35 |
| matty_dubs | sudo lsof -i :4000 works too, to show what's on port 4000 | 20:35 |
| matty_dubs | I think netstat is the more 'normal' way though | 20:35 |
| NobodyCam | i use lsof to see who (what process) has a file open | 20:38 |
| NobodyCam | haven't used it for port... | 20:39 |
| linggao | matty_dubs, NobodyCam http://paste.openstack.org/show/75361/ | 20:39 |
| linggao | matty_dubs, NobodyCam, that's what ironic produced. | 20:40 |
| matty_dubs | linggao: On a whim, do the two files (background= and the .pw file in /tmp) exist on disk? | 20:41 |
| NobodyCam | linggao: nc -z 127.0.0.1 4202 | 20:41 |
| linggao | matty_dubs, NobodyCam, I placed http://<hostname>:4202 there, it shows "File not found". | 20:41 |
| linggao | NobodyCam, nc -z 127.0.0.1 4202 returns nothing. | 20:42 |
| matty_dubs | linggao: Yeah, I wonder if it's complaining because it can't find /tmp/f62a481d-7f9c-4693-9914-c6596190a991.pw or /usr/ling/statck/ironic/ironic/console/f62a481d-7f9c-4693-9914-c6596190a991.pid | 20:42 |
| NobodyCam | linggao: http://paste.openstack.org/show/BHIAVu4WCuGTWwu1kmsV/ | 20:43 |
| linggao | matty_dubs, those files are there. | 20:43 |
| NobodyCam | linggao: firewall | 20:43 |
| matty_dubs | Hrm, there goes my theory! | 20:44 |
| linggao | NobodyCam nc -z 127.0.0.1 22 returns nothing for me either. | 20:45 |
| NobodyCam | do you have ssh running? | 20:45 |
| NobodyCam | sshd :-p | 20:45 |
| linggao | If I kill those 2 processes and run the same exact command manually, the console works. | 20:45 |
| NobodyCam | oh' | 20:46 |
| NobodyCam | could neutron be zapping the ports somehow? | 20:46 |
| *** max_lobur has quit IRC | 20:46 | |
| NobodyCam | if it works manually thats somehting else | 20:47 |
| linggao | NobodyCam, there are no other OpenStack processes, only the ironic venv. | 20:47 |
| linggao | there is a /usr/sbin/sshd -D running | 20:47 |
| NobodyCam | odd about nc not returning something... can you try nc -z 127.0.0.1 22;echo $? | 20:48 |
| matty_dubs | Are you running it as root normally? | 20:52 |
| matty_dubs | Err, rephrase: Are you running shellinaboxd as root? | 20:52 |
| matty_dubs | Or some other suer? | 20:52 |
| matty_dubs | *user | 20:52 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Adding a reference driver for the agent https://review.openstack.org/84795 | 20:52 |
| NobodyCam | matty_dubs: When started as root, the server drops most privileges at start up. Unless overridden by the --group option, | 20:55 |
| NobodyCam | from https://code.google.com/p/shellinabox/wiki/shellinaboxd_man | 20:55 |
| matty_dubs | Yeah, I don't think you actually need root. It's probably dangerous, in fact | 20:55 |
| NobodyCam | ++ | 20:56 |
| *** mrda_away is now known as mrda | 20:57 | |
| matty_dubs | I was just trying to think of _something_ that would explain the behavior | 20:57 |
| NobodyCam | good morning mrda | 20:57 |
| NobodyCam | :-p | 20:57 |
| *** mrda is now known as Guest52851 | 20:58 | |
| linggao | NobodyCam, nc -z 127.0.0.1 22;echo $? | 20:59 |
| linggao | returns 0 | 20:59 |
| NobodyCam | ya.. the working fact that it works when manually run is whats getting me... Like maybe a timing issue? or something else it interrecating | 20:59 |
| Guest52851 | Morning NobodyCam | 20:59 |
| linggao | matty_dubs, yes, I ran it as root. | 20:59 |
| NobodyCam | linggao:ahh so 22 is open | 20:59 |
| NobodyCam | linggao: try 4202 like that | 21:00 |
| matty_dubs | What if you run as a non-root user by hand? Does that work? | 21:00 |
| linggao | NobodyCam: same. 0 | 21:01 |
| NobodyCam | zero is good... means its open | 21:01 |
| *** Guest52851 is now known as mrda | 21:02 | |
| NobodyCam | linggao: your using hostname... can you try the url with the ip | 21:02 |
| linggao | matty_dubs, I am trying... | 21:03 |
| linggao | NobodyCam, I am using the ip. | 21:03 |
| NobodyCam | ok :) | 21:03 |
| mrda | hey NobodyCam, I was wondering if you (or any other core) would like to express an opinion on https://review.openstack.org/#/c/84374/ There's 3 +1's (from cores) but a reluctance to +2 it :-) | 21:04 |
| mrda | ...if you don't mind | 21:04 |
| NobodyCam | lol several of them where not core when they +1 | 21:05 |
| * NobodyCam looking | 21:05 | |
| *** stevehuang has quit IRC | 21:05 | |
| linggao | matty_dubs, I ran as a non-root user manually, the browser got the following: Access denied! Session closed. | 21:06 |
| linggao | 21:06 | |
| matty_dubs | A... different error. Weird. | 21:06 |
| mrda | NobodyCam: this is true, but its good leverege for asking you :) | 21:06 |
| linggao | I think that's from the bmc. | 21:06 |
| matty_dubs | Oh, access denied to the tmp file or something, maybe? | 21:06 |
| *** jdob has quit IRC | 21:07 | |
| matty_dubs | Err, huh. That's weird. | 21:07 |
| linggao | matty_dubs, maybe the .pw file | 21:07 |
| linggao | I am just guessing. | 21:07 |
| *** datajerk has quit IRC | 21:07 | |
| matty_dubs | Me too :) | 21:08 |
| NobodyCam | good catch matty_dubs :) | 21:08 |
| matty_dubs | Oh! Who owns the pid and pw files? | 21:09 |
| matty_dubs | Are they root-owned, and nobody can't read them? | 21:09 |
| * matty_dubs concocts random theories until one work | 21:09 | |
| matty_dubs | *works | 21:09 |
| linggao | matty_dubs, the root owns them | 21:10 |
| NobodyCam | ahh | 21:10 |
| NobodyCam | chmod 777 | 21:10 |
| matty_dubs | What if you delete them, and let the Ironic process recreate theM? | 21:10 |
| NobodyCam | fix all | 21:10 |
| matty_dubs | Which is how it should work. | 21:10 |
| NobodyCam | mrda: I can see lucas-dinner point. but I also feel that validate needs to be sync and there for should not issue ipmi commands... but that is another can-o-worms | 21:12 |
| mrda | NobodyCam: so how do you think I should proceed? | 21:14 |
| linggao | matty_dubs, I chmod 777 to the files, and restarted shellinaboxd as a non-root user manually. and still got Acess denied. | 21:14 |
| linggao | matty_dubs NobodyCam, I have to go. thanks a lot for your time and patience. I'll sleep on it tonight. Hope the problem will go away magically tomorrow. :-) | 21:16 |
| NobodyCam | mrda: reviewing now | 21:16 |
| mrda | NobodyCam: thanks, much appreciated | 21:16 |
| matty_dubs | No problem! | 21:16 |
| matty_dubs | linggao: I think you just want to delete the two files. They should only last as long as the session anyway. | 21:17 |
| NobodyCam | :) linggao have a good night :) let us know tomorrow :) | 21:17 |
| matty_dubs | And then Ironic will create them as something not root-owned | 21:17 |
| matty_dubs | In any case, good night! Catch you later. | 21:17 |
| linggao | matty_dubs, I'll try it tomorrow. | 21:17 |
| linggao | thanks a lot! | 21:17 |
| *** linggao has quit IRC | 21:18 | |
| openstackgerrit | Vladimir Kozhukalov proposed a change to openstack/ironic-python-agent: Added disk partitioner https://review.openstack.org/86163 | 21:20 |
| NobodyCam | mrda: reviewed | 21:21 |
| NobodyCam | lucas-dinner: see my comments on https://review.openstack.org/#/c/84374 | 21:21 |
| NobodyCam | need to do quick walkies... brb | 21:22 |
| mrda | Thanks NobodyCam! | 21:23 |
| *** romcheg has quit IRC | 21:28 | |
| NobodyCam | mrda: :) | 21:35 |
| NobodyCam | mrda: there still need to be a talk about what validate() should do. | 21:36 |
| NobodyCam | *needs | 21:37 |
| mrda | NobodyCam: and I'm happy to help out there | 21:37 |
| NobodyCam | :) | 21:37 |
| devananda | hmm | 21:37 |
| devananda | interesting | 21:38 |
| mrda | now that patch is going to land (thanks devananda!) off to do some reviews and look at ther bugs :) | 21:38 |
| devananda | just learned how openstack tracks back ports | 21:38 |
| devananda | to milestone proposed | 21:38 |
| devananda | it's not wha ti expected | 21:38 |
| devananda | and | 21:38 |
| devananda | it complciates the parted backport thing | 21:38 |
| devananda | so i'm glad we haven't landed that yet | 21:38 |
| NobodyCam | lol :/ | 21:38 |
| adam_g | anyone have thoughts on https://bugs.launchpad.net/ironic/+bug/1304673 ? might it be a good candidate for a juno blueprint? | 21:44 |
| *** russellb has quit IRC | 21:44 | |
| *** jbjohnso_ has quit IRC | 21:45 | |
| *** russellb has joined #openstack-ironic | 21:45 | |
| *** stevehuang has joined #openstack-ironic | 21:47 | |
| NobodyCam | adam_g: ya.. but I need to see how it wouldbe done. | 21:49 |
| NobodyCam | I wouldn't want a neutron.vender_passthru api :-p | 21:49 |
| adam_g | NobodyCam, right. totally up for discussion. the approach nova took seems like reasonable base to work from | 21:50 |
| adam_g | oh, cool. tempest ironic exercise finally +A'd https://review.openstack.org/#/c/81958/ | 21:50 |
| NobodyCam | w00t \o/ | 21:51 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Add version 1 of agent payload https://review.openstack.org/86173 | 21:53 |
| NobodyCam | gah I alway forget to put a space between no and bug for rechecks :-p | 21:53 |
| devananda | adam_g: grats! | 21:54 |
| devananda | hmm | 21:56 |
| devananda | so, folks, for the sfdisk->parted change | 21:57 |
| devananda | which I think we should backport to icehouse | 21:57 |
| devananda | at least a small fix of it | 21:57 |
| devananda | in looking at https://review.openstack.org/#/c/84396/2/ironic/drivers/modules/deploy_utils.py again i have concerns | 21:57 |
| devananda | that this isn't going to erase an existing partition table and will break in very unexpected ways | 21:57 |
| devananda | if one exists | 21:57 |
| NobodyCam | devananda: because offset always starts at one | 21:59 |
| NobodyCam | ? | 21:59 |
| devananda | NobodyCam: no. because it *only* calls mkpart | 21:59 |
| devananda | it never checks if there are existing partitions | 21:59 |
| NobodyCam | ya if there is already a ephemeral partision set then I can see it getting wacked | 22:00 |
| devananda | i'm not a parted (or sfdisk) expert | 22:00 |
| devananda | but what if there's already a primary part? | 22:00 |
| NobodyCam | *partition | 22:00 |
| NobodyCam | should overwrite i believe | 22:01 |
| devananda | will parted barf creating a second primary partition? (i think not, but this is stil lthe wrong behavior) | 22:01 |
| lucas-dinner | NobodyCam, will do | 22:02 |
| lucas-dinner | russell_h, thanks will take a look | 22:02 |
| devananda | lucas-dinner: hi! around for a minute? | 22:02 |
| lucas-dinner | devananda, yes | 22:02 |
| lucas-dinner | devananda, so the mklabel msdos command | 22:02 |
| lucas-dinner | will erease the previous partition table | 22:02 |
| devananda | lucas-dinner: i'm revisiting your parted fix for icehouse and just learned the process for actually doing a backport and tagging an RC2 | 22:03 |
| *** russellb has quit IRC | 22:03 | |
| lucas-dinner | devananda, ah nice | 22:03 |
| *** russellb has joined #openstack-ironic | 22:03 | |
| devananda | lucas-dinner: ahhh, i see. that is run before mkpart, but it appears in the code after it | 22:03 |
| NobodyCam | lucas-dinner: what would happen if there is already a ephemeral partition and this is a redepoly? | 22:03 |
| devananda | NobodyCam: same as before -- part table is recreated, but the data *should* line up, and if preserve-ephemeral, then that part is not recreated | 22:04 |
| lucas-dinner | devananda, http://paste.openstack.org/show/75370/ | 22:05 |
| lucas-dinner | yes | 22:05 |
| lucas-dinner | NobodyCam, yeah, we delete it and then create it on the same offset | 22:05 |
| lucas-dinner | we don't touch the filesystem, only the partition table | 22:05 |
| NobodyCam | gulp | 22:05 |
| devananda | lucas-dinner: ok, so, process | 22:05 |
| devananda | AIUI ... it is ... | 22:06 |
| devananda | we need two bugs, one targeted to icehosue-rc2, the other to master | 22:06 |
| devananda | we fix the bug in master | 22:06 |
| devananda | fix has to land | 22:06 |
| devananda | that bug is marked FixCommitted | 22:06 |
| devananda | we backport the fix (or some part thereof) to the milestone-proposed branch | 22:07 |
| devananda | when that lands, we mark that bug as FixReleased | 22:07 |
| devananda | and ttx tags it at that point | 22:07 |
| lucas-dinner | hmm I see, more complicated than I thought | 22:07 |
| devananda | i dont think the patch you have up follows that process... want me to do it? | 22:07 |
| devananda | yea, much mroe complex than i thought too | 22:07 |
| devananda | which is why it's good for us to learn | 22:08 |
| devananda | now, while we're still in incubation :) | 22:08 |
| lucas-dinner | heh +1 | 22:08 |
| NobodyCam | yes | 22:08 |
| devananda | lucas-dinner: so we can land your quick fix in master, then refactor it AND backport it | 22:08 |
| lucas-dinner | devananda, ack sounds like a plan | 22:09 |
| lucas-dinner | want me to remove the WIP? | 22:09 |
| devananda | yep, thanks | 22:09 |
| lucas-dinner | done | 22:09 |
| *** lucas-dinner is now known as lucas-semi-afk | 22:09 | |
| lucas-semi-afk | heh | 22:09 |
| devananda | lucas-semi-afk: want to rebase it on mp branch? | 22:09 |
| lucas-semi-afk | devananda, ack | 22:10 |
| NobodyCam | lucas-dinner: just asking if you have tested data in a EP? | 22:10 |
| lucas-semi-afk | NobodyCam, I think I did tested this case yes | 22:10 |
| NobodyCam | :) | 22:10 |
| NobodyCam | TY | 22:10 |
| lucas-semi-afk | I can give it a go quickly here to see | 22:10 |
| lucas-semi-afk | I tried to cross test it | 22:10 |
| NobodyCam | you awesome :) | 22:10 |
| lucas-semi-afk | with diff combinations | 22:10 |
| NobodyCam | s/you/your/ | 22:10 |
| devananda | NobodyCam: EP ? | 22:11 |
| NobodyCam | just stick a file in the EP and make sure it is still valid | 22:11 |
| lucas-semi-afk | ephemeral partition | 22:11 |
| devananda | ah | 22:11 |
| lucas-semi-afk | NobodyCam, ack | 22:11 |
| NobodyCam | :) | 22:11 |
| devananda | lucas-semi-afk: i think what you do is: git checkout gerrit/milestone-proposed ;; make canges ;; git review | 22:12 |
| devananda | i think ... | 22:12 |
| NobodyCam | I have concerns about if the EP is lvm but that is something else | 22:12 |
| lucas-semi-afk | devananda, lemme try | 22:13 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Replace sfdisk with parted https://review.openstack.org/84396 | 22:14 |
| devananda | lucas-semi-afk: ah, so you apparently also need to explicitly push to that branch in gerrit?? maybe? | 22:14 |
| lucas-semi-afk | err master?! | 22:14 |
| lucas-semi-afk | I guess yea | 22:14 |
| devananda | yea | 22:14 |
| devananda | lucas-semi-afk: git review milestone-proposed | 22:15 |
| lucas-semi-afk | oh wait maybe it was the Change-id since I did a "git am" | 22:15 |
| devananda | no -- jsut tell git the branch name | 22:15 |
| lucas-semi-afk | devananda, https://review.openstack.org/#/c/86179/ | 22:19 |
| lucas-semi-afk | seems now it worked | 22:19 |
| lucas-semi-afk | the problem was my Change-id | 22:19 |
| devananda | ahh | 22:19 |
| devananda | great, thanks! | 22:20 |
| lucas-semi-afk | I removed the old one with the ref to the master branch | 22:20 |
| devananda | lucas-semi-afk: so followign the process, we need to land the fix in master first | 22:21 |
| lucas-semi-afk | devananda, ack gimme 10 min to test the ep thing | 22:21 |
| lucas-semi-afk | I think I tested that case | 22:21 |
| lucas-semi-afk | but want to confirm | 22:21 |
| devananda | lucas-semi-afk: any objection to landing the same quick fix in both places, then basing all your DiskPartitioner work on that? | 22:21 |
| lucas-semi-afk | devananda, no problem for me | 22:22 |
| NobodyCam | humm anyone happen to know if check-tripleo-ironic-seed-precise is broken? | 22:24 |
| *** dwalleck_ has joined #openstack-ironic | 22:24 | |
| devananda | NobodyCam: ask in tripleo? I'm not sure | 22:25 |
| NobodyCam | ya | 22:25 |
| NobodyCam | just checking | 22:25 |
| NobodyCam | :-p | 22:25 |
| *** dwalleck has quit IRC | 22:27 | |
| adam_g | russell_h, https://etherpad.openstack.org/p/IronicCI updated with notes on everything that is currently in flight | 22:33 |
| russell_h | adam_g: awesome, thanks! | 22:34 |
| russell_h | dwalleck_: ^ | 22:34 |
| dwalleck_ | sweet! I'll have a look | 22:35 |
| *** vkozhukalov has quit IRC | 22:37 | |
| *** eguz has joined #openstack-ironic | 22:39 | |
| lucas-semi-afk | devananda, oh wait with the parted patch, dmitry submitted a new review setting the offset to 1 | 22:39 |
| lucas-semi-afk | and I didn't have that change in my local branch | 22:39 |
| *** dwalleck has joined #openstack-ironic | 22:41 | |
| devananda | lucas-semi-afk: ah. yea. please propose the same code (or as much as same as possible) to both master and milestone | 22:41 |
| dwalleck | adam_g: You need any help with fixing these testing without isolation issues? That's pretty much the only way I test in some environments, so I need it anyway | 22:41 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Replace sfdisk with parted https://review.openstack.org/84396 | 22:41 |
| lucas-semi-afk | devananda, will do | 22:41 |
| devananda | process wise, we need to land it on master first, even if you're proposing both at the same time | 22:41 |
| adam_g | dwalleck, so the patches i have up should address everything that is failing, at least in the ironic job (with that regex applied to omit irrelevant tests) | 22:42 |
| devananda | i'll need to step away soon for a meeting and dinner, but will try to look again later tonight | 22:42 |
| lucas-semi-afk | done | 22:43 |
| adam_g | dwalleck, there is a periodic jenkins job that runs tempest against neutron isolated creds disabled that is meant to catch these failures. there are still a few others outside of the ironic case that fail | 22:43 |
| *** eghobo has quit IRC | 22:43 | |
| adam_g | dwalleck, http://logs.openstack.org/periodic-qa/periodic-tempest-dsvm-neutron-full-non-isolated-master/dbcb96b/logs/testr_results.html.gz | 22:43 |
| *** dwalleck_ has quit IRC | 22:43 | |
| NobodyCam | lol devananda percona 404'ing | 22:44 |
| devananda | note to self and others - we need to create https://wiki.openstack.org/wiki/Ironic/ReleaseNotes/Icehouse | 22:45 |
| adam_g | dwalleck, im hoping that once we have the ironic job passing for us, we can enable it in the tempest pipeline to get some pre-commit visibility there, since the periodic jobs dont seem to get much attention | 22:45 |
| JoshNang | when someone gets a chance, can I get some reviews on https://review.openstack.org/#/c/81391/? I'll need it for the agent driver. | 22:45 |
| devananda | adam_g: yea. once it passes, we can move it from experimental to -nv on every commit | 22:45 |
| devananda | adam_g: we jsut cant make our tempest tests voting on tempest until we graduate ... | 22:45 |
| devananda | but that'll at least give us the ability to search in logstash for when something breaks :) | 22:46 |
| devananda | *to find the cause when ... | 22:46 |
| adam_g | it might not cover non-isolated creds 100% for tempest, but would catch a lot of stuff that shouldn't have gotten thru the gate | 22:46 |
| devananda | ok, really need to go now ... bbl | 22:46 |
| adam_g | cheers | 22:46 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Drivers may expose a top-level passthru API https://review.openstack.org/81919 | 22:46 |
| jroll | devananda and other cores: can we please either get this merged or discuss a better way to accomplish it? https://review.openstack.org/#/c/81919/ | 22:47 |
| jroll | mailing list thread went nowhere | 22:47 |
| dwalleck | adam_g: Well, I've got time. I'm afraid of a lot of the flakiness is just really test bugs | 22:47 |
| jroll | and we have a bunch of stuff hanging on that | 22:47 |
| *** matty_dubs is now known as matty_dubs|gone | 22:47 | |
| adam_g | dwalleck, yeah | 22:48 |
| *** epim has quit IRC | 22:53 | |
| openstackgerrit | Russell Haering proposed a change to openstack/ironic: Version agent lookup payloads https://review.openstack.org/85228 | 22:58 |
| lucas-semi-afk | NobodyCam, ep part preserves the partition [confirmed] | 23:01 |
| lucas-semi-afk | NobodyCam, http://paste.openstack.org/show/75374/ | 23:03 |
| NobodyCam | :) ty lucas-semi-afk your awesome | 23:03 |
| lucas-semi-afk | NobodyCam, np! | 23:04 |
| NobodyCam | !! | 23:04 |
| openstack | NobodyCam: Error: "!" is not a valid command. | 23:04 |
| lucas-semi-afk | hah | 23:04 |
| NobodyCam | doh | 23:04 |
| lucas-semi-afk | aye! sleeping time for me | 23:04 |
| NobodyCam | night lucas-semi-afk :) | 23:04 |
| lucas-semi-afk | have a good night everyone! night NobodyCam devananda | 23:04 |
| lucas-semi-afk | see you tomorrow | 23:05 |
| *** lucas-semi-afk is now known as lucas-afk | 23:05 | |
| * NobodyCam is waiting for gate jobs to come back.. fix just merged | 23:05 | |
| *** dwalleck has quit IRC | 23:35 | |
| *** lucas-afk has quit IRC | 23:36 | |
| openstackgerrit | David Shrewsbury proposed a change to openstack/ironic: Encapsulate Ironic client retry logic https://review.openstack.org/83105 | 23:44 |
| openstackgerrit | A change was merged to openstack/ironic-python-agent: Version node lookup payload https://review.openstack.org/86141 | 23:45 |
| Shrews | grrr.... the addition of the new driver tests to our tox runs revealed my subsequent breakage of the tests | 23:45 |
| NobodyCam | thats not a bad thing | 23:46 |
| NobodyCam | lol | 23:46 |
| * NobodyCam ducks | 23:46 | |
| Shrews | NobodyCam: so, the only difference in 83105 is the blank line you wanted, and fixing the tests | 23:46 |
| NobodyCam | :) will take a look see in a bitdinner just about here :) | 23:47 |
| Shrews | no rush. eat | 23:47 |
| *** eguz has quit IRC | 23:51 | |
| *** eghobo has joined #openstack-ironic | 23:51 | |
| *** derekh has joined #openstack-ironic | 23:56 | |
| *** mrda is now known as mdavies | 23:58 | |
| *** mdavies is now known as mrda | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!