Tuesday, 2019-07-02

*** cdearborn has quit IRC00:11
*** hblixt has quit IRC00:12
*** hamzy has quit IRC00:35
*** gyee has quit IRC00:37
*** ijw has quit IRC00:43
openstackgerritVanou Ishii proposed openstack/ironic master: Deal with iPXE boot interface incompatibility in Train  https://review.opendev.org/66809702:04
openstackgerritzhaixiaojun proposed openstack/ironic master: Bump openstackdocstheme to 1.20.0  https://review.opendev.org/66347402:08
*** hwoarang has quit IRC02:09
openstackgerritzhaixiaojun proposed openstack/ironic master: Bump openstackdocstheme to 1.20.0  https://review.opendev.org/66347402:18
openstackgerritzhaixiaojun proposed openstack/python-ironicclient master: Bump openstackdocstheme to 1.20.0  https://review.opendev.org/66347102:23
openstackgerritzhaixiaojun proposed openstack/ironic-python-agent master: Bump openstackdocstheme to 1.20.0  https://review.opendev.org/66347502:37
*** hwoarang has joined #openstack-ironic02:42
*** hamzy has joined #openstack-ironic03:18
*** whoami-rajat has joined #openstack-ironic03:42
*** igordc has quit IRC03:51
*** rachit7 has joined #openstack-ironic04:08
*** ricolin has joined #openstack-ironic04:25
*** stendulker has joined #openstack-ironic04:33
*** zaneb has quit IRC04:34
*** zaneb has joined #openstack-ironic04:35
*** hblixt has joined #openstack-ironic04:43
*** zbitter has joined #openstack-ironic04:57
*** zaneb has quit IRC04:59
*** dansmith has quit IRC05:04
*** dansmith has joined #openstack-ironic05:22
*** yaawang has quit IRC05:24
*** diga has joined #openstack-ironic05:28
digaGood Morning Ironic!05:28
digastendulker Hi05:28
stendulkerGood morning diga05:29
digastendulker - My OOB introspection patch is passing all the zuul checks, please review - https://review.opendev.org/#/c/648360/05:29
patchbotpatch 648360 - ironic (stable/rocky) - Fix OOB introspection to use pxe_enabled flag in i... - 20 patch sets05:29
stendulkerdiga: Does this change applies only to rocky?05:31
digastendulker Yes rocky and previous releases05:31
digaNot only for rocky05:32
stendulkerok, not required on master?05:32
digaBut current change is applies only for rocky05:32
digaMaster change is already merged05:32
stendulkerIs this fix same as in master?05:33
digaYes, let me post a link here for master change05:33
digastendulker https://review.opendev.org/#/c/617951/05:37
patchbotpatch 617951 - ironic - Fix OOB introspection to use pxe_enabled flag in i... (MERGED) - 17 patch sets05:37
digaThis is the master change already merged05:37
stendulkerok05:38
digaActually it requires dracclient version == 1.5.0 to list the nics settings but previous release only support dracclient==1.3.0 which does'n;t have list_nic_Setings method support05:38
digaso I brought in all the required changes in this patch instead of upgrading previous releases dracclient version05:39
stendulkerok05:39
*** zbitter has quit IRC05:44
*** yaawang has joined #openstack-ironic06:04
*** mkowalski has joined #openstack-ironic06:06
openstackgerritpengyuesheng proposed openstack/ironic-ui master: Follow the new PTI for document build  https://review.opendev.org/66728206:15
*** jtomasek has joined #openstack-ironic06:15
*** yedongcan has joined #openstack-ironic06:29
*** yaawang has quit IRC06:34
*** yaawang has joined #openstack-ironic06:35
*** Emine has joined #openstack-ironic06:36
*** Emine has quit IRC06:37
arne_wiebalckgood morning, ironic!06:57
*** tssurya has joined #openstack-ironic07:13
*** tesseract has joined #openstack-ironic07:15
*** rachit7 has quit IRC07:20
*** sri_ has quit IRC07:31
*** ricolin_ has joined #openstack-ironic07:35
*** ricolin has quit IRC07:37
*** e0ne has joined #openstack-ironic07:49
*** ricolin__ has joined #openstack-ironic07:49
*** ricolin_ has quit IRC07:51
*** rpittau|afk is now known as rpittau07:54
rpittaugood morning ironic! o/07:54
*** lucasagomes has joined #openstack-ironic07:59
*** pcaruana has joined #openstack-ironic08:01
*** hblixt has quit IRC08:10
*** trident has quit IRC08:10
*** dougsz has joined #openstack-ironic08:12
*** trident has joined #openstack-ironic08:13
iurygregorymorning rpittau o/08:18
rpittauhey iurygregory :)08:18
openstackgerritparesh sao proposed openstack/ironic master: [WIP]oob disk erase patch  https://review.opendev.org/64158208:30
openstackgerritKaifeng Wang proposed openstack/ironic-inspector master: Use oslo_db.sqlalchemy.test_fixtures  https://review.opendev.org/66863308:36
openstackgerritWill Szumski proposed openstack/bifrost master: Make sure deploy_logs_local_path exists  https://review.opendev.org/66852908:57
*** ricolin_ has joined #openstack-ironic09:04
*** ricolin__ has quit IRC09:07
*** Lucas_Gray has joined #openstack-ironic09:11
*** weystrom has joined #openstack-ironic09:17
*** priteau has joined #openstack-ironic09:17
*** diga has quit IRC09:29
*** ricolin_ is now known as ricolin09:33
*** diga has joined #openstack-ironic09:40
*** whoami-rajat has quit IRC09:42
*** ricolin_ has joined #openstack-ironic09:47
mkraiHi Ironic, good morning o/09:49
*** ricolin has quit IRC09:49
mkraiCan someone please help review https://review.opendev.org/#/c/659550/09:49
mkraihttps://review.opendev.org/#/c/662980/?09:49
patchbotpatch 659550 - ironic - Add IntelIPMIHardware - 8 patch sets09:49
patchbotpatch 662980 - ironic - Add documentation for IntelIPMI hardware - 5 patch sets09:49
*** priteau has quit IRC09:57
*** dougsz has quit IRC10:01
*** dougsz has joined #openstack-ironic10:03
*** priteau has joined #openstack-ironic10:06
*** diga has quit IRC10:08
*** rachit7 has joined #openstack-ironic10:16
*** sri_ has joined #openstack-ironic10:23
openstackgerritKaifeng Wang proposed openstack/ironic-inspector master: Use oslo_db.sqlalchemy.test_fixtures  https://review.opendev.org/66863310:51
*** yedongcan has quit IRC11:17
openstackgerritMerged openstack/ironic master: devstack: Install arch specific debs only when deploying to that arch  https://review.opendev.org/66818611:27
*** stendulker has quit IRC11:28
*** rh-jelabarre has joined #openstack-ironic11:39
*** priteau has quit IRC11:41
*** dougsz has quit IRC11:46
*** sri_ has quit IRC12:00
openstackgerritVarsha Verma proposed openstack/sushy master: Refactor DurableName identifier and Protocol fields  https://review.opendev.org/66866212:07
TheJuliagood morning everyone12:26
TheJuliamkrai: they are on the list for this week12:27
rpittauhi TheJulia :)12:28
*** jcoufal has joined #openstack-ironic12:28
*** jaypipes has joined #openstack-ironic12:46
*** bobmel has quit IRC12:49
*** bfournie has joined #openstack-ironic12:52
*** sthussey has joined #openstack-ironic12:53
*** priteau has joined #openstack-ironic12:56
*** ricolin_ is now known as ricolin12:56
*** cdearborn has joined #openstack-ironic12:57
openstackgerritJulia Kreger proposed openstack/ironic-inspector master: Comment out race prone sqlite db model sync test  https://review.opendev.org/66867313:03
TheJuliarpittau: and any cores around, this should fix the ironic-inspector gate and allow us to get patches moving again13:04
TheJulia^^^^^13:04
mkraiTheJulia: good morning, thank you13:07
*** priteau has quit IRC13:13
*** priteau has joined #openstack-ironic13:13
*** dougsz has joined #openstack-ironic13:13
*** mbeierl has joined #openstack-ironic13:18
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add Thermal resource emulation  https://review.opendev.org/66672713:22
*** rloo has joined #openstack-ironic13:22
rpittauoh thanks TheJulia !13:23
openstackgerritMerged openstack/ironic master: Remove deprecated app.wsgi script  https://review.opendev.org/66851513:23
*** zaneb has joined #openstack-ironic13:35
*** zaneb has quit IRC13:43
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add Indicator LED resource emulation  https://review.opendev.org/65593613:44
*** zaneb has joined #openstack-ironic13:46
*** iurygregory_ has joined #openstack-ironic13:49
*** iurygregory has quit IRC13:51
*** rachit7 has quit IRC14:02
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add Virtual Media support  https://review.opendev.org/65679214:13
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add `get/set_boot_image` to libvirt driver  https://review.opendev.org/65693814:13
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add Thermal resource emulation  https://review.opendev.org/66672714:13
*** Lucas_Gray has quit IRC14:32
*** Lucas_Gray has joined #openstack-ironic14:33
*** sri_ has joined #openstack-ironic14:56
sri_rpioso, morning15:04
*** gyee has joined #openstack-ironic15:04
sri_rpioso, while doing the introspection I am capturing traffic with tcpdump I see this output : IP undercloud.localdomain.38149 > undercloud.localdomain.tftp:  47 RRQ "map-file" octet tsize 0 blksize 512 timeout 315:06
*** Lucas_Gray has quit IRC15:10
TheJuliagood morning15:17
sri_rpioso, here is my full log output, instackenv.json and undercloud.conf file http://paste.openstack.org/show/753768/15:19
*** e0ne has quit IRC15:27
*** tssurya has quit IRC15:40
*** gyee has quit IRC16:12
*** iurygregory_ has quit IRC16:16
rpittaugood night! o/16:21
*** rpittau is now known as rpittau|afk16:21
arne_wiebalckbye, everyone o/16:23
*** gyee has joined #openstack-ironic16:25
TheJuliaCan I get another core reviewer to take a look at https://review.opendev.org/#/c/668673/ ?16:27
patchbotpatch 668673 - ironic-inspector - Comment out race prone sqlite db model sync test - 1 patch set16:27
TheJuliajroll: rloo ^^^16:27
rlooTheJulia: am in meetings. Will look at it later today when I get a break.16:29
*** lucasagomes has quit IRC16:32
TheJuliak, its just commenting out a racey unit test. Commit message should explain it completely.16:44
*** dougsz has quit IRC16:47
*** priteau has quit IRC16:48
jrollTheJulia: +W16:48
openstackgerritJulia Kreger proposed openstack/ironic-inspector master: Lets temporarily disable broken templates so we can fix gate  https://review.opendev.org/66871216:49
openstackgerritJulia Kreger proposed openstack/ironic-inspector master: Lets temporarily disable broken templates so we can fix gate  https://review.opendev.org/66871216:49
openstackgerritJulia Kreger proposed openstack/ironic-inspector master: Revert "Lets temporarily disable broken templates so we can fix gate"  https://review.opendev.org/66871316:50
TheJuliaOkay, that is a little extreme, but that will at least allow us to get the bindep fix in plaec to fix everything else16:50
TheJuliaA sea of chickens and eggs16:50
TheJuliajroll: if you wouldn't mind doing the quick honors, above for the commenting out, I'll recheck the bindep fix, and once that merges I'll approve the revert16:51
jrollTheJulia: that's what I meant I did :)16:54
TheJuliayeah, except the job can't be commented out without bindep change merging, which can't land without the other change merging and bindep also has backports stacked up16:56
*** tesseract has quit IRC17:01
*** bobmel has joined #openstack-ironic17:04
*** ricolin_ has joined #openstack-ironic17:05
*** ricolin has quit IRC17:07
*** ricolin_ has quit IRC17:14
rpiososri_: Good morning17:15
* TheJulia pushes buttons steps away to run errand17:15
jrollTheJulia: bah, I misunderstood, didn't realize there was another commenting out thing17:31
*** efried has joined #openstack-ironic17:32
TheJuliaJroll: no worries! :) I didn't realize the complete quagmire until I spotted a comment from riccardo17:46
*** cdearborn_ has joined #openstack-ironic17:50
*** cdearborn_ has quit IRC17:50
*** gyee has quit IRC17:52
*** efried has left #openstack-ironic17:54
*** ijw has joined #openstack-ironic17:58
*** gyee has joined #openstack-ironic18:06
*** igordc has joined #openstack-ironic18:13
openstackgerritBrian Haley proposed openstack/ironic master: DNM: test patch to use f29 nodeset from devstack  https://review.opendev.org/66872518:17
*** kaiokmo has joined #openstack-ironic18:29
openstackgerritMark Beierl proposed openstack/ironic master: Fix potential race condition on node power on and reboot  https://review.opendev.org/63383318:31
*** zaneb has quit IRC19:01
rpiososri_: Is map-file in the correct location? What about its perms and size?19:13
*** zaneb has joined #openstack-ironic19:49
*** bnemec has quit IRC19:54
*** mmethot has quit IRC20:04
*** mmethot has joined #openstack-ironic20:05
*** bnemec has joined #openstack-ironic20:07
*** johnsag has joined #openstack-ironic20:08
*** johnsag has quit IRC20:14
NobodyCamGood Afternoon Ironic'ers20:25
openstackgerritJulia Kreger proposed openstack/ironic-inspector master: Lets temporarily disable broken templates so we can fix gate  https://review.opendev.org/66871220:29
openstackgerritJulia Kreger proposed openstack/ironic-inspector master: Revert "Lets temporarily disable broken templates so we can fix gate"  https://review.opendev.org/66871320:30
*** ijw has quit IRC20:32
*** Lucas_Gray has joined #openstack-ironic20:34
*** bfournie has quit IRC20:36
*** Lucas_Gray has quit IRC20:37
*** tssurya has joined #openstack-ironic20:37
*** ijw has joined #openstack-ironic20:37
mbeierlTheJulia: is https://review.opendev.org/#/c/668712/3 the cause of openstack-tox-py36 and openstack-tox-py37 showing RETRY_LIMIT?20:49
patchbotpatch 668712 - ironic-inspector - Lets temporarily disable broken templates so we ca... - 3 patch sets20:49
*** Lucas_Gray has joined #openstack-ironic20:52
TheJuliambeierl: no, the cause is the bindep files that are missing in that repo20:54
TheJuliaso a tempoary disable of the tests will let us merge the two things that need to be able to be merged to unbrick the gate20:54
mbeierlTheJulia: ok, so the patch I submitted is not broken, but I will have to do something to kick off a new Zuul check sometime later?20:55
TheJuliambeierl: just leave a "recheck" comment20:55
TheJuliathat will fire zuul back up20:55
mbeierlis it ok to do so now, or should I wait?20:56
TheJuliambeierl: if there is a specific patch, I can trigger it for you if you want20:56
mbeierlI can do the recheck comment.  It's https://review.opendev.org/#/c/63383320:56
patchbotpatch 633833 - ironic - Fix potential race condition on node power on and ... - 7 patch sets20:56
TheJuliawait until it has merged, but I'd prefer to only merge the two things that are mutually needed20:56
TheJuliaoh... THAT20:56
mbeierlhaha, yes, that!20:56
TheJuliajust go ahead and try recheck now20:57
mbeierlok20:57
*** mmethot has quit IRC20:58
*** pcaruana has quit IRC20:58
*** mmethot has joined #openstack-ironic20:58
mbeierlThanks for your help!21:01
*** Lucas_Gray has quit IRC21:03
*** Lucas_Gray has joined #openstack-ironic21:04
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: manual introspection trigger command  https://review.opendev.org/66619821:06
*** priteau has joined #openstack-ironic21:07
*** priteau has quit IRC21:07
*** jcoufal_ has joined #openstack-ironic21:12
*** ijw has quit IRC21:13
*** ijw has joined #openstack-ironic21:14
*** jcoufal has quit IRC21:16
*** ijw has quit IRC21:19
TheJuliambeierl: no problem, happy to help! :)21:21
*** jcoufal_ has quit IRC21:22
*** bnemec has quit IRC21:25
*** bnemec has joined #openstack-ironic21:27
stevebakermorning21:36
TheJuliaGood morning!21:37
*** tssurya has quit IRC21:50
TheJuliaLooks like we're just pending a grenade job before I'll be able to recheck the two patches to fix-up ironic-inspector22:11
openstackgerritMerged openstack/ironic stable/stein: redfish: handle missing Bios attribute  https://review.opendev.org/66586622:29
*** jcoufal has joined #openstack-ironic22:39
TheJuliahmm. so ironic-inspector is running a full tempest sweep instead of scenarios22:40
TheJulias/of/of just/22:40
*** sthussey has quit IRC23:32
*** cxc99 has quit IRC23:44

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