Friday, 2017-10-27

*** lindycoder has quit IRC00:00
*** ppiela has joined #openstack-ironic00:04
*** ppiela_ has quit IRC00:06
*** AlexeyAbashkin has joined #openstack-ironic00:10
*** catintheroof has joined #openstack-ironic00:11
*** AlexeyAbashkin has quit IRC00:14
*** rcernin has joined #openstack-ironic00:21
*** catintheroof has quit IRC00:32
*** rama_y has quit IRC00:33
*** catintheroof has joined #openstack-ironic00:38
*** catintheroof has quit IRC00:42
*** catintheroof has joined #openstack-ironic00:44
*** AlexeyAbashkin has joined #openstack-ironic00:46
*** litao__ has joined #openstack-ironic00:49
*** AlexeyAbashkin has quit IRC00:51
*** hjensas has quit IRC00:55
*** catintheroof has quit IRC00:55
*** catintheroof has joined #openstack-ironic00:56
*** zzzeek has quit IRC00:58
*** charcol has quit IRC01:00
*** zzzeek has joined #openstack-ironic01:00
*** phuongnh has joined #openstack-ironic01:02
*** charcol has joined #openstack-ironic01:04
*** catintheroof has quit IRC01:20
*** namnh has joined #openstack-ironic01:21
*** catintheroof has joined #openstack-ironic01:21
*** ppiela_ has joined #openstack-ironic01:22
*** ppiela has quit IRC01:23
TheJuliazigo: it has taken many packages :(01:24
*** catintheroof has quit IRC01:24
TheJuliazigo: this feels like a dependency issue, although it may be an out of order execution of the steps. i've got python-oslo.db 4.13.3 and the db tests are passing locally. I am getting three failing ilo driver unit tests. The installed version of the proliantutils lib was 2.1.4, not 2.4.1 which  is current. bumped it to the correct version and the tests all passed01:37
TheJuliafwiw, current oslodb for pike should be >=4.2401:39
*** rcernin has quit IRC01:43
*** rcernin has joined #openstack-ironic01:44
*** zhangjl has joined #openstack-ironic01:55
*** catintheroof has joined #openstack-ironic01:55
*** zshi__ has joined #openstack-ironic01:58
*** catintheroof has quit IRC02:00
*** gcb has joined #openstack-ironic02:16
openstackgerritJulia Kreger proposed openstack/bifrost master: Bump ansible version to latest stable  https://review.openstack.org/50519102:17
openstackgerritJulia Kreger proposed openstack/bifrost master: Clean up jinja2 templates on conditional checks  https://review.openstack.org/51556802:17
*** ppiela has joined #openstack-ironic02:20
*** ppiela_ has quit IRC02:22
*** catintheroof has joined #openstack-ironic02:28
*** lindycoder has joined #openstack-ironic02:29
*** rama_y has joined #openstack-ironic02:29
*** rama_y has quit IRC02:31
*** catintheroof has quit IRC02:33
*** zhangfei has joined #openstack-ironic02:33
*** fragatina has quit IRC02:40
*** fragatina has joined #openstack-ironic02:45
*** AlexeyAbashkin has joined #openstack-ironic02:47
*** lindycoder has quit IRC02:49
*** fragatina has quit IRC02:50
*** AlexeyAbashkin has quit IRC02:52
*** Nisha_Agarwal has joined #openstack-ironic02:55
Nisha_Agarwalpmannidi, hi02:55
Nisha_Agarwalmorning ironic!!!02:56
pmannidiNisha_Agarwal, good morning02:56
Nisha_Agarwalpmannidi, any update?02:56
pmannidino update from customer, i'll ask again and let you know by monday02:57
Nisha_Agarwalok02:58
*** pmannidi is now known as pmannidi|brb03:00
*** AlexeyAbashkin has joined #openstack-ironic03:08
*** AlexeyAbashkin has quit IRC03:12
*** rbudden has joined #openstack-ironic03:18
*** rbudden has quit IRC03:20
*** Nisha_Agarwal has quit IRC03:33
*** charcol-v2 has joined #openstack-ironic03:41
*** charcol has quit IRC03:41
*** zhangfei has quit IRC03:44
*** Nisha_Agarwal has joined #openstack-ironic03:57
*** fragatina has joined #openstack-ironic03:57
*** fragatina has quit IRC03:58
*** udesale has joined #openstack-ironic03:58
*** fragatina has joined #openstack-ironic03:58
*** catintheroof has joined #openstack-ironic04:01
*** dsneddon has quit IRC04:03
*** catintheroof has quit IRC04:05
*** yangyape_ has joined #openstack-ironic04:09
*** yangyapeng has quit IRC04:11
*** rama_y has joined #openstack-ironic04:14
*** ppiela_ has joined #openstack-ironic04:14
*** fragatina has quit IRC04:16
*** ppiela has quit IRC04:17
*** logan- has quit IRC04:38
*** pmannidi|brb has quit IRC04:38
*** pmannidi|brb has joined #openstack-ironic04:39
*** pmannidi|brb is now known as pmannidi04:39
*** pmannidi has quit IRC04:39
*** pmannidi has joined #openstack-ironic04:40
*** logan- has joined #openstack-ironic04:42
*** rama_y has quit IRC05:10
*** masuberu has joined #openstack-ironic05:15
*** masber has quit IRC05:18
*** masber has joined #openstack-ironic05:20
*** masuberu has quit IRC05:23
*** jaganathan has joined #openstack-ironic05:25
*** yolanda has joined #openstack-ironic05:42
*** yolanda__ has joined #openstack-ironic06:02
openstackgerritNguyen Van Trung proposed openstack/ironic master: Add additional capabilities discovery for iRMC driver  https://review.openstack.org/41734706:07
*** mjura has joined #openstack-ironic06:10
*** vsaienk0 has joined #openstack-ironic06:17
*** spectr has joined #openstack-ironic06:22
*** zhangfei has joined #openstack-ironic06:23
*** kzaitsev1pi has joined #openstack-ironic06:38
*** kzaitsev_pi has quit IRC06:40
*** ccamacho has quit IRC06:47
*** mjura has quit IRC06:49
*** mjura has joined #openstack-ironic06:50
*** mrch has joined #openstack-ironic07:06
*** serlex has joined #openstack-ironic07:08
*** vsaienk0 has quit IRC07:20
*** vsaienk0 has joined #openstack-ironic07:20
openstackgerritfengzhr proposed openstack/ironic-inspector master: Use database to store introspection_data  https://review.openstack.org/51455207:21
*** tesseract has joined #openstack-ironic07:22
*** parlos has joined #openstack-ironic07:25
*** yolanda__ has quit IRC07:27
*** karimb has joined #openstack-ironic07:32
*** charcol-v2 has quit IRC07:34
*** zshi__ has quit IRC07:34
*** namnh has quit IRC07:34
*** milan has joined #openstack-ironic07:35
milanmorning Ironic! :)07:35
milan#pixiesay -mflexing clean-up-the-patches-day!07:36
PixieBootsᕙʕ⇀ᴥ⇀ʔᕗ: clean-up-the-patches-day!07:36
vsaienk0morning milan, aarefiev and all Ironic'ers!07:42
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Fix state discrepancy in process._finish  https://review.openstack.org/51535207:42
milanmorning aarefiev, vsaienk0! :)07:42
Nisha_Agarwalmorning vsaienk0, milan aarefiev07:43
milanmorning Nisha_Agarwal! :)07:43
aarefievmorning vsaienk0, milan, Nisha_Agarwal and Ironic!07:44
*** e0ne has joined #openstack-ironic07:45
milanaarefiev, mind some proofreading on https://review.openstack.org/#/c/510928/3?07:46
patchbotpatch 510928 - ironic-inspector - Use db transactions when error-finish is called07:46
milanit's quite tiny07:46
milanbut should enhance UX in HA mode a lot07:46
milanas well https://review.openstack.org/#/c/514854/207:47
patchbotpatch 514854 - ironic-inspector - Make _abort set error and finished_at in fms_event07:47
milanand https://review.openstack.org/#/c/514854/207:47
patchbotpatch 514854 - ironic-inspector - Make _abort set error and finished_at in fms_event07:47
milanall three share same "features"07:47
milanoops07:47
aarefievmilan: I'm looking07:48
milanhttps://review.openstack.org/#/c/515352/207:48
patchbotpatch 515352 - ironic-inspector - Fix state discrepancy in process._finish07:48
milanaarefiev, thx!07:48
aarefievmilan: I have trivial patch for your review list too https://review.openstack.org/#/c/513768/07:49
*** trungnv has quit IRC07:49
patchbotpatch 513768 - python-ironic-inspector-client - Remove deprecated setting ipmi creds feature07:49
aarefiev:)07:49
milanaarefiev, sure thing!07:49
* milan looks07:49
*** ralonsoh has joined #openstack-ironic07:50
*** AlexeyAbashkin has joined #openstack-ironic07:56
openstackgerritVasyl Saienko proposed openstack/ironic master: Add test to check local boot with partitioned image  https://review.openstack.org/50288607:57
*** rcernin has quit IRC07:58
*** makowals has joined #openstack-ironic07:58
*** gcb has quit IRC08:00
*** gcb has joined #openstack-ironic08:03
*** trungnv has joined #openstack-ironic08:09
*** ccamacho has joined #openstack-ironic08:10
*** zhangfei has quit IRC08:15
openstackgerritVasyl Saienko proposed openstack/ironic-lib master: Move openstack-tox-cover job to project config  https://review.openstack.org/51564108:21
*** lucas-afk is now known as lucasagomes08:26
*** udesale has quit IRC08:35
openstackgerritMerged openstack/python-ironic-inspector-client master: Remove deprecated setting ipmi creds feature  https://review.openstack.org/51376808:44
milanaarefiev, w/r https://review.openstack.org/#/c/510928/3 would be a bit refactoring, but no prob with me...08:44
patchbotpatch 510928 - ironic-inspector - Use db transactions when error-finish is called08:44
*** derekh has joined #openstack-ironic08:45
aarefievmilan: why? seems we handle all errors in fsm_transition08:45
milancause it's not part of the bug, so more like an unrelated change to remove the redundant handling?08:46
milanbut as I said I don't mind :)08:46
milanaarefiev, I'll update in a bit08:47
milan*post update08:47
*** amarao has joined #openstack-ironic08:48
aarefievmilan: yeah, it could go to separate patch, but I thought there is nothing to update, only dropping this handle08:48
aarefievmaybe I'm missing something08:49
milanaarefiev, no worries, I'll post the update, let's see how it's going to look like08:49
milanthanks for checking08:49
* milan glad it's not a -2 thing ;)08:49
milandtantsur|afk, around? :DDDD ^08:50
*** namnh has joined #openstack-ironic08:51
*** udesale has joined #openstack-ironic08:53
*** racedo has joined #openstack-ironic08:54
openstackgerritVasyl Saienko proposed openstack/ironic master: Add test to check local boot with partitioned image  https://review.openstack.org/50288608:55
*** pmannidi has quit IRC08:55
openstackgerritVasyl Saienko proposed openstack/ironic master: Add test to check local boot with partitioned image  https://review.openstack.org/50288608:58
*** stendulker has joined #openstack-ironic09:01
*** Nisha_Agarwal has quit IRC09:05
openstackgerritMilan Kováčik proposed openstack/ironic-specs master: [WIP] Introduce inspect wait state  https://review.openstack.org/51565709:16
openstackgerritMilan Kováčik proposed openstack/ironic-specs master: [WIP] Introduce inspect wait state  https://review.openstack.org/51565709:19
milanaarefiev, dtantsur|afk, sambetts|afk me and rloo thought would be probably better to have a spec for the inspect wait state. ^ is the brief version.09:20
* milan will post missing sections if folks agree this is the way to go09:21
stendulkerGood morning milan :)09:31
stendulkermilan: Having a spec would help.09:31
milanmorning stendulker! :)09:36
vdrokgood morning all, milan stendulker09:36
milanmorning vdrok! :)09:37
stendulkerGood morning vdrok :)09:37
vdrokand good morning aarefiev and vsaienk009:37
vsaienk0morning vdrok09:37
*** gcb has quit IRC09:37
milanstendulker, yup, mind checking the one I proposed? ;)09:37
aarefievmorning vdrok :)09:37
stendulkerGood morning aarefiev, vsaienk0 :)09:38
vsaienk0o/ stendulker09:38
aarefievmorning stendulker09:38
stendulkermilan: Do you want to add 'abort' also in this spec?09:38
*** hw_wutianwei has quit IRC09:39
milanstendulker, not really; I'd say this is more about the missing timeout protection and making the state machine consistent w/r the inspect state09:39
stendulkermilan: Also its not clear, what is the difference in behavior if one 'abort' or just wait for timeout to happen09:40
milan* inspect state compared to the other "async" states09:40
stendulkermilan: You have added tht in the 'Proposed change'. There is no abort functionality to inspection today09:40
milanstendulker, the abort would trigger the interface's abort implementation09:40
milanstendulker, yeah, fair note!09:41
*** links has quit IRC09:42
stendulkermilan: also you need to capture how things would be conveyed to inspector in case of timeout09:42
openstackgerritVasyl Saienko proposed openstack/ironic master: Add test to check local boot with partitioned image  https://review.openstack.org/50288609:44
milanstendulker, yup, inspector has the periodic sync with ironic, so that is possible, but you're right, the spec should mention that explicitly09:44
stendulkermilan: ok09:44
milanstendulker, thanks for reading!09:45
stendulkermilan: you are welcome :)09:45
*** fragatina has joined #openstack-ironic09:48
*** zhangjl has quit IRC09:50
*** zhangjl has joined #openstack-ironic09:51
openstackgerritVasyl Saienko proposed openstack/ironic master: Add test to check local boot with partitioned image  https://review.openstack.org/50288609:56
*** ppiela__ has joined #openstack-ironic10:03
*** amarao1 has joined #openstack-ironic10:03
*** ppiela_ has quit IRC10:07
*** lindycoder has joined #openstack-ironic10:10
*** Goneri has joined #openstack-ironic10:20
*** mjura has quit IRC10:24
*** phuongnh has quit IRC10:24
* milan is away: bbib10:30
*** namnh has quit IRC10:36
*** jaganathan has quit IRC10:37
*** vsaienk0 has quit IRC10:37
*** Goneri has quit IRC10:53
*** Goneri has joined #openstack-ironic10:55
*** trungnv has quit IRC10:57
*** vsaienk0 has joined #openstack-ironic11:00
*** robcresswell has quit IRC11:03
*** lucasagomes is now known as lucas-hungry11:07
openstackgerritfengzhr proposed openstack/ironic-inspector master: Use database to store introspection_data  https://review.openstack.org/51455211:14
*** mat128 has joined #openstack-ironic11:17
*** dyasny has quit IRC11:18
openstackgerritfengzhr proposed openstack/ironic-inspector master: Use database to store introspection_data  https://review.openstack.org/51455211:18
*** marlinc has quit IRC11:28
*** Goneri has quit IRC11:30
*** hamzy has joined #openstack-ironic11:30
*** baoli has joined #openstack-ironic11:34
*** Goneri has joined #openstack-ironic11:36
*** litao__ has quit IRC11:36
*** baoli has quit IRC11:37
*** marlinc has joined #openstack-ironic11:38
*** baoli has joined #openstack-ironic11:38
*** zhangjl has quit IRC11:39
*** sambetts|afk is now known as sambetts11:41
nicodemosmorning, ironic11:53
*** dtantsur|afk is now known as dtantsur11:58
dtantsurmorning/afternoon ironic, says your PTL fighting with cold :(11:59
*** athomas has joined #openstack-ironic12:01
sambetts:(12:04
*** dprince has joined #openstack-ironic12:05
*** fellypefca has joined #openstack-ironic12:05
*** stendulker has quit IRC12:06
*** parlos has quit IRC12:09
*** jaypipes has joined #openstack-ironic12:12
*** lucas-hungry is now known as lucasagomes12:14
vsaienk0folks please add to your review queue https://review.openstack.org/#/c/515641/ to finish zuulv3 migration for ironic-lib12:15
patchbotpatch 515641 - ironic-lib - Move openstack-tox-cover job to project config12:15
*** trown|outtypewww is now known as trown12:18
dtantsureeeeek http://logs.openstack.org/26/514426/1/gate/openstack-tox-validate/6880139/job-output.txt.gz#_2017-10-27_00_07_06_34683212:25
dtantsurthis is on openstack/releases when trying to release IPA12:26
*** jcoufal has joined #openstack-ironic12:30
*** parlos has joined #openstack-ironic12:30
openstackgerritMerged openstack/ironic-specs master: Add spec for XClarity Driver  https://review.openstack.org/47692212:31
*** baoli has quit IRC12:32
openstackgerritHugo Nicodemos proposed openstack/ironic master: Migrate python-oneviewclient validations to Ironic OneView drivers  https://review.openstack.org/46842812:33
*** baoli has joined #openstack-ironic12:37
openstackgerritMerged openstack/ironic-python-agent master: Fix post upload images task  https://review.openstack.org/51548412:45
*** vsaienk0 has quit IRC12:46
openstackgerritHugo Nicodemos proposed openstack/ironic master: Get a new OneView client when needed  https://review.openstack.org/51064412:50
*** edleafe is now known as figleaf13:02
*** jaypipes is now known as leakypipes13:02
*** catintheroof has joined #openstack-ironic13:04
*** udesale has quit IRC13:08
*** baoli has quit IRC13:09
sambettsdtantsur: yeah release-openstack-server is missing from the IPA config in projects.yaml...13:12
dtantsursambetts: which, if added, will stop publishing to pypi13:12
dtantsurit's seems that our simplest solution is to just the releases repo (see #openstack-release scrollback)13:12
sambettsdtantsur: do we need to change our release-type then?13:12
dtantsur14:39 <dhellmann> dtantsur : if you add include_pypi_link = yes it should fix things13:13
*** efried is now known as fried_rice13:14
*** vsaienk0 has joined #openstack-ironic13:18
openstackgerritVasyl Saienko proposed openstack/ironic master: Add test to check local boot with partitioned image  https://review.openstack.org/50288613:19
*** fdegir has quit IRC13:19
*** fdegir has joined #openstack-ironic13:19
*** cdearborn has joined #openstack-ironic13:20
*** dansmith is now known as superdan13:30
*** catintheroof has quit IRC13:34
*** Goneri has quit IRC13:35
openstackgerritMerged openstack/ironic-lib master: Move openstack-tox-cover job to project config  https://review.openstack.org/51564113:38
* dtantsur mentions bad karma of milan on #tripleo :D13:39
*** Goneri has joined #openstack-ironic13:41
TheJuliaGood morning everyone13:42
dtantsurmorning TheJulia13:42
*** dyasny has joined #openstack-ironic13:42
*** catintheroof has joined #openstack-ironic13:42
TheJuliaNobodyCam: If you have some free braincells today, there are a few relatively quick bifrost reviews13:43
dtantsurbraaain, BRAAAAIIIINN13:44
* Tengu sends some coffee to dtantsur 13:45
dtantsurthanks :) today no coffee is enough13:45
TheJuliano coffee is enough?13:46
TheJuliaconfusing!?!?13:46
* TheJulia fires up the espresso machine13:46
dtantsurmy cold keeps progressing :( sore throat and confused brain13:47
*** MattMan has quit IRC13:47
*** MattMan has joined #openstack-ironic13:47
*** spectr has quit IRC13:49
* dtantsur tries to understand why exactly one (!) ironic release hit pypi13:50
* Tengu passes some sweet tea with honey and some spices to dtantsur 13:50
dtantsurdoing exactly this right now :)13:50
Tengu:)13:51
milandtantsur, hey whaaat? :D13:51
dtantsurhahaha13:52
milanmorning sambetts, Tengu, TheJulia! :)13:52
dtantsurmilan: the patch introducing ironic-inspector in docker into THT cannot merge for many weeks already. no matter what CI condition is, it always hits some transient error13:52
dtantsurwe blame your karma :D13:52
milantransient :D13:52
milanlol13:52
milandefinitely my karma!13:52
dtantsurETOOMANYDECORATORS13:53
*** rbudden has joined #openstack-ironic13:53
milanmy karma is all about being transient :d13:53
milan:D13:53
TheJuliadtantsur: go get some rest13:53
milanlol13:53
dtantsurTheJulia: does going to IKEA count?13:53
milannooooo]13:54
milanikea == too many decorators!13:54
TheJuliadtantsur: that is like 2xNOT_REST13:54
dtantsurI'd really love some rest, but 1. I just got the keys for the new flat, 2. I need to figure out WTF is with our releases13:54
openstackgerritVladyslav Drok proposed openstack/ironic-lib stable/pike: Move legacy ironic-lib jobs in-tree  https://review.openstack.org/51571613:54
milanEWAYTOOMANYDECORATORS.raise(dtansur)13:54
dtantsuron the bright side, Weekend Is Coming! (+ Tue is a holiday)13:54
openstackgerritVladyslav Drok proposed openstack/ironic-lib stable/pike: Move legacy ironic-lib jobs in-tree  https://review.openstack.org/51571613:54
milanHappy Fenstertag, dtantsur!13:55
dtantsurmilan: except TooManyDecoratorsErrors: dtantsur.apply_vote(-2)13:55
dtantsurheh13:55
milan:D13:55
*** serlex has left #openstack-ironic13:55
* milan hopes it's still a thing, the Fenstertag13:55
TheJuliadtantsur: what seems to be the issue with the release?13:55
milandtantsur, then watch out Ikea, cause there's a lot of decorative stuff per square meter :P13:56
dtantsurTheJulia: #1. ironic 6.3.0 (newton) somehow got published to PyPI, we dunno why; #2. IPA release requests fail validation: they don't like the combination of our release type and the fact that we publish it to PyPI13:56
dtantsur(I've found #1 while debugging #2 actually)13:56
dtantsurTheJulia: see scrollback on #openstack-release13:56
* TheJulia blinks13:57
*** rbudden has quit IRC14:00
*** baoli has joined #openstack-ironic14:01
Tengu'afternoon milan ;)14:02
dtantsurmilan: don't worry, I'll come home and -2 a few of your patches to calm down =^_^=14:03
TheJuliadtantsur: I'm fairly sure it has happened before only by accident, and someone had to get pypi folks to clean it up14:03
*** Dinesh_Bhor has quit IRC14:04
milanlol very nice :D14:04
dtantsurTheJulia: well, 6.3.0 is a recent release, it was only done a few days ago14:04
dtantsur(final release for newton)14:04
TheJuliathere was an ironic 0.0 record from april 2016 with no software \o/14:04
*** mjturek has joined #openstack-ironic14:04
*** d0ugal_ has joined #openstack-ironic14:04
dtantsurI'm afraid we'll start publishing stuff to pypi now14:04
dtantsuror not? who knows :)14:04
dtantsure.g. we publish ironic-inspector and IPA there historically14:04
*** d0ugal has quit IRC14:05
TheJulialooks like the ipa post also needs to be nuked14:06
TheJuliaonly one version, and I remember it being discussed long ago. wayback machine shows no record of the page in the past14:06
openstackgerritDmitry Tantsur proposed openstack/ironic master: [devstack] stop using properties  https://review.openstack.org/51545214:07
dtantsurCollecting ironic-python-agent==99914:07
dtantsur  Could not find a version that satisfies the requirement ironic-python-agent==999 (from versions: 1.0.0, 1.0.1, 1.0.2, 1.0.3, 1.0.4, 1.0.5, 1.1.0, 1.2.0, 1.2.1, 1.2.2, 1.2.3, 1.3.0, 1.4.0, 1.5.0, 1.5.1, 1.5.2, 2.0.0, 2.0.1, 2.1.0, 2.2.0, 2.2.1, 3.0.0)14:07
dtantsurTheJulia: this is how I check pypi :)14:07
TheJuliainteresting...14:08
TheJuliathe webui is acting like there has only ever been one version :(14:08
dtantsuryep. I remember previously it was possible to find old versions14:09
*** spectr has joined #openstack-ironic14:09
TheJuliaThe description of False is amusing as well14:09
dtantsur\o/14:10
dtantsurwhat is truth in our world anyway?14:10
dtantsurthat's too ironic to even mention it!14:10
* dtantsur converts ginger tea to puns currently14:10
openstackgerritYushiro FURUKAWA proposed openstack/ironic-specs master: Support a new hardware type for PRIMEQUEST MMB  https://review.openstack.org/51571714:11
TheJuliadtantsur: that takes skill14:12
sambettsPyPi like to hide releases from the UI, and as the owner of the package you can hide all the releases if you want, but it only hides it from the UI you can't hide it from pip14:13
sambettsdiscovered this when trying to hide old date based releases of networking-cisco14:13
TheJuliafun!14:14
sambettsI can make them vanish in the GUI, but pip will still install 2015.x.x ...14:14
*** exodusftw has quit IRC14:19
*** ElCoyote_ has joined #openstack-ironic14:24
*** baoli has quit IRC14:29
*** exodusftw has joined #openstack-ironic14:30
openstackgerritfengzhr proposed openstack/ironic-inspector master: Use database to store introspection_data  https://review.openstack.org/51455214:31
*** vsaienk0 has quit IRC14:31
milandtantsur, ginger tea? whaat? In the city of Barn? ;)14:31
milanbut I agree ironic.release == True would just be too lame ;)14:31
dtantsurmilan: I mean, the proper one, not a cold bottle of hell-knows-what :)14:32
milans/release/description :D14:32
milanheh :)14:32
*** baoli has joined #openstack-ironic14:35
*** rbudden has joined #openstack-ironic14:37
*** vsaienk0 has joined #openstack-ironic14:38
*** spectr has quit IRC14:39
*** baoli has quit IRC14:42
*** fellypefca has quit IRC14:42
*** sasha2 has quit IRC14:43
*** sasha2 has joined #openstack-ironic14:43
*** e0ne has quit IRC14:45
*** baoli has joined #openstack-ironic14:45
*** baoli has quit IRC14:48
*** sasha2 has quit IRC14:49
*** sasha2 has joined #openstack-ironic14:49
*** e0ne has joined #openstack-ironic14:53
*** e0ne has quit IRC14:53
*** e0ne has joined #openstack-ironic14:54
milanfolks, https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L2452 actually there is a timeout protection on the inspecting state, correct?14:54
*** e0ne has quit IRC14:54
*** e0ne has joined #openstack-ironic14:55
*** e0ne has quit IRC14:56
*** amarao has quit IRC14:56
*** e0ne has joined #openstack-ironic14:56
*** e0ne has quit IRC14:57
*** amarao1 has quit IRC14:57
TheJuliamy memory says yes15:01
* TheJulia looks15:01
*** ijw has joined #openstack-ironic15:01
*** sasha2 has quit IRC15:01
*** ijw_ has joined #openstack-ironic15:02
TheJuliayeah, and that is done consistently with the existing timeouts15:03
* milan suspicious https://review.openstack.org/#/c/515657/ sucks a bit more actually :D15:03
patchbotpatch 515657 - ironic-specs - [WIP] Introduce inspect wait state15:03
milanTheJulia, thx15:04
* milan needs to rethink this wait state15:04
TheJuliamilan: yeah, you need to be microversion guarded15:04
TheJuliawoot!15:04
*** ijw has quit IRC15:06
*** baoli has joined #openstack-ironic15:07
TheJuliamilan: or it could make it simpler, as long as the reply is guarded and i think we have a method that gets called for data updates before being sent to the api client15:07
milanhmm yup I also have to work on the reasoning why we need it then, I guess15:08
rpiosoGood morning, ironicers.  Happy Friday!15:09
TheJuliamilan: interlocking15:09
TheJuliamilan: consistency15:09
TheJulialess magic, moar consistency == good!15:10
milanTheJulia, makes sense15:10
milanplus the merger might be simpler later on I guess15:10
vsaienk0sambetts: dtantsur aarefiev https://review.openstack.org/#/c/513294 - move jobs to ironic-inspector tree15:10
patchbotpatch 513294 - ironic-inspector - Add zuul3 jobs in-tree15:10
openstackgerritYushiro FURUKAWA proposed openstack/ironic-specs master: Support a new hardware type for PRIMEQUEST MMB  https://review.openstack.org/51571715:10
*** SerenaFeng has joined #openstack-ironic15:12
*** AlexeyAbashkin has quit IRC15:13
*** SerenaFeng has quit IRC15:17
TheJuliaIf anyone wants to get a last review in on 487410, it has 2x+215:18
*** SerenaFeng has joined #openstack-ironic15:18
*** links has joined #openstack-ironic15:19
openstackgerritJulia Kreger proposed openstack/bifrost master: Move legacy bifrost jobs in-tree  https://review.openstack.org/51551415:19
*** SerenaFeng has quit IRC15:21
jlvillalGood morning Ironic15:22
TheJuliagood morning jlvillal15:22
jlvillal:)15:23
*** sasha2 has joined #openstack-ironic15:25
*** ricardoas has joined #openstack-ironic15:36
ricardoashi ironicers!15:37
*** Goneri has quit IRC15:39
TheJuliagood morning ricardoas15:40
ricardoasmorning TheJulia15:41
dtantsurvsaienk0: will add to my review queue, but not for today15:42
dtantsurmorning jlvillal, rpioso15:42
jlvillal:)15:43
ricardoasHey... I need to pass a json content to the oneview driver when launching an instance, can drivers access to customization script content (maybe on ´task´ object?) or is it only intended to be used by cloud init? I´m a bit confused because the content appears to be encoded...15:47
TheJulianicodemos: A few minor things on 468428 :)15:47
TheJuliaricardoas: I think we need a little more detail of exactly what your trying to achieve15:48
TheJuliaricardoas: you mean the config drive metadata?15:48
rpiosodtantsur: Morning.  Sorry to learn you're still not feeling well :(15:50
ricardoasTheJulia: i´m trying to use oneview with image streamer... at least for a poc15:50
openstackgerritJohn L. Villalovos proposed openstack/ironic-python-agent master: Detect when log messages are mis-formatted  https://review.openstack.org/51106615:50
ricardoasTheJulia: i mapped deployment plans to glance images so when the user selects this image (with an special property with oneview´s deployment plan uuid) it will do something similar to bfv15:51
*** derekh has quit IRC15:52
ricardoasTheJulia: the thing is that these delpoyment plans can have custom attributes, and they´re all used before the machine is powered on, when we allocate the machine so cloud-init is not an option in this scenario15:52
ricardoasTheJulia: yeah... i though of put this custom attributed (it´s a json) on the config drive metadata15:53
ricardoasTheJulia: but i have the feeling i´m doing something ¨wrong¨ :(15:54
*** Goneri has joined #openstack-ironic15:55
rpiosoI'm working on a presentation for Sydney.  My proposal was accepted in the second wave as a 10 minute long session.  The organizers have classified it as a project update.  It'll cover ironic and Redfish.  The current description is at https://www.openstack.org/summit/sydney-2017/summit-schedule/events/20039/baremetal-server-management-like-shooting-redfish-in-a-barrel15:55
*** vsaienk0 has quit IRC15:57
rpiosoIs there anything folks would like me to present?15:57
*** SerenaFeng has joined #openstack-ironic15:57
*** SerenaFeng has quit IRC15:58
*** parlos has quit IRC15:58
*** rama_y has joined #openstack-ironic16:03
openstackgerritMerged openstack/ironic master: Reference architecture: common bits  https://review.openstack.org/48741016:04
*** fried_rice has quit IRC16:05
*** lucasagomes is now known as lucas-afk16:13
openstackgerritJohn L. Villalovos proposed openstack/ironic-python-agent master: Detect when log messages are mis-formatted  https://review.openstack.org/51106616:18
openstackgerritJohn L. Villalovos proposed openstack/ironic-python-agent master: Fix a mis-formatted log message  https://review.openstack.org/51576216:18
*** fried_rice has joined #openstack-ironic16:18
*** ijw_ has quit IRC16:18
*** robcresswell has joined #openstack-ironic16:19
*** e0ne has joined #openstack-ironic16:21
*** catintheroof has quit IRC16:23
*** catintheroof has joined #openstack-ironic16:27
*** ccamacho has quit IRC16:29
*** Goneri has quit IRC16:34
dtantsurw00000000000000000000000000000t!!16:34
dtantsur(re ref arch first bit merging)16:35
*** catintheroof has quit IRC16:35
*** ccamacho has joined #openstack-ironic16:35
dtantsurrpioso: please sync with TheJulia re project updates strategy16:36
dtantsurand have a good weekend16:36
*** dtantsur is now known as dtantsur|afk16:36
*** catintheroof has joined #openstack-ironic16:36
rpiosodtantsur|afk: Ack!  You, too :)16:36
*** links has quit IRC16:38
*** catintheroof has quit IRC16:39
*** catintheroof has joined #openstack-ironic16:39
* sambetts --> home16:41
sambettsnight all16:41
*** sambetts is now known as sambetts|afk16:41
*** ccamacho has quit IRC16:41
TheJuliaokay, sorry folks, my bank called me and wanted to have a chat16:50
* TheJulia rewinds brain16:50
TheJuliaricardoas: so is it something in the OS that needs that metadata? It seems like the deployed machine should be unaware of the "deployment plan" or method, so if it is driver side for ironic, we have instance_info, but I guess I'm still lacking an understanding of the mechanics your tryign to fit in to, I just know it is the image streamer that makes magic happen :)16:52
*** AlexeyAbashkin has joined #openstack-ironic16:53
*** trown is now known as trown|lunch16:54
*** mrch has quit IRC16:56
TheJuliarpioso: so I have the project update slot that is 40 minutes long https://www.openstack.org/summit/sydney-2017/summit-schedule/events/19966/ironic-project-update but we'll be covering things at a fairly high level, with the exception of diving into bfv. It is later that day, if you want to recommend folks that might have more generic questions about ironic, that they could attend the longer session at the16:56
TheJuliaend of the day.16:56
*** AlexeyAbashkin has quit IRC16:57
TheJuliaadditionally, if there is anything that you want to mention at a high level, or that you want a callback to, happy to work it in16:57
TheJuliaalso, send me your gmail address and I'll share with you the in-progress deck16:58
*** ralonsoh has quit IRC17:00
*** fragatina has quit IRC17:03
*** dyasny has quit IRC17:14
*** racedo_ has joined #openstack-ironic17:15
*** Goneri has joined #openstack-ironic17:17
*** racedo has quit IRC17:17
rpiosoTheJulia: Thank you.  I'm going to head to lunch.  How about we continue this outside of the channel after I return?17:17
*** rpioso is now known as rpioso|eat17:17
*** ianychoi_ has quit IRC17:17
*** fragatina has joined #openstack-ironic17:21
openstackgerritAnton Arefiev proposed openstack/ironic-inspector master: Centralize config options  https://review.openstack.org/51578617:21
openstackgerritAnton Arefiev proposed openstack/ironic-inspector master: Add keystoneauth adapters  https://review.openstack.org/51578717:21
*** aarefiev is now known as aarefiev_afk17:21
TheJuliarpioso|eat: sure17:39
*** dsneddon has joined #openstack-ironic17:39
TheJuliaNobodyCam: you around today?17:43
NobodyCamHi TheJulia I will after 1... sorry supper bust this morning17:43
NobodyCambusy even17:44
TheJuliano worries17:44
NobodyCamI will take a look at the reviews after my meeting is over17:44
NobodyCamoh and Good Morning Ironic'ers17:44
ricardoasTheJulia: the user will - at instance lauch time - give a (property,value) pair; oneview driver wil set this pair on the server profile which will be applied to the available server (during prepare phase); the image streamer makes magic happen and then after that the server will finally be powered on with a configured SO17:45
ricardoasTheJulia: my problem is to figure out how to make this pair go from end-user to the (poc) oneview deploy interface17:47
TheJuliawhat is the property/value that is to be passed in?17:48
ricardoasTheJulia: anything configurable in the SO uploaded to image streamer... username, password, nic cards, dns...17:49
*** penick has joined #openstack-ironic17:50
TheJuliaso.. sounds like the streamer is going to want the configuration drive17:50
TheJuliaif memory serves, it is an iso that has been gzip'ed and base64 encoded.17:51
TheJuliaricardoas: hopefully that helps with some of the confusion?17:53
ricardoasTheJulia: definitely! I´ll take a look on the config driver then... thank you very much!17:54
* TheJulia ponders going and finding lunch17:59
*** rpioso|eat is now known as rpioso18:02
TheJuliawb rpioso18:04
*** harlowja has quit IRC18:05
*** harlowja has joined #openstack-ironic18:05
*** e0ne has quit IRC18:15
*** rama_y has quit IRC18:22
*** dyasny has joined #openstack-ironic18:22
openstackgerritMilan Kováčik proposed openstack/ironic-specs master: [WIP] Introduce inspect wait state  https://review.openstack.org/51565718:24
milanmorning NobodyCam, TGIF! :D18:25
NobodyCamhey hey good morning milan :)18:27
NobodyCamand oh ya...18:27
NobodyCamdoing prep got a big meeting18:27
milan:D18:27
milanhave a good one, NobodyCam! :)18:27
NobodyCamTY :)18:27
*** vsaienk0 has joined #openstack-ironic18:28
*** rama_y has joined #openstack-ironic18:28
jesusaurI'm having some trouble with bifrost and static dhcp, dnsmasq tells me the dhcp-range is invalid18:31
milan w/r https://review.openstack.org/515657 aarefiev_afk, sambetts|afk, dtantsur|afk, (missing rloo ;) I've realised sooner today that the introspection is actually guarded by a timeout already, as TheJulia confirmed. I've also implemented the suggestions from stendulker. Would <3 some early feedback (before I add all the missing sections)... Thx!18:32
patchbotpatch 515657 - ironic-specs - [WIP] Introduce inspect wait state18:32
jesusaurare there any examples out there of using bifrost with static dhcp?18:32
milanjesusaur, wdym by static dhcp?18:33
jesusaurI want each host to always get the same ip address based on its mac address18:34
jesusaurreading dnsmasq's man page, the dhcp range should either have an end address or the static keyword, but not both18:35
milanhmm18:35
jesusaurbut setting `dhcp_inventory: true` in bifrost gives both an end address and the static keyword in the dhcp-range18:35
jesusaurwhoops, I meant `inventory_dhcp: true`18:36
* milan unfamiliar with bifrost but sound a bit like a bug18:37
milanjesusaur, mind reporting? Worst case it's gonna be closed ;]18:37
jesusaursure, should I use launchpad or storyboard?18:40
jesusaurlooks like launchpad, I don't see bifrost in storyboard18:41
milanyeah just wanted to say https://bugs.launchpad.net/bifrost18:43
*** vsaienk0 has quit IRC18:43
milanjesusaur, thanks for reporting!18:43
*** trown|lunch is now known as trown18:44
zigohttp://paste.openstack.org/show/624848/ <--- Proliantutils has a unit test failure with Python 3.6.18:44
zigoWhere should I bug-report it?18:44
zigoLaunchpad ?18:45
milanzigo, https://bugs.launchpad.net/ironic I guess18:46
milanzigo, there seems to be some iLo bugs already, so hopefully I'm correct ;)18:47
*** fragatina has quit IRC18:48
*** dprince has quit IRC18:48
zigomilan: It seems there's a proliantutils project in Launchpad too...18:48
milanaha18:48
milanzigo, most important is to report, component can be always changed ;)18:48
milanzigo, too, thanks for reporting! ;)18:49
zigohttps://bugs.launchpad.net/proliantutils/+bug/172812618:49
openstackLaunchpad bug 1728126 in proliantutils "Fails test under python 3.6" [Undecided,New]18:49
milaninteresting, seems like some mock leaked... sometimes these issues happen when tests are (involuntarily) interdependent18:55
milanand with that news I'm calling it a day! :)18:55
milan#pixiesay -mflexing Night Ironic, sleep well!18:56
PixieBootsᕙʕ⇀ᴥ⇀ʔᕗ: Night Ironic, sleep well!18:56
*** milan has quit IRC18:56
TheJulia:)19:06
TheJuliajesusaur: we have a couple job that should be using it based off of inventory DHCP, but... Hmmm19:07
TheJuliazigo: the project is correct for the bug19:08
TheJuliajesusaur: sounds like an issue with the template :\ are you looking at the leases definition? Or just dnsmasq.conf?19:11
*** dprince has joined #openstack-ironic19:13
*** athomas has quit IRC19:14
*** mgoddard has quit IRC19:22
*** ChubYann has joined #openstack-ironic19:23
*** Goneri has quit IRC19:25
*** ChubYann has left #openstack-ironic19:28
*** catintheroof has quit IRC19:29
*** e0ne has joined #openstack-ironic19:29
*** e0ne has quit IRC19:30
openstackgerritDmitry Tantsur proposed openstack/ironic master: [devstack] stop using properties  https://review.openstack.org/51545219:31
*** tesseract has quit IRC19:33
*** catintheroof has joined #openstack-ironic19:41
jlvillaldtantsur|afk: TheJulia: https://review.openstack.org/#/c/513697/  Is needed to get the stable/ocata branch working again.19:42
patchbotpatch 513697 - ironic (stable/ocata) - Move ironic legacy jobs into the ironic tree19:42
jlvillalNothing urgent pressing at the moment, that I know of.19:43
*** catintheroof has quit IRC19:45
*** rcernin has joined #openstack-ironic19:57
*** mat128 has quit IRC20:06
jesusaurTheJulia: just dnsmasq.conf20:10
*** ijw has joined #openstack-ironic20:11
*** penick has quit IRC20:11
*** ijw has quit IRC20:16
zigoTheJulia: What I found out is that ironic build-conflicts with something. I'm still looking for what this something is though ...20:16
zigoTheJulia: It should be easy to find out, I've saved the list of python packages that were installed on my system when building.20:17
*** dprince has quit IRC20:27
*** sasha2 has quit IRC20:30
*** ijw has joined #openstack-ironic20:35
*** ijw has quit IRC20:35
*** ijw has joined #openstack-ironic20:36
*** sasha2 has joined #openstack-ironic20:47
*** lindycoder has quit IRC20:53
TheJuliajesusaur: so, I feel like there is a static lease override facility that we use20:54
TheJuliazigo: from what I could tell, it was oslo.db in that what was getting pulled in was too old, but you might already have your head wrapped around that.20:55
TheJuliajlvillal: +2'ed20:57
*** fragatina has joined #openstack-ironic21:04
*** trown is now known as trown|outtypewww21:06
openstackgerritRamamani Yeleswarapu proposed openstack/ironic master: Documentation update for booting from volume  https://review.openstack.org/50374421:10
*** rama_y_ has joined #openstack-ironic21:13
*** rama_y has quit IRC21:13
*** leakypipes has quit IRC21:23
jlvillalTheJulia: Thanks :)21:25
openstackgerritMerged openstack/ironic master: ironic-dbsync: check object versions  https://review.openstack.org/49770321:25
*** penick has joined #openstack-ironic21:25
*** rcernin has quit IRC21:28
*** shoaib has joined #openstack-ironic21:31
shoaibhi, not sure if anyone is still here, i see an IPMI error when trying to enroll a node. Manually issuing an ipmitool command works fine, but ironic uses this command "Command: ipmitool -I lanplus -H http://128.224.64.212 -L ADMINISTRATOR -U root -R 12 -N 5 -f /tmp/tmpq7l1qW power status"21:32
shoaibThe -f <file> path doesnt even exist... Just wondering if i need to configure something in ironic not to do that ? Thanks21:33
TheJuliashoaib: ironic uses a temporary file to pass in the password for ipmitool, so we don't log it21:35
shoaibActually i just looked at it again, the error in ironic-conductor.log, is this " Stderr: u'Address lookup for http://128.224.64.212 failed\nCould not open socket!\n' ", but manually trying to do " ipmitool -I lanplus -H 128.224.64.212 -U root -P root power status", works just fine21:39
TheJuliaheh21:39
TheJuliayeah, so that should just be an IP address, not a URL21:39
shoaibi used this command " ironic --ironic-api-version latest node-create -d pxe_ipmitool -i ipmi_address=http://128.224.64.212 "21:40
TheJuliashoaib: what version of ironic-api/ironic-conductor are you using?21:40
shoaibhow do i check that ? i was just using 'latest' against every command21:40
shoaibmy client is  " python-ironic-inspector-client-1.10.0-1. "21:41
shoaibsorry, i meant " python2-ironicclient-1.7.1-0. ", dont even use ironic-inspector21:42
*** baoli has quit IRC21:42
*** rama_y_ has quit IRC21:44
TheJuliayou have to do it on the conductor21:46
TheJuliaif you just installed it from master, then you might want to look at our documentation regarding hardware tpes, since pxe_ipmitool is a classic driver21:47
TheJuliayour command should really be "ironic --ironic-api-version latest node-create -d pxe_ipmitool -i ipmi_address=128.224.64.21221:47
TheJulia"21:47
TheJuliaand naturally, you'll want to ensure you have the ipmi_username and ipmi_password set21:48
shoaibok Thanks Julia, resolved now, changing the url to just an ip-address fixed the problem... Have a good weekend :)21:48
TheJuliano problem, have a wonderful weekend!21:48
*** shoaib has quit IRC21:49
*** figleaf is now known as edleafe22:01
NobodyCamokay meeting done!22:03
*** ijw has quit IRC22:05
*** ijw has joined #openstack-ironic22:05
TheJuliawoot!22:12
NobodyCamTheJulia: looking at 51551422:13
TheJuliathanks22:14
*** jtomasek has quit IRC22:14
NobodyCamlots of the jobs have the name: name: Autoconverted job legacy- BLAH.22:14
TheJuliayeah, I din't want to go change much around on just doing the move over22:15
NobodyCamokay, maybe I'll toss up a patch to remove the `Autoconverted job` bits so its just the actual job name22:16
NobodyCamwon't have time today. but next week I should22:16
TheJuliaokay, the whole long term idea is re-do as just native roles22:17
TheJuliawhich... actually has appeal since the main test is actually just a playbook now22:17
TheJuliaNobodyCam: there is another rev also up to fix some syntax issues for ansible 2.4 compatability22:18
NobodyCamokay. was that the bracket quote one?22:19
NobodyCamlol I always find my self having to think about the job names: bifrost-integration-dibipa-debian-centos-722:21
NobodyCamdib ipa on debian centos 7 system?22:21
*** karimb has quit IRC22:25
*** cdearborn has quit IRC22:34
openstackgerritMerged openstack/bifrost master: Unquote empty braces inside jinja blocks  https://review.openstack.org/51554622:35
TheJulia NobodyCam bingo22:37
*** mjturek has quit IRC22:44
*** jcoufal has quit IRC22:44
openstackgerritJulia Kreger proposed openstack/bifrost master: Clean up jinja2 templates on conditional checks  https://review.openstack.org/51556822:48
*** ijw has quit IRC22:48
*** ijw has joined #openstack-ironic22:49
openstackgerritMerged openstack/bifrost master: Move legacy bifrost jobs in-tree  https://review.openstack.org/51551422:49
openstackgerritJulia Kreger proposed openstack/bifrost master: Clean up jinja2 templates on conditional checks  https://review.openstack.org/51556822:53
*** karimb has joined #openstack-ironic23:19
*** fried_rice is now known as efried23:22
*** penick has quit IRC23:25
*** srart has quit IRC23:35
*** karimb has quit IRC23:37
*** oanson has quit IRC23:54
*** oanson has joined #openstack-ironic23:55
*** ianychoi has joined #openstack-ironic23:56

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