Wednesday, 2017-10-18

*** namnh has joined #openstack-ironic00:00
*** namnh has quit IRC00:00
*** charcol has quit IRC00:02
*** ijw has quit IRC00:03
*** charcol has joined #openstack-ironic00:07
*** karimb has quit IRC00:18
*** jlabarre has quit IRC00:19
*** rbudden has quit IRC00:21
-openstackstatus- NOTICE: due to unscheduled restart of zuulv3.o.o you will need to 'recheck' your jobs that were last running. Sorry for the inconvenience.00:32
*** zshi__ has joined #openstack-ironic00:32
*** ijw has joined #openstack-ironic00:45
*** yangyapeng has quit IRC00:51
*** yangyapeng has joined #openstack-ironic00:51
*** TuanLA has joined #openstack-ironic00:54
*** rama_y has quit IRC00:56
*** yangyapeng has quit IRC00:56
*** phuongnh has joined #openstack-ironic01:11
*** crushil has quit IRC01:19
*** AlexeyAbashkin has joined #openstack-ironic01:23
*** AlexeyAbashkin has quit IRC01:27
*** ijw has quit IRC01:30
openstackgerritzenghui.shi proposed openstack/ironic master: Add db api layer for node bios operations  https://review.openstack.org/51140201:31
openstackgerritzenghui.shi proposed openstack/ironic master: Add Node BIOS support - REST API  https://review.openstack.org/51257901:31
*** yangyapeng has joined #openstack-ironic01:48
*** yangyape_ has joined #openstack-ironic01:49
*** yangyapeng has quit IRC01:52
*** dpawar has joined #openstack-ironic02:01
*** rama_y has joined #openstack-ironic02:07
*** AlexeyAbashkin has joined #openstack-ironic02:23
*** AlexeyAbashkin has quit IRC02:27
*** dpawar has quit IRC02:30
*** rama_y has quit IRC02:33
*** namnh has joined #openstack-ironic02:46
*** chlong has quit IRC02:47
*** TuanLA has quit IRC02:47
*** jcoufal has quit IRC02:47
*** TuanLA has joined #openstack-ironic02:48
*** zhangfei has joined #openstack-ironic02:49
*** namnh has quit IRC02:56
*** TuanLA has quit IRC02:56
*** namnh has joined #openstack-ironic02:57
*** TuanLA has joined #openstack-ironic02:57
*** phuongnh has quit IRC03:01
*** phuongnh has joined #openstack-ironic03:03
*** rbudden has joined #openstack-ironic03:09
*** rbudden has quit IRC03:16
*** openstackgerrit has quit IRC03:22
*** AlexeyAbashkin has joined #openstack-ironic03:23
*** hwoarang has quit IRC03:26
*** AlexeyAbashkin has quit IRC03:27
*** rbudden has joined #openstack-ironic03:30
*** ijw has joined #openstack-ironic03:31
*** hwoarang has joined #openstack-ironic03:31
*** rama_y has joined #openstack-ironic03:33
*** sasha2 has quit IRC03:37
*** links has joined #openstack-ironic03:46
*** udesale has joined #openstack-ironic04:09
*** udesale has quit IRC04:14
*** udesale has joined #openstack-ironic04:15
*** hshiina has joined #openstack-ironic04:23
*** zhangfei has quit IRC04:42
*** ijw has quit IRC04:48
*** zhangfei has joined #openstack-ironic04:58
*** rbudden has quit IRC05:01
*** snaz has quit IRC05:05
*** rama_y has quit IRC05:12
*** pmannidi has quit IRC05:15
*** e0ne has joined #openstack-ironic05:15
*** yolanda has quit IRC05:15
*** e0ne has quit IRC05:19
*** ChubYann has quit IRC05:28
*** snaz has joined #openstack-ironic05:28
*** vsaienk0 has joined #openstack-ironic05:46
*** mjura has joined #openstack-ironic05:47
TenguMorning Ironic :)05:53
Tengusmall question: I'd like to build some kind of "lab" infra for our tripleO setup. In order to do so, I'm thinking about putting that lab infra in our production openstack. Would Ironic be able to speak "openstack" in order to manage the VMs (reboot, deploy, and so on) ?05:54
*** marios has joined #openstack-ironic05:55
Tenguhmm. I think I have my answer reading the blueprint for quintupleO (tripleo on openstack): no pxe capability for nova, no power management capability for now as well, and some stuff in the network. So nope, not possible yet.05:59
Tengubut that reminds me of the redhat formation I followed, they actually had a virtual "power manager" that could emulate IPMI. hmm.06:01
*** TuanLA has quit IRC06:11
*** parlos has joined #openstack-ironic06:19
*** zshi__ has quit IRC06:29
*** TuanLA has joined #openstack-ironic06:33
*** trungnv has joined #openstack-ironic06:34
*** hjensas has joined #openstack-ironic06:36
*** hjensas has quit IRC06:36
*** hjensas has joined #openstack-ironic06:36
*** jtomasek has joined #openstack-ironic06:41
*** zhangfei has quit IRC06:43
*** jtomasek has quit IRC06:44
*** jtomasek has joined #openstack-ironic06:45
*** pcaruana has joined #openstack-ironic06:45
*** zhangfei has joined #openstack-ironic06:56
*** hshiina has quit IRC06:58
*** charcol has quit IRC07:12
*** tesseract has joined #openstack-ironic07:17
*** rbartal has joined #openstack-ironic07:31
*** hoonetorg has joined #openstack-ironic07:32
*** hw_wutianwei has joined #openstack-ironic07:33
*** gcb has quit IRC07:34
*** ccamacho has joined #openstack-ironic07:34
*** mikal has quit IRC07:40
*** karimb has joined #openstack-ironic07:41
*** mikal has joined #openstack-ironic07:41
*** udesale has quit IRC07:55
*** AlexeyAbashkin has joined #openstack-ironic07:57
*** amarao has joined #openstack-ironic07:59
*** ppiela_ has joined #openstack-ironic08:04
*** ppiela has quit IRC08:04
*** ricardoas has quit IRC08:04
*** ricardoas has joined #openstack-ironic08:06
*** lucas-afk has quit IRC08:07
*** rook has quit IRC08:07
*** weshay|ruck has quit IRC08:07
*** rook has joined #openstack-ironic08:08
*** rook is now known as Guest8382808:09
*** weshay has joined #openstack-ironic08:09
*** lucasagomes has joined #openstack-ironic08:09
*** mgoddard has joined #openstack-ironic08:14
*** openstackgerrit has joined #openstack-ironic08:17
openstackgerritKaifeng Wang proposed openstack/ironic master: Detailed sequence-diagram for deploy process  https://review.openstack.org/41451208:17
*** e0ne has joined #openstack-ironic08:18
*** e0ne_ has joined #openstack-ironic08:21
*** ralonsoh has joined #openstack-ironic08:23
*** ralonsoh_ has joined #openstack-ironic08:26
*** mgoddard_ has joined #openstack-ironic08:27
*** e0ne has quit IRC08:28
*** mgoddard has quit IRC08:28
*** ralonsoh has quit IRC08:29
*** jrist has quit IRC08:30
*** fragatina has joined #openstack-ironic08:30
*** fragatina has quit IRC08:31
*** fragatina has joined #openstack-ironic08:31
openstackgerritJavier Peña proposed openstack/ironic-python-agent stable/ocata: Mock _wait_for_interface in test_run_with_sleep  https://review.openstack.org/51296908:33
openstackgerritJavier Peña proposed openstack/ironic-python-agent stable/ocata: Mock _wait_for_interface in test_run_with_sleep  https://review.openstack.org/51296908:34
*** leyal has quit IRC08:38
openstackgerritKaifeng Wang proposed openstack/ironic master: Detailed sequence-diagram for deploy process  https://review.openstack.org/41451208:39
openstackgerritKaifeng Wang proposed openstack/ironic master: Workflow of iPXE usage in Deploy Process  https://review.openstack.org/45537708:39
*** dtantsur|afk is now known as dtantsur08:40
dtantsurTheJulia, mgagne, I don't mind it, but that's against what we've been doing with microversions all the time08:41
*** derekh has joined #openstack-ironic08:45
*** leyal has joined #openstack-ironic08:45
*** ralonsoh_ is now known as ralonsoh08:48
*** jtomasek has quit IRC08:48
*** jtomasek has joined #openstack-ironic08:49
*** vsaienk0 has quit IRC08:52
*** karthiks has joined #openstack-ironic08:55
openstackgerritNguyen Van Trung proposed openstack/ironic master: [WIP] Implement raid configuration for BM via irmc drive (OOB-1)  https://review.openstack.org/51297909:00
*** vsaienk0 has joined #openstack-ironic09:04
*** pmannidi has joined #openstack-ironic09:09
dtantsurTheJulia: I think I know what to do about "latest" version, will try to make a patch today09:16
*** gcb has joined #openstack-ironic09:25
*** milan has joined #openstack-ironic09:26
milanmorning Ironic! :)09:27
milan#pixiesay -mflexing work-work09:28
PixieBootsᕙʕ⇀ᴥ⇀ʔᕗ: work-work09:28
pas-hamorning ironic :)09:30
*** racedo has joined #openstack-ironic09:33
*** hw_wutianwei has quit IRC09:38
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: [WIP] Allow re-negotiation of the latest version supplied by CLI  https://review.openstack.org/51298609:38
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: [WIP] Allow re-negotiation of the latest version supplied by CLI  https://review.openstack.org/51298609:44
dtantsurmorning milan, pas-ha09:47
milanmorning dtantsur, pas-ha! :)09:47
*** AlexeyAbashkin has quit IRC09:49
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: [WIP] Set the default API version of OSC CLI to "latest"  https://review.openstack.org/51298909:51
openstackgerritfupingxie proposed openstack/ironic master: stop console when unnecessary  https://review.openstack.org/51142209:57
dtantsurTengu: that's maybe what you're looking for http://openstack-virtual-baremetal.readthedocs.io/en/latest/index.html10:01
*** pmannidi has quit IRC10:02
dtantsurthis is quintupleO reborn, and it's used in the tripleo CI10:02
*** yangyape_ has quit IRC10:09
*** baoli has joined #openstack-ironic10:14
*** hjensas has quit IRC10:14
Tengudtantsur: aahh, thanks!10:15
Tengudtantsur: thanks a lot, I think that will help me.10:15
*** vsaienk0 has quit IRC10:18
*** Guest83828 is now known as rook10:18
*** parlos has quit IRC10:18
*** baoli has quit IRC10:19
*** TuanLA has quit IRC10:24
*** vsaienk0 has joined #openstack-ironic10:25
*** mat128 has joined #openstack-ironic10:33
Tengudtantsur: hmmm, the BMC VM makes me think this is what redhat uses in their openstack formation, right?10:40
Tengudtantsur: it's a kind of proxy, transposing ipmi commands to openstack commands in order to reboot vms10:40
dtantsurTengu: yeah, the BMC VM runs some code for ipmi-to-nova convertion10:42
dtantsurTengu: I don't know a lot of details, you can ping bnemec on #tripleo for that10:42
Tengudtantsur: hmm, but I just see "may not be appropriate for production clouds" - not really sure it's a good idea, seing how we must patch both nova and configration.10:42
TenguI'll check with Ben.10:43
*** AlexeyAbashkin has joined #openstack-ironic10:55
*** chlong has joined #openstack-ironic11:01
*** namnh has quit IRC11:04
*** vsaienk0 has quit IRC11:05
*** jrist has joined #openstack-ironic11:09
*** zzzeek has quit IRC11:10
*** zhangfei has quit IRC11:12
*** zzzeek has joined #openstack-ironic11:14
openstackgerritNguyen Hung Phuong proposed openstack/ironic master: Add additional capabilities discovery for iRMC driver  https://review.openstack.org/41734711:16
*** vsaienk0 has joined #openstack-ironic11:19
*** hjensas has joined #openstack-ironic11:20
*** hjensas has joined #openstack-ironic11:20
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Add ansible deploy interface  https://review.openstack.org/51052211:21
*** ktibi has joined #openstack-ironic11:22
*** phuongnh has quit IRC11:23
*** parlos has joined #openstack-ironic11:29
*** vsaienk0 has quit IRC11:29
*** lucasagomes is now known as lucas-hungry11:38
*** vsaienk0 has joined #openstack-ironic11:40
*** gcb has quit IRC11:45
*** yangyapeng has joined #openstack-ironic11:45
*** fellypefca has joined #openstack-ironic11:46
*** gcb has joined #openstack-ironic11:47
*** vsaienk0 has quit IRC11:50
nicodemosgood morning, ironic.11:51
*** vsaienk0 has joined #openstack-ironic11:53
*** litao__ has quit IRC11:53
dtantsurmorning nicodemos11:56
*** dtantsur is now known as dtantsur|brb12:01
*** jcoufal has joined #openstack-ironic12:05
*** trown|outtypewww is now known as trown12:05
*** Dinesh_Bhor has quit IRC12:09
*** ppiela_ has quit IRC12:10
*** dprince has joined #openstack-ironic12:12
*** jlabarre has joined #openstack-ironic12:12
*** parlos has quit IRC12:25
*** lucas-hungry is now known as lucasagomes12:30
*** jhesketh_ has joined #openstack-ironic12:38
*** jhesketh has quit IRC12:43
*** vsaienk0 has quit IRC12:43
*** links has quit IRC12:44
*** vsaienk0 has joined #openstack-ironic12:44
*** cdearborn has joined #openstack-ironic12:44
*** mat128 has quit IRC12:45
*** hw_wutianwei has joined #openstack-ironic12:52
*** vdrok has quit IRC13:02
*** rloo has joined #openstack-ironic13:02
*** vdrok has joined #openstack-ironic13:02
*** sasha2 has joined #openstack-ironic13:03
*** fdegir has quit IRC13:04
*** fdegir has joined #openstack-ironic13:07
*** SerenaFeng has joined #openstack-ironic13:11
*** jaypipes has joined #openstack-ironic13:12
*** bfournie has joined #openstack-ironic13:20
*** SerenaFeng has quit IRC13:24
*** SerenaF__ has joined #openstack-ironic13:24
pas-hahave we lost the multinode job on gates? zuul vote does not list it13:27
pas-haalso, why is (what I thought would be the most neede test) bios_agent_ipmitool non-voting?13:28
pas-haseems like something changed / lost while migrating to zuul313:29
*** SerenaF__ has quit IRC13:32
*** gcb has quit IRC13:38
*** gcb has joined #openstack-ironic13:38
* milan relocates13:41
milanbbib13:42
*** MattMan has quit IRC13:43
*** MattMan has joined #openstack-ironic13:44
*** hjensas has quit IRC13:44
*** milan has quit IRC13:47
*** mat128 has joined #openstack-ironic13:47
*** cdearborn has quit IRC13:50
*** parlos has joined #openstack-ironic13:51
*** cdearborn has joined #openstack-ironic13:56
rloopas-ha: that is odd. i just looked at https://review.openstack.org/#/c/509334/, multinode is there but ipa-wholedisk-agent_ipmitool-tinyipa is missing14:08
*** mjturek has joined #openstack-ironic14:08
patchbotpatch 509334 - ironic - Add rescue_interface to node DB table14:08
rloopas-ha: which patch were you looking at?14:08
*** crushil has joined #openstack-ironic14:08
pas-halatest zuul vote for my ansible-deploy patch, https://review.openstack.org/#/c/510522/, patchset 514:09
patchbotpatch 510522 - ironic - Add ansible deploy interface14:09
pas-havote is from yesterday, so may be something got changed in between14:09
rloopas-ha: this one is hot off the press, also missing the ipa-wholedisk-agent... one14:09
rloohttps://review.openstack.org/#/c/497685/14:10
patchbotpatch 497685 - ironic - Disallow rolling upgrade from Ocata to Queens14:10
rloopas-ha: that ansible one has grenade-multinode?14:10
pas-hayes, but not the standard multinode14:11
rloopas-ha: but it too is missing the ipa-wholedisk-agent...14:11
rloopas-ha: which ons ie the standard multinode?14:11
pas-halegacy-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa is there, or you mean something else?14:11
pas-habut it is non-voting14:12
pas-hain jenkins we had `gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial`14:12
pas-hanope, not that one, that's grenade14:12
rloopas-ha: i mean the one corresponding to jenkins' gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode-ubuntu-xenial14:12
*** snaz has quit IRC14:12
pas-hayes, the standard multinode, it is that14:13
pas-hathe non-grenade multinode14:13
jlvillalFYI: Seems like we lost our: gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode-ubuntu-xenial job in the Zuul transition14:13
pas-hayep14:13
jlvillalvsaienk0 ^^^14:13
rloojlvillal: yes, that's what i was saying ^^14:13
jlvillalOh, you are already talking about that. Heh :)14:14
sambetts|afkI think I spotted this while going through this patch: https://review.openstack.org/#/c/511267/14:14
patchbotpatch 511267 - ironic - Move ironic legacy jobs into the ironic tree14:14
*** sambetts|afk is now known as sambetts14:14
rloojlvillal: well, that and pas-ha said we're missing a non-grenade multinode job. i'm not sure which one that is.14:14
vsaienk0jlvillal: I saw it, I'm sure pas-ha is looking :)14:14
sambettsI think there was a mistake with the job name processing becuase we have two jobs that have the same name in the openstack-zuul-jobs repo14:15
sambettsI was trying to work out why and I think you've just ansered it14:15
*** ppiela has joined #openstack-ironic14:15
rloosambetts: so would the best thing to do be to just fix it in your patch that moves the legacy jobs to our tree? i mean, add the missing test?14:16
rloosambetts: or land your patch, then add the missing test in a sep patch?14:16
*** yolanda has joined #openstack-ironic14:16
rloobtw, hi and shocking good morning everyone :)14:17
sambettsrloo: it would be easy to add to my patch14:17
rloosambetts: yeah, but just thinking about it. i think i'd prefer a sep patch. cuz it isn't actually in zuul, and your patch is to migrate them.14:18
rloosambetts: are we close to landing your patch? I see some -1. do you have time to update?14:18
*** baoli has joined #openstack-ironic14:19
sambettsrloo: yup I'll get it updated asap14:19
rloosambetts: that'd be great! i'm fine short-term if we're missing that one test (crosses my fingers...)14:20
rloosambetts: hmm, even when we move those tests to ironic... what about our stable branches? do we have to backport the patch?14:21
*** baoli has quit IRC14:21
sambettsrloo: we have to backport the projects.yaml file but not the jobs themselves14:21
rloosambetts: ok, we'll want to do that asap too, cuz that one test is missing from stable/ocata too14:22
*** ppiela_ has joined #openstack-ironic14:28
*** ppiela has quit IRC14:28
*** baoli has joined #openstack-ironic14:28
*** cdearborn has quit IRC14:31
*** jcoufal_ has joined #openstack-ironic14:42
*** jcoufal_ has quit IRC14:42
openstackgerritSam Betts proposed openstack/ironic master: Move ironic legacy jobs into the ironic tree  https://review.openstack.org/51126714:43
sambettsrloo, pas-ha, vsaienk0: ^^ turns out the issue was that the branch filters had been setup wrong, so the multitenent job was only enabled on stable/newton+ocata14:44
sambettseasy fix to change the branch filter14:44
*** jcoufal has quit IRC14:45
sambettsand I answered the other comments too, so hopefully we can get a positive result from zuul and merge that patch ASAP to enable the correct jobs14:46
vsaienk0sambetts: syntax error on last patchset14:46
sambettsah nuts...14:46
*** jcoufal has joined #openstack-ironic14:46
sambettsoooo I know what the issue is14:46
openstackgerritSam Betts proposed openstack/ironic master: Move ironic legacy jobs into the ironic tree  https://review.openstack.org/51126714:48
*** rbartal has quit IRC14:48
sambettsvsaienk0: at least zuul is fast with syntax errors14:49
jlvillalsambetts, What about the idea of zuul/playbooks/  instead of playbooks/ ?14:51
jlvillalsambetts, Or do that as a follow-up? Or don't do it?14:51
*** chlong has quit IRC14:52
sambettsjlvillal: there are several assumtions made in zuul about the location of playbooks as well as a few other special directories14:53
jlvillalsambetts, No, not for playbooks.14:54
jlvillalsambetts, I asked on infra and they said playbooks can go anywhere.14:54
jlvillalsambetts, Well anywhere except zuul.d/14:54
sambetts"However, the main playbook for the job may either be explicitly specified (with the run: attribute) or if that is omitted, an implied value of playbooks/<jobname> is used." from the zuul migration guide14:54
*** markguz_ has joined #openstack-ironic14:54
jlvillalsambetts, Now maybe they told me incorrect information14:54
*** cdearborn_ has joined #openstack-ironic14:55
sambettsbasically if we leave playbooks, we can have the job name == the playbook name, or we have to be explict on every job14:55
sambettsabout the location of the playbooks14:55
jlvillalsambetts, Okay. I'm okay with leaving it as playbooks/ for now. We can migrate it later if we want. Thanks.14:55
TheJuliaGood morning everyone14:58
sambettsyeah as we create migrate our jobs we may or maybe see a need to hide the ansible stuff behind a zuul directory, I guess if we start adding custom roles, we might not want a roles dir at the top level14:58
sambettst14:58
sambettstoo14:58
sambettshey TheJulia14:58
*** baoli has quit IRC14:59
*** baoli has joined #openstack-ironic15:00
*** _milan_ has joined #openstack-ironic15:00
*** mjura has quit IRC15:00
*** cdearborn__ has joined #openstack-ironic15:01
TheJuliadtantsur|brb: re microversion discussion with mgagne, I don't see how it is different if we're maintaining the same error code to go against what we've been doing given that the change guarded is ability to send configdrive data upon rebuild, but given cold meds, I might not be groking things properly15:02
*** baoli has quit IRC15:02
*** baoli has joined #openstack-ironic15:02
TheJuliadtantsur|brb: re latest, awesome. Unless we've changed that code substantially, I think it would only be in one place actually, and a minor change at that. Just... the fun of the overall impact :)15:03
*** cdearborn has joined #openstack-ironic15:07
*** penick has joined #openstack-ironic15:08
*** cdearborn_ has quit IRC15:11
*** trown is now known as trown|brb15:11
jlvillalTheJulia, dtantsur|brb: If you have time to review these two stable backports: https://review.openstack.org/#/c/508321/ & https://review.openstack.org/#/c/508322/15:11
patchbotpatch 508321 - ironic (stable/pike) - Reduce complexity of node_power_action() function15:11
patchbotpatch 508322 - ironic (stable/pike) - ipmitool: reboot: Don't power off node if already off15:11
*** catintheroof has joined #openstack-ironic15:13
TheJuliadtantsur|brb: ugh, I didn't realize the osc pugin had some different logic wrapped around there, but just replied to rloo's comments and am in complete agreement15:13
TheJuliajlvillal: adding to the hopper15:14
rlooTheJulia: thx!15:14
openstackgerritSam Betts proposed openstack/ironic master: Move ironic legacy jobs into the ironic tree  https://review.openstack.org/51126715:15
rloodtantsur|brb: question about https://review.openstack.org/#/c/512969/ -- is it needed for stable/pike too?15:15
patchbotpatch 512969 - ironic-python-agent (stable/ocata) - Mock _wait_for_interface in test_run_with_sleep15:15
*** trown|brb is now known as trown15:15
rloodtantsur|brb: ok, sorry, i just saw the comment you made in that patch.15:16
rloodtantsur|brb: i think that comment should be in the commit msg15:16
*** cdearborn__ has quit IRC15:17
*** pcaruana has quit IRC15:19
*** cdearborn_ has joined #openstack-ironic15:20
*** tesseract has quit IRC15:20
jlvillalTheJulia, thanks15:20
TheJuliajlvillal: just to confirm my interpretation, refactor required to land a backported bug fix?15:21
*** cdearborn_ has quit IRC15:22
TheJuliarloo: I think it would be good to land, just the tracking tag for backporting is not on it15:22
rlooTheJulia: yeah, i'm going to update the commit first though. just looking for original commit.15:23
rlooTheJulia: but original commit code looks wrong... i am somewhat puzzled...15:23
TheJuliaoh, so direct change...15:23
TheJuliahmmmm15:24
rlooTheJulia: well, regardless, i'll update this commit, then look at the actual fix on master. maybe i need more caffeine or something...15:24
TheJuliaI bet it got added as part of some larger refactoring, but that got noticed in and in the grander scheme of things the author, I guess decided just to copy the mock in to place so things don't break15:24
TheJuliarloo: I have decongestants, those may help or hurt15:25
rlooTheJulia: thx for the offer but I think I'll suffer instead :D15:26
TheJuliaheh15:26
*** Sukhdev_ has joined #openstack-ironic15:26
openstackgerritRuby Loo proposed openstack/ironic-python-agent stable/ocata: Mock _wait_for_interface in test_run_with_sleep  https://review.openstack.org/51296915:27
jlvillalTheJulia, Correct on refactor required. I added a note at the bottom of the commit messages, dtantsur|brb had requested that comment.15:29
TheJuliaack ack15:30
lucasagomes<ttx> PTG confirmed in Dublin, Ireland the week of Feb 26, 201815:32
lucasagomeslooking fwd to see y'all around :-)15:32
jlvillallucasagomes, cool. and hello :)15:32
lucasagomesjlvillal, hi there :D15:32
*** amarao has quit IRC15:32
*** cdearborn has quit IRC15:32
TheJulia\o/15:32
*** srart has quit IRC15:33
jlvillalI see the webpage is updated: https://www.openstack.org/ptg/15:33
lucasagomesah nice!15:33
openstackgerritRuby Loo proposed openstack/ironic-python-agent master: Unit test has incorrect mock order  https://review.openstack.org/51307115:34
rloolucasagomes: awesome. Not sure I'll be there, but I hope you'll see some of us.15:35
lucasagomesoh noes rloo, you have to come!15:36
*** dtantsur|brb is now known as dtantsur15:36
rloolucasagomes: depends on my company...15:37
* lucasagomes fingers crossed15:37
dtantsurmorning rloo, jlvillal; go back to bed TheJulia :)15:37
dtantsurlucasagomes: \o/15:38
rloolucasagomes: thx, i hope you have the luck of the irish :D15:38
lucasagomesdtantsur, hey :D15:38
jlvillalHi dtantsur :)15:38
rloojlvillal: if you have a few minutes, vdrok +2'd, if you can also and +A, that'd be great: https://review.openstack.org/51126715:38
patchbotpatch 511267 - ironic - Move ironic legacy jobs into the ironic tree15:38
lucasagomesrloo, hah not sure about that, but let's hope :D15:38
rloolucasagomes: :)15:39
TheJuliadtantsur: oh alright :)15:41
*** fragatina has quit IRC15:42
*** fragatina has joined #openstack-ironic15:42
NobodyCamGood Morning Ironic'ers15:43
dtantsurmorning NobodyCam15:44
NobodyCamhey hey dtantsur, how is home hunting going?15:44
dtantsurNobodyCam: done! moving on, hopefully, on 1st Nov15:45
NobodyCam:) w00t!15:45
NobodyCamcongrats!15:45
*** srart has joined #openstack-ironic15:47
*** parlos has quit IRC15:47
*** Sukhdev has joined #openstack-ironic15:51
openstackgerritSam Betts proposed openstack/ironic master: Move ironic legacy jobs into the ironic tree  https://review.openstack.org/51126715:52
openstackgerritMerged openstack/ironic-python-agent stable/ocata: Mock _wait_for_interface in test_run_with_sleep  https://review.openstack.org/51296915:53
dtantsurthanks NobodyCam :)15:53
NobodyCam:)15:53
*** links has joined #openstack-ironic15:55
openstackgerritMerged openstack/ironic stable/pike: Fix mis-formatted log messages  https://review.openstack.org/51019916:00
*** e0ne_ is now known as e0ne16:01
*** rama_y has joined #openstack-ironic16:03
*** marios has quit IRC16:03
*** hw_wutianwei has quit IRC16:06
*** ijw has joined #openstack-ironic16:14
openstackgerritMerged openstack/ironic master: Disallow rolling upgrade from Ocata to Queens  https://review.openstack.org/49768516:14
*** lucasagomes is now known as lucas-afk16:15
*** baoli has quit IRC16:15
*** trown is now known as trown|lunch16:21
dtantsurrloo: any other must-haves for the coming release? ironic dbsync change?16:23
rloodtantsur: ironic release?16:23
rloodtantsur: that one yes. let me think about it... (lunching now)16:24
*** alex_xu has quit IRC16:24
rloodtantsur: i16:25
rloodtantsur: i'd like to get pas-ha's patches in for the keystone catalog stuff.16:26
dtantsurrloo: that may be quite a reviewing challenge16:26
rloodtantsur: cuz we have landed some of it, might be odd not to land them all16:26
dtantsur:)16:26
rloopas-ha: ^^ where are we at with it? seems like you were going to update the next patch?16:28
pas-harloo, need to rebase them first, and edit for some comments. will do tomorrow.16:28
*** links has quit IRC16:28
rloopas-ha: and they're easy to review, right? :D16:28
pas-hawell, the second-to-last (neutron) is not so :(16:29
*** baoli has joined #openstack-ironic16:30
*** baoli has quit IRC16:30
*** alex_xu has joined #openstack-ironic16:31
rloodtantsur: we should get this doc update in too: https://review.openstack.org/#/c/511240/. i'll review today.16:31
patchbotpatch 511240 - ironic - Finish the guide on upgrading to hardware types16:31
dtantsuryeah, I'd not mind this :)16:31
*** baoli has joined #openstack-ironic16:32
rloodtantsur, others, reminder that there is the nova spec that might be worth reviewing before tomorrow's nova feature spec freeze: https://review.openstack.org/#/c/449155/16:34
patchbotpatch 449155 - nova-specs - Ironic: Resize and cold migration support16:34
rloolucas-afk: qq, why did you abandon this: https://review.openstack.org/#/c/327642/16:37
patchbotpatch 327642 - ironic - Add power_transition_started_at to the nodes table (ABANDONED)16:37
*** mat128 has quit IRC16:38
openstackgerritMerged openstack/ironic master: Remove 'next' for GET /nodes?limit=1&instance_uuid=  https://review.openstack.org/50620516:39
rloodtantsur: we have a bunch of bugs marked 'high'. Sigh. I didn't plan on doing bug triaging today. We should probably do a better job (more frequently?) reviewing these high ones.16:42
*** derekh has quit IRC16:42
rloodtantsur: eg, https://bugs.launchpad.net/ironic/+bug/1722268 has a patch that needs reviews: https://review.openstack.org/#/c/510644/16:42
openstackLaunchpad bug 1722268 in Ironic "OneView client sessions expires after 24 hours" [High,In progress] - Assigned to Stenio Araujo (stenioaraujo)16:42
patchbotpatch 510644 - ironic - Get a new OneView client when needed16:42
rloodtantsur: maybe we should add links to those in our whiteboard, under the bugs section?16:43
* dtantsur looks at the bug count16:43
dtantsur38 high bugs for ironic only. that's sad indeed, and I don't think adding 38 links anywhere will help much16:43
rloodtantsur: oh, i only mean links that have patches that are ready to review16:44
rloodtantsur: quite a few of these are NOT in progress, and some that were, I've changed16:44
dtantsurthanks rloo! maybe we should pick some number of ready-to-be-fixed bugs every week?16:44
dtantsurhow many patches do we have now?16:44
rloodtantsur: that one i mentioned above. another one has stalled - i might bring that one up in meeting. i'm still looking :-)16:45
rloofellypefca: is this fixed now? https://bugs.launchpad.net/ironic/+bug/171602316:45
openstackLaunchpad bug 1716023 in Ironic "OneView driver is not working for multitenancy scenario in ocata" [High,In progress] - Assigned to Fellype Cavalcante (fellypefca)16:45
dtantsuryeah, the only you mentioned is on my radar16:45
*** srart has quit IRC16:46
rloodtantsur: i did skip the ones that were test-related though. did you want me to do a thorough job? i was just looking wrt the ironic release16:46
fellypefcarloo yes16:47
dtantsurrloo: if you have some time - yes please. otherwise, we should establish some procedure around reducing the bug backlog16:47
*** AlexeyAbashkin has quit IRC16:47
rloodtantsur: i've got a meeting in a few min. i'm going to update our whiteboard with what i find, under the bugs, and we can see/think about how to deal with them/process in the future16:48
*** Sukhdev has quit IRC16:50
*** srart has joined #openstack-ironic16:51
dtantsurcool16:51
*** mgoddard_ has quit IRC16:51
*** parlos has joined #openstack-ironic16:57
*** e0ne has quit IRC16:59
*** ralonsoh has quit IRC17:01
*** leyal has quit IRC17:04
*** leyal has joined #openstack-ironic17:05
*** vsaienk0 has quit IRC17:16
*** dtantsur is now known as dtantsur|afk17:24
*** mat128 has joined #openstack-ironic17:29
*** _milan_ has quit IRC17:34
*** Sukhdev_ has quit IRC17:36
*** trown|lunch is now known as trown17:37
*** e0ne has joined #openstack-ironic17:41
* sambetts --> home17:46
sambettsnight all17:46
*** sambetts is now known as sambetts|afk17:46
rloogoodnight sambetts|afk17:52
*** catinthe_ has joined #openstack-ironic17:53
*** catintheroof has quit IRC17:53
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Allow to set default ifaces in DevStack  https://review.openstack.org/51315217:57
*** catintheroof has joined #openstack-ironic17:59
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Allow to set default ifaces in DevStack  https://review.openstack.org/51315217:59
*** parlos has quit IRC18:00
*** catinthe_ has quit IRC18:01
*** baoli has quit IRC18:02
*** baoli has joined #openstack-ironic18:04
*** catinthe_ has joined #openstack-ironic18:05
*** catintheroof has quit IRC18:07
rloohey jlvillal, is this https://review.openstack.org/#/c/489762/, to address this bug: https://bugs.launchpad.net/ironic/+bug/1660606 ?18:09
openstackLaunchpad bug 1660606 in Ironic "Ironic CI refactoring" [High,In progress] - Assigned to John L. Villalovos (happycamp)18:09
patchbotpatch 489762 - ironic-tempest-plugin - Script to extract the ironic_tempest_plugin/ direc...18:09
openstackgerritJohn L. Villalovos proposed openstack/ironic-python-agent master: Have unit tests all derive from our base unit test class  https://review.openstack.org/51315518:09
jlvillalrloo: I don't think so18:10
jlvillalLet me look for the bug18:10
jlvillalrloo: https://bugs.launchpad.net/ironic/+bug/171716518:11
openstackLaunchpad bug 1717165 in Ironic "Move tempest test to ironic-tempest-plugin repo" [High,In progress] - Assigned to John L. Villalovos (happycamp)18:11
rloojlvillal: ok, maybe add that bug then to the patch. or did i miss it?18:11
rloojlvillal: i ask, cuz of this patch: https://review.openstack.org/#/c/489790/18:11
patchbotpatch 489790 - ironic-tempest-plugin - Add Ironic standalone tests (ABANDONED)18:11
rloojlvillal: thought maybe it was the same thing18:11
jlvillalrloo: Probably a mistake on my part.18:11
rloojlvillal: and 489790 was associated with that bug. no worries, i'll just change the status of that bug.18:12
jlvillalrloo: Oh wait. Standalone tests is something we want. Some have been added.18:12
rloojlvillal: that bug is assigned to you :)18:12
jlvillalrloo: So what happened, when I was attempting to move the patches over. I proposed already merged patches to ironic_tempest_plugin18:12
jlvillalrloo: As part of the transition process. But we have decided to not do it that way.18:13
*** baoli has quit IRC18:13
rloojlvillal: but the bug is still valid? or not? it is assigned to you so please do whatever with it.18:13
jlvillalrloo: We will work with infra to just copy a repo in with all the patches in place.18:13
jlvillalrloo: I gave it back to vsaienk0 :)18:13
rloojlvillal: ha ha.18:14
*** baoli has joined #openstack-ironic18:14
rloojlvillal: although you should probably ask him about it, or just unassign yourself.18:14
jlvillalrloo: He was the owner before my patch submission changed it.18:14
*** catintheroof has joined #openstack-ironic18:14
rloojlvillal: anyway, i'm just going through high bugs, to see if there are any patches that need reviewing. so... moving on to the next bug...18:15
jlvillalrloo: Thanks for doing that18:15
*** catinthe_ has quit IRC18:15
rloojlvillal: :)18:15
*** AlexeyAbashkin has joined #openstack-ironic18:19
*** Sukhdev has joined #openstack-ironic18:20
*** AlexeyAbashkin has quit IRC18:24
*** rwsu has quit IRC18:25
*** mat128 has quit IRC18:25
*** ijw has quit IRC18:27
*** weshay is now known as weshay|ruck|brb18:30
openstackgerritJohn L. Villalovos proposed openstack/ironic-python-agent master: Improve the catching of calls to 'execute' related functions  https://review.openstack.org/46610018:30
* jlvillal just realized he needs to update his Gerrit dashboard, as now it is Zuul voting instead of Jenkins18:37
rloojlvillal: do you have time to review this: https://review.openstack.org/#/c/511267/18:40
patchbotpatch 511267 - ironic - Move ironic legacy jobs into the ironic tree18:40
jlvillalrloo: Yeah, I can do that.18:43
rloojlvillal: thx, would be good to land that today. will make it faster for us to fix things.18:43
*** ChubYann has joined #openstack-ironic18:44
*** crushil has quit IRC18:46
*** ccamacho has quit IRC18:52
jlvillalrloo: +2. Waiting for the gate to pass.18:54
jlvillalrloo: If you look at http://zuulv3.openstack.org/ you can see it in action. About 20+ jobs running. as running old and new.18:55
rloojlvillal: sweet18:55
* jlvillal decides it is lunch time18:55
rloojlvillal: thx and enjoy your lunch.18:56
radezTheJulia: ping, I got that error again.19:01
radezhttps://paste.fedoraproject.org/paste/7~0LhJTxWioH3IuSgoA-MQ19:01
radezit's actually from the nova log, I skipped introspection19:01
radezwhat confuses me is that the ironic node shows enough disk eventhough the filter bombs on it https://paste.fedoraproject.org/paste/qEmTbB-SqaULCnB2WneJww19:03
*** ppiela has joined #openstack-ironic19:04
*** ppiela_ has quit IRC19:04
*** dprince has quit IRC19:08
*** rama_y has quit IRC19:09
*** rama_y has joined #openstack-ironic19:10
*** jcoufal has quit IRC19:16
*** weshay|ruck|brb is now known as weshay|ruck19:18
*** AlexeyAbashkin has joined #openstack-ironic19:19
*** AlexeyAbashkin has quit IRC19:24
*** fellypefca has quit IRC19:31
openstackgerritAnup Navare proposed openstack/ironic-ui master: Allow selecting interfaces while enrolling nodes  https://review.openstack.org/47395719:32
*** hjensas has joined #openstack-ironic19:35
*** hjensas has joined #openstack-ironic19:35
*** yolanda has quit IRC19:36
*** rwsu has joined #openstack-ironic19:37
*** sasha2 has quit IRC19:40
*** sasha2 has joined #openstack-ironic19:40
*** rama_y has quit IRC19:45
*** rama_y has joined #openstack-ironic19:48
*** crushil has joined #openstack-ironic19:57
*** penick has quit IRC19:57
*** markguz__ has joined #openstack-ironic20:00
*** penick has joined #openstack-ironic20:01
*** markguz_ has quit IRC20:01
*** jkilpatr has quit IRC20:10
*** openstackgerrit has quit IRC20:17
*** dsneddon has quit IRC20:19
*** Sukhdev has quit IRC20:21
*** jkilpatr has joined #openstack-ironic20:26
*** patchbot has quit IRC20:38
*** patchbot has joined #openstack-ironic20:39
*** catintheroof has quit IRC20:53
*** catintheroof has joined #openstack-ironic20:54
*** catintheroof has quit IRC20:58
*** jlabarre has quit IRC20:59
*** markguz__ has quit IRC21:02
*** markguz_ has joined #openstack-ironic21:03
*** markguz__ has joined #openstack-ironic21:05
*** markguz__ has quit IRC21:05
*** markguz__ has joined #openstack-ironic21:05
*** markguz_ has quit IRC21:07
*** trown is now known as trown|outtypewww21:10
*** ktibi has quit IRC21:16
*** ijw has joined #openstack-ironic21:18
*** AlexeyAbashkin has joined #openstack-ironic21:19
*** yolanda has joined #openstack-ironic21:20
*** AlexeyAbashkin has quit IRC21:24
*** e0ne has quit IRC21:25
*** e0ne has joined #openstack-ironic21:26
*** e0ne has quit IRC21:27
*** e0ne has joined #openstack-ironic21:28
*** e0ne has quit IRC21:28
*** Sukhdev has joined #openstack-ironic21:29
*** e0ne has joined #openstack-ironic21:29
*** e0ne has quit IRC21:30
*** e0ne has joined #openstack-ironic21:31
*** e0ne has quit IRC21:31
*** e0ne has joined #openstack-ironic21:33
*** e0ne has quit IRC21:33
*** baoli has quit IRC21:35
*** jtomasek has quit IRC21:40
*** rwsu has quit IRC22:05
*** openstackgerrit has joined #openstack-ironic22:09
openstackgerritMathieu Gagné proposed openstack/ironic master: Add ability to provide configdrive when rebuilding  https://review.openstack.org/50176122:09
*** patchbot has quit IRC22:19
*** patchbot has joined #openstack-ironic22:21
*** penick has quit IRC22:34
*** Sukhdev has quit IRC23:00
*** rwsu has joined #openstack-ironic23:01
*** ppiela has quit IRC23:08
*** rloo has quit IRC23:10
*** rloo has joined #openstack-ironic23:11
*** ijw has quit IRC23:13
*** rloo has quit IRC23:17
*** AlexeyAbashkin has joined #openstack-ironic23:19
*** karimb has quit IRC23:21
*** AlexeyAbashkin has quit IRC23:23
*** rloo has joined #openstack-ironic23:25
*** markguz__ has quit IRC23:27
openstackgerritMathieu Gagné proposed openstack/ironic master: Add ability to provide configdrive when rebuilding  https://review.openstack.org/50176123:30
*** mjturek has quit IRC23:31
*** ijw has joined #openstack-ironic23:34
*** ijw has quit IRC23:38
*** ijw has joined #openstack-ironic23:44
*** rloo has quit IRC23:50
*** ijw has quit IRC23:51

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!