Friday, 2020-01-03

openstackgerritJulia Kreger proposed openstack/bifrost master: Tell the prep playbook where to find IPA  https://review.opendev.org/70095800:03
TheJuliaNobodyCam: ci is done :)00:27
*** rh-jelabarre has quit IRC00:29
NobodyCammerging onow00:33
NobodyCamnow even00:33
openstackgerritJulia Kreger proposed openstack/bifrost master: Python3 changes for Ubuntu  https://review.opendev.org/70096600:36
openstackgerritJulia Kreger proposed openstack/bifrost master: Use pip3!  https://review.opendev.org/70096700:36
* TheJulia is on a roll00:36
openstackgerritMerged openstack/bifrost master: Disable bifrost jobs so CI can be fixed  https://review.opendev.org/70095200:39
*** gyee has quit IRC00:58
*** k_mouza has quit IRC02:30
openstackgerritnorman shen proposed openstack/networking-generic-switch stable/rocky: DNM: test upperbound  https://review.opendev.org/70098102:41
openstackgerritMerged openstack/ironic-python-agent-builder master: Use variables for syslinux in build-iso script  https://review.opendev.org/69763803:34
openstackgerritMerged openstack/ironic-python-agent-builder master: Upgrade syslinux version  https://review.opendev.org/69827203:34
*** Lucas_Gray has joined #openstack-ironic04:16
*** bobmel has joined #openstack-ironic04:50
*** goldyfruit has joined #openstack-ironic04:54
*** bobmel has quit IRC04:55
*** ociuhandu has joined #openstack-ironic05:06
*** ociuhandu has quit IRC05:11
*** Lucas_Gray has quit IRC05:29
*** ociuhandu has joined #openstack-ironic05:30
*** ociuhandu has quit IRC05:35
*** tzumainn has quit IRC05:51
openstackgerritWilliam Caban proposed openstack/virtualbmc master: Task: 37947 Story: 2007066  https://review.opendev.org/70099005:55
openstackgerritWilliam Caban proposed openstack/virtualbmc master: Task: 37947 Story: 2007066  https://review.opendev.org/70099005:57
openstackgerritWilliam Caban proposed openstack/virtualbmc master: Task: 37947 Story: 2007066  https://review.opendev.org/70099106:30
openstackgerritzhufl proposed openstack/ironic master: Fix missing job_id parameter in the log message  https://review.opendev.org/70099206:48
openstackgerritKaifeng Wang proposed openstack/ironic master: Make qemu hook compatible with python3  https://review.opendev.org/70099407:57
iurygregorygood morning Ironic!07:58
*** jtomasek has joined #openstack-ironic08:04
*** tesseract has joined #openstack-ironic08:14
kaifenghey iurygregory o/08:16
*** ociuhandu has joined #openstack-ironic08:22
openstackgerritKaifeng Wang proposed openstack/ironic master: Remove the deprecated [glance]glance_num_retries  https://review.opendev.org/70099808:22
*** ociuhandu has quit IRC08:27
*** Lucas_Gray has joined #openstack-ironic08:34
*** pcaruana has joined #openstack-ironic08:35
*** rpittau|afk is now known as rpittau08:41
rpittaugood morning ironic! o/08:41
iurygregorykaifeng, rpittau o/08:46
rpittauhey iurygregory :)08:47
*** Lucas_Gray has quit IRC08:48
kaifenghi rpittau o/08:49
*** bobmel has joined #openstack-ironic08:52
rpittauhey kaifeng :)08:52
*** bobmel has quit IRC08:57
*** Lucas_Gray has joined #openstack-ironic08:58
*** lucasagomes has joined #openstack-ironic09:13
*** khansa has joined #openstack-ironic09:19
openstackgerritRiccardo Pittau proposed openstack/ironic master: [DNM] test multinode job  https://review.opendev.org/69869609:26
*** Lucas_Gray has quit IRC09:26
openstackgerritRiccardo Pittau proposed openstack/ironic master: [DNM] test multinode job  https://review.opendev.org/69869609:28
*** ociuhandu has joined #openstack-ironic09:32
*** alexmcleod has joined #openstack-ironic09:42
*** Lucas_Gray has joined #openstack-ironic09:44
*** derekh has joined #openstack-ironic10:00
*** dtantsur|afk is now known as dtantsur10:02
dtantsurmorning ironic10:02
*** ociuhandu has quit IRC10:06
*** ociuhandu has joined #openstack-ironic10:07
openstackgerritRiccardo Pittau proposed openstack/ironic-lib master: Remove deprecated [disk_utils]iscsi_verify_attempts  https://review.opendev.org/70100810:16
khansadtantsur good morning o/10:17
dtantsuro/10:17
*** khansa has quit IRC10:25
*** k_mouza has joined #openstack-ironic10:25
*** ociuhandu has quit IRC10:38
*** khansa has joined #openstack-ironic10:41
*** Lucas_Gray has quit IRC11:13
*** ociuhandu has joined #openstack-ironic11:17
*** Lucas_Gray has joined #openstack-ironic11:26
*** Lucas_Gray has quit IRC11:31
*** Lucas_Gray has joined #openstack-ironic11:32
*** ociuhandu has quit IRC11:33
*** Lucas_Gray has quit IRC11:34
*** Lucas_Gray has joined #openstack-ironic11:34
*** ociuhandu has joined #openstack-ironic11:37
*** ociuhandu has quit IRC11:38
*** ociuhandu has joined #openstack-ironic11:40
*** ociuhandu has quit IRC11:40
*** ociuhandu has joined #openstack-ironic11:42
*** bobmel has joined #openstack-ironic11:47
*** ociuhandu has quit IRC11:48
*** Lucas_Gray has quit IRC11:58
*** Lucas_Gray has joined #openstack-ironic12:11
*** Lucas_Gray has quit IRC12:20
*** ociuhandu has joined #openstack-ironic12:28
dtantsurhmm, this doesn't look great http://zuul.openstack.org/builds?job_name=ironic-standalone12:29
dtantsurand stable jobs are down because of python 3 >_< https://7616be2cd45ed9ef42cc-2bf09648209a5bc535e8d47f070e968b.ssl.cf5.rackcdn.com/700915/1/check/ironic-tempest-ipa-partition-uefi-pxe_ipmitool-tinyipa/c8f3fa0/controller/logs/screen-virtualbmc.txt.gz12:31
iurygregoryouch =(12:33
iurygregorybut for stable we need to change the version in requirements?12:34
iurygregorysince virtualbmc doesn't have stable branches12:35
mgoddardhi dtantsur12:36
mgoddarddo you remember the reason for changing .vmlinuz to .kernel in https://review.opendev.org/#/c/692200/1 ?12:36
patchbotpatch 692200 - bifrost (stable/train) - Switch to IPA-builder for building ironic-python-a... (MERGED) - 1 patch set12:36
openstackgerritDmitry Tantsur proposed openstack/ironic stable/train: [stable] consume virtualbmc from pip packages  https://review.opendev.org/70102012:37
dtantsurTheJulia: ^^^12:37
dtantsurmgoddard: .vmlinuz has been deprecated for many years IIRC12:37
dtantsurand is not supported by IPA-builder at all12:38
mgoddarddtantsur: :(12:38
mgoddarddeprecated by who?12:38
dtantsurmgoddard: https://opendev.org/openstack/diskimage-builder/src/branch/master/diskimage_builder/elements/ironic-agent/cleanup.d/99-ramdisk-create#L5612:39
dtantsurhttps://opendev.org/openstack/diskimage-builder/commit/ba808fa6ecfb71cd85976e153d7c7755b7a082d0 from 4 years ago12:39
dtantsurIPA-builder has never produced .vmlinuz at all12:39
mgoddardthat's a pain12:40
dtantsurwe can re-introduce it if needed.. although I'd rather not to12:40
mgoddardbifrost used .vmlinuz, so all existing nodes will have driver_info.deploy_kernel=*.vmlinuz12:41
dtantsurmaybe we should create the alias in bifrost itself?12:41
mgoddardcould do12:42
mgoddardseems that tinyipa still uses vmlinuz: https://tarballs.openstack.org/ironic-python-agent/tinyipa/files/12:42
dtantsurand .gz instead of .initramfs. consistency...12:43
iurygregoryafter i get the VM with bifrost (using Vagrant) i to see if everything is ok i just need to run ./scripts/test-bifrost.sh ?12:44
mgoddarddtantsur: thanks for info, I'll think a bit more about the best approach12:45
dtantsurnp12:46
*** ociuhandu has quit IRC12:52
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Always log via Flask application  https://review.opendev.org/69363912:54
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Run publishing jobs with CentOS 8 instead of 7  https://review.opendev.org/70102112:55
dtantsurreview appreciated ^^12:55
*** bobmel has quit IRC13:11
iurygregorydtantsur, lgtm13:15
*** ociuhandu has joined #openstack-ironic13:17
*** ociuhandu has quit IRC13:21
*** ociuhandu has joined #openstack-ironic13:23
*** rh-jelabarre has joined #openstack-ironic13:26
*** rh-jelabarre has quit IRC13:31
*** rh-jelabarre has joined #openstack-ironic13:32
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Always log via Flask application  https://review.opendev.org/69363913:34
*** zaneb has joined #openstack-ironic13:34
TheJuliagood morning14:05
rpittaugood morning TheJulia :)14:05
TheJuliadtantsur: we need "if master branch do x, if not use y" config :(14:07
mgoddarddtantsur: is the old ironic-agent element expected to work in train?14:08
iurygregorygood morning TheJulia o/14:11
*** ociuhandu has quit IRC14:13
*** ociuhandu has joined #openstack-ironic14:14
TheJuliamgoddard: I feel like it kind of was, but definitely replaced by the builder14:14
mgoddardTheJulia: thanks. Would be a bit mean to remove in one cycle14:15
mgoddardI'll try with ironic-agent14:15
iurygregoryTheJulia, I've tested your changes on bifrost, I got the VM and the only test failling is the one for MySQL when I run (./scripts/test-bifrost.sh) trying to figure out how to fix it14:19
TheJuliaiurygregory: on it already :)14:19
TheJuliaone moment14:19
iurygregoryhttp://paste.openstack.org/show/788036/14:19
iurygregoryoh cool =)14:19
openstackgerritJulia Kreger proposed openstack/bifrost master: Python3 changes for Ubuntu  https://review.opendev.org/70096614:23
TheJuliaoh hmm...14:23
TheJuliadifferent from what I was expecting14:23
*** tzumainn has joined #openstack-ironic14:26
TheJuliaI'm wondering why that one is happening, I hit that last night too14:27
iurygregorymaybe something on the package =(14:27
openstackgerritMerged openstack/ironic-python-agent master: Permit hashed passwords to be sent to IPA.  https://review.opendev.org/69137214:27
iurygregorywhen I was using other box with virtual box I got a strange issue where I had to add commands to re-install mariadb14:27
openstackgerritJulia Kreger proposed openstack/bifrost master: Fix/update Vagrant image  https://review.opendev.org/70091114:27
TheJuliadefinite possibility14:27
TheJuliaI actually just uncommented those....14:28
TheJuliahoping they work14:28
TheJuliaI'll need to restack all of the patches a little later14:28
TheJuliaI did get ansible to exit cleanly yesterday, but it reached image building and stopped14:28
TheJuliaactually, your error is the exact same last one I got14:30
iurygregorygot it14:30
TheJuliathe mysql module author has made changes over the years, at least the ?last? maintainer that did some funky things around auth behavior, but since we've been pinned to stable 2.6 for ansible, I wouldn't expect that is what breaks this.14:31
TheJuliait is definitely obsessing over a config_file though14:31
dtantsurmorning TheJulia14:31
TheJuliaand that is a code path in the module which is "source of truth" or "nothing"14:31
iurygregoryyeah14:31
dtantsurTheJulia: I was thinking about it, but it requires more code in devstack to handle it in addition to the current logic. dunno..14:32
dtantsurand I hope that we won't have a similar breakage any soon14:33
TheJuliadtantsur: yeah... we don't break that stuff _THAT_ often either14:33
dtantsurright :)14:33
dtantsurrpittau: if you have a minute: https://review.opendev.org/#/c/701020/14:33
patchbotpatch 701020 - ironic (stable/train) - [stable] consume virtualbmc from pip packages - 1 patch set14:33
rpittauTheJulia, iurygregory: I believe the error you're seeing is caused by ansible looking for the default location of the mysql config file, even if auth parameters are set in the ansible playbook14:42
rpittauit might be a bug, in theory the module will try to read .my.cnf only if the credentials are not specified14:47
rpittauiurygregory, TheJulia: I would check if mysql_user and mysql_pass are defined in your environment14:51
iurygregoryrpittau, ack14:52
TheJulialogin_user and login_password, maybe the parameters were changed at some point14:52
rpittauTheJulia: login_user and login_password are set by the playbook based on mysql_user and mysql_pass env variables14:53
TheJuliaand they are defined, so :\14:54
rpittauwell then it shouldn't fall back to .my.cnf :/14:54
TheJuliait shouldn't, but there have been cases where the maintainer changed it14:54
* TheJulia has fixed it in the ansible codebase once or twice14:55
*** henriqueof has joined #openstack-ironic14:57
*** andreaFFF has joined #openstack-ironic15:01
*** henriqueof has quit IRC15:03
*** andreaFFF has quit IRC15:03
*** henriqueof has joined #openstack-ironic15:03
*** afasano_ has joined #openstack-ironic15:06
iurygregoryTheJulia, bfournie was able to test and it's working \o/ \o/15:33
iurygregoryjust a problem in a LOG, I've fixed and I'm going to remove the comment things XD15:34
*** bnemec is now known as beekneemech15:47
iurygregorywell, almost =( he said to early, after he checked logs grub2 install was there.15:58
dtantsurTheJulia: I found a slight issue with long-running ramdisks: we cache root_device on the 1st lookup. If it gets updated later, we won't notice.16:13
dtantsura typical TripleO flow is introspect -> populate root_device -> deploy16:14
*** ociuhandu has quit IRC16:19
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Allow reading root_device from instance_info  https://review.opendev.org/70104316:30
*** k_mouza has quit IRC16:37
*** khansa has quit IRC16:38
*** lucasagomes has quit IRC16:40
*** afasano_ has quit IRC16:43
*** henriqueof has quit IRC16:44
TheJuliaiurygregory: \o/ that is awesome16:50
TheJuliadtantsur: I was actually pondering that as well recently and I think we do that through lookup. I wonder if lookup should periodicially continue to occur with the agent token work....16:51
TheJuliabtw, which works awesomely16:51
dtantsuryep, we do it through lookup16:51
dtantsurmaybe we should return configuration in heartbeats?16:52
dtantsuralthough it exposes the node information in plain text unnecessary..16:52
TheJuliadon't know which would be better, I just like the idea of at least validating that the request and settings is validated somehow.16:53
openstackgerritMerged openstack/ironic stable/train: [stable] consume virtualbmc from pip packages  https://review.opendev.org/70102016:53
openstackgerritMerged openstack/ironic-python-agent master: Run publishing jobs with CentOS 8 instead of 7  https://review.opendev.org/70102116:53
openstackgerritDmitry Tantsur proposed openstack/ironic stable/stein: [stable] consume virtualbmc from pip packages  https://review.opendev.org/70104716:53
*** ociuhandu has joined #openstack-ironic16:55
*** ociuhandu has quit IRC16:56
*** ociuhandu has joined #openstack-ironic16:58
*** dtantsur is now known as dtantsur|afk16:58
dtantsur|afkhave a great weekend16:58
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Allow reading root_device from instance_info  https://review.opendev.org/70104817:00
openstackgerritDmitry Tantsur proposed openstack/ironic stable/train: redfish-vmedia: correctly pass ipa-debug  https://review.opendev.org/70091517:01
*** ociuhandu has quit IRC17:01
*** ociuhandu has joined #openstack-ironic17:01
*** gyee has joined #openstack-ironic17:03
*** ociuhandu has quit IRC17:12
openstackgerritJulia Kreger proposed openstack/bifrost master: Remove the explicit python interpreter use  https://review.opendev.org/70095017:14
openstackgerritJulia Kreger proposed openstack/bifrost master: Change install-scripts for py3  https://review.opendev.org/70091217:14
*** ociuhandu has joined #openstack-ironic17:14
openstackgerritJulia Kreger proposed openstack/bifrost master: Change install-scripts for py3 with Ubuntu  https://review.opendev.org/70091217:16
openstackgerritJulia Kreger proposed openstack/bifrost master: Explicitly set python interpreter for vagrant exec  https://review.opendev.org/70095317:16
openstackgerritJulia Kreger proposed openstack/bifrost master: Make a copy of requirements for local testers  https://review.opendev.org/70095417:16
openstackgerritJulia Kreger proposed openstack/bifrost master: Tell the prep playbook where to find IPA  https://review.opendev.org/70095817:16
openstackgerritJulia Kreger proposed openstack/bifrost master: Python3 changes for Ubuntu  https://review.opendev.org/70096617:17
openstackgerritJulia Kreger proposed openstack/bifrost master: Use pip3!  https://review.opendev.org/70096717:17
*** iurygregory has quit IRC17:18
rpittauhave a nice weekend, bye! o/17:18
TheJuliao/17:18
*** rpittau is now known as rpittau|afk17:18
*** ociuhandu has quit IRC17:23
*** ociuhandu has joined #openstack-ironic17:24
*** ociuhandu has quit IRC17:25
*** ociuhandu has joined #openstack-ironic17:25
*** ociuhandu has quit IRC17:31
*** ociuhandu has joined #openstack-ironic17:32
*** ociuhandu has quit IRC17:36
*** hwoarang has quit IRC17:49
*** alexmcleod has quit IRC17:50
*** hwoarang has joined #openstack-ironic17:50
*** derekh has quit IRC18:00
openstackgerritWilliam Caban proposed openstack/virtualbmc master: Fix error triggered by reading intergers as string from virtualbmc.conf  https://review.opendev.org/70099018:03
openstackgerritJulia Kreger proposed openstack/bifrost master: Fix/update Vagrant image  https://review.opendev.org/70091118:25
* TheJulia sighs and grumbles at mysql18:40
*** bobmel has joined #openstack-ironic18:46
*** bobmel has quit IRC18:46
*** bobmel has joined #openstack-ironic18:46
*** Lucas_Gray has joined #openstack-ironic18:48
*** henriqueof has joined #openstack-ironic19:21
*** Lucas_Gray has quit IRC20:00
openstackgerritMerged openstack/ironic stable/stein: [stable] consume virtualbmc from pip packages  https://review.opendev.org/70104720:19
openstackgerritTzu-Mainn Chen proposed openstack/ironic-specs master: Create node lessee field  https://review.opendev.org/70106120:20
openstackgerritMerged openstack/ironic stable/train: redfish-vmedia: correctly pass ipa-debug  https://review.opendev.org/70091520:44
openstackgerritTzu-Mainn Chen proposed openstack/ironic-specs master: Create node lessee field  https://review.opendev.org/70106120:55
*** ociuhandu has joined #openstack-ironic22:02
openstackgerritJulia Kreger proposed openstack/bifrost master: Only set openstack_citest credientials when in openstack ci  https://review.opendev.org/70106322:02
*** bobmel has quit IRC22:05
*** ociuhandu has quit IRC22:06
*** TxGirlGeek has joined #openstack-ironic22:35
openstackgerritJulia Kreger proposed openstack/bifrost master: Add ebtables to bindep  https://review.opendev.org/70106522:50
*** tesseract has quit IRC22:54
*** pcaruana has quit IRC22:59
*** TxGirlGeek has quit IRC23:28
openstackgerritJulia Kreger proposed openstack/bifrost master: WIP: Hopefully the last patch to fix ubuntu for python3  https://review.opendev.org/70106723:58
*** gyee has quit IRC23:58

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