Wednesday, 2017-06-21

*** chlong has quit IRC00:03
*** Nisha_ has joined #openstack-ironic00:07
*** Nisha_Agarwal has quit IRC00:07
*** ijw has joined #openstack-ironic00:07
*** makowals has quit IRC00:10
*** ijw has quit IRC00:12
*** ijw has joined #openstack-ironic00:12
*** amotoki_away is now known as amotoki00:23
*** makowals has joined #openstack-ironic00:27
*** retr0h has quit IRC00:37
*** retr0h has joined #openstack-ironic00:37
*** Nisha_ has quit IRC00:39
*** makowals has quit IRC00:47
*** ijw has quit IRC00:54
*** ijw has joined #openstack-ironic00:57
*** rloo has quit IRC00:57
*** makowals has joined #openstack-ironic01:11
*** SerenaFeng has joined #openstack-ironic01:15
*** shuyingya has joined #openstack-ironic01:18
*** ijw has quit IRC01:25
openstackgerritHironori Shiina proposed openstack/ironic master: DevStack: Add configuration for boot-from-volume  https://review.openstack.org/46633301:35
*** exodusftw has quit IRC02:00
*** adisky__ has quit IRC02:03
*** exodusftw has joined #openstack-ironic02:05
*** zhugaoxiao has joined #openstack-ironic02:13
*** baoli has joined #openstack-ironic02:38
*** makowals has quit IRC02:40
*** baoli has quit IRC02:56
openstackgerritMerged openstack/ironic master: Move _abort_attach_volumes functionality to detach_volumes  https://review.openstack.org/46776603:02
*** SerenaFeng has quit IRC03:05
*** mikal_ is now known as mikal03:15
*** baoli has joined #openstack-ironic03:16
*** tuanluong has joined #openstack-ironic03:18
*** makowals has joined #openstack-ironic03:19
*** baoli has quit IRC03:21
*** SerenaFeng has joined #openstack-ironic03:21
*** udesale has joined #openstack-ironic03:40
*** SerenaFeng has quit IRC04:02
*** links has joined #openstack-ironic04:07
*** makowals has quit IRC04:14
*** vmud213 has joined #openstack-ironic04:18
*** makowals has joined #openstack-ironic04:22
openstackgerritVu Cong Tuan proposed openstack/ironic master: Replace the usage of 'manager' with 'os_primary'  https://review.openstack.org/47599604:29
*** adisky__ has joined #openstack-ironic04:40
*** jerrygb has joined #openstack-ironic04:56
*** mvpnitesh has joined #openstack-ironic05:00
*** jerrygb has quit IRC05:01
*** makowals has quit IRC05:03
*** links has quit IRC05:07
*** adreznec has quit IRC05:08
*** bnemec has quit IRC05:08
*** makowals has joined #openstack-ironic05:09
*** links has joined #openstack-ironic05:23
*** SerenaFeng has joined #openstack-ironic05:25
*** ChubYann has quit IRC05:27
*** mvpnitesh has quit IRC05:40
*** e0ne has joined #openstack-ironic05:43
*** bnemec has joined #openstack-ironic05:46
*** e0ne has quit IRC05:47
*** mjura has joined #openstack-ironic05:52
*** mvpnitesh has joined #openstack-ironic06:00
*** adreznec has joined #openstack-ironic06:00
*** mjura has quit IRC06:01
*** rama_y has joined #openstack-ironic06:11
*** mjura has joined #openstack-ironic06:14
mkraiHi Ironic team. I have a shellinaboxd process running to server console of my baremetal instance.06:16
mkraiIs there any API to stop this console?06:16
*** jaganathan has joined #openstack-ironic06:28
*** jtomasek_ has joined #openstack-ironic06:28
*** mvpnitesh has quit IRC06:31
*** pcaruana has joined #openstack-ironic06:31
openstackgerritMadhuri Kumari proposed openstack/ironic master: [Trivial] Update install-guide to be consistent  https://review.openstack.org/47601306:33
*** mjura has quit IRC06:35
*** jerrygb has joined #openstack-ironic06:36
*** jtomasek_ has quit IRC06:45
openstackgerritparesh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get one time boot  https://review.openstack.org/47409306:45
*** mjura has joined #openstack-ironic06:47
openstackgerritVu Cong Tuan proposed openstack/ironic master: Replace the usage of 'manager' with 'os_primary'  https://review.openstack.org/47599606:52
*** rcernin has joined #openstack-ironic06:59
*** b00tcat has joined #openstack-ironic07:02
*** thaj has joined #openstack-ironic07:04
*** SerenaFeng has quit IRC07:06
openstackgerritparesh sao proposed openstack/proliantutils master: Adds 'get_pending_boot_mode' in ILO redfish  https://review.openstack.org/46798207:09
openstackgerritparesh sao proposed openstack/proliantutils master: Adds 'update_persistent_boot' in ILO redfish  https://review.openstack.org/47529307:09
openstackgerritparesh sao proposed openstack/proliantutils master: Adds 'get_persistent_boot_device' in ILO redfish  https://review.openstack.org/47526807:09
openstackgerritparesh sao proposed openstack/proliantutils master: Adds 'set_one_time_boot' in ILO redfish  https://review.openstack.org/47529507:10
*** SerenaFeng has joined #openstack-ironic07:12
*** tesseract has joined #openstack-ironic07:15
*** ralonsoh has joined #openstack-ironic07:15
pas-hamorning ironic :)07:20
openstackgerritparesh sao proposed openstack/ironic master: Added a condition for ILO for pxe hardware type  https://review.openstack.org/47534207:21
*** SerenaFeng has quit IRC07:22
*** athomas has joined #openstack-ironic07:24
openstackgerritNisha Agarwal proposed openstack/sushy master: Add local_gb discovery to sushy  https://review.openstack.org/45511407:27
*** links has quit IRC07:31
*** jerrygb has quit IRC07:34
*** ccamacho has joined #openstack-ironic07:34
*** links has joined #openstack-ironic07:44
*** SerenaFeng has joined #openstack-ironic07:45
*** udesale__ has joined #openstack-ironic07:54
*** mgoddard has joined #openstack-ironic07:56
*** udesale has quit IRC07:57
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:02
*** hjensas has quit IRC08:02
*** b00tcat has quit IRC08:10
*** b00tcat has joined #openstack-ironic08:10
*** e0ne has joined #openstack-ironic08:10
*** e0ne has quit IRC08:13
*** udesale has joined #openstack-ironic08:19
*** udesale__ has quit IRC08:21
vdrokgood morning ironic, mkrai pas-ha08:25
*** racedo has joined #openstack-ironic08:25
vdrokmkrai: I guess you mean this? https://developer.openstack.org/api-ref/baremetal/?expanded=start-stop-console-detail#start-stop-console08:26
*** e0ne has joined #openstack-ironic08:27
*** milan has joined #openstack-ironic08:30
*** e0ne has quit IRC08:33
*** rcernin has quit IRC08:45
*** lucas-afk is now known as lucasagomes08:47
*** rcernin has joined #openstack-ironic08:51
*** paramite has joined #openstack-ironic08:54
*** dtantsur|afk is now known as dtantsur08:56
dtantsurhi ironic! I'm in a training for the next 4 hours, please expect delays in responses08:57
*** priteau has joined #openstack-ironic08:58
*** hjensas has joined #openstack-ironic08:58
openstackgerritAnshul Jain proposed openstack/ironic master: Add manual clean step ironic standalone test  https://review.openstack.org/42977009:02
openstackgerritparesh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get one time boot  https://review.openstack.org/47409309:09
tuanluongMorning ironic09:09
*** b00tcat_ has joined #openstack-ironic09:13
*** b00tcat has quit IRC09:14
*** SerenaFeng has quit IRC09:17
*** e0ne has joined #openstack-ironic09:18
pas-hamorning vdrok dtantsur tuanluong09:19
tuanluongmorning pas-ha09:20
*** mjura has quit IRC09:21
*** mjura has joined #openstack-ironic09:22
*** SerenaFeng has joined #openstack-ironic09:24
*** bcornec1 has joined #openstack-ironic09:37
*** arnewiebalck has joined #openstack-ironic09:37
openstackgerritAparna proposed openstack/proliantutils master: Change Logs for proliantutils 2.3.0 release  https://review.openstack.org/47606609:39
*** tuanluong_ has joined #openstack-ironic09:47
vdrokmorning dtantsur tuanluong09:47
*** tuanluong has quit IRC09:50
*** e0ne has quit IRC09:50
*** e0ne has joined #openstack-ironic09:52
*** sambetts|afk is now known as sambetts09:55
sambettsMorning all09:55
*** hjensas has quit IRC10:07
aarefievmilan: hey, mind to make sanity check when you have time https://review.openstack.org/#/c/463768/10:14
patchbotpatch 463768 - ironic-inspector - Use declarative reader/writer separation10:14
aarefievas for general idea10:14
milanaarefiev, morning sure!10:14
aarefievthanks10:14
milanmorning Ironic! :)10:14
aarefievmorning milan :)10:14
openstackgerritAnshul Jain proposed openstack/ironic master: Add manual clean step ironic standalone test  https://review.openstack.org/42977010:18
*** mat128 has joined #openstack-ironic10:19
*** mjura has quit IRC10:24
openstackgerritAnshul Jain proposed openstack/ironic master: Add manual clean step ironic standalone test  https://review.openstack.org/42977010:29
*** udesale__ has joined #openstack-ironic10:31
*** fragatina has joined #openstack-ironic10:32
*** udesale has quit IRC10:33
*** fragatina has quit IRC10:34
*** fragatina has joined #openstack-ironic10:34
*** mjura has joined #openstack-ironic10:36
*** SerenaFeng has quit IRC10:36
*** SerenaFeng has joined #openstack-ironic10:38
*** racedo has quit IRC10:45
*** sai has quit IRC10:46
*** rook has quit IRC10:46
*** fragatina has quit IRC10:48
*** jkilpatr has quit IRC10:49
*** SerenaFeng has quit IRC10:50
*** fragatina has joined #openstack-ironic10:51
*** sai has joined #openstack-ironic10:51
*** racedo has joined #openstack-ironic10:51
*** SerenaFeng has joined #openstack-ironic10:51
*** rook has joined #openstack-ironic10:52
*** e0ne has quit IRC10:52
*** rook is now known as Guest4501210:52
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata  https://review.openstack.org/47573910:56
*** mjura has quit IRC10:57
*** udesale has joined #openstack-ironic11:02
*** udesale__ has quit IRC11:03
dtantsurok, I'm done much earlier than expected :)11:05
dtantsurmorning sambetts, aarefiev, milan, vdrok, pas-ha, tuanluong_11:06
milanmorning dtantsur! :)11:06
aarefievmorning dtantsur11:06
*** ccamacho is now known as ccamacho|lunch11:06
tuanluong_mroning dtantsur11:06
vdrokgood morning sambetts aarefiev and milan11:07
milanmorning tuanluong_ vdrok! :)11:07
nicodemosmorning Ironic!11:07
*** jkilpatr has joined #openstack-ironic11:08
vdrokgood morning nicodemos11:08
*** Guest45012 is now known as rook-ugh11:09
*** mjura has joined #openstack-ironic11:09
*** SerenaFeng has quit IRC11:14
*** fellypefca has joined #openstack-ironic11:28
*** mgoddard has quit IRC11:35
*** mvpnitesh has joined #openstack-ironic11:37
*** udesale has quit IRC11:39
openstackgerritAnton Arefiev proposed openstack/ironic-inspector master: Introduce oslo.messaging RPC  https://review.openstack.org/46286111:43
*** Nisha_Agarwal has joined #openstack-ironic11:49
*** amarao has joined #openstack-ironic11:50
*** vmud213 has quit IRC11:50
TheJuliagood morning everyone11:56
*** e0ne has joined #openstack-ironic11:56
openstackgerritAparna proposed openstack/ironic master: Add API and conductor methods for [un]rescue  https://review.openstack.org/35083111:57
openstackgerritAparna proposed openstack/ironic master: Add rescuewait timeout periodic task  https://review.openstack.org/35315611:58
vdrokgood morning TheJulia !11:59
*** hjensas has joined #openstack-ironic12:01
*** hjensas has quit IRC12:01
*** hjensas has joined #openstack-ironic12:01
*** skazi has quit IRC12:01
*** litao__ has quit IRC12:04
*** skazi has joined #openstack-ironic12:04
*** fragatina has quit IRC12:04
*** skazi has quit IRC12:06
*** skazi has joined #openstack-ironic12:07
*** shuyingya has quit IRC12:10
*** shuyingya has joined #openstack-ironic12:11
*** trown|outtypewww is now known as trown12:12
*** bcornec1 has quit IRC12:14
dtantsurmorning TheJulia12:14
TheJuliajlvillal: I replied to 413324, so you may want to take a look once you get some coffee :)12:18
openstackgerritAparna proposed openstack/ironic master: Agent driver rescue implementation  https://review.openstack.org/40043712:19
*** ccamacho|lunch is now known as ccamacho12:20
openstackgerritAparna proposed openstack/ironic master: WIP Rescue mode tempest tests  https://review.openstack.org/45230812:21
openstackgerritJulia Kreger proposed openstack/ironic master: iPXE template support for iSCSI  https://review.openstack.org/41332412:24
openstackgerritJulia Kreger proposed openstack/ironic master: Logic for skipping deployment with BFV  https://review.openstack.org/45424312:25
openstackgerritFellype Cavalcante proposed openstack/ironic master: Adds onetime boot in oneview drivers  https://review.openstack.org/43646912:25
TheJulialooks like the volume connection crud changes now merge conflict with some of the stuff that has landed. Rebased the above BFV patches so I can rebase them.12:26
*** mgoddard has joined #openstack-ironic12:28
*** derekh has joined #openstack-ironic12:29
*** tuanluong_ has quit IRC12:34
*** bfournie has quit IRC12:35
*** dprince has joined #openstack-ironic12:35
*** derekh has quit IRC12:36
*** bcornec1 has joined #openstack-ironic12:39
*** derekh has joined #openstack-ironic12:43
*** derekh has quit IRC12:48
*** derekh has joined #openstack-ironic12:49
*** rook-ugh is now known as rook12:49
*** mvpnitesh has quit IRC12:50
*** mvpnitesh has joined #openstack-ironic12:50
*** sasha2 has joined #openstack-ironic12:55
openstackgerritMerged openstack/ironic-ui master: Imported Translations from Zanata  https://review.openstack.org/47573912:58
*** jerrygb has joined #openstack-ironic13:00
*** goldyfruit has joined #openstack-ironic13:02
*** shuyingya has quit IRC13:02
*** jaganathan has quit IRC13:02
*** jaganathan has joined #openstack-ironic13:06
openstackgerritJulia Kreger proposed openstack/ironic master: Add CRUD notification objects for volume connector and volume target  https://review.openstack.org/46393013:09
openstackgerritJulia Kreger proposed openstack/ironic master: Add REST API for volume connector and volume target operation  https://review.openstack.org/21458613:09
openstackgerritJulia Kreger proposed openstack/ironic master: Add functional API tests for volume connector and volume target  https://review.openstack.org/47371713:10
*** rloo has joined #openstack-ironic13:10
openstackgerritJulia Kreger proposed openstack/ironic master: Enable cinder storage interface for generic hardware  https://review.openstack.org/46390813:10
openstackgerritJulia Kreger proposed openstack/ironic master: Add storage interface field to node-related notifications  https://review.openstack.org/46397213:10
openstackgerritJulia Kreger proposed openstack/ironic master: DevStack: Add configuration for boot-from-volume  https://review.openstack.org/46633313:11
openstackgerritJulia Kreger proposed openstack/ironic master: Add a guide for Devstack configuration for boot-from-volume  https://review.openstack.org/46656813:11
*** baoli has joined #openstack-ironic13:12
TheJuliaclean rebases after 46393013:12
*** lucasagomes is now known as lucas-brb13:13
*** Goneri has joined #openstack-ironic13:14
*** shuyingya has joined #openstack-ironic13:15
*** shuyingya has quit IRC13:15
*** baoli has quit IRC13:16
*** bfournie has joined #openstack-ironic13:16
*** Nisha_Agarwal has quit IRC13:16
*** trown is now known as trown|brb13:16
*** jcoufal has joined #openstack-ironic13:17
*** shuyingya has joined #openstack-ironic13:17
*** trown|brb is now known as trown13:19
*** baoli has joined #openstack-ironic13:20
*** mvpnitesh has quit IRC13:31
*** jaganathan has quit IRC13:34
*** lucas-brb is now known as lucasagomes13:36
fellypefcadtantsur Hey! Yesterday ricardoas and I talked to vdrok about problems that you pointed on onetime patch and showed that the solution were already being made.13:36
fellypefcadtantsur Just to update you on this :) Thanks!13:36
*** mjturek has joined #openstack-ironic13:43
*** shuyingya has quit IRC13:46
*** racedo has quit IRC13:51
*** racedo has joined #openstack-ironic13:51
*** racedo has quit IRC13:52
*** adisky__ has quit IRC13:59
pas-hasooo, it looks like this time has come - glance have moved to apache, and ironic is predictably broken o_O14:01
TheJulia:(14:02
pas-haI have a patch up https://review.openstack.org/#/c/467728/ but currently re-writing/rebasing it, will push a new patch-set in a couple of hours14:02
patchbotpatch 467728 - ironic - Rework keystone auth for glance14:02
TheJuliapas-ha: Thank you, I'll keep an eye out for it14:02
* TheJulia realizes that should be thanks, and just gives up on $words14:03
*** jerrygb has quit IRC14:04
dtantsurpas-ha: I think we need to revert their patch, and politely ask to communicate first14:04
*** crushil has joined #openstack-ironic14:04
*** chlong has joined #openstack-ironic14:07
*** litao__ has joined #openstack-ironic14:13
litao__hi guys, I deployed my devstack and the vms as baremetal started for a while and stopped by itself, who knows why?14:14
litao__so i could not deploy the baremetal  vms14:15
dtantsurvsaienk0: hi! as networking-baremetal is under our governance, could you please move https://launchpad.net/networking-baremetal under ironic-drivers?14:15
vsaienk0dtantsur: yes it is under ironic governance14:16
*** hjensas has quit IRC14:16
vsaienk0dtantsur: done14:17
dtantsurthanks14:17
*** mjura has quit IRC14:24
vdrokdtantsur: 10 emails at once, you did scare me :)14:26
dtantsurhaha14:26
dtantsurI wish we had just one place to define the core team, then everything would just use it..14:27
* TheJulia hasn't heard a specific chatter in a while, wonders if a thing is still a thing... and decides it is best to leave alone for now.14:29
*** ralonsoh has quit IRC14:40
jlvillalTheJulia, mjturek: I replied. Basically my brain gets confused by the unit tests :)14:40
davidlenwellI heard it was somebodies birthday today14:41
mjturekjlvillal: lol no worries14:42
TheJuliajlvillal: no problem :)14:42
TheJuliajlvillal: to be fair, I looked at it for a minute and went "hey, wait a second!"14:42
TheJuliadavidlenwell: is there someone who we should, or should not be sending congratulations to?14:43
davidlenwellTheJulia: my calenar says its NobodyCam s birthday today14:43
TheJuliai feel like a bad friend because I can neither confirm nor deny14:44
davidlenwell:) he doesn’t go telling people14:44
*** mrtenio has joined #openstack-ironic14:44
davidlenwellbut google plus doomed him with my knowledge that stuck with my google calendar long after google plus ate it14:45
davidlenwellthat is unless he just lies on social media.. one can never tell.. it might have been his clever plan all along.14:45
NobodyCamhey hey good morning davidlenwell it is! that and the trip to DMV is why I took PTO today :)14:48
davidlenwellha14:48
NobodyCam:)14:48
davidlenwellhappy birthday.. here is some government bureaucracy for you14:48
davidlenwellhappy birthday.. you’re so old now that we require an eye test before we’ll let you keep your license14:50
TheJuliaNobodyCam: happy birthday then!14:51
NobodyCam:) thank you both :)14:52
davidlenwellI hope you made an appointment14:52
TheJuliajealous of those who can actually make appointments14:57
dtantsurNobodyCam: happy birthday and good morning :)14:58
NobodyCam:) I did :)14:58
NobodyCammorning dtantsur Thank you :)14:59
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Rework keystone auth for glance  https://review.openstack.org/46772815:01
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Introcude keystoneauth adapters for clients  https://review.openstack.org/46962715:01
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for neutronclient  https://review.openstack.org/47617015:01
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for cinderclient  https://review.openstack.org/47617115:01
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for inspectorclient  https://review.openstack.org/47617215:01
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Fix swiftclient creation  https://review.openstack.org/47617315:01
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for cinderclient  https://review.openstack.org/47617115:01
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for neutronclient  https://review.openstack.org/47617015:01
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Rework keystone auth for glance  https://review.openstack.org/46772815:01
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Fix swiftclient creation  https://review.openstack.org/47617315:01
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for inspectorclient  https://review.openstack.org/47617215:01
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Introduce keystoneauth adapters for clients  https://review.openstack.org/46962715:01
*** athomas has quit IRC15:02
pas-hatypo in the first commit title in the first chain... /me facepalms15:02
pas-hahey NobodyCam, happy birthday!15:03
NobodyCamThank you pas-ha :)15:03
milanhappy b-day NobodyCam! :)15:04
TheJuliapas-ha: so the entire stack to fix the issues encountered by glance going to apache?15:04
pas-hanope, just first two15:05
*** jaosorior has quit IRC15:05
NobodyCam:) thank you milan15:05
rloohi and good morning everyone, pas-ha, milan, TheJulia, davidlenwell, dtantsur, and everyone else I may have missed15:05
pas-haI took this opportuity to re-write how we instentiate clients once again, using keystoneauth adapters where possible15:05
milanmorning rloo! :)15:06
rlooand a very happy birthday to NobodyCam! Glad you are taking the day off!!15:06
pas-haso the first patch is adding those to config, and the second re-writes glance15:06
davidlenwellhi rloo15:06
NobodyCammorning rloo :)15:06
pas-hamorning rloo :)15:06
NobodyCamand thank you :)15:06
*** rcernin has quit IRC15:07
rloopas-ha: thx for the patches (I think!) Is our gate broken w/o your fix?15:07
*** ralonsoh has joined #openstack-ironic15:07
rloopas-ha: wondering if we need to update our white board15:07
pas-hayep, glance-api moved to apache apparently, but we still hardcode glance_host:glance_port in config15:10
dtantsurmorning rloo15:10
TheJuliarloo: we will need the devstack gate fix reverted first I think15:11
TheJuliaerr15:11
pas-hamy second patch in the series should fix this, but I haven't tested it yet locally after rebase+some re-writes, so waiting for the jenkins :)15:11
TheJulianot gate, openstack-dev/devstack15:11
rloopas-ha: ok, so gate is broken until your fix(es) or someone gets glance to revert?15:12
pas-hayes15:12
*** pcaruana has quit IRC15:12
*** fellypefca has quit IRC15:13
*** rpioso has joined #openstack-ironic15:15
*** mrtenio has quit IRC15:15
vdrokgood morning rloo and happy birthday NobodyCam :)15:15
*** Sukhdev has joined #openstack-ironic15:16
*** skazi_ has joined #openstack-ironic15:16
rloopas-ha et al. i've updated the whiteboard - please correct/add more info. pas-ha, those patches are err not short :-) Also, would be nice to have a bug or something to associate with them. wondering if other projects are also broken.15:18
*** skazi_ has quit IRC15:20
*** rama_y has quit IRC15:25
*** skazi_ has joined #openstack-ironic15:29
pas-harloo: not so sure about other projects - the code in ironic that had to be fixed are from nova-baremetal times :-/15:30
openstackgerritHieu LE proposed openstack/ironic master: Enable OSProfiler support in Ironic  https://review.openstack.org/35814215:30
rloopas-ha: OH. then maybe not fair to ask glance to revert; they may have announced a deprecation...15:30
rloopas-ha: i am so sad that you had to touch so many files. but i guess it is our fault for not reviewing/addressing it sooner. let me know when it is good to review and i'll try to look.15:31
*** Nisha_Agarwal has joined #openstack-ironic15:32
NobodyCammorning vdrok and Thank you :)15:32
pas-hawell, at least in the first patch the biggest change is config sample :)15:32
rloopas-ha: :)15:33
*** skazi_ has quit IRC15:33
pas-haI'm also proposing to deprecate quite a number of options in those patches, together with support for 'glance://...15:34
pas-haimage url scheme15:34
pas-haso please take a look15:34
anupnHi milan, are you around?  I am trying to enroll node using auto-discovery but kinda not clear about the steps, I created a node and it is in enroll state so do i need to call inspect on other node and inspector would identify the new node during inspection or put this newly created node in manage->inspect state15:35
milananupn, if you create the node manually it's not autodiscovered15:36
anupnmilan, so if i create with ironic node-create then it won't be discovered?15:36
rloopas-ha: if you deprecate, will definitely need bugs, and of course, release notes15:36
milananupn, it won't but it can be inspected still15:36
anupnmilan, so do i need to use virsh or similar15:36
milananupn, if you're testing, yeah15:37
anupnmilan: hmm, interesting. Is there some pointer that will guide me through the steps for auto-discovery?15:38
milananupn, yeah there's the deep dive from dtantsur15:39
* milan scans the mailbox for a link15:39
anupnmilan, Also the problem with inspecting the manually created node is that it has the power state None, hence it is unable to reboot it during the process of inspection15:39
milananupn, mind setting up proper driver15:40
anupnmilan: i created using agent_ipmitool15:40
anupnand then updated the driver_info with ipmi details15:40
anupnusername, password and port15:41
milananupn, https://www.youtube.com/watch?v=wJkDxxjL3NQ15:41
milananupn, but you're testing with virsh?15:41
milanI mean virtual nodes?15:41
* milan keeps forgetting what we use in the gate15:41
* milan checks15:42
anupnmilan, great! thanks for the links15:42
anupnmilan, in gate we use virsh15:43
anupnmilan, https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/test/inspector_tempest_plugin/tests/test_discovery.py#L11815:43
milananupn, http://logs.openstack.org/03/439603/22/check/gate-tempest-dsvm-ironic-inspector-ubuntu-xenial/6cd3426/logs/local.conf.txt.gz15:43
anupnmilan, ah sorry for the wrong link15:43
* pas-ha just realized that the first patch can not be merged until glance is fixed, so either revert glance, or squash two first commits into (huge) single one15:43
pas-haplus some neutron code issues in the first patch..15:44
*** crushil has quit IRC15:44
dtantsur>_<15:44
dtantsurI think it's not unfair to ask to revert and land in a few days15:44
milananupn, yeah virsh and agent_ipmitool driver15:45
anupnmilan, let me try creating with virsh15:45
milananupn, yeah may be some trial and error exercise I'm afraid15:46
anupnmilan, i thought of creating with ironic because then the node will be in enroll state otherwise i am not sure with virsh how will it be identified15:48
milananupn, so the testcase you found workarounds it as in the note by aarefiev15:49
rloopas-ha: I agree with dtantsur. Can't hurt to ask anyway.15:49
anupnmilan, oh yeah15:50
* anupn completely missed the note15:50
anupnmilan, so that mean create and then change the power state to ON and then delete15:51
milananupn, yup that's the trick15:52
aarefievmilan, anupn: if you are talking about discovery tempest test, yep it's workaround, so "don't try this at home"15:52
milanaarefiev, +115:52
anupnHi aarefiev15:52
aarefievo/15:52
*** crushil has joined #openstack-ironic15:52
milananupn, please also check you enable the node missing hook in the inspector conf and upload a rule to inspector to configure the discovered node (driver)15:53
anupnaarefiev, ok15:53
aarefievanupn: what are you trying to achieve?15:53
anupnmilan, i have set node_not_found_hook=enroll15:54
aarefievI'm confused, auto-discovery the node ?15:54
anupnbut i don't have changed or uploaded ay rule to inspector15:54
milancool15:54
anupnaarefiev, i am just trying to discover the node using node_not_found_hook15:54
anupnaarefiev, i have been asked to see how this feature works so kinda playing with it15:55
aarefievanupn: and you are using devstack with virtual nodes, right15:55
anupnaarefiev, yes15:55
anupnthat's right15:55
aarefievanupn: so you could drop one of the node from ironic, configure inspector with node_not_found_hook=enroll and boot the via virsh15:56
aarefiev* boot the node via virs15:56
*** ralonsoh has quit IRC15:57
pas-harloo: https://bugs.launchpad.net/ironic/+bug/169954215:57
openstackLaunchpad bug 1699542 in Ironic "ironic fails when glance-api is deployed under uwsgi as vhost" [Undecided,New] - Assigned to Pavlo Shchelokovskyy (pshchelo)15:57
milananupn, actually the value-added thing of the autodiscovery is the rules that configure the node15:57
*** Sukhdev_ has joined #openstack-ironic15:57
anupnaarefiev, why to drop node from ironic15:57
anupnaarefiev, i mean i can just create a new node with virsh15:57
anupnwithout dropping existing one15:57
aarefievanupn: yeah, you could that too15:57
milananupn, but then you end up with ironic not being able to boot it?15:58
aarefievbut dropping existing one is faster15:58
milaneasiest is to delete it from ironic, it will be recreated after all with a different uuid15:58
milan* supposing all goes well O:-)15:58
aarefievyeah, should be, it's virtual env15:59
milanworks in the CI (tm)15:59
anupnmilan: a bit confused so virsh will boot it with ironic? how is dropping an existing one and creating new node via virsh related?15:59
* anupn disclaimer i have not used virsh anytime16:00
milanwell you said yourself you cannot power on the newly created node with ironic16:00
anupnmilan, yeah that is when i have right now created the node with ironic and i am trying to put that node in inspect state16:01
anupnmilan, but if i create using virsh i am not going to put the node in inspect state16:01
milanso easiest for you will be: create the rule as the test case does, power on th virt node and remove it from ironic, it will boot inspection IPA and inspector will recreate the node in ironic for you16:01
*** tesseract has quit IRC16:03
milanor you can copy the config of the driver to the node you created manually and  make sure you can power it on, but that won't test autodiscovery as the node_not_found hook won't be run (the node exist in ironic after all)16:03
anupnmilan, yeah let me try that.16:03
anupnmilan, yeah that would not test autodiscovery16:04
rloothx pas-ha, i updated whiteboard with that bug16:04
*** makowals has quit IRC16:05
anupnmilan: i will follow the test case, put the node into manage->inspect and wait for it to power on and then delete it, lets see how it goes16:06
anupnmilan, aarefiev: thanks16:06
anupn:)16:06
milananupn, cool! good luck!16:06
*** aarefiev is now known as aarefiev_afk16:07
pas-harloo: also for bookkeeping - an rfe for rewriting some auth/clients parts and deprecating some options https://bugs.launchpad.net/ironic/+bug/169954716:08
openstackLaunchpad bug 1699547 in Ironic "[RFE] unify endpoint discovery for services by using keystoneauth adapters" [Undecided,New]16:08
*** dtantsur is now known as dtantsur|afk16:09
dtantsur|afko/16:09
rloopas-ha: thx, looking...16:09
rloohave a good evening dtantsur|afk16:10
*** jaganathan has joined #openstack-ironic16:10
* milan be gone16:12
milannight Ironic! :)16:12
*** milan has quit IRC16:12
sambettsmilan be gone is that a type of clean product for removing milans16:12
sambetts;)16:13
*** rama_y has joined #openstack-ironic16:16
*** rama_y_ has joined #openstack-ironic16:17
*** rama_y has quit IRC16:17
rloopas-ha: wrt the rfe and email discussion, it isn't clear to me whether we ought to wait for some stuff to get into ksa, like nova's get_service_url().16:19
rloopas-ha: and wrt the rfe itself, the common configuration scheme -- is that a scheme used by other projects already? or a scheme that you are going to define -- and if so, then it ought to be in the rfe...16:21
*** mrtenio has joined #openstack-ironic16:21
*** fellypefca has joined #openstack-ironic16:23
*** hjensas has joined #openstack-ironic16:24
*** livelace has joined #openstack-ironic16:28
*** sambetts is now known as sambetts|afk16:28
sambetts|afknight all!16:28
jlvillalsambetts|afk: ciao16:29
pas-harloo, not sure WDYM for get_Service_url - it is handled by ksa.Adapter.get_endpoint() with adapter instance loaded from options in conf.16:31
pas-hare configurartion scheme - it is based on conf options exported by ksa auth plugins (we use those already), sessions (we have those but do not use everywhere) and adapters (the new thing for our config)16:33
pas-hathe adapter options specify service_type, service_name, interface and region_name to find appropriate endpoint in catalog, or endpoint_override to set the endpoint manually16:33
pas-haI'm pretty sure nova is moving to this direction already16:34
*** joe___ has joined #openstack-ironic16:37
*** joe___ has quit IRC16:37
rloopas-ha: i was referring to eric fried's work on nova, and his comment about moving some of the nova code to ksa: http://lists.openstack.org/pipermail/openstack-dev/2017-May/117435.html16:40
rloopas-ha: am wondering what minimal set of this is needed to fix the glance issue now, and am guessing most of the rfe.16:40
*** trown is now known as trown|lunch16:45
pas-haI could probably hack up a minimal fixer for glance issue at hand... but honestly see little point since 've been busy with the 'right' way to solve it for about a week...16:47
rloopas-ha: yeah, i think if we're going to spend the time, might as well do it 'right'16:47
pas-haas for that nova patch - it abstracts getting the url for service, but ironic is a bit different - we rearly actually pass the user token, and most of the time AFAIU are using service clients, so there is no need to get the service url in ironic code, as sessions with service user auth will do it themselves. we also have a bunch of options to keep until deprecated completely, so I went with a bit more verbose way..16:50
*** mgoddard has quit IRC16:50
*** fellypefca has quit IRC16:51
rloopas-ha: ok. i'm good with the rfe (w/o a spec). although it would be good to be explicit in the rfe as to the new configs and deprecated configs.16:51
*** Sukhdev_ has quit IRC16:52
pas-haok, I'll add a comment trying to remember _all_ options to be deprecated :)16:52
jlvillalmjturek: I left a comment in: https://review.openstack.org/#/c/413324/16:56
patchbotpatch 413324 - ironic - iPXE template support for iSCSI16:56
jlvillalmjturek: Basically I can post a follow-up patch with my suggested change. Or you could incorporate it yourself. Either works for me.16:56
jlvillalmjturek: Of course we are blocked by the gate breakage at the moment.16:56
mjturekjlvillal: I can update today!16:56
jlvillalmjturek: Thanks!16:56
mjturekjlvillal: seems fair, basically you want to incorporate using lists and the plural case in the direct call testing16:58
jlvillalmjturek: Yes please :)16:58
*** derekh has quit IRC16:58
*** bcornec1 has quit IRC16:59
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for cinderclient  https://review.openstack.org/47617116:59
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for neutronclient  https://review.openstack.org/47617016:59
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Rework keystone auth for glance  https://review.openstack.org/46772816:59
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Fix swiftclient creation  https://review.openstack.org/47617316:59
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for inspectorclient  https://review.openstack.org/47617216:59
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Introduce keystoneauth adapters for clients  https://review.openstack.org/46962716:59
jlvillalGo go go pas-ha ! :)17:00
pas-hahehe17:00
jlvillalpas-ha: So hopefully we don't need all six to fix the gate.17:01
* jlvillal goes to get some hydration17:02
*** harlowja has joined #openstack-ironic17:07
openstackgerritNisha Agarwal proposed openstack/sushy master: Add local_gb discovery to sushy  https://review.openstack.org/45511417:07
*** livelace has quit IRC17:08
*** e0ne has quit IRC17:11
*** lucasagomes is now known as lucas-afk17:12
*** links has quit IRC17:15
*** makowals has joined #openstack-ironic17:16
openstackgerritMichael Turek proposed openstack/ironic master: iPXE template support for iSCSI  https://review.openstack.org/41332417:17
*** bcornec1 has joined #openstack-ironic17:18
*** amarao has quit IRC17:19
*** Sukhdev has quit IRC17:25
jlvillalmjturek: Thanks!17:27
*** ijw has joined #openstack-ironic17:28
*** joe___ has joined #openstack-ironic17:36
*** makowals has quit IRC17:39
*** joe___ has quit IRC17:45
*** mjturek has quit IRC17:45
*** bcornec1 has quit IRC17:51
*** oanson has quit IRC17:52
*** oanson has joined #openstack-ironic17:52
*** trown|lunch is now known as trown17:57
*** fellypefca has joined #openstack-ironic18:02
*** Nisha_Agarwal has quit IRC18:08
*** ijw has quit IRC18:14
*** ijw has joined #openstack-ironic18:18
jlvillalpas-ha: What is the plan to fix the gate? I'm a bit confused.18:18
rloojlvillal: i put some notes in the whiteboard, did you see that?18:19
jlvillalI notice https://review.openstack.org/469627 is failing in the gate when I look at   http://status.openstack.org/zuul/18:19
patchbotpatch 469627 - ironic - Introduce keystoneauth adapters for clients18:19
jlvillalrloo: I did, and I am confused.18:19
rloojlvillal: although pas-ha said that for his first patch to actually pass, we need the glance change reverted18:19
jlvillalrloo: Ah18:19
rloojlvillal: unless pas-ha combines both patches into one.18:19
jlvillalrloo: Is there a patch up for that somewhere? Has someone talked to glance?18:20
rloojlvillal: i'm not sure what pas-ha decided to do18:20
rloojlvillal: i don't know if anyone has talked to glance. no one said that they had.18:20
jlvillalrloo: Okay. I will see if anyone is over there18:20
* jlvillal goes to join #openstack-glance18:20
rloojlvillal: thx18:20
*** ijw has quit IRC18:23
rloojlvillal: i updated etherpad, hope it is less confusing now.18:23
*** ijw has joined #openstack-ironic18:24
jlvillalrloo: Thanks. I'm trying to figure out what patch in glance broke it.18:24
rloojlvillal: dunno. wondering if it is devstack that made the switch.18:25
jlvillalrloo: ding ding ding!18:26
* rloo can interpret that in many ways. look out, fire trucks comin' :)18:26
*** mjturek has joined #openstack-ironic18:28
rama_y_rloo, jlvillal, this patch merge in devstack could be the reason: https://github.com/openstack-dev/devstack/commit/7fef90aff9b023709428977c41a8db5893675c8118:31
TheJuliaThere is a proposed revert for it that has a -118:33
jlvillalTheJulia: Yep. I voted +1 :)18:34
jlvillalhttps://review.openstack.org/#/c/476108/18:34
patchbotpatch 476108 - openstack-dev/devstack - Revert "Use uwsgi for glance-api"18:34
jlvillalrama_y_: Thanks18:34
rama_y_:)18:35
rloothx rama_y_, jlvillal, TheJulia. I have, once again, updated our white board18:42
TheJuliajlvillal: I threw a +1 on there as well, I think Matthew is on the other side of the planet right now, but I have a call later today that he might be on. If so, I'll bring it up.18:42
rloojlvillal, TheJulia: also, for pas-ha's fixes, we need to approve the rfe: https://bugs.launchpad.net/ironic/+bug/1699547. so if either of you want to take a look18:43
openstackLaunchpad bug 1699547 in Ironic "[RFE] unify endpoint discovery for services by using keystoneauth adapters" [Wishlist,Triaged] - Assigned to Pavlo Shchelokovskyy (pshchelo)18:43
rlooand thx TheJulia!18:43
*** Sukhdev has joined #openstack-ironic18:43
* TheJulia reads the rfe18:43
TheJuliaheh, pas-ha proposed a noauth plugin for ksa18:46
jlvillalTheJulia: Thanks18:48
TheJuliarloo: posted feedback to the rfe18:54
*** ijw has quit IRC18:55
TheJuliaalso, this is a defect at this point :(18:55
*** ijw has joined #openstack-ironic18:55
rlooTheJulia: thx. well, it is both, i think pas-ha was covering his bases, cuz he has a bug and an rfe.18:55
rlooTheJulia: the code is there :-)18:55
rlooTheJulia: so you good with approving the rfe?18:56
rlooTheJulia: the reason it is good to have an rfe is cuz of the config deprecations.18:56
rlooTheJulia: well, you said conceptually you are good with it. so I'm going to approve it :-)18:57
TheJuliarloo: about to context switch to the code, All good with approving.18:58
rlooTheJulia: thx. I'm just trying to streamline, fast-path, whatever this :-)18:58
*** ijw has quit IRC18:59
TheJulia:)19:03
*** priteau has quit IRC19:07
*** priteau has joined #openstack-ironic19:07
*** priteau has quit IRC19:12
*** patriciadomin has quit IRC19:22
*** ijw has joined #openstack-ironic19:23
*** makowals has joined #openstack-ironic19:29
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-ui master: Add support for editing node's resource_class  https://review.openstack.org/46385219:29
*** mat128 has quit IRC19:33
*** ijw has quit IRC19:36
*** fragatina has joined #openstack-ironic19:39
*** bfournie has quit IRC19:39
*** rloo has quit IRC19:42
*** rloo has joined #openstack-ironic19:42
*** patriciadomin has joined #openstack-ironic19:47
*** ChubYann has joined #openstack-ironic19:48
*** fellypefca has quit IRC19:48
*** crushil has quit IRC19:50
*** makowals has quit IRC19:53
*** crushil has joined #openstack-ironic20:02
*** openstackgerrit has quit IRC20:03
*** Sukhdev has quit IRC20:06
*** fragatina has quit IRC20:07
*** mrtenio has quit IRC20:12
*** Goneri has quit IRC20:14
*** priteau has joined #openstack-ironic20:14
*** jcoufal has quit IRC20:15
*** bfournie has joined #openstack-ironic20:19
*** bfournie has quit IRC20:22
*** fxpester has quit IRC20:30
*** dprince has quit IRC20:32
*** ijw has joined #openstack-ironic20:32
*** goldyfruit has quit IRC20:33
*** trown is now known as trown|outtypewww20:51
*** amotoki is now known as amotoki_away20:52
*** crushil has quit IRC20:53
*** priteau has quit IRC20:56
*** baoli has quit IRC21:07
*** crushil has joined #openstack-ironic21:09
*** openstackgerrit has joined #openstack-ironic21:11
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-ui master: Add support for editing node's resource_class  https://review.openstack.org/46385221:11
*** chlong has quit IRC21:22
*** makowals has joined #openstack-ironic21:43
*** jkilpatr has quit IRC21:56
*** ijw has quit IRC22:03
*** ijw has joined #openstack-ironic22:03
*** goldyfruit has joined #openstack-ironic22:04
*** makowals has quit IRC22:10
*** silvrax has quit IRC22:14
*** MattMan_1 has quit IRC22:15
*** MattMan_1 has joined #openstack-ironic22:15
*** silvrax has joined #openstack-ironic22:21
*** Nisha_Agarwal has joined #openstack-ironic22:28
*** Sukhdev has joined #openstack-ironic22:28
*** makowals has joined #openstack-ironic22:53
openstackgerritNisha Agarwal proposed openstack/ironic master: py3.5:Workaround fix for forcing virtualbmc installation with pip2  https://review.openstack.org/47391322:57
*** makowals has quit IRC22:58
Nisha_Agarwalvsaienk0, whats the review comment on https://review.openstack.org/#/c/473913/22:58
patchbotpatch 473913 - ironic - py3.5:Workaround fix for forcing virtualbmc instal...22:58
Nisha_Agarwalvsaienk0, i see ur negative vote but no comment22:59
*** sasha2 has quit IRC23:06
openstackgerritPeter Piela proposed openstack/ironic-ui master: Set current boot device on a node  https://review.openstack.org/47392623:14
openstackgerritNisha Agarwal proposed openstack/ironic master: Fixes typeerror for py3.5  https://review.openstack.org/47388823:28
*** hoonetorg has quit IRC23:39
*** Nisha_Agarwal has quit IRC23:43
*** makowals has joined #openstack-ironic23:48
*** hoonetorg has joined #openstack-ironic23:56

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