Tuesday, 2020-09-01

*** janders has quit IRC00:47
*** janders has joined #openstack-ironic00:49
openstackgerritSteve Baker proposed openstack/ironic master: New argument validate decorator  https://review.opendev.org/74279501:28
openstackgerritSteve Baker proposed openstack/ironic master: Add status_code argument to @method.expose  https://review.opendev.org/74367701:28
openstackgerritSteve Baker proposed openstack/ironic master: Add @method.body decorator  https://review.opendev.org/74367801:28
openstackgerritSteve Baker proposed openstack/ironic master: Duplicate trait validation with jsonschema  https://review.opendev.org/74577801:28
openstackgerritSteve Baker proposed openstack/ironic master: Convert allocation endpoint to plain JSON  https://review.opendev.org/74367901:28
openstackgerritSteve Baker proposed openstack/ironic master: Convert bios endpoint to plain JSON  https://review.opendev.org/74547501:28
openstackgerritSteve Baker proposed openstack/ironic master: Convert chassis endpoint to plain JSON  https://review.opendev.org/74547601:28
openstackgerritSteve Baker proposed openstack/ironic master: Convert conductor endpoint to plain JSON  https://review.opendev.org/74577901:28
openstackgerritSteve Baker proposed openstack/ironic master: Convert deploy_templates endpoint to plain JSON  https://review.opendev.org/74578001:28
openstackgerritSteve Baker proposed openstack/ironic master: Convert drivers endpoint to plain JSON  https://review.opendev.org/74186501:28
openstackgerritSteve Baker proposed openstack/ironic master: Convert event endpoint to plain JSON  https://review.opendev.org/74662301:29
openstackgerritSteve Baker proposed openstack/ironic master: Convert node endpoint to plain JSON  https://review.opendev.org/74782001:29
*** rh-jelabarre has quit IRC01:40
*** rh-jelabarre has joined #openstack-ironic01:40
*** Qianbiao has joined #openstack-ironic01:52
*** ijw has quit IRC02:24
*** ijw has joined #openstack-ironic02:25
*** rh-jelabarre has quit IRC02:28
*** ijw_ has joined #openstack-ironic02:29
*** ijw has quit IRC02:30
*** ijw_ has quit IRC02:34
*** rcernin has quit IRC02:40
*** ijw has joined #openstack-ironic02:43
*** rh-jelabarre has joined #openstack-ironic02:47
*** rh-jelabarre has quit IRC02:52
*** gyee has quit IRC03:13
*** rcernin has joined #openstack-ironic03:13
*** rcernin has quit IRC03:19
*** rcernin has joined #openstack-ironic03:19
*** Qianbiao has quit IRC03:43
openstackgerritNisha Agarwal proposed openstack/ironic master: Adds few of the security dashboard parameters to capabilities  https://review.opendev.org/74641604:51
*** hjensas has joined #openstack-ironic05:24
*** tzumainn has quit IRC05:31
*** Qianbiao has joined #openstack-ironic05:44
*** cdearborn has quit IRC05:56
*** belmoreira has joined #openstack-ironic06:51
*** belmoreira has quit IRC06:53
*** akahat is now known as akahat|rover07:05
rpittaugood morning ironic! o/07:06
*** jtomasek has joined #openstack-ironic07:12
Qianbiaomorning rpittau07:15
rpittauhey Qianbiao :)07:15
Qianbiaohey o/07:16
Qianbiaoiurygregory is late today :)07:16
*** tosky has joined #openstack-ironic07:21
*** rcernin has quit IRC07:36
*** fmuyassarov has joined #openstack-ironic07:38
iurygregorygood morning rpittau Qianbiao o/07:41
rpittauhey iurygregory :)07:41
iurygregoryI was finishing my breakfast =)07:41
iurygregorycold weather make me sleep more time hehe07:41
Qianbiaoiurygregory morning o/07:47
QianbiaoHello, when run setup ironic by devstack, which dhcp will pxe uses.07:49
Qianbiaothrough shared network?07:50
Qianbiao^^ http://paste.openstack.org/show/797317/07:50
*** belmoreira has joined #openstack-ironic07:51
QianbiaoAnd where can i read the dhcp log ~~07:53
iurygregoryjournalctl for neutron07:53
iurygregoryit should be there I would say07:53
iurygregoryq-dhcp07:54
iurygregoryhttps://docs.openstack.org/devstack/latest/systemd.html can be useful07:55
Qianbiaoiurygregory thanks, seems right07:55
*** dougsz has joined #openstack-ironic07:56
openstackgerritRiccardo Pittau proposed openstack/tenks master: Add skip list to ansible lint  https://review.opendev.org/74920107:58
*** rcernin has joined #openstack-ironic07:59
*** rcernin has quit IRC08:02
Qianbiaoneutron did not update ironic node port to dhcp leases file08:04
*** priteau has joined #openstack-ironic08:04
Qianbiaobut in log, it display it writes, any idea?08:04
Qianbiaohttp://paste.openstack.org/show/797320/08:04
QianbiaoFor mac: 28:a6:db:a9:92:2808:04
openstackgerritMerged openstack/bifrost bugfix/8.3: Make /var/lib/ironic/{,images,master_images} readable by nginx  https://review.opendev.org/74902108:05
openstackgerritMerged openstack/bifrost stable/ussuri: Make /var/lib/ironic/{,images,master_images} readable by nginx  https://review.opendev.org/74902208:05
*** Lucas_Gray has joined #openstack-ironic08:10
jandersg'day Ironic o/08:11
iurygregoryhey janders o/08:14
*** lucasagomes has joined #openstack-ironic08:15
rpittauhey janders :)08:17
rpittaueasy review if anyone has a minute :) https://review.opendev.org/74788308:18
patchbotpatch 747883 - bifrost - ansible-lint: fix error 206 - 4 patch sets08:18
iurygregoryack08:19
iurygregoryI'm wondering if you went file by file adding the spaces rpittau08:19
iurygregoryWUT 602 is also working?08:20
rpittauiurygregory: I'd rather not revail my secrets :P08:20
rpittauiurygregory: yep08:20
iurygregoryhaha08:20
iurygregorysed is a good one XD08:21
rpittaujokes aside, IDEs and text editors help a lot in these cases08:21
iurygregoryyeah08:21
*** fmuyassarov has quit IRC08:21
*** sshnaidm|bbl is now known as sshnaidm08:36
*** derekh has joined #openstack-ironic08:46
openstackgerritMerged openstack/bifrost master: ansible-lint: fix error 206  https://review.opendev.org/74788308:50
*** Nisha_Agarwal has joined #openstack-ironic08:53
openstackgerritMerged openstack/bifrost master: ansible-lint: fix error 504  https://review.opendev.org/74792508:56
*** Lucas_Gray has quit IRC08:59
*** dtantsur|afk is now known as dtantsur09:05
dtantsurmorning ironic09:06
dtantsurTheJulia: no particular reason (re dib in metalsmith CI), the CI was created before we switched to DIB.09:06
*** Lucas_Gray has joined #openstack-ironic09:06
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Stop using --system-site-packages  https://review.opendev.org/74908109:09
iurygregorymorning dtantsur09:09
openstackgerritMerged openstack/bifrost master: ansible-lint: fix error 602  https://review.opendev.org/74859709:12
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Workaround for setuptools 50.0  https://review.opendev.org/74920709:13
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Stop using --system-site-packages  https://review.opendev.org/74908109:15
jandersgood morning dtantsur09:21
Nisha_Agarwaldtantsur, good morning :)09:21
mnasiadkaGood morning09:22
mnasiadkadtantsur: what's the plan for https://review.opendev.org/#/c/749014/1? I don't think I'm so familiar with all bifrost CI jobs I could help :)09:22
patchbotpatch 749014 - bifrost (stable/stein) - [Train and older] Fix several CI problems - 1 patch set09:22
openstackgerritRiccardo Pittau proposed openstack/tenks master: Use victoria tests  https://review.opendev.org/74920809:23
openstackgerritRiccardo Pittau proposed openstack/bifrost master: Fix error 601  https://review.opendev.org/74920909:26
openstackgerritRiccardo Pittau proposed openstack/bifrost master: Fix error 601  https://review.opendev.org/74920909:26
dtantsurmnasiadka: is it blocking you? I was going to take a look but I have more pressing issues right now.09:31
mnasiadkadtantsur: well, it blocks Kolla builds/publishes of fresh Stein Ironic containers, not pressing a lot, but would be nice to fix it soon09:31
dtantsurmnasiadka: okay, I'll move it higher up my priority list09:32
openstackgerritRiccardo Pittau proposed openstack/tenks master: Fix ansiblelint 208  https://review.opendev.org/74921109:33
Qianbiaohi, is devstack problem fixed09:35
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Workaround for setuptools 50.0  https://review.opendev.org/74920709:39
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Stop using --system-site-packages  https://review.opendev.org/74908109:40
dtantsurQianbiao: the workaround has landed. I'm not sure our multinode job has been fixed though09:43
rpittauI rechecked a trivial change around 2 hours ago to see how it's going09:44
rpittauhere should end soon, multinode has not finished yet https://review.opendev.org/74897809:46
patchbotpatch 748978 - ironic - [trivial] remove emacs config from devstack script - 1 patch set09:46
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Replace oslo's loopingcall with tenacity  https://review.opendev.org/74921409:54
Qianbiaodtantsur thanks.09:54
*** tkajinam has quit IRC09:54
*** tkajinam has joined #openstack-ironic09:55
Qianbiaodtantsur Is ironic devstack for virt nodes works for hardware bmc too?09:55
QianbiaoI mean it's installed with option "IRONIC_IS_HARDWARE=False"09:55
Qianbiaocan i just add a hardware node directly09:56
dtantsurQianbiao: ironic should work, but networking may not09:59
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: [PoC] Do not use eventlet and oslo.service for serving API  https://review.opendev.org/74902009:59
Qianbiaodtantsur I guess so, bmc pxe could not dhcp for now.10:00
Qianbiaodtantsur what if i set "IRONIC_IS_HARDWARE=True" and add hardware_info defines, is it ready for use?10:01
*** antotala has joined #openstack-ironic10:08
openstackgerritMerged openstack/tenks master: Add skip list to ansible lint  https://review.opendev.org/74920110:29
*** Nisha_Agarwal has quit IRC10:36
*** mgagne has quit IRC10:39
mnasiadkadtantsur: can you also propose bifrost 7.2.2 for Train with the merged fix?10:42
dtantsuriurygregory or rpittau or anyone if you have time ^^10:42
iurygregorydtantsur, I can10:43
openstackgerritAija Jaunteva proposed openstack/sushy master: Add BIOS update apply time and maintenance window  https://review.opendev.org/74923610:43
iurygregory[Train and older] Fix several CI problems with this one right?10:45
*** vmud213 has joined #openstack-ironic10:45
dtantsuriurygregory: yeah10:46
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Stop using --system-site-packages  https://review.opendev.org/74908110:46
iurygregorydtantsur, done10:49
dtantsurthx!10:49
iurygregorynp10:49
openstackgerritAija Jaunteva proposed openstack/ironic master: Add Redfish BIOS internface to idrac HW type  https://review.opendev.org/74924010:54
openstackgerritAija Jaunteva proposed openstack/ironic master: Add Redfish BIOS interface to idrac HW type  https://review.opendev.org/74924010:55
*** belmoreira has quit IRC10:57
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Stop using --system-site-packages  https://review.opendev.org/74908111:05
*** tonyb has quit IRC11:05
dtantsurdevstack seems fixed, we're back to broken multinode..11:06
iurygregoryyeah =(11:11
* iurygregory brb lunch time11:11
*** antotala has quit IRC11:12
*** Lucas_Gray has quit IRC11:14
openstackgerritMerged openstack/tenks master: Use victoria tests  https://review.opendev.org/74920811:17
*** tonyb has joined #openstack-ironic11:31
Qianbiao<dtantsur> devstack still has bug.11:38
Qianbiaowhen setting USE_PYTHON3=True, still fails on ubuntu 18.04 for branch train11:38
*** Lucas_Gray has joined #openstack-ironic11:40
arne_wiebalckGood morning, ironic!11:43
*** Qianbiao has quit IRC11:46
*** Qianbiao has joined #openstack-ironic11:46
*** fmuyassarov has joined #openstack-ironic11:58
*** chenhaw has quit IRC12:00
*** rh-jelabarre has joined #openstack-ironic12:03
dtantsurQianbiao: in the ironic plugin or in devstack itself? if the latter, try asking on #openstack-qa12:03
*** rh-jelabarre has quit IRC12:04
*** rh-jelabarre has joined #openstack-ironic12:04
Qianbiaodtantsur devstack12:05
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Stop using --system-site-packages  https://review.opendev.org/74908112:05
dtantsurfolks, this ^^ is getting close but it's so ugly I'm not sure we should merge it :(12:05
dtantsurbut system-site-packages is also bad..12:05
dtantsuropinions?12:05
dtantsurthe alternative is https://review.opendev.org/749207 which fixes the setuptools issue via a workaround I don't understand12:06
patchbotpatch 749207 - bifrost - Workaround for setuptools 50.0 - 2 patch sets12:06
*** Nisha_Agarwal has joined #openstack-ironic12:10
Qianbiaodtantsur https://github.com/pypa/setuptools/blob/04e3df22df840c6bb244e9b27bc56750c44b7c85/CHANGES.rst12:15
Qianbiaomaybe changelog will help you12:15
iurygregorygood afternoon arne_wiebalck =)12:16
arne_wiebalckyeah ... dentist this morning :-)12:16
iurygregoryI hope it was ok =)12:17
arne_wiebalckyep12:17
*** thiagop has joined #openstack-ironic12:20
openstackgerritNisha Agarwal proposed openstack/ironic master: Adds few of the security dashboard parameters to capabilities  https://review.opendev.org/74641612:22
rpittaudtantsur: re: setuptools hell, this is how I see it: the workaround with SETUPTOOLS_USE_DISTUTILS is tight to the issue and it's temporary until a proper fix is provided, so maybe it's still the way to go12:26
dtantsurI've unblocked https://review.opendev.org/#/c/749207/12:27
patchbotpatch 749207 - bifrost - Workaround for setuptools 50.0 - 2 patch sets12:27
*** cdearborn has joined #openstack-ironic12:27
dtantsurI'll probably abandon the system-site-packages patch12:27
*** Nisha_Agarwal has quit IRC12:27
rpittauok :/12:28
openstackgerritQianBiao Ng proposed openstack/ironic master: Feat: add ibmc hardware info support for devstack  https://review.opendev.org/74884112:33
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Remove unnecessary overrides of ansible_python_interpreter  https://review.opendev.org/74926212:36
*** derekh has quit IRC12:39
bfourniehi dtantsur!  so I was hitting some problems testing Bifrost booting a baremetal node, it couldn't reach the inspector on the Bifrost node from the BM node because of firewalld.  I just disabled firewalld for testing, I wonder if its a problem with the zone setup here in this case https://review.opendev.org/#/c/738042/9/playbooks/roles/bifrost-ironic-install/tasks/inspector_bootstrap.yml12:50
patchbotpatch 738042 - bifrost - Use firewalld to open ports on CentOS and RHEL. (MERGED) - 9 patch sets12:50
bfourniethis is with Centos on Bifrost node12:50
dtantsurbfournie: you mean, your nodes are not in the 'public' zone?12:52
dtantsur(rather the network interface, but you get it)12:52
*** derekh has joined #openstack-ironic12:54
dtantsurbfournie: firewall-cmd --get-active-zones?12:55
dtantsurfirewall-cmd --list-all --zone public?12:55
bfourniedtantsur: yeah, I'm using port eno49 which isn't in any zones - http://pastebin.test.redhat.com/898247, I guess I should be adding this to a zone12:56
bfourniedtantsur: via ./bifrost-cli install --network-interface eno49 --dhcp-pool 192.168.24.5-192.168.24.2412:58
dtantsurbfournie: try it. maybe bifrost should do it automatically, I'm not sure. I don't understand firewalld well enough, I assumed it would add all interfaces to some zone.12:59
bfourniedtantsur: I will try it, thanks12:59
rpittauready for the second day of Ironic Victoria Virtual Midcycle Meetup ? -> https://meetpad.opendev.org/Ironic-Victoria-midcycle12:59
*** fmuyassarov has quit IRC13:00
dtantsurbfournie: hmmm, judging by quick googling, it should be handled by NM13:00
TheJuliagood morning13:00
openstackgerritIvan Kolodyazhny proposed openstack/ironic-ui master: [goal] Migrate testing to ubuntu focal  https://review.opendev.org/74470813:00
iurygregorygood morning TheJulia =)13:00
bfourniedtantsur: interesting13:01
dtantsurmorning TheJulia13:01
*** Lucas_Gray has quit IRC13:02
*** trandles has joined #openstack-ironic13:05
jandersbfournie: I had a colleague from another team at RHAT test out bifrost on real baremetal and he did hit some firewalld hurdles. I can ask him tomorrow morning if you like (he's also East Coast Australia ) - let's touch base when I get online13:05
jandershi TheJulia o/13:05
*** chenhaw has joined #openstack-ironic13:05
bfourniejanders: sounds good, thanks!13:05
trandleshi janders13:06
trandlesjanders, I hit some firewalld issues with bifrost that dtantsur fixed13:07
trandlesmight be some overlap13:07
dtantsurI guess ideally we should create our own zone in bifrost, add our network interface to it and then modify only it13:11
dtantsurbut I'm not sure I understand the patterns of firewalld well13:12
* TheJulia needs much more coffeeeeee13:12
dtantsurbfournie: if I prepare a proof-of-concept patch today/tomorrow will you be able to test it?13:15
dtantsurI don't have real bare metal at hand13:15
trandlesI can test since all I have is bare metal13:16
dtantsurokay. I want to try the idea with a separate 'bifrost' zone because my gut feeling we should do it instead of modifying 'public'13:17
*** dsneddon has quit IRC13:17
*** chenhaw has quit IRC13:18
jandersdtantsur bfournie I will check IRC logs tomorrow morning and unless you already smashed it out by then I will get the info off my Aussie colleague13:20
janderstrandles hi! :) and thanks for helping with this!13:21
jandersit's getting really late here - time for me to knock off13:21
janderssee you tomorrow Ironic o/13:21
trandleslater janders13:21
*** Goneri has joined #openstack-ironic13:27
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Create our own firewalld zone and use it on real bare metal  https://review.opendev.org/74928513:33
dtantsurtrandles, bfournie, janders ^^^13:33
dtantsurI'd especially appreciate if you could test that it still works after a reboot (if it works at all)13:33
*** janders5 has joined #openstack-ironic13:34
bfourniedtantsur: nice, thanks, I will test it out13:35
*** janders has quit IRC13:37
*** janders5 is now known as janders13:37
*** chenhaw has joined #openstack-ironic13:37
*** tzumainn has joined #openstack-ironic13:48
openstackgerritVishal Manchanda proposed openstack/ironic-ui master: [goal] Migrate testing to ubuntu focal  https://review.opendev.org/74470813:51
JayFIs the gate fixed? It looks like ironic-whiteboard was never updated for the last outage so I can't tell13:52
dtantsurJayF: partly. devstack is back, but we have https://zuul.openstack.org/builds?job_name=ironic-tempest-ipa-wholedisk-direct-tinyipa-multinode13:52
dtantsurthe IPA CI should be fine, if that's what you're interested in13:52
JayFack, ty, and I should've used that to check my  jobs too :D13:52
openstackgerritChristopher Dearborn proposed openstack/sushy master: WIP: Allow monitoring progress of a firmware update  https://review.opendev.org/74595013:57
JayFttps://docs.openstack.org/ironic/latest/install/configdrive.html we document a 64MB max size13:59
zer0c00ldtantsur: Quick question on "Link to firmware update's SimpleUpdate asynchronous behavior in the spec:14:10
zer0c00lSorry " The parent patch should be updated with this change.14:10
dtantsuryep. git commit --amend && git review :)14:11
zer0c00lShould i squash the commits and push a new change?14:11
zer0c00li see okay14:11
zer0c00lThanks.14:11
dtantsurmerge the changes from the 2nd commit into the 1st and push a new change14:11
JayFSwitching between Github flow and Gerrit flow can give you whiplash :|14:12
zer0c00lyeah, github doesn't like git commit --amend, allows you to squash when you merge the PR14:13
*** fmuyassarov has joined #openstack-ironic14:14
*** fmuyassarov has quit IRC14:15
*** Lucas_Gray has joined #openstack-ironic14:18
*** sshnaidm has quit IRC14:34
*** vmud213 has quit IRC14:47
JayFzer0c00l: fwiw you can --amend with github, you just have to force push your branch14:47
*** ijw has quit IRC14:48
*** vmud213 has joined #openstack-ironic14:48
dtantsurgit push --force \o/14:50
JayFWe have "big brother" watching our enterprise github. So if you accidentally did that to the default branch, the entire world would come crashing down upon you :P14:52
JayFnever git push --force without a branch specifier. If you're running old enough git, or have it configured strangely, it'll happily push *all your branches *14:52
*** ijw has joined #openstack-ironic14:53
*** sshnaidm has joined #openstack-ironic14:53
jrolland besides all that, a force push on github removes any way to see the diff between previous "change set" and the next. have to review the whole patch again. so annoying14:55
*** trandles has quit IRC14:55
*** trandles_ has joined #openstack-ironic14:55
*** trandles_ is now known as trandles14:55
clarkbjroll: public github keeps the diff contexts now (but not the refs so you can't expand context further)14:56
*** dtantsur is now known as dtantsur|afk15:18
dtantsur|afkgoing a bit earlier today, see you tomorrow15:18
QianbiaoHello, anyone has experience with ironic devstack.15:24
QianbiaoI can not clean baremetal node due to pxe dhcp not reach neutron service.15:25
Qianbiaoand could not figure out why :(15:25
Qianbiaothis is my local.conf http://paste.openstack.org/show/797341/, anything else i should config?15:27
*** chihhsin has quit IRC15:28
openstackgerritVishal Manchanda proposed openstack/ironic-ui master: [goal] Migrate testing to ubuntu focal  https://review.opendev.org/74470815:32
openstackgerritLucas Alvares Gomes proposed openstack/metalsmith master: Explicitly set jobs to ML2/OVS  https://review.opendev.org/74934615:35
*** antotala has joined #openstack-ironic15:38
antotalaHi All, I have written a ironic-inspector plugin using this as a reference:: https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/plugins/example.py .. no I want ironic-inspector to detect this new plugin.. for this I am adding this parameter in ironic-inspector.conf  -- under [processing] default_processing_hooks =15:43
antotalaramdisk_error,root_disk_selection,scheduler,validate_interfaces,capabilities,pci_devices15:43
antotalais it the right configuration or am I making any mistake here ? one more thing the new_plugin code, it should present on controller node or in the ramdisk ?15:44
JayFI'm not an  expert in Ironic Inspector, but it looks like you should check https://github.com/openstack/ironic-inspector/blob/master/setup.cfg -- search for "example", and add entry points for your items just like they have for the example items (in the package you have your custom plugin)15:47
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Move ironic-standalone-ipa-src to Ubuntu Focal  https://review.opendev.org/74821315:49
antotalaJayF I have entry point there, I am just not sure on where should my package be installed, based on my reading and understanding it should be on controller node..15:50
*** sshnaidm is now known as sshnaidm|bbl15:50
JayFack, then I'm not sure how to help. Sorry! Good luck.15:51
antotalaThanks JayF15:52
JayFOK, so actually going to try and help15:53
JayFI think that needs to be installed on the ramdisk15:53
JayFbut again not 100% sure15:53
JayFFor instance, https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/plugins/lldp_basic.py performs commands that only make sense to run on the target node in a ramdisk15:53
antotalaThanks JayF , I am gonna try that and do you see any issue with the configuration that needs to be in ironic-inspector.conf that I shared above15:57
JayFas long as that matches the name in the entry point you /should/ be OK. But I am *not* an inspector expert, so that's just a guess15:57
*** zzzeek has quit IRC15:59
*** zzzeek has joined #openstack-ironic16:00
antotalaThanks JayF , I am gonna try that16:01
JayFgood luck16:02
openstackgerritJulia Kreger proposed openstack/metalsmith master: Replace the Centos7 image job with a Centos8 UEFI based job  https://review.opendev.org/74831516:02
*** Lucas_Gray has quit IRC16:04
*** zzzeek has quit IRC16:05
*** zzzeek has joined #openstack-ironic16:06
*** bnemec has quit IRC16:06
*** bnemec has joined #openstack-ironic16:06
*** vmud213 has quit IRC16:08
*** Qianbiao has quit IRC16:08
*** lucasagomes has quit IRC16:11
*** dougsz has quit IRC16:14
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP] Do not allocate initial space for virtual volumes  https://review.opendev.org/74935716:15
*** Lucas_Gray has joined #openstack-ironic16:25
iurygregoryI've pushed a patch to grenade that will offer the possibility to specify services we wish to skip their upgrade16:27
openstackgerritRiccardo Pittau proposed openstack/ironic master: Replace retrying with tenacity  https://review.opendev.org/37657416:34
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Native zuulv3 grenade multinode multitenant  https://review.opendev.org/70503016:35
*** gyee has joined #openstack-ironic16:38
iurygregorylet's see how it goes =)16:38
* iurygregory fingers crossed that at least I will see the message about skipping nova16:39
*** ijw_ has joined #openstack-ironic16:42
TheJuliaiurygregory: awesome!16:43
TheJuliarpittau: why not allocate initial space? Is there someplace running out of initial space? The downside to not doing so we end up having to do more IO at deploy time, so it may shift race conditions or outright failures into view16:44
*** ijw__ has joined #openstack-ironic16:45
*** ijw has quit IRC16:45
rpittauTheJulia: when migrating ipa jobs to ubuntu focal I stumbled into the issue where we consume all the space available when creating the volumes16:45
rpittauTheJulia:  https://review.opendev.org/74821316:46
patchbotpatch 748213 - ironic-python-agent - Move ironic-standalone-ipa-src to Ubuntu Focal - 8 patch sets16:46
TheJuliarpittau: on / ?16:46
TheJuliaor /opt?16:46
rpittauon /opt16:46
TheJuliawow16:46
rpittauwe pass from 35 G to over 70G16:46
TheJuliaand /opt is variable on the clouds16:46
TheJuliafunky :\16:46
*** sshnaidm|bbl is now known as sshnaidm16:46
*** Lucas_Gray has quit IRC16:47
rpittauI think it's because of a change in qemu or virsh, I don't think we're actually allocating space now, you can check it here https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_01e/748269/4/check/ironic-standalone-ipa-src/01e0d1d/du-opt.txt16:47
*** antotala has quit IRC16:47
rpittauit might be that by default the allocation is 0 until a certain version, but I couldn't find it documented anywhere16:48
*** ijw_ has quit IRC16:49
*** Lucas_Gray has joined #openstack-ironic16:49
*** derekh has quit IRC16:51
TheJuliaI think it was doing a thin allocation16:57
TheJuliasparse16:57
*** priteau has quit IRC16:59
*** mushrushu has joined #openstack-ironic17:01
openstackgerritJulia Kreger proposed openstack/metalsmith master: Replace the Centos7 image job with a Centos8 UEFI based job  https://review.opendev.org/74831517:01
*** sshnaidm is now known as sshnaidm|afk17:11
TheJuliaThere are many subtle differences in IO behavior there sadly :(17:16
*** jtomasek has quit IRC17:26
*** k_mouza has joined #openstack-ironic17:41
*** tonyb has quit IRC17:44
*** JamesBenson has quit IRC17:44
*** mushrushu has quit IRC17:44
*** ddurst has quit IRC17:44
*** Lucas_Gray has quit IRC17:44
*** mushrushu has joined #openstack-ironic17:46
*** tonyb has joined #openstack-ironic17:46
*** JamesBenson has joined #openstack-ironic17:46
*** ddurst has joined #openstack-ironic17:46
JayFhttps://review.opendev.org/#/c/748767/ is +2 and Verified, pretty basic bugfix would love to get it reviewed17:48
patchbotpatch 748767 - ironic-python-agent - Make WSGI server respect listen_* directives - 5 patch sets17:48
TheJuliasure!17:54
TheJuliastevebaker: looks like metalsmith may be broken :\17:54
TheJuliahttps://www.irccloud.com/pastebin/s0HtBIM5/17:54
JayFhttps://review.opendev.org/#/c/747193/ also has 1 +1 and is passing CI ,would love review son it17:55
patchbotpatch 747193 - ironic-python-agent - If listen_tls is true, enable TLS on wsgi server - 8 patch sets17:55
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Add fstab pointer to EFI partition  https://review.opendev.org/74826017:56
*** k_mouza has quit IRC17:56
*** dsneddon has joined #openstack-ironic17:58
openstackgerritMerged openstack/bifrost master: Move services to internal_ip by default and refactor endpoint creation  https://review.opendev.org/74823017:59
*** dsneddon has quit IRC18:04
openstackgerritMerged openstack/bifrost master: Workaround for setuptools 50.0  https://review.opendev.org/74920718:06
openstackgerritMerged openstack/bifrost master: Update keystone documentation  https://review.opendev.org/74839218:06
openstackgerritMerged openstack/bifrost master: Remove support for the legacy CSV format  https://review.opendev.org/74789318:06
openstackgerritVerification of a change to openstack/bifrost failed: Use packaged iPXE ROMs for openSUSE  https://review.opendev.org/74789418:06
openstackgerritMerged openstack/bifrost master: Remove support for OpenStack MetaData version 2012-08-10  https://review.opendev.org/74790018:06
openstackgerritMerged openstack/bifrost master: Trivial: remove a reference to update-rc.d  https://review.opendev.org/74823718:06
* thiagop thinks zuul is working on the double today18:07
*** ociuhandu has joined #openstack-ironic18:19
TheJuliabifrost jobs go fairly quick18:34
TheJulia)18:34
TheJuliaJayF: approverated for the first one :)18:35
*** ociuhandu has quit IRC18:36
trandlesping TheJulia18:52
TheJuliapong18:55
trandlesTheJulia: thanks, setting up a new IRC client with persistent logging18:56
trandlesI haven't taken IRC seriously in like a decade...sigh...I've forgotten The Old Ways18:56
TheJuliaheh18:57
*** openstackgerrit has quit IRC19:21
rpiosoTheJulia: In light of the feedback we received during the discussion of https://storyboard.openstack.org/#!/story/2003594 during this morning's virtual meetup, would it be acceptable to proceed with a Dell EMC-specific implementation without a spec? We would use Swift to encrypt and store the "configuration molds".19:36
TheJuliarpioso: I suspect that will be okay. I'm sure someone will want to use that functionality with some other file/data storage mechanism but that can be figured out as we go or after the fact.19:42
rpiosoTheJulia: Should we send a message to the ML to inform the community?19:45
*** thiagop has quit IRC19:53
TheJuliarpioso: I suspect that would be good, just to raise awareness20:00
*** dsneddon has joined #openstack-ironic20:03
*** openstackgerrit has joined #openstack-ironic20:18
openstackgerritJulia Kreger proposed openstack/metalsmith master: Attempt to workaround apparent pip issues  https://review.opendev.org/74939120:18
openstackgerritVerification of a change to openstack/ironic-python-agent failed: Make WSGI server respect listen_* directives  https://review.opendev.org/74876720:20
TheJuliaI hope that fixes metalsmith since it seems to be presenting like site packages weirdness20:21
*** Lucas_Gray has joined #openstack-ironic20:22
TheJuliaFyi everyone, I submitted a project update for the upcoming summit.20:33
TheJuliaOf course, I haven't made slides yet, but figure it is good to get that submitted.20:33
*** Lucas_Gray has quit IRC20:35
openstackgerritVerification of a change to openstack/ironic-python-agent failed: Increase memory for tinyipa jobs  https://review.opendev.org/74763420:37
*** Lucas_Gray has joined #openstack-ironic20:42
TheJuliaReminder: Doodle to discuss kexec - https://doodle.com/poll/ihbbtns36b2rkhfd20:49
stevebakermorning20:51
openstackgerritMerged openstack/networking-baremetal master: Add missing keystoneauth1 and oslo.service to requirements  https://review.opendev.org/74793620:57
stevebakerTheJulia: it looks like setuptools repeatedly breaks things and maybe metalsmith should follow the global requirements to exclude known bad ones https://review.opendev.org/#/c/749257/1/global-requirements.txt21:00
patchbotpatch 749257 - requirements (stable/ussuri) - Blacklist setuptools 50.0.0 - 1 patch set21:00
JayFTheJulia: ack, ty21:18
TheJuliastevebaker: Well, I think the conundrum is the requirements get installed and I suspect because we're not verbose logging it is also upgrading setuptools under the hood quietly21:23
clarkbalso keep in mind there tends to be a chicken and egg issue with restricting setuptools versions21:28
clarkbdepending on where things break you may need to preinstall setuptools too21:29
openstackgerritArun S A G proposed openstack/ironic-specs master: Add support for anaconda deploy interface  https://review.opendev.org/74850321:36
TheJuliaI'd assume it would get pulled in well before this too... and there is also the headache with how requirements processing occurs differently21:41
*** uzumaki has joined #openstack-ironic22:11
openstackgerritChristopher Dearborn proposed openstack/sushy master: WIP: Allow monitoring progress of a firmware update  https://review.opendev.org/74595022:20
*** JamesBenson has quit IRC22:27
*** Goneri has quit IRC22:37
*** tosky has quit IRC22:43
*** JamesBenson has joined #openstack-ironic22:58
*** rcernin has joined #openstack-ironic23:01
*** hjensas has quit IRC23:02
*** JamesBenson has quit IRC23:03
openstackgerritJulia Kreger proposed openstack/ironic master: WIP: Remove token-less agent support  https://review.opendev.org/74940723:11
openstackgerritJulia Kreger proposed openstack/metalsmith master: Use upper constraints on metalsmith installation  https://review.opendev.org/74939123:12
openstackgerritSteve Baker proposed openstack/metalsmith master: Document metalsmith_instances  https://review.opendev.org/73813123:14
openstackgerritSteve Baker proposed openstack/metalsmith master: metalsmith_deployment role switch to metalsmith_instances  https://review.opendev.org/73788823:14
*** Lucas_Gray has quit IRC23:19
*** Lucas_Gray has joined #openstack-ironic23:22
uzumakihello ironic o/ anybody here?23:25
uzumakia quick question, the docs say that if you have a single software raid volume, it must be RAID1, does that mean we will have an error if it isn't? or will it just be a 'bad practice'?23:26
*** JamesBenson has joined #openstack-ironic23:35
*** JamesBenson has quit IRC23:39
*** Lucas_Gray has quit IRC23:50

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!