Friday, 2018-03-16

*** rcernin has joined #openstack-ironic00:21
*** mjturek has quit IRC00:28
*** AlexeyAbashkin has joined #openstack-ironic00:32
*** AlexeyAbashkin has quit IRC00:36
*** hshiina has joined #openstack-ironic00:37
*** harlowja has quit IRC00:49
*** liuzz has joined #openstack-ironic00:50
*** hamzy__ has joined #openstack-ironic00:57
*** phuongnh has joined #openstack-ironic01:03
*** itlinux has joined #openstack-ironic01:13
*** tiendc has joined #openstack-ironic01:17
*** tiendc has joined #openstack-ironic01:18
*** phuongnh has quit IRC01:32
*** phuongnh has joined #openstack-ironic01:33
*** OctopusZhang has joined #openstack-ironic01:41
*** AlexeyAbashkin has joined #openstack-ironic01:52
*** AlexeyAbashkin has quit IRC01:56
*** phuongnh has quit IRC01:58
*** phuongnh has joined #openstack-ironic01:58
*** mikal_ has joined #openstack-ironic02:03
*** mikal has quit IRC02:06
openstackgerritRuby Loo proposed openstack/ironic master: Copy port[group] VIF info from extra to internal_info  https://review.openstack.org/51163602:28
*** tuanla____ has joined #openstack-ironic02:28
*** OctopusZhang__ has joined #openstack-ironic02:29
*** OctopusZhang has quit IRC02:29
openstackgerritRuby Loo proposed openstack/ironic master: Copy port[group] VIF info from extra to internal_info  https://review.openstack.org/51163602:42
*** hieulq has quit IRC02:54
*** Goneri has quit IRC02:55
openstackgerritZachary Ma proposed openstack/networking-generic-switch master: Huawei vrpv8 switch support port group  https://review.openstack.org/55192202:55
*** itlinux has quit IRC02:56
*** fragatina has quit IRC03:02
*** rbudden has quit IRC03:05
*** itlinux has joined #openstack-ironic03:09
openstackgerritZachary Ma proposed openstack/ironic master: add warning log about empty pxe_enabled_ports  https://review.openstack.org/53132003:11
openstackgerritZachary Ma proposed openstack/ironic master: add warning log about empty pxe_enabled_ports  https://review.openstack.org/53132003:24
openstackgerritzenghui.shi proposed openstack/ironic master: Add DB model for bios setting table  https://review.openstack.org/51116203:27
openstackgerritOpenStack Proposal Bot proposed openstack/networking-generic-switch master: Updated from global requirements  https://review.openstack.org/55367203:29
*** r-daneel has quit IRC03:36
*** OctopusZhang__ has quit IRC03:39
*** OctopusZhang__ has joined #openstack-ironic03:40
*** OctopusZhang__ has quit IRC03:41
*** r-daneel has joined #openstack-ironic03:53
openstackgerritOpenStack Proposal Bot proposed openstack/virtualbmc master: Updated from global requirements  https://review.openstack.org/55241103:55
*** r-daneel_ has joined #openstack-ironic03:55
*** crushil has joined #openstack-ironic03:57
*** r-daneel has quit IRC03:57
*** r-daneel_ is now known as r-daneel03:57
*** fragatina has joined #openstack-ironic04:08
*** fragatina has quit IRC04:10
*** mikal_ has quit IRC04:17
*** mikal has joined #openstack-ironic04:20
openstackgerritzenghui.shi proposed openstack/ironic master: Add DB api layer for bios setting operations  https://review.openstack.org/51140204:20
*** OctopusZhang__ has joined #openstack-ironic04:26
*** links has joined #openstack-ironic04:32
*** harlowja has joined #openstack-ironic04:35
openstackgerritNguyen Hai proposed openstack/ironic master: Follow the new PTI for document build  https://review.openstack.org/55347305:02
*** harlowja has quit IRC05:22
openstackgerritNguyen Hai proposed openstack/ironic master: Follow the new PTI for document build  https://review.openstack.org/55347305:28
*** moshele has joined #openstack-ironic05:32
*** dpawar has joined #openstack-ironic05:34
*** moshele has quit IRC05:40
*** jtomasek has joined #openstack-ironic05:41
*** hw_wutianwei has quit IRC05:45
*** hw_wutianwei has joined #openstack-ironic05:45
*** jaganathan has joined #openstack-ironic05:45
openstackgerritKaifeng Wang proposed openstack/ironic-specs master: Support baremetal inspection abort operation  https://review.openstack.org/48286705:47
openstackgerritZachary Ma proposed openstack/networking-generic-switch master: Huawei vrpv8 switch support port group  https://review.openstack.org/55192205:48
openstackgerritZachary Ma proposed openstack/ironic master: add warning log about empty pxe_enabled_ports  https://review.openstack.org/53132006:00
*** dpawar has quit IRC06:01
*** skazi has quit IRC06:12
openstackgerritNguyen Hai proposed openstack/ironic master: Follow the new PTI for document build  https://review.openstack.org/55347306:14
openstackgerritKaifeng Wang proposed openstack/ironic-specs master: Support baremetal inspection abort operation  https://review.openstack.org/48286706:23
*** crushil has quit IRC06:25
*** dpawar has joined #openstack-ironic06:27
*** phuongnh has quit IRC06:28
*** phuongnh has joined #openstack-ironic06:29
*** AlexeyAbashkin has joined #openstack-ironic06:36
*** fragatina has joined #openstack-ironic06:43
openstackgerritMerged openstack/networking-generic-switch master: Use save_config provided by netmiko  https://review.openstack.org/54823806:43
*** fragatina has quit IRC06:43
*** fragatina has joined #openstack-ironic06:43
*** fragatin_ has joined #openstack-ironic06:44
openstackgerritNguyen Hai proposed openstack/ironic master: Follow the new PTI for document build  https://review.openstack.org/55347306:48
*** fragatina has quit IRC06:48
*** OctopusZhang has joined #openstack-ironic06:49
*** OctopusZhang__ has quit IRC06:49
*** OctopusZhang is now known as yufei06:49
*** OctopusZhang has joined #openstack-ironic06:49
*** AlexeyAbashkin has quit IRC06:51
*** jtomasek has quit IRC06:52
*** yufei has quit IRC06:53
*** OctopusZhang is now known as yufei06:53
*** mjura has joined #openstack-ironic06:58
*** jcoufal has joined #openstack-ironic07:12
TheJuliadtantsur|afk: Take a look at https://review.openstack.org/553699 when you have a chance.07:14
patchbotpatch 553699 - nova - WIP: Update Ironic UTs to enable REST migration07:14
*** dpawar has quit IRC07:16
*** dpawar has joined #openstack-ironic07:16
*** yolanda_ is now known as yolanda07:16
* TheJulia goes and gets some sleep07:17
*** rcernin has quit IRC07:20
*** marios has joined #openstack-ironic07:24
*** phuongnh has quit IRC07:27
*** phuongnh has joined #openstack-ironic07:27
*** zshi_ has quit IRC07:38
*** zshi_ has joined #openstack-ironic07:39
*** skazi has joined #openstack-ironic07:42
*** itlinux has quit IRC07:46
*** OctopusZhang_ has joined #openstack-ironic07:47
*** yufei has quit IRC07:47
*** OctopusZhang has joined #openstack-ironic07:48
*** AlexeyAbashkin has joined #openstack-ironic07:50
*** OctopusZhang_ has quit IRC07:52
*** OctopusZhang is now known as yufei07:52
*** jcoufal_ has joined #openstack-ironic08:09
*** jcoufal has quit IRC08:10
*** hoonetorg has quit IRC08:11
*** etingof has joined #openstack-ironic08:19
*** stendulker has joined #openstack-ironic08:25
*** hoonetorg has joined #openstack-ironic08:27
*** tesseract has joined #openstack-ironic08:30
*** moshele has joined #openstack-ironic08:32
*** OctopusZhang_ has joined #openstack-ironic08:45
*** yufei has quit IRC08:45
*** OctopusZhang has joined #openstack-ironic08:46
*** OctopusZhang is now known as yufei08:50
*** moshele has quit IRC08:50
*** OctopusZhang_ has quit IRC08:50
pas-hamorning ironic :-)09:01
etingofo/09:07
*** phuongnh has quit IRC09:09
*** MattMan has quit IRC09:13
*** MattMan has joined #openstack-ironic09:13
openstackgerritAnshul Jain proposed openstack/ironic-specs master: Update OOB RAID for iLO5 based HPE Proliant Servers  https://review.openstack.org/53083809:18
*** hshiina has quit IRC09:19
*** jistr is now known as jistr|mtg09:24
*** mgoddard_ has joined #openstack-ironic09:38
*** lucas-afk is now known as lucasagomes09:38
*** derekh has joined #openstack-ironic09:40
*** dtantsur|afk is now known as dtantsur09:42
dtantsurmorning ironic09:42
dtantsurTheJulia: sure, will do09:42
dtantsurlucasagomes: hey, quick question. do you remember if we're able to store ramdisk logs only on deploy or also on cleaning?09:45
*** etingof has quit IRC09:45
*** liuzz has quit IRC09:47
lucasagomesdtantsur, only deployment, we call it at the method that turn down the machine AFAIR09:48
lucasagomesfinish_deploy() or something like that09:48
dtantsuroh, that's bad09:48
dtantsurthanks lucasagomes09:48
lucasagomes(or when something errors out as well)09:48
openstackgerritzenghui.shi proposed openstack/ironic master: BIOS Settings: Add DB model  https://review.openstack.org/51116209:48
openstackgerritzenghui.shi proposed openstack/ironic master: BIOS Settings: Add DB API  https://review.openstack.org/51140209:48
lucasagomesit's easy to extend, the method should be quite generic09:48
lucasagomesit's an ipa command (collect_system_logs)09:48
lucasagomes:q!09:48
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: Use the main introspection test instead of the smoke one  https://review.openstack.org/55347709:51
dtantsurlucasagomes: one cannot just quit vim!!09:53
lucasagomesdtantsur, lol09:53
lucasagomestried to quit prior to switching windows... morning man09:53
dtantsurheh, yeah :)09:54
openstackgerritDao Cong Tien proposed openstack/ironic master: Follow-up patch for rescue mode devstack change  https://review.openstack.org/55373409:55
dtantsurvdrok, pas-ha, sambetts|afk or whoever remembers: do we run this test https://github.com/openstack/ironic-tempest-plugin/blob/master/ironic_tempest_plugin/tests/scenario/test_baremetal_multitenancy.py#L86 anywhere in the CI?09:57
*** early has quit IRC09:57
pas-haAFAIR on the multinode job09:57
dtantsurpas-ha: no, we run smoke tests there. and I suspect it's a bug09:58
*** jtomasek has joined #openstack-ironic09:59
*** jtomasek has quit IRC10:00
*** jtomasek has joined #openstack-ironic10:00
*** mgoddard_ has quit IRC10:01
*** early has joined #openstack-ironic10:01
openstackgerritDmitry Tantsur proposed openstack/ironic master: Gate: run ironic tests in the regular multinode job  https://review.openstack.org/55373610:01
dtantsurlet's try this ^^^10:01
dtantsurTheJulia: ^^^10:01
openstackgerritzenghui.shi proposed openstack/ironic master: BIOS Settings: Add DB model  https://review.openstack.org/51116210:03
openstackgerritzenghui.shi proposed openstack/ironic master: BIOS Settings: Add DB API  https://review.openstack.org/51140210:03
*** jcoufal has joined #openstack-ironic10:06
*** jcoufal_ has quit IRC10:09
dtantsurpas-ha: see my comment on https://review.openstack.org/#/c/553736/ I think you may be confusing something10:17
patchbotpatch 553736 - ironic - Gate: run ironic tests in the regular multinode job10:17
*** e0ne has joined #openstack-ironic10:21
*** mgoddard has joined #openstack-ironic10:26
*** etingof has joined #openstack-ironic10:27
*** andymccr_ is now known as andymccr10:27
*** Nisha_Agarwal has joined #openstack-ironic10:28
openstackgerritMerged openstack/ironic-specs master: Introduce inspect wait state  https://review.openstack.org/51565710:44
*** jaganathan has quit IRC10:48
*** Nisha_ has joined #openstack-ironic10:51
*** Nisha_Agarwal has quit IRC10:51
*** Nisha_Agarwal has joined #openstack-ironic10:54
*** Nisha_ has quit IRC10:54
openstackgerritAija proposed openstack/sushy-tools master: Make simulators proper system daemons  https://review.openstack.org/55198410:55
*** ajya has joined #openstack-ironic10:59
*** jistr|mtg is now known as jistr11:07
*** stendulker has quit IRC11:11
openstackgerritDmitry Tantsur proposed openstack/ironic master: Prevent overwriting of last_error on cleaning failures  https://review.openstack.org/54665611:12
*** Nisha_Agarwal has quit IRC11:12
*** Nisha_ has joined #openstack-ironic11:12
openstackgerritAnkit Kumar proposed openstack/ironic master: Add 'inspect_ports' as a 'driver_info' parameter to iLO drivers  https://review.openstack.org/32523011:15
openstackgerritMerged openstack/ironic master: Updated from global requirements  https://review.openstack.org/55318111:19
*** Nisha_away has joined #openstack-ironic11:27
*** Nisha_ has quit IRC11:27
*** Nisha_ has joined #openstack-ironic11:34
*** Nisha_away has quit IRC11:34
*** Nisha_away has joined #openstack-ironic11:36
*** Nisha_ has quit IRC11:36
openstackgerritAija proposed openstack/sushy-tools master: Make simulators proper system daemons  https://review.openstack.org/55198411:59
openstackgerritJacek Tomasiak proposed openstack/ironic stable/pike: Use internal Swift endpoint by default  https://review.openstack.org/55333012:10
*** edleafe is now known as figleaf12:12
olivierbourdon38hello ironicers, basic question, I tried to bootstrap 2 VMs using Bifrost/Ironic each with 2 disks 1 being 30Gb wide, other being 60Gb big. The  VMs only differ with the "order" of the disks aka vda smaller in VM1 vdb smaller on VM2. The QCOW2 image I am trying to provision being 50Gb I was very surprised that the IPA returned "properties/local_gb": "29" in both cases and that both deployment fails with write_image.sh12:13
olivierbourdon38 saying "Device is too small"12:13
olivierbourdon38is this expected ? do I have a way to solve this somehow ?12:14
dtantsurolivierbourdon38: this is the default behavior, as weird as it may be sometimes. you need https://docs.openstack.org/ironic/latest/install/advanced.html#specifying-the-disk-for-deployment-root-device-hints12:14
*** sambetts|afk is now known as sambetts12:15
sambettsolivierbourdon38: default behaviour in IPA is select the first HDD bigger than 4gb as the root disk12:16
sambettsdtantsur: I'm surprised we don't try to select the first disk that is big enough to fit the image in...12:17
dtantsursambetts: well, on introspection we don't know the image size...12:18
olivierbourdon38ok, got it, very surprising indeed but I am wondering what "first" means in this case sambetts as I mentioned the disks are vda 30Gb vdb 60Gb in VM1 and vda 60Gb vdb 30Gb as  mentioned above12:18
olivierbourdon38therfore first should not be the same across these 2 VMs12:18
dtantsurolivierbourdon38: the rule is "the first *smallest* disk that is larger than 4 GiB"12:18
dtantsurlegacy from.. hmm.. long ago12:19
olivierbourdon38ok got it12:19
olivierbourdon38thx for the help12:19
sambettsdtantsur: we don't select the HDD on inspection do we? we select it at deploy image12:19
dtantsursambetts: we do. we populate local_gb, for example12:19
sambettsdtantsur: but that is (was) just used for scheduling but for selecting the disk at deploy time12:21
etingofsambetts, hey, thanks for the idea on running `qemu-img check`! I'm thinking to add that as a function to ironic-lib, WDYT? -- https://review.openstack.org/#/c/544839/12:21
patchbotpatch 544839 - ironic - Raw image size estimation improved12:21
sambettsnot for selecting the disk at the deploy time12:21
sambettsetingof: no problem, ideally we'll be able to use qemu-img measure in the future, but thats only avaiable in newer version of qemu utils12:22
dtantsursambetts: yeah, but we use local_gb to validate image size even before we boot IPA12:23
*** tiendc has quit IRC12:23
sambettsdtantsur: ah...12:24
sambettsyeah12:24
sambettsthe rabbit hole goes deeper12:24
etingofsambetts, should we handle the case of older qemu tooling that does not support the check feature? or it is somehow insured by the dependencies?12:25
dtantsurI guess we should check the versions in the popular distros12:26
sambettsetingof: check has been around for long time from what I can tell, and all the major distros have it now from what I can tell12:26
sambettsetingof: but qemu-img measure hasn't made it out to all of them yet12:27
sambettsnot sure if we state min versions for our bindeps though12:27
etingofsambetts, ah, I see. so should we leave that multiply factor as a fallback?12:27
etingofsambetts, alternatively, we can just use the virtual image size as we used to do12:28
sambettsI think i'd prefer that12:28
dtantsurI'd prefer to fall back to the old logic, as it seems paranoid, but safe12:29
sambettsbasically work out the image's sparsness if we can, otherwise assume its not sparse12:29
sambettsand use the virtual size12:30
*** tuanla____ has quit IRC12:30
etingofalright, thanks!12:31
*** dprince has joined #openstack-ironic12:34
*** NightMonkeydk has joined #openstack-ironic12:35
dtantsursambetts: wdyt about https://review.openstack.org/#/c/553477/ ? I don't remember if we discussed it yesterday or not12:35
patchbotpatch 553477 - ironic-tempest-plugin - Use the main introspection test instead of the smo...12:35
sambettsdtantsur: personally I think it makes sense12:36
dtantsurcool12:40
dtantsursambetts: also see https://review.openstack.org/553736 if I get it right, we're not really testing multitenancy in the CI now12:41
patchbotpatch 553736 - ironic - Gate: run ironic tests in the regular multinode job12:41
*** trown|outtypewww is now known as trown12:48
*** moshele has joined #openstack-ironic12:49
*** trown is now known as trown|ruck12:49
sambettsdtantsur: I think as pas-ha said the default tempest smoke suite tests cover our multi-tenancy for us12:52
sambettsbecause we don't run with the ironic regex, we get the neutron multi-node multi-tenancy test for free12:52
dtantsursambetts: as I said to pas-ha, it does not seem to be true12:54
*** moshele has quit IRC12:54
dtantsurby default the smoke tests are run, and they don't include the actual multi-tenancy as far as I understand12:54
sambettsdtantsur: http://logs.openstack.org/15/548215/5/check/ironic-tempest-dsvm-ipa-wholedisk-agent_ipmitool-tinyipa-multinode/ccd71ee/job-output.txt.gz#_2018-02-28_18_22_38_07881412:54
dtantsursambetts: this is basic_ops, right?12:55
sambettshuh... hold on have they changed something...12:55
*** ElCoyote_ has joined #openstack-ironic12:55
dtantsuralso, what's the reason to run smoke tests, given that they are really long for ironic?12:56
dtantsurwe do it in grenade because we're forced to, but why here?12:56
dtantsurand it seems that https://github.com/openstack/ironic-tempest-plugin/blob/master/ironic_tempest_plugin/tests/scenario/test_baremetal_multitenancy.py dpes mpt run anywhere because of this12:56
dtantsur* does not12:56
sambettsjust trying to work out what I'd do in my multinode third party CI because I'm sure I run the neutron one12:57
*** jlabarre has quit IRC12:58
dtantsurok12:58
dtantsurin the end, we either have to remove https://github.com/openstack/ironic-tempest-plugin/blob/master/ironic_tempest_plugin/tests/scenario/test_baremetal_multitenancy.py and fully rely on neutron, or switch to our tests12:58
dtantsurI'd prefer the latter, but I can be convinced otherwise12:59
*** jlabarre has joined #openstack-ironic13:00
jrolltest_connectivity_between_vms_on_different_networks has been skipped for two years O_O13:01
jroll(morning)13:01
dtantsurmorning jroll, right13:01
dtantsurand it's not a smoke test in any case13:01
dtantsurso we either have to explicitly use it or explicitly use our tests13:01
jrollyeah, I hit the blame to see if/when it was a smoke test in the past and noticed that13:01
*** gyankum has joined #openstack-ironic13:01
jrolldtantsur: just curious, what makes it run smoke tests instead of all tests?13:02
dtantsurjroll: it seems to be the default if no regex is provided; I did not dive deeper13:02
jrollah ok13:02
dtantsurbut i'm pretty sure we don't want tempest-full :D13:02
jrollsure :)13:03
sambettsit depends on which tempest env is used13:03
jrolljust the job definition didn't say smoke, so I assumed it'd be full, which would be pretty cool if we'd been passing it :)13:03
sambettsif you don't provide a regex to devstack-gate it used tox -e smoke13:03
dtantsurI don't like this "smoke" thingy exactly because you never know which tests will be run, and you don't notice when tests come and go13:03
sambettsotherwise you have to set either tempest_full13:03
sambettsor a regex13:04
sambettsthen it uses tox -e all_plugin13:04
jrollsambetts: right, smoke is the default, got it. that's the bit I didn't know13:04
sambettsdefault in devstack-gate not for tempest itself because infra backmagic13:04
sambettsblackmagic*13:05
jrollright13:05
dtantsurheh13:05
sambettsI'm happy to turn on that test then13:05
sambettsI actually have a patch for it too https://review.openstack.org/#/c/541383/13:06
patchbotpatch 541383 - ironic-tempest-plugin - Add [baremetal] available_nodes configuration option13:06
*** lucasagomes is now known as lucas-hungry13:06
sambettsdtantsur: https://review.openstack.org/#/c/552890/5/ironic_lib/disk_utils.py wrt your comment here, what OS is your VM?13:07
patchbotpatch 552890 - ironic-lib - Use blkid instead of hexdump to get mbr disk id13:07
dtantsursambetts: centos713:07
*** Goneri has joined #openstack-ironic13:07
dtantsursambetts: I can make my patch depend on yours to make sure it's covered13:07
sambettsdtantsur: ffs... why does centos7 insist on being 1000 years out of date on every package13:09
sambettsblkid version on centos7 is from 201513:09
openstackgerritDmitry Tantsur proposed openstack/ironic master: Gate: run ironic tests in the regular multinode job  https://review.openstack.org/55373613:09
dtantsursambetts: with depends-on on your patch ^^^13:09
dtantsurjroll: ^^^13:09
dtantsursambetts: loooool13:09
sambettsI hit this on another package recently13:09
dtantsurwell.. stability?13:09
dtantsur;)13:09
jrollolder software isn't stable13:10
sambettsthe other packge I hit it was a bug that had been fix for 5 years13:10
* jroll gives his juno environments a mean look13:10
sambettsbut centos7 was running a 8 year old version13:10
dtantsursigh, ikr?13:10
dtantsurit reminds me of the ipxe ROM from 2015 we used for some time13:11
sambettshmm ok I  might make it so that we try a couple of different methods to find the Part table ID, because on centos7 udevadm info shows it13:12
sambettsbut I hit issues with tiny core's udevadm not showing it13:12
dtantsurcomputers \o/13:12
sambettsits honestly a if you want something doing do it yourself ... I'm seriously thinking of writing python that reads the partition table without using another tool13:14
dtantsurthat's what hexdump does, right? :)13:14
sambettsnot having hexdump in IPA is the problem I'm trying to work around13:15
dtantsuryep13:15
jrollwhy don't we have hexdump in ipa?13:15
jrollw/don't/can't13:15
jrolls/^w/s/13:15
dtantsurTheJulia: when you're aware and caffeinated, could you please check my response on https://review.openstack.org/#/c/546273/4/ironic/conductor/utils.py ?13:16
patchbotpatch 546273 - ironic - Rework logic handling reserved orphaned nodes in t...13:16
sambettsnot sure, TheJulia raised it as not a hard dependency for us, I think because you don't need it installed if your are using iscsi deploy but you do if you are using agent deploy13:16
*** dpawar has quit IRC13:16
sambettsI also have a concern about that code only working for MBR13:17
sambettsbut GPT partition deploys13:17
sambettsbut not*13:17
jrollthe first seems like an odd reason, I'd just list it as a dependency13:17
jrollthe second would be valid13:17
jrollbut fixable13:18
jroll(I assume)13:18
openstackgerritMerged openstack/ironic master: devstack: enabled fake-hardware and fake interfaces  https://review.openstack.org/55081113:18
sambettsI was trying to swtich to one of the tools we already have a hard dependecy on like blkid or udevadm to solve the first issue (but obviously there are derpy issues with that :(), those tools also are a step in the right direction to solving the second issue as they will return either the GPT uuid or mbr serial number13:20
sambettsbut ties to MBR go deeper into ironic than just IPA from what I've found13:20
jrollindeed13:20
jrolldo we expect people to build rhel 7 IPA images?13:21
dtantsurwe have people using rhel 7 IPA images :)13:21
dtantsursince Liberty13:21
* jroll asks, thinking the answer might be yes in his own downstream13:21
jrollokay, figured as much :)13:21
dtantsurtripleo CI uses centos7 IPA images which are public13:21
jrollah13:21
sambettstbh i'd like to move away from reading the part table uuid or serial number, we only need it for chainloading to the hdd (and we don't even use it for chain loading with iPXE)13:22
sambettsI wonder if we'd be safer working out the "hd0" or "hd1" disk id as seen by the bios13:23
*** Nisha_away has quit IRC13:23
*** Nisha_ has joined #openstack-ironic13:23
sambettsthen it doesn't matter whats actually on the disk13:24
sambettsin terms of partitioning13:24
dtantsurjroll: if you wanna see some really fat IPA images :) https://images.rdoproject.org/master/rdo_trunk/current-tripleo/13:27
jrollheh13:27
dtantsurneed 2G RAM even to start :)13:30
*** mjturek has joined #openstack-ironic13:31
*** r-daneel has quit IRC13:34
* dtantsur brb13:34
*** etingof has quit IRC13:36
*** Nisha_ has quit IRC13:37
*** e0ne has quit IRC13:40
*** e0ne has joined #openstack-ironic13:40
*** rbudden has joined #openstack-ironic13:48
openstackgerritMerged openstack/ironic master: Add meaningful exception in Neutron port show  https://review.openstack.org/53364013:49
*** moshele has joined #openstack-ironic13:55
*** links has quit IRC13:56
*** moshele has quit IRC13:58
*** lucas-hungry is now known as lucasagomes14:06
*** baha has joined #openstack-ironic14:06
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: devstack: enable "fake" inspection implementation  https://review.openstack.org/55379714:09
dtantsursambetts: fixed the inspector gate (sigh) ^^^14:09
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: Switch documentation to hardware types  https://review.openstack.org/54528514:10
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: devstack: enable "fake" inspection implementation  https://review.openstack.org/55379714:11
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: Switch documentation to hardware types  https://review.openstack.org/54528514:11
* dtantsur has 41 open patches now, and should probably stop producing more14:14
jrollwow14:15
jrollso much free time since you quit PTL-ship :P14:15
dtantsurexactly!14:15
TheJuliagood morning everyone14:16
dtantsurmorning TheJulia14:16
jrollhey TheJulia :)14:16
* TheJulia attempts to wake up14:16
*** d0ugal has quit IRC14:19
*** crushil has joined #openstack-ironic14:19
openstackgerritMerged openstack/networking-generic-switch master: Updated from global requirements  https://review.openstack.org/55367214:20
openstackgerritMerged openstack/bifrost master: Updated from global requirements  https://review.openstack.org/55274114:20
openstackgerritMerged openstack/bifrost master: Add missing release note for custom partitioning  https://review.openstack.org/55229614:20
*** etingof has joined #openstack-ironic14:21
TheJuliadtantsur: wait... you reviewed the nova unit test changes....14:21
TheJuliadtantsur: and you didn't scream, go crazy, and run for the hills?14:21
*** dansmith is now known as superdan14:22
jrollhe did that a long time ago already14:22
openstackgerritMark Goddard proposed openstack/ironic-tempest-plugin master: Validate scheduling fields in basic ops scenario  https://review.openstack.org/54529714:22
*** r-daneel has joined #openstack-ironic14:23
TheJuliaoh...14:23
TheJuliac'est la vie14:23
*** derekh has quit IRC14:24
*** jcoufal has quit IRC14:25
*** crushil has quit IRC14:26
*** crushil has joined #openstack-ironic14:27
*** gyankum has quit IRC14:29
*** larsks has joined #openstack-ironic14:29
jroll(I meant the screaming, going crazy, and running, to be clear)14:29
jroll:P14:29
larsksHowdy.  How do I talk to ironic running in noauth mode? The 'ironic' and 'openstack baremetal' commands insist on keystone credentials.14:30
jrolllarsks: something with --os-auth-plugin, trying to find what the value would be14:31
jrolllooks like it's probably just 'noauth'14:32
dtantsurTheJulia: jroll is right, yes14:32
jrolloh, it's 'none' https://github.com/openstack/keystoneauth/blob/master/setup.cfg#L4014:32
larsksjroll: hmm, neither ironic nor openstackclient claim to know about the --os-auth-plugin option.14:33
jrollO_o14:34
larsksjroll: looks like I've got python2-ironicclient-2.2.0, if that helps at all.14:35
jrolllarsks: oh, --os-auth-type14:35
jrollat least for openstackclient14:35
larskshuh, openstackclient does appear to accept that, although it doesn't show up in help output :/. But now it says "Set a cloud-name with --os-cloud or OS_CLOUD".14:39
larsksIt looks like I should be able to put together a clouds.yml file that works, I guess. Let me give that a shot.14:40
*** derekh has joined #openstack-ironic14:40
dtantsurI remember it did not work the way you expect it to work last time I checked..14:41
jrollI got it via 'openstack --help | grep auth', fwiw14:43
jrollI'm surprised you need to set --os-cloud14:43
*** dpawar has joined #openstack-ironic14:48
*** links has joined #openstack-ironic14:48
TheJuliathe credential processing code that gets used for sessions really really really wants an OS_CLOUD :(14:51
larsksTheJulia: do you have any idea what's necessary to get either of the clis to talk to a standalone ironic in noauth mode?14:52
larsksI have set OS_CLOUD and I've tried to set up a clouds.yaml, but so far no luck.14:52
dtantsurlast time it was something like --os-token fake --os-endpoint https://ironic14:52
dtantsurwhich is super ugly, I know14:52
TheJuliayeah, larsks what do you have defined in your clouds.yaml and is it in ~/.config/openstack/clouds.yaml?14:53
TheJuliasetting the environment variables still works for ironic cli, osc demands config from clouds.yaml14:53
openstackgerritIlya Etingof proposed openstack/ironic-lib master: Added `get_raw_image_mb()`  https://review.openstack.org/55380714:54
*** milan has joined #openstack-ironic14:54
larsksTheJulia: It's in ~/clouds.yaml, and strace confirms that it's being read.  But I can move it.  It has just an auth_type and ironic_url in a single cloud definition, http://termbin.com/6oq1k14:55
*** bnemec is now known as beekneemech14:55
TheJuliahttps://git.openstack.org/cgit/openstack/bifrost/tree/env-vars is for ironic-cli, environment vars14:55
TheJuliaI think folks fixed bifrost's clouds.yaml, grabbing that url14:55
larsksTheJulia: with what I'm using, openstackclient still says "Set a cloud-name with --os-cloud or OS_CLOUD" (even with OS_CLOUD set).  Looking at the code, it looks like this message can actually mean things like "I don't know how to auth".14:56
TheJuliahttps://git.openstack.org/cgit/openstack/bifrost/tree/playbooks/roles/bifrost-keystone-client-config/templates/clouds.yaml.j2 this supposidly works for basic clouds.yaml format. Keep in mind that the link is a jinja2 template based on conditionals https://git.openstack.org/cgit/openstack/bifrost/tree/playbooks/roles/bifrost-keystone-client-config/templates/clouds.yaml.j214:56
larsksTheJulia: so, it looks like I want the '{% else %}' section there.  That gives me this: http://termbin.com/lk3s14:58
larsks...which results in the same behavior.14:58
TheJuliatry s/url/endpoint_override/... which should be standardied override name14:59
TheJuliathe session code that gets fired up takes me a day dig through anytime I touch it :(14:59
larsksStill complaining about OS_CLOUD.15:00
larsksTo be clear, I'm running: openstack --os-cloud sandbox   baremetal node list15:00
* TheJulia wonders if it is wanting the envvar15:00
*** dprince has quit IRC15:00
larsksTheJulia: I have tried that, too :/15:00
*** dprince has joined #openstack-ironic15:01
*** r-daneel has quit IRC15:01
*** r-daneel has joined #openstack-ironic15:01
* TheJulia looks for one of her noauth vms to reproduce15:02
larsksTheJulia: aha! Looks like --os-auth-type noauth, not --os-auth-type none.15:03
larsks(because of course)'15:03
openstackgerritPavlo Shchelokovskyy proposed openstack/networking-generic-switch master: WIP Use native Zuul job  https://review.openstack.org/55381315:03
*** crushil has quit IRC15:04
TheJuliaugh, and doing the same with clouds.yaml gets different results15:05
TheJuliawoot15:05
larsksUgh. Well.  Now the 'openstack baremetal list' command complains that it wnats endpoint and user-id.  Trying to figure those out...15:05
TheJuliait might be endpoint_override on the command line, in clouds.yaml it is endpoint I believe15:06
TheJuliaat least, I Got that error to go away then15:06
larsksTheJulia: running: openstack --os-auth-type noauth --os-user-id root --os-endpoint http://localhost:6385 baremetal node list15:06
larsksI get: 'CinderNoAuthPlugin' object has no attribute 'get_auth_ref'15:06
TheJuliaYeah, and that is not a plugin in keystoneauth115:06
* TheJulia wonders if someone did something...15:07
larsksTHis is queens on centos 7.15:07
TheJuliaugh https://github.com/openstack/python-cinderclient/blob/master/setup.cfg#L3715:07
TheJulianone is the actual noauth plugin included with keystoneauth115:08
*** melwitt is now known as jgwentworth15:08
larsksOkay.  Well, using os-auth-type=none I can't get passed that whole OS_CLOUD error.15:09
larsksBleargh.15:09
larsksDoes anybody here have this working?15:09
TheJuliaI know people have mentioned it works, looking at the keystoneauth1 code at the moment15:10
larsksHold on, I think I've got it.15:10
larsksopenstack --os-auth-type none  --os-endpoint http://localhost:6385 baremetal node list15:10
larsksNot to figure out what the corresponding clouds.yaml should look like.15:10
*** mwhahaha has joined #openstack-ironic15:10
*** mgoddard has quit IRC15:11
larsksTheJulia: There we go! http://termbin.com/pufy15:12
openstackgerritMerged openstack/ironic-python-agent master: Updated from global requirements  https://review.openstack.org/55318415:12
openstackgerritMerged openstack/ironic-lib master: Updated from global requirements  https://review.openstack.org/55318315:12
openstackgerritPavlo Shchelokovskyy proposed openstack/networking-generic-switch master: WIP Use native Zuul job  https://review.openstack.org/55381315:12
TheJuliaI have quite literally the same locally, and i'm still getting the rrro15:12
TheJuliawell, same if s/sandbox/bifrost/15:13
openstackgerritKaifeng Wang proposed openstack/ironic-specs master: Follow up of inspect wait state spec  https://review.openstack.org/55382215:18
*** moshele has joined #openstack-ironic15:19
*** dpawar has quit IRC15:20
*** moshele has quit IRC15:24
*** jcoufal has joined #openstack-ironic15:24
*** crushil has joined #openstack-ironic15:31
*** links has quit IRC15:32
*** d0ugal has joined #openstack-ironic15:36
*** d0ugal has quit IRC15:36
*** d0ugal has joined #openstack-ironic15:36
*** derekh has quit IRC15:39
*** AlexeyAbashkin has quit IRC15:48
*** r-daneel has quit IRC15:53
*** ianychoi_ has joined #openstack-ironic15:53
openstackgerritMerged openstack/ironic master: Add Error Codes  https://review.openstack.org/51174315:54
*** IgorYozhikov has quit IRC15:55
*** IgorYozhikov_ has joined #openstack-ironic15:55
*** IgorYozhikov_ is now known as IgorYozhikov15:56
*** mjura has quit IRC15:57
*** peitolm_ has joined #openstack-ironic15:57
*** dsneddon_ has joined #openstack-ironic15:59
*** jistr_ has joined #openstack-ironic16:01
*** ianychoi has quit IRC16:03
*** dsneddon has quit IRC16:03
*** lifeless has quit IRC16:03
*** BadCub has quit IRC16:03
*** rajinir has quit IRC16:03
*** davidlenwell has quit IRC16:03
*** sambetts has quit IRC16:03
*** jistr has quit IRC16:03
*** peitolm has quit IRC16:03
*** harlowja has joined #openstack-ironic16:03
*** sambetts_ has joined #openstack-ironic16:03
*** sambetts_ is now known as sambetts16:03
*** jistr_ is now known as jistr16:04
*** baha has quit IRC16:04
*** lifeless has joined #openstack-ironic16:05
openstackgerritDmitry Tantsur proposed openstack/ironic master: Gate: stop setting IRONIC_ENABLED_INSPECT_INTEFACES=inspector  https://review.openstack.org/55384116:10
*** rajinir has joined #openstack-ironic16:10
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: devstack: enable "fake" inspection implementation  https://review.openstack.org/55379716:10
*** BadCub has joined #openstack-ironic16:10
*** davidlenwell has joined #openstack-ironic16:11
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: Switch documentation to hardware types  https://review.openstack.org/54528516:11
dtantsurdamn, we should have looked at the inspector job going red back then >_<16:11
dtantsurTheJulia: one of the patches I was talking about: https://review.openstack.org/#/c/553477/16:13
patchbotpatch 553477 - ironic-tempest-plugin - Use the main introspection test instead of the smo...16:13
*** e0ne_ has joined #openstack-ironic16:20
*** serlex has joined #openstack-ironic16:20
*** serlex has quit IRC16:20
*** e0ne has quit IRC16:21
*** e0ne has joined #openstack-ironic16:22
*** e0ne_ has quit IRC16:25
dtantsurTheJulia: and this is the other: https://review.openstack.org/55373616:25
patchbotpatch 553736 - ironic - Gate: run ironic tests in the regular multinode job16:25
*** dtantsur is now known as dtantsur|afk16:31
dtantsur|afkhave a great weekend16:31
*** dsneddon_ is now known as dsneddon16:34
hjensasTheJulia: when I have some free cycles I will go trough the RFC and some ICS DHCP/dnsmasq docs and probably post some questions to the bug reporter.16:35
*** derekh has joined #openstack-ironic16:35
TheJuliahjensas: awesome, thanks16:35
hjensasTheJulia: afaict now, having the options we already used combined with this bootfile-url should'nt be an issue according to RFC. It is possible it should be fixed elsewhere.16:36
TheJuliahjensas: on the safe side, you might want to just double check what the ipxe code does, because it might be honoring it by default so that might cause some weirdness16:38
hjensasTheJulia: thanks, will do.16:40
*** d0ugal has quit IRC16:45
*** e0ne has quit IRC16:46
*** trown|ruck is now known as trown|lunch16:47
*** baha has joined #openstack-ironic16:51
*** milan has quit IRC16:52
*** yolanda has quit IRC16:55
*** harlowja has quit IRC17:05
*** gyee has joined #openstack-ironic17:05
openstackgerritNguyen Hai proposed openstack/ironic master: Follow the new PTI for document build  https://review.openstack.org/55347317:09
jlvillalSo who is in charge the HPE Proliant CI?17:14
jlvillalThey keep saying merge failure on patches. Well the one patch I see.17:14
jrolljlvillal: https://wiki.openstack.org/wiki/Ironic/Drivers17:14
jroll(spoiler: nisha and stendulker)17:15
jlvillaljroll, I was hoping they were here on the channel :)17:15
jrollheh17:15
jrolldtantsur|afk: ^ found another doc to update for hardware types :P17:15
jrolljlvillal: I've found I get better response from most CI teams via email, in the past17:16
jlvillalOkay17:16
*** marios has quit IRC17:16
jrollprobably copy julia17:16
jlvillaljroll, Also I guess it is Saturday for them17:16
jrolljlvillal: true17:16
jrollif it's a known issue they'll likely respond anyway17:16
*** AlexeyAbashkin has joined #openstack-ironic17:17
*** fragatin_ has quit IRC17:18
*** AlexeyAbashkin has quit IRC17:22
*** ianychoi__ has joined #openstack-ironic17:24
jrolldtantsur|afk: actually, that wiki page is super out of date, you can probably ignore that last ping17:24
*** ianychoi_ has quit IRC17:27
*** jtomasek has quit IRC17:33
*** lucasagomes is now known as lucas-pto17:35
*** ajya has quit IRC17:43
*** weshay is now known as weshay_brb17:47
*** r-daneel has joined #openstack-ironic17:59
*** harlowja has joined #openstack-ironic18:01
*** harlowja_ has joined #openstack-ironic18:04
*** trown|lunch is now known as trown|ruck18:04
*** harlowja has quit IRC18:06
*** fragatina has joined #openstack-ironic18:26
*** fragatina has quit IRC18:26
*** fragatina has joined #openstack-ironic18:26
*** deadnull has joined #openstack-ironic18:35
* sambetts --> home 18:48
sambettsnight all o/18:48
*** sambetts is now known as sambetts|afk18:48
*** AlexeyAbashkin has joined #openstack-ironic18:50
*** crushil has quit IRC18:50
*** AlexeyAbashkin has quit IRC18:54
*** weshay_brb is now known as weshay18:59
*** tesseract has quit IRC19:00
*** crushil has joined #openstack-ironic19:00
*** deadnull has quit IRC19:13
*** crushil_ has joined #openstack-ironic19:15
*** crushil has quit IRC19:18
NobodyCamUgh travel days19:44
*** jtomasek has joined #openstack-ironic19:44
NobodyCamGood morning ironicers19:44
*** jtomasek has quit IRC19:45
*** r-daneel_ has joined #openstack-ironic19:45
*** r-daneel has quit IRC19:47
*** r-daneel_ is now known as r-daneel19:47
*** AlexeyAbashkin has joined #openstack-ironic19:50
*** AlexeyAbashkin has quit IRC19:55
*** crushil_ has quit IRC19:55
jrollhey, morning NobodyCam :)20:10
*** moshele has joined #openstack-ironic20:28
*** dprince has quit IRC20:29
*** hw_wutianwei1 has joined #openstack-ironic20:32
*** hw_wutianwei has quit IRC20:33
*** hw_wutianwei1 is now known as hw_wutianwei20:33
*** jcoufal has quit IRC20:34
*** athomas has quit IRC20:41
*** Goneri has quit IRC20:41
openstackgerritMerged openstack/ironic master: Add optional healthcheck middleware  https://review.openstack.org/54299220:46
openstackgerritMerged openstack/ironic master: Add 'Other considerations' to security doc  https://review.openstack.org/46672920:46
openstackgerritMerged openstack/ironic master: Rework logic handling reserved orphaned nodes in the conductor  https://review.openstack.org/54627320:46
*** moshele has quit IRC20:51
*** trown|ruck is now known as trown|outtypewww21:00
*** bfournie has quit IRC21:09
*** baha has quit IRC21:15
openstackgerritBen Nemec proposed openstack/networking-generic-switch master: Remove config.get_config()  https://review.openstack.org/55390221:21
*** e0ne has joined #openstack-ironic21:25
*** AlexeyAbashkin has joined #openstack-ironic21:29
*** itlinux has joined #openstack-ironic21:31
*** AlexeyAbashkin has quit IRC21:34
*** d0ugal has joined #openstack-ironic21:47
*** d0ugal has quit IRC21:47
*** d0ugal has joined #openstack-ironic21:47
*** itlinux has quit IRC21:48
*** itlinux has joined #openstack-ironic22:03
*** lifeless has quit IRC22:07
*** skazi has quit IRC22:07
*** marlinc has quit IRC22:07
*** krtaylor has quit IRC22:07
*** olivierbourdon38 has quit IRC22:07
*** lifeless has joined #openstack-ironic22:07
*** skazi has joined #openstack-ironic22:07
*** marlinc has joined #openstack-ironic22:07
*** krtaylor has joined #openstack-ironic22:07
*** olivierbourdon38 has joined #openstack-ironic22:07
*** lifeless has quit IRC22:08
*** lifeless has joined #openstack-ironic22:08
*** r-daneel has quit IRC22:08
*** IgorYozhikov has quit IRC22:09
*** d0ugal has quit IRC22:10
*** e0ne has quit IRC22:13
*** IgorYozhikov has joined #openstack-ironic22:13
*** e0ne has joined #openstack-ironic22:15
*** figleaf is now known as edleafe22:16
NobodyCamHey hey jroll 😃22:17
*** e0ne has quit IRC22:18
*** e0ne has joined #openstack-ironic22:18
*** e0ne has quit IRC22:20
*** hw_wutianwei has quit IRC22:46
*** hw_wutianwei has joined #openstack-ironic22:46
*** superdan is now known as dansmith22:58
*** ElCoyote_ has quit IRC23:09
*** r-daneel has joined #openstack-ironic23:47
*** openstackgerrit has quit IRC23:48
*** jgwentworth is now known as melwitt23:48
*** r-daneel has quit IRC23:51
*** gyee has quit IRC23:55

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