Thursday, 2018-07-26

*** rcernin has joined #openstack-ironic00:03
*** zhangfei has joined #openstack-ironic00:15
*** zhangfei has quit IRC00:19
*** livelace2 has quit IRC00:24
*** livelace2 has joined #openstack-ironic00:24
*** mjturek has joined #openstack-ironic00:34
*** rpioso is now known as rpioso|afk00:37
*** hamzy has joined #openstack-ironic00:38
*** bdodd has joined #openstack-ironic00:56
*** namnh has joined #openstack-ironic00:56
*** namnh has quit IRC01:01
*** phuongnh has joined #openstack-ironic01:02
openstackgerritMichael Turek proposed openstack/ironic master: Pass prep_boot_part_uuid to install_bootloader for ppc64* partition images  https://review.openstack.org/57915901:03
*** mjturek has quit IRC01:04
*** gyee has quit IRC01:08
*** ddurst has quit IRC01:09
*** ddurst has joined #openstack-ironic01:09
*** trungnv has quit IRC01:13
*** trungnv has joined #openstack-ironic01:13
*** tiendc has joined #openstack-ironic01:26
*** rloo has quit IRC01:33
*** zhangfei has joined #openstack-ironic01:45
openstackgerritMerged openstack/ironic master: Add conductor_group to node notifications  https://review.openstack.org/58537101:46
*** namnh has joined #openstack-ironic01:47
*** namnh has quit IRC01:50
*** namnh has joined #openstack-ironic01:50
openstackgerritMerged openstack/ironic master: Add deploy_step to NodePayload.SCHEMA  https://review.openstack.org/58541301:52
*** namnh has quit IRC01:52
*** zhangfei has quit IRC01:57
*** zhangfei has joined #openstack-ironic02:10
*** openstack has joined #openstack-ironic02:49
*** ChanServ sets mode: +o openstack02:49
*** zhangfei has quit IRC03:01
*** zhangfei has joined #openstack-ironic03:14
*** edleafe has quit IRC03:14
*** edleafe has joined #openstack-ironic03:15
*** Sukhdev has joined #openstack-ironic03:27
openstackgerritNguyen Van Trung proposed openstack/ironic master: Fix for failure in cleaning in iRMC driver  https://review.openstack.org/58593603:29
openstackgerritvinay kumar muddu proposed openstack/ironic master: Documentation update of iLO BIOS settings  https://review.openstack.org/58520203:37
openstackgerritNguyen Van Trung proposed openstack/ironic master: Follow-up to improve pep8 checking with hacking  https://review.openstack.org/58593903:39
*** gkadam has joined #openstack-ironic03:49
*** mjura has joined #openstack-ironic04:40
*** Sukhdev has quit IRC04:44
*** Sukhdev has joined #openstack-ironic04:47
*** jaganathan has joined #openstack-ironic04:48
*** jaganathan has quit IRC04:49
*** jaganathan has joined #openstack-ironic04:49
*** e0ne has joined #openstack-ironic05:06
*** mraineri has quit IRC05:06
*** rwsu has quit IRC05:23
*** Sukhdev has quit IRC05:31
*** zhangfei has quit IRC05:33
*** gkadam has quit IRC05:39
*** zhangfei has joined #openstack-ironic05:47
*** links has joined #openstack-ironic05:48
*** e0ne has quit IRC06:00
*** rbartal has joined #openstack-ironic06:02
*** mraineri has joined #openstack-ironic06:12
*** skazi has joined #openstack-ironic06:26
*** serlex has joined #openstack-ironic06:41
*** mgoddard has joined #openstack-ironic06:41
*** exodusftw has quit IRC06:42
*** exodusftw has joined #openstack-ironic06:42
*** mgoddard has quit IRC06:50
openstackgerritvinay kumar muddu proposed openstack/ironic master: Documentation update of iLO BIOS settings  https://review.openstack.org/58520206:52
openstackgerritvinay kumar muddu proposed openstack/ironic master: Documentation update of iLO BIOS settings  https://review.openstack.org/58520206:55
*** hoonetorg has quit IRC06:57
openstackgerrityolanda.robla proposed openstack/ironic master: Add automated_clean field  https://review.openstack.org/58579507:03
*** gkadam has joined #openstack-ironic07:04
*** liuzz has joined #openstack-ironic07:11
*** tssurya has joined #openstack-ironic07:12
*** hoonetorg has joined #openstack-ironic07:14
*** racedo has joined #openstack-ironic07:21
*** pcaruana has joined #openstack-ironic07:25
*** ajya has joined #openstack-ironic07:27
openstackgerritNguyen Van Trung proposed openstack/ironic master: Refactor RAID configuration via iRMC driver  https://review.openstack.org/58597707:30
*** mgoddard has joined #openstack-ironic07:31
*** AlexeyAbashkin has joined #openstack-ironic07:33
*** zhangfei has quit IRC07:42
*** dougsz has joined #openstack-ironic07:43
*** dtantsur|afk is now known as dtantsur07:46
dtantsurmorning ironic07:46
dtantsuredleafe: likely the conductor did not start. tripleo has a lot of drivers by default.07:46
*** zhangfei has joined #openstack-ironic07:55
*** mjura has quit IRC07:58
*** mjura has joined #openstack-ironic07:58
*** jiapei has joined #openstack-ironic08:13
*** e0ne has joined #openstack-ironic08:17
openstackgerritNguyen Van Trung proposed openstack/ironic master: Update docs to portgroup with creating windows images  https://review.openstack.org/58561308:19
*** dougsz has quit IRC08:29
openstackgerrityolanda.robla proposed openstack/ironic master: Add automated_clean field to the node object  https://review.openstack.org/58599108:30
*** ajya has quit IRC08:32
*** ajya has joined #openstack-ironic08:32
*** skazi has quit IRC08:33
openstackgerritShivanand Tendulker proposed openstack/ironic master: Follow-up patch for 7c5a04c1149f14900f504f32e000a7b4e69e661f  https://review.openstack.org/58599408:33
openstackgerritOpenStack Release Bot proposed openstack/python-ironic-inspector-client stable/rocky: Update .gitreview for stable/rocky  https://review.openstack.org/58599508:34
openstackgerritOpenStack Release Bot proposed openstack/python-ironic-inspector-client stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky  https://review.openstack.org/58599608:34
openstackgerritOpenStack Release Bot proposed openstack/python-ironic-inspector-client master: Update reno for stable/rocky  https://review.openstack.org/58599708:34
*** tesseract has joined #openstack-ironic08:42
openstackgerrityolanda.robla proposed openstack/ironic master: Add automated_clean field  https://review.openstack.org/58579508:42
openstackgerritMerged openstack/python-ironic-inspector-client master: Fix setup.cfg for release  https://review.openstack.org/58585708:42
openstackgerritWill Miller proposed openstack/ironic-inspector master: Allow nested action value formatting  https://review.openstack.org/58577908:43
*** derekh has joined #openstack-ironic08:43
*** namnh has joined #openstack-ironic08:49
*** sambetts|afk has quit IRC08:57
*** sambetts_ has joined #openstack-ironic08:59
*** stendulker has joined #openstack-ironic09:02
openstackgerritAnkit Kumar proposed openstack/ironic-tempest-plugin master: Test cases for iLO drivers in standalone ironic  https://review.openstack.org/53565209:03
*** pmannidi has quit IRC09:08
openstackgerrityolanda.robla proposed openstack/ironic master: Add automated_clean field to the node object  https://review.openstack.org/58599109:10
*** namnh has quit IRC09:16
*** zhangfei has quit IRC09:19
*** d0ugal has joined #openstack-ironic09:21
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Reorder unittest assertions  https://review.openstack.org/58601709:21
*** dougsz has joined #openstack-ironic09:25
openstackgerritHamdy Khader proposed openstack/ironic master: Add support for smartNICs  https://review.openstack.org/58357309:27
*** zhangfei has joined #openstack-ironic09:31
openstackgerritDerek Higgins proposed openstack/ironic master: Switch ironic-tempest-dsvm-pxe_ipmitool-postgres  https://review.openstack.org/58318509:37
openstackgerritDerek Higgins proposed openstack/ironic master: Switch ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa  https://review.openstack.org/58333709:37
openstackgerritDerek Higgins proposed openstack/ironic master: Switch ironic-tempest-dsvm-ipa-wholedisk-bios-pxe_snmp-tinyipa  https://review.openstack.org/58354609:37
openstackgerritDerek Higgins proposed openstack/ironic master: Switch ironic-tempest-dsvm-ipa-partition-uefi-pxe_ipmitool-tinyipa  https://review.openstack.org/58355109:37
openstackgerritDerek Higgins proposed openstack/ironic master: Switch ironic-tempest-dsvm-ipa-partition-pxe_ipmitool-tinyipa-python3  https://review.openstack.org/58355509:37
*** stendulker_ has joined #openstack-ironic09:38
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Clean up libvirt unit tests  https://review.openstack.org/58602909:38
*** stendulker has quit IRC09:41
openstackgerritAija Jaunteva proposed openstack/sushy master: Add DictionaryField class to resource base  https://review.openstack.org/58603109:41
etingofthis job seems to occasionally fail elsewhere as well, but what's cause? -- http://logs.openstack.org/25/449425/8/check/ironic-tempest-dsvm-ipa-wholedisk-agent_ipmitool-tinyipa-multinode/bb55c31/job-output.txt.gz#_2018-07-24_17_08_04_49965709:43
dtantsuretingof: it's pretty unstable overall, but I haven't looked into why it fails now09:47
etingofwell, it's not just "now" ;)09:48
*** d0ugal has quit IRC09:52
dtantsurwell, for example some of test nodes are slower than the others. and this test is particularly complex and resource-sensitive09:53
etingofso should I recheck or better wait while the situation improves?09:55
dtantsuretingof: recheck09:55
dtantsurthe slow nodes will never improve. and other potential problems won't improve without someone looking into them (you can be this someone ;)09:56
openstackgerritMadhuri Kumari proposed openstack/ironic master: Implement basic interfaces for GraphicalConsole Interface  https://review.openstack.org/54735609:57
openstackgerrityolanda.robla proposed openstack/ironic master: Add documentation for BIOS settings  https://review.openstack.org/58532209:58
*** namnh has joined #openstack-ironic10:06
*** namnh has quit IRC10:06
*** namnh has joined #openstack-ironic10:07
*** namnh_ has joined #openstack-ironic10:07
openstackgerritMerged openstack/ironic-python-agent stable/queens: fix uname mock arguments for get-pip.py  https://review.openstack.org/58581110:09
openstackgerritMerged openstack/ironic-python-agent stable/pike: fix uname mock arguments for get-pip.py  https://review.openstack.org/58581010:09
openstackgerritMerged openstack/ironic-python-agent stable/ocata: fix uname mock arguments for get-pip.py  https://review.openstack.org/58580910:09
openstackgerritMadhuri Kumari proposed openstack/ironic master: Implement basic interfaces for GraphicalConsole Interface  https://review.openstack.org/54735610:10
*** namnh has quit IRC10:11
*** stendulker has joined #openstack-ironic10:16
*** stendulker_ has quit IRC10:19
openstackgerritMadhuri Kumari proposed openstack/ironic master: Implement basic interfaces for GraphicalConsole Interface  https://review.openstack.org/54735610:22
*** racedo has quit IRC10:24
*** namnh_ has quit IRC10:26
*** namnh has joined #openstack-ironic10:29
*** namnh has quit IRC10:30
*** d0ugal has joined #openstack-ironic10:32
*** d0ugal has quit IRC10:34
*** phuongnh has quit IRC10:35
*** namnh has joined #openstack-ironic10:37
*** liuzz has quit IRC10:40
*** stendulker_ has joined #openstack-ironic10:44
*** stendulker has quit IRC10:47
openstackgerrityolanda.robla proposed openstack/ironic master: WIP: Add automated_clean field to the node object and API  https://review.openstack.org/58599110:53
yolandadtantsur, i have a question on this error10:55
yolandahttp://logs.openstack.org/22/585322/7/check/build-openstack-api-ref/e4382e4/job-output.txt.gz#_2018-07-26_10_43_49_97228310:55
yolandafor the response of this call, we just return a dictionary, and the key is variable10:55
yolandai tried to reflect that, but i hit this error10:55
yolandahow shall we reflect it?10:55
*** dougsz has quit IRC10:58
*** linkmark has joined #openstack-ironic10:58
*** dougsz has joined #openstack-ironic11:01
*** stendulker has joined #openstack-ironic11:03
*** rcernin has quit IRC11:04
*** dougsz has quit IRC11:06
*** stendulker_ has quit IRC11:06
*** namnh has quit IRC11:07
*** dougsz has joined #openstack-ironic11:08
*** namnh has joined #openstack-ironic11:09
*** namnh has quit IRC11:13
dtantsuryolanda: not sure. maybe just in textual form in the description?11:14
yolandaso maybe i could just set <key>: d_bios_setting there?11:20
yolandai need to pass a name: value there11:20
dtantsuryeah, maybe11:20
dtantsurmaybe we should not have done it this way..11:21
yolandalooks as it's not following the standards11:21
yolandathat's the only output i could find with this format11:21
*** stendulker has quit IRC11:24
openstackgerrityolanda.robla proposed openstack/ironic master: Add documentation for BIOS settings  https://review.openstack.org/58532211:26
*** liuzz has joined #openstack-ironic11:26
openstackgerritHamdy Khader proposed openstack/ironic master: Add support for smartNICs  https://review.openstack.org/58357311:26
TheJuliaGood morning11:35
*** zhangfei has quit IRC11:35
openstackgerrityolanda.robla proposed openstack/ironic master: WIP: Add automated_clean field to the node object and API  https://review.openstack.org/58599111:36
*** skazi has joined #openstack-ironic11:43
dtantsurmorning TheJulia11:51
TheJuliagood morning dtantsur11:55
dtantsurso, I guess our client releases went fine, modulo inspector-client's setup.cfg?11:56
TheJuliayup11:57
dtantsursweet!11:57
TheJuliaI need to take a look at https://review.openstack.org/#/c/578959/ again this morning11:58
patchbotpatch 578959 - ironic - Change PXE logic to always link macs with UEFI11:59
TheJuliaso it is WF-1 at the moment11:59
*** MattMan has quit IRC12:08
*** MattMan has joined #openstack-ironic12:09
*** liuzz_ has joined #openstack-ironic12:09
*** liuzz has quit IRC12:10
*** hamzy has quit IRC12:14
*** rh-jelabarre has joined #openstack-ironic12:18
*** hamzy has joined #openstack-ironic12:19
*** tiendc has quit IRC12:21
yolandahi, i'm getting this error when adding the automated_clean... http://logs.openstack.org/95/585795/3/check/openstack-tox-py27/f547ead/job-output.txt.gz#_2018-07-26_09_21_28_78496312:21
yolandaseems to be some problem between tinyint/boolean, not sure what's the problem, as i defined the type as boolean in the model and on migration...12:22
*** dprince has joined #openstack-ironic12:24
*** mjturek has joined #openstack-ironic12:32
*** jcoufal has joined #openstack-ironic12:32
*** jiapei has quit IRC12:33
*** openstackgerrit has quit IRC12:36
*** links has quit IRC12:46
TheJuliayolanda: have you compard it to another boolean field?12:49
TheJuliaYou really can't null a boolean either, although that should imply false I guess12:49
*** namnh has joined #openstack-ironic12:50
yolandamm, then it will imply it will always have a value12:54
*** namnh has quit IRC12:54
yolandai'm thinking on how that should work, if we have that value set to False , and the general automated_cleanup set to True...12:55
yolandaTheJulia, i see it quite risky12:56
yolandamaybe we should change the datatype then?12:56
*** Sukhdev has joined #openstack-ironic12:56
*** Sukhdev has quit IRC13:01
*** rloo has joined #openstack-ironic13:03
*** bdodd has quit IRC13:05
*** namnh has joined #openstack-ironic13:05
yolandafor what we need to do we need to have none, true, false. I'd try to change to tinyint13:05
TheJuliadtantsur: yeah, I need to revise https://review.openstack.org/#/c/578959/5 :\13:06
patchbotpatch 578959 - ironic - Change PXE logic to always link macs with UEFI13:06
TheJuliacomment inline13:06
TheJuliayolanda: You almost need three options then13:07
TheJuliaWhat are we really trying to achieve with this option?13:08
yolandaTheJulia, None will mean the node takes the option from general config. True/False means it takes the option from the node, and does automated cleanup/not13:09
yolandaso integer with some constraint, or some enum13:09
TheJuliaSo a boolean option would only be False, not none13:09
yolandayep, that doesn't fit13:10
TheJuliamay have been better in driver_info since it is all driver code13:10
yolandasomehow i was thinking i could have a difference between null and false13:10
TheJuliaif it stored and treated as a string and then cast...13:17
TheJuliathere could be13:17
dtantsurTheJulia: did not quite get it, to be honest13:22
TheJuliadtantsur: regarding?13:23
dtantsurTheJulia: well, both regarding the UEFI patch and the problem with boolean, actually :)13:24
TheJuliadtantsur: no 01- for grub :\13:24
*** bdodd has joined #openstack-ironic13:24
TheJuliaand : instead of -13:24
dtantsurah, I see now13:24
TheJuliayeah :\13:24
dtantsurTheJulia: also, what's the problem with NULL boolean?13:26
TheJuliare the boolean, it sounds like we want an independent dual override for cleaning that just completely disregards the conductor default setting if set. So that an operator can choose to disable or enable for that specific node and disregard the conductor wide setting. Unless we want to default it to the present automated clean value and just migrate away from the global setting....13:26
TheJuliaNULL should == None which should == False13:27
TheJuliaYou can't have three options there :\13:27
*** mjturek has quit IRC13:27
dtantsurno, automated_clean = None | True | false13:27
dtantsurI don't see problems still13:27
*** mbeierl has joined #openstack-ironic13:27
TheJuliadtantsur: I agree there, but it can't be a boolean then, it has to be a string13:27
TheJuliaand then cast13:27
dtantsurwhy?13:27
*** mjturek has joined #openstack-ironic13:27
TheJuliaor just a string maybe if sqlalchemy will do it for us13:28
* dtantsur is lost13:28
*** med_ has quit IRC13:28
TheJuliadtantsur: blame sqlalchemy13:28
dtantsursure, if it does not work with NULL, we have to fix it13:28
TheJuliaI don't think it will because the existing maintenance state is a boolean13:29
TheJuliaand I think it is always default false as a result of not being set13:29
dtantsurI don't think so13:29
* TheJulia cracks open her local db13:29
dtantsurwhat you're saying is that NULL never works. I quite doubt it, to be honest..13:29
*** jistr is now known as jistr|mtg13:29
TheJuliadtantsur: in the db, it is being set as a 0/113:30
TheJuliayup, tinyint(1)13:30
TheJuliaso at the DB level, I can null it out13:31
dtantsurTheJulia: well, at least sqlite works correctly itself13:31
*** mjturek has quit IRC13:31
TheJuliaand..... apparently requests is gone from my systme :\13:32
*** rbartal_ has joined #openstack-ironic13:32
TheJuliaField `maintenance' cannot be None (HTTP 500)13:33
dtantsursure, it's not nullable13:33
dtantsurbut you can create a nullable field, no?13:33
dtantsurso, it's wsme problem, not sqlalchemy?13:33
dtantsurI think yolanda's problem is simply because sqlite does not have proper types13:33
dtantsurso boolean is emulated as tinyint (I can reproduce it with sqlite)13:34
TheJulianot sure, you should always be able to actually13:34
*** rbartal has quit IRC13:34
TheJuliayeah, that i likely what is going on then13:34
dtantsurstill, it can be NULL on database level (just checked)13:34
dtantsurit's just NULL | 0 | 1 internally13:34
TheJuliayeah, that is what I did13:34
yolandaso i'm trying now creating an enum type13:35
dtantsuryolanda: please stick with boolean13:35
yolandadtantsur, but then how do we avoid this problem?13:35
TheJuliaenum won't work in sqllite either13:35
TheJuliawe tried to do that for deva a long time ago and it blew up on us13:36
*** mjturek has joined #openstack-ironic13:36
dtantsurTheJulia: with wsme you cannot have boolean = None, but you can use 'remove' PATCH action13:36
TheJuliadtantsur: \o/13:36
dtantsurTheJulia: we do have enums in ironic-inspector. I guess they're also emulated13:36
dtantsuryolanda: just check that the column is present, not assert its type?13:36
TheJuliaYeah, I think that is what it was, we went to actually do somehting and it all went *boom* and had to backtrack13:36
dtantsurI mean, as TheJulia said, the type check will probably fail on enum as well. in sqlite everything is a number or text IIRC13:37
yolandaok let me avoid that check of type... so for mysql, it will be safe to assume that null will exist?13:37
dtantsuryolanda: null exists everywhere, it's a part of standard13:37
dtantsurevery column that is not declared as 'not null' can be null13:38
yolandai mean, will not be confused, converted, etc13:38
yolandaNone with False13:38
TheJuliadtantsur: except pkey columns ;)13:38
dtantsurTheJulia: yeah, they're explicitly 'not null' (I hope!)13:38
dtantsurbut sqlite is... funny..13:38
yolandai also could not even generate the migration using sqlalchemy13:39
yolandait complains that cannot upgrade the db because it cannot alter a constraint13:39
yolandadue to some of the migrations in the history13:39
yolandaif had to do it on a devstack where i had mysql installed13:39
dtantsuryolanda: yep, sqlite cannot change constraints13:39
TheJuliadtantsur: I'm not going to call the in-brain select to find the rows (instances) due to painful database memories13:39
dtantsurheh13:40
dtantsuryolanda: there is a trick to handle it. we have a ironic-dbsync command to create a schema without going through migrations13:40
*** r-daneel has joined #openstack-ironic13:40
dtantsurlemme find docs for you13:40
dtantsurmeh, I cannot find docs. but it's something like ironic-dbsync create_schema13:42
dtantsurthen you can apply your change and then generate the migration13:42
dtantsuryolanda: ^^^13:42
yolandadtantsur, TheJulia , if we have all these issues with sqlite... why do we list that as an option?13:42
dtantsuryolanda: it's only for local testing and unit tets13:42
dtantsurotherwise we'd have to require mysql to run unit tests13:42
*** namnh has quit IRC13:43
yolandai see13:44
* dtantsur remembers ironic-inspector only supporting sqlite.. fun times.13:44
yolandadoh13:44
dtantsurit was kilo, time to prune it out of my memory :)13:44
*** rook-- is now known as rook13:45
*** markguz has joined #openstack-ironic13:45
*** hongbin has joined #openstack-ironic13:52
TheJuliaso I searched for ironic on the talk voting and 55 results....13:53
TheJuliadtantsur: heh13:53
dtantsuryep, quite a lot of things13:53
etingof...out of 746 talk proposals13:54
TheJuliaI had no idea it was that many13:54
TheJuliaThat is really good numbers13:55
etingofthat's what it reports on empty search string13:55
TheJuliaInteresting to see the vendor technology integration pitches again13:56
TheJulia"tragedy of ironic and nova" :\14:00
TheJuliaIt is well written14:00
* TheJulia finally gets the humor14:01
etingofwith bonus: "one or two things about Shakespeare"14:03
TheJulia++14:03
*** arnewiebalck_ has joined #openstack-ironic14:06
*** rnoriega has quit IRC14:07
*** weshay_PTO has quit IRC14:07
*** rnoriega has joined #openstack-ironic14:07
*** weshay has joined #openstack-ironic14:08
*** arnewiebalck_ has quit IRC14:14
dtantsuryep14:16
*** diga has joined #openstack-ironic14:16
*** etingof has quit IRC14:20
*** tssurya has quit IRC14:23
*** arnewiebalck_ has joined #openstack-ironic14:25
*** rpioso|afk is now known as rpioso14:27
rpiosoGood morning :)14:28
*** rbartal_ has quit IRC14:28
*** arnewiebalck_ has quit IRC14:32
*** etingof has joined #openstack-ironic14:33
*** hongbin has quit IRC14:46
*** Sukhdev has joined #openstack-ironic14:50
NobodyCamGood Morning Ironic'ers14:52
rpiosoNobodyCam: Mornin'14:55
NobodyCammorning rpioso :)14:56
*** jistr|mtg is now known as jistr14:56
*** vmud213 has joined #openstack-ironic15:04
*** alexpilotti has quit IRC15:14
*** Sukhdev has quit IRC15:17
*** vmud213 has quit IRC15:22
*** mjturek has quit IRC15:33
*** pcaruana has quit IRC15:38
*** skazi has quit IRC15:39
*** hamzy has quit IRC15:41
*** hamzy has joined #openstack-ironic15:41
*** livelace2 has quit IRC15:44
ajyagerrit-irc hook seems to be down, is this discussed somewhere?15:44
ajyaalso for a new code review Zuul was not added as a reviewer, I added manually. Should I add 'recheck' to trigger it?15:45
dtantsurajya: it's fine, it adds itself when it first votes15:46
*** hongbin has joined #openstack-ironic15:49
*** mjturek has joined #openstack-ironic15:53
mgoddardanyone have time to look at https://review.openstack.org/#/c/530537/? It has a +2 already15:55
patchbotpatch 530537 - networking-generic-switch - Add a netmiko device driver for Juniper switches15:55
mgoddard(and a +1 from someone from Juniper)15:55
TheJuliaohh15:56
TheJuliahmm looks like vdrok is idle and he would be ideal I think15:57
*** gyee has joined #openstack-ironic15:59
clarkbajya: yes, in #openstack-infra. Long story short freenode spam killer bot decided it was spamming and killed it. My hunch is that we may just leave it in that state until the spam attack goes away and we can rerun it without getting caught by spam killers15:59
dtantsuroh, that's funny. we're spamming with out patches :D16:02
dtantsurs/out/our/16:02
*** e0ne has quit IRC16:02
*** alexpilotti has joined #openstack-ironic16:10
*** AlexeyAbashkin has quit IRC16:10
ajyaclarkb, thanks16:10
ajyabeing too productive :)16:10
*** alexpilotti has quit IRC16:14
*** skazi has joined #openstack-ironic16:16
*** dprince has quit IRC16:16
*** verdurin has quit IRC16:21
*** verdurin has joined #openstack-ironic16:24
* TheJulia bangs her head against the pxe interface16:27
*** andymccr_ has joined #openstack-ironic16:29
*** ajya has quit IRC16:29
* etingof slides a beer stein full of cool sparkling beer to TheJulia 16:30
TheJuliammmmmm16:30
TheJuliaBeer sounds really good at the moment16:30
etingofand it mixes with coffee \o/16:30
TheJulianot coffee beer?16:31
etingofno, the mug I slide is just pure Czech beer16:32
* TheJulia feels sad suddenly16:32
etingofbut next time you will have coffee stout \o/16:32
*** dprince has joined #openstack-ironic16:32
TheJuliaI do need to go to the post office....16:32
TheJuliaHmmmm16:32
* etingof almost typed `stdout`, time to wrap up16:32
TheJulialol16:33
TheJuliagoodnight etingof16:33
*** baha has joined #openstack-ironic16:33
*** andymccr_ has quit IRC16:33
etingofo/16:33
*** andymccr_ has joined #openstack-ironic16:33
*** andymccr_ has quit IRC16:34
dtantsurmmm, coffee stout16:34
dtantsurwe have good ones in brewdog16:34
*** andymccr_ has joined #openstack-ironic16:34
* dtantsur is pondering spending tomorrow working from there16:34
TheJuliadtantsur: would you be up for looking at mgoddard's n-g-s patch?16:34
dtantsurTheJulia: not today, sorry. I'm boiling :(16:37
*** diga has quit IRC16:37
TheJuliaI think tomorrow is fine in all honesty16:37
dtantsuractually I was about to say g'night16:37
dtantsuryeah, tomorrow quite probably16:37
*** dtantsur is now known as dtantsur|afk16:38
dtantsur|afkc u16:38
TheJuliaI'm banging my head against this pxe fix righ tnow thinking it is time to take a beer break16:38
*** andymccr_ has quit IRC16:38
dtantsur|afk++ go ahead16:38
* TheJulia senses she is going to be spending lots of time with this file open over the next few months16:38
*** andymccr_ has joined #openstack-ironic16:38
*** tesseract has quit IRC16:39
*** andymccr_ has quit IRC16:39
mgoddardif you have time tomorrow dtantsur, would be appreciated16:41
mgoddardalso boiling here16:41
*** med_ has joined #openstack-ironic16:42
*** andymccr_ has joined #openstack-ironic16:44
*** andymccr_ has quit IRC16:53
*** andymccr_ has joined #openstack-ironic16:54
*** andymccr_ has quit IRC16:55
*** andymccr_ has joined #openstack-ironic16:55
*** d0ugal has joined #openstack-ironic16:55
*** d0ugal has quit IRC16:55
*** d0ugal has joined #openstack-ironic16:55
TheJuliaand the moment you realize the code is right, it is the test that is wrong16:57
*** derekh has quit IRC17:00
*** markguz has quit IRC17:03
*** mjturek has quit IRC17:05
*** openstackgerrit has joined #openstack-ironic17:05
openstackgerritJulia Kreger proposed openstack/ironic master: Change PXE logic to always link macs with UEFI  https://review.openstack.org/57895917:05
*** mjturek has joined #openstack-ironic17:07
openstackgerritMerged openstack/sushy-tools master: Add unittests for OpenStack nova driver  https://review.openstack.org/56863617:08
openstackgerritMerged openstack/sushy-tools master: Reorder unittest assertions  https://review.openstack.org/58601717:08
*** r-daneel_ has joined #openstack-ironic17:10
*** r-daneel has quit IRC17:10
*** r-daneel_ is now known as r-daneel17:10
*** Sukhdev has joined #openstack-ironic17:12
*** r-daneel has quit IRC17:14
*** dougsz has quit IRC17:16
*** serlex has quit IRC17:16
*** e0ne has joined #openstack-ironic17:20
*** d0ugal has quit IRC17:21
*** r-daneel has joined #openstack-ironic17:22
openstackgerritJulia Kreger proposed openstack/ironic master: Adds ramdisk deploy driver  https://review.openstack.org/56894017:22
*** mjturek has quit IRC17:24
*** jtomasek has quit IRC17:29
*** trown is now known as trown|lunch17:29
*** med_ has quit IRC17:29
*** baha has quit IRC17:33
*** mjturek has joined #openstack-ironic17:33
*** mjturek has quit IRC17:33
*** e0ne has quit IRC17:37
TheJuliamgoddard: thanks for the +217:42
mgoddardTheJulia: np, thanks for the driver17:43
TheJuliamgoddard: also, https://review.openstack.org/#/c/585818/ is a quick review17:46
patchbotpatch 585818 - ironic - Deprecate xclarity hardware type17:46
* TheJulia runs into town to go to the post office17:47
mgoddarddone17:48
TheJuliathanks17:48
*** baoli has joined #openstack-ironic17:52
*** jaganathan has quit IRC18:16
*** e0ne has joined #openstack-ironic18:22
*** baoli has quit IRC18:41
*** skazi has quit IRC18:42
*** mgoddard has quit IRC18:45
openstackgerritJulia Kreger proposed openstack/ironic master: Add decoratores to ramdisk deploy interface  https://review.openstack.org/58632718:55
*** hamzy has quit IRC19:02
*** hamzy_ has joined #openstack-ironic19:02
openstackgerritJulia Kreger proposed openstack/ironic master: Ramdisk deploy driver doc  https://review.openstack.org/58548619:03
*** mjturek has joined #openstack-ironic19:07
TheJuliao/19:09
*** med_ has joined #openstack-ironic19:10
*** med_ has quit IRC19:10
*** med_ has joined #openstack-ironic19:10
*** baha has joined #openstack-ironic19:10
*** ianychoi_ has joined #openstack-ironic19:15
*** pcaruana has joined #openstack-ironic19:17
*** ianychoi has quit IRC19:18
*** r-daneel has quit IRC19:21
*** r-daneel has joined #openstack-ironic19:22
edleafedtantsur|afk: TheJulia: Figured out my issue. dtantsur|afk was correct; the Ironic conductor wasn't running. Digging into the logs I saw an exception which was the clue. I've summarized it here: http://paste.openstack.org/show/726710/19:28
TheJuliaoh, I should have thought about that19:31
TheJuliaGood to know :)19:31
*** trown|lunch is now known as trown19:43
*** pcaruana has quit IRC19:45
openstackgerrityolanda.robla proposed openstack/ironic master: Add automated_clean field  https://review.openstack.org/58579519:46
openstackgerrityolanda.robla proposed openstack/ironic master: Add automated_clean field  https://review.openstack.org/58579519:51
openstackgerrityolanda.robla proposed openstack/ironic master: Add automated_clean field  https://review.openstack.org/58579519:52
*** dprince has quit IRC20:03
*** alexpilotti has joined #openstack-ironic20:08
*** alexpilo_ has joined #openstack-ironic20:12
*** e0ne has quit IRC20:13
openstackgerritMerged openstack/ironic master: Deprecate xclarity hardware type  https://review.openstack.org/58581820:14
openstackgerritMerged openstack/ironic master: Documentation update of iLO BIOS settings  https://review.openstack.org/58520220:14
*** alexpilotti has quit IRC20:17
*** med_ has quit IRC20:18
*** dougsz has joined #openstack-ironic20:20
openstackgerritJulia Kreger proposed openstack/ironic master: Prepare for Rocky release  https://review.openstack.org/58634220:22
*** med_ has joined #openstack-ironic20:22
*** med_ has quit IRC20:22
*** med_ has joined #openstack-ironic20:22
*** alexpilo_ has quit IRC20:45
*** alexpilotti has joined #openstack-ironic20:46
*** dougsz has quit IRC20:47
*** rh-jelabarre has quit IRC20:56
*** r-daneel_ has joined #openstack-ironic20:59
*** e0ne has joined #openstack-ironic20:59
*** alexpilotti has quit IRC20:59
*** e0ne has quit IRC21:00
*** r-daneel has quit IRC21:00
*** r-daneel_ is now known as r-daneel21:00
*** trown is now known as trown|outtypewww21:03
*** jcoufal has quit IRC21:15
*** r-daneel has quit IRC21:17
*** markguz has joined #openstack-ironic21:30
*** skazi has joined #openstack-ironic21:31
TheJuliaSpeaker voting closes tonight! https://www.openstack.org/summit/berlin-2018/vote-for-speakers#/21:31
*** mjturek has quit IRC21:33
*** markguz has quit IRC21:34
*** med_ has quit IRC21:48
*** markguz has joined #openstack-ironic21:51
*** skazi has quit IRC21:52
*** baha has quit IRC21:54
*** Sukhdev has quit IRC22:08
*** rcernin has joined #openstack-ironic22:19
*** gkadam has quit IRC22:30
*** alexpilotti has joined #openstack-ironic22:36
*** alexpilo_ has joined #openstack-ironic22:41
*** alexpilotti has quit IRC22:44
*** namnh has joined #openstack-ironic22:46
*** rpioso is now known as rpioso|afk22:48
*** bdodd has quit IRC22:56
*** alexpilotti has joined #openstack-ironic22:57
*** bdodd has joined #openstack-ironic22:57
*** hongbin has quit IRC23:00
*** lbragstad_ has joined #openstack-ironic23:01
*** alexpilo_ has quit IRC23:01
*** lbragstad has quit IRC23:01
*** med_ has joined #openstack-ironic23:19
*** med_ has quit IRC23:19
*** med_ has joined #openstack-ironic23:19
*** namnh has quit IRC23:41

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