*** janders has quit IRC | 00:47 | |
*** janders has joined #openstack-ironic | 00:49 | |
openstackgerrit | Steve Baker proposed openstack/ironic master: New argument validate decorator https://review.opendev.org/742795 | 01:28 |
---|---|---|
openstackgerrit | Steve Baker proposed openstack/ironic master: Add status_code argument to @method.expose https://review.opendev.org/743677 | 01:28 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Add @method.body decorator https://review.opendev.org/743678 | 01:28 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Duplicate trait validation with jsonschema https://review.opendev.org/745778 | 01:28 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert allocation endpoint to plain JSON https://review.opendev.org/743679 | 01:28 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert bios endpoint to plain JSON https://review.opendev.org/745475 | 01:28 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert chassis endpoint to plain JSON https://review.opendev.org/745476 | 01:28 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert conductor endpoint to plain JSON https://review.opendev.org/745779 | 01:28 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert deploy_templates endpoint to plain JSON https://review.opendev.org/745780 | 01:28 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert drivers endpoint to plain JSON https://review.opendev.org/741865 | 01:28 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert event endpoint to plain JSON https://review.opendev.org/746623 | 01:29 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert node endpoint to plain JSON https://review.opendev.org/747820 | 01:29 |
*** rh-jelabarre has quit IRC | 01:40 | |
*** rh-jelabarre has joined #openstack-ironic | 01:40 | |
*** Qianbiao has joined #openstack-ironic | 01:52 | |
*** ijw has quit IRC | 02:24 | |
*** ijw has joined #openstack-ironic | 02:25 | |
*** rh-jelabarre has quit IRC | 02:28 | |
*** ijw_ has joined #openstack-ironic | 02:29 | |
*** ijw has quit IRC | 02:30 | |
*** ijw_ has quit IRC | 02:34 | |
*** rcernin has quit IRC | 02:40 | |
*** ijw has joined #openstack-ironic | 02:43 | |
*** rh-jelabarre has joined #openstack-ironic | 02:47 | |
*** rh-jelabarre has quit IRC | 02:52 | |
*** gyee has quit IRC | 03:13 | |
*** rcernin has joined #openstack-ironic | 03:13 | |
*** rcernin has quit IRC | 03:19 | |
*** rcernin has joined #openstack-ironic | 03:19 | |
*** Qianbiao has quit IRC | 03:43 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Adds few of the security dashboard parameters to capabilities https://review.opendev.org/746416 | 04:51 |
*** hjensas has joined #openstack-ironic | 05:24 | |
*** tzumainn has quit IRC | 05:31 | |
*** Qianbiao has joined #openstack-ironic | 05:44 | |
*** cdearborn has quit IRC | 05:56 | |
*** belmoreira has joined #openstack-ironic | 06:51 | |
*** belmoreira has quit IRC | 06:53 | |
*** akahat is now known as akahat|rover | 07:05 | |
rpittau | good morning ironic! o/ | 07:06 |
*** jtomasek has joined #openstack-ironic | 07:12 | |
Qianbiao | morning rpittau | 07:15 |
rpittau | hey Qianbiao :) | 07:15 |
Qianbiao | hey o/ | 07:16 |
Qianbiao | iurygregory is late today :) | 07:16 |
*** tosky has joined #openstack-ironic | 07:21 | |
*** rcernin has quit IRC | 07:36 | |
*** fmuyassarov has joined #openstack-ironic | 07:38 | |
iurygregory | good morning rpittau Qianbiao o/ | 07:41 |
rpittau | hey iurygregory :) | 07:41 |
iurygregory | I was finishing my breakfast =) | 07:41 |
iurygregory | cold weather make me sleep more time hehe | 07:41 |
Qianbiao | iurygregory morning o/ | 07:47 |
Qianbiao | Hello, when run setup ironic by devstack, which dhcp will pxe uses. | 07:49 |
Qianbiao | through shared network? | 07:50 |
Qianbiao | ^^ http://paste.openstack.org/show/797317/ | 07:50 |
*** belmoreira has joined #openstack-ironic | 07:51 | |
Qianbiao | And where can i read the dhcp log ~~ | 07:53 |
iurygregory | journalctl for neutron | 07:53 |
iurygregory | it should be there I would say | 07:53 |
iurygregory | q-dhcp | 07:54 |
iurygregory | https://docs.openstack.org/devstack/latest/systemd.html can be useful | 07:55 |
Qianbiao | iurygregory thanks, seems right | 07:55 |
*** dougsz has joined #openstack-ironic | 07:56 | |
openstackgerrit | Riccardo Pittau proposed openstack/tenks master: Add skip list to ansible lint https://review.opendev.org/749201 | 07:58 |
*** rcernin has joined #openstack-ironic | 07:59 | |
*** rcernin has quit IRC | 08:02 | |
Qianbiao | neutron did not update ironic node port to dhcp leases file | 08:04 |
*** priteau has joined #openstack-ironic | 08:04 | |
Qianbiao | but in log, it display it writes, any idea? | 08:04 |
Qianbiao | http://paste.openstack.org/show/797320/ | 08:04 |
Qianbiao | For mac: 28:a6:db:a9:92:28 | 08:04 |
openstackgerrit | Merged openstack/bifrost bugfix/8.3: Make /var/lib/ironic/{,images,master_images} readable by nginx https://review.opendev.org/749021 | 08:05 |
openstackgerrit | Merged openstack/bifrost stable/ussuri: Make /var/lib/ironic/{,images,master_images} readable by nginx https://review.opendev.org/749022 | 08:05 |
*** Lucas_Gray has joined #openstack-ironic | 08:10 | |
janders | g'day Ironic o/ | 08:11 |
iurygregory | hey janders o/ | 08:14 |
*** lucasagomes has joined #openstack-ironic | 08:15 | |
rpittau | hey janders :) | 08:17 |
rpittau | easy review if anyone has a minute :) https://review.opendev.org/747883 | 08:18 |
patchbot | patch 747883 - bifrost - ansible-lint: fix error 206 - 4 patch sets | 08:18 |
iurygregory | ack | 08:19 |
iurygregory | I'm wondering if you went file by file adding the spaces rpittau | 08:19 |
iurygregory | WUT 602 is also working? | 08:20 |
rpittau | iurygregory: I'd rather not revail my secrets :P | 08:20 |
rpittau | iurygregory: yep | 08:20 |
iurygregory | haha | 08:20 |
iurygregory | sed is a good one XD | 08:21 |
rpittau | jokes aside, IDEs and text editors help a lot in these cases | 08:21 |
iurygregory | yeah | 08:21 |
*** fmuyassarov has quit IRC | 08:21 | |
*** sshnaidm|bbl is now known as sshnaidm | 08:36 | |
*** derekh has joined #openstack-ironic | 08:46 | |
openstackgerrit | Merged openstack/bifrost master: ansible-lint: fix error 206 https://review.opendev.org/747883 | 08:50 |
*** Nisha_Agarwal has joined #openstack-ironic | 08:53 | |
openstackgerrit | Merged openstack/bifrost master: ansible-lint: fix error 504 https://review.opendev.org/747925 | 08:56 |
*** Lucas_Gray has quit IRC | 08:59 | |
*** dtantsur|afk is now known as dtantsur | 09:05 | |
dtantsur | morning ironic | 09:06 |
dtantsur | TheJulia: no particular reason (re dib in metalsmith CI), the CI was created before we switched to DIB. | 09:06 |
*** Lucas_Gray has joined #openstack-ironic | 09:06 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Stop using --system-site-packages https://review.opendev.org/749081 | 09:09 |
iurygregory | morning dtantsur | 09:09 |
openstackgerrit | Merged openstack/bifrost master: ansible-lint: fix error 602 https://review.opendev.org/748597 | 09:12 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Workaround for setuptools 50.0 https://review.opendev.org/749207 | 09:13 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Stop using --system-site-packages https://review.opendev.org/749081 | 09:15 |
janders | good morning dtantsur | 09:21 |
Nisha_Agarwal | dtantsur, good morning :) | 09:21 |
mnasiadka | Good morning | 09:22 |
mnasiadka | dtantsur: 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 |
patchbot | patch 749014 - bifrost (stable/stein) - [Train and older] Fix several CI problems - 1 patch set | 09:22 |
openstackgerrit | Riccardo Pittau proposed openstack/tenks master: Use victoria tests https://review.opendev.org/749208 | 09:23 |
openstackgerrit | Riccardo Pittau proposed openstack/bifrost master: Fix error 601 https://review.opendev.org/749209 | 09:26 |
openstackgerrit | Riccardo Pittau proposed openstack/bifrost master: Fix error 601 https://review.opendev.org/749209 | 09:26 |
dtantsur | mnasiadka: is it blocking you? I was going to take a look but I have more pressing issues right now. | 09:31 |
mnasiadka | dtantsur: well, it blocks Kolla builds/publishes of fresh Stein Ironic containers, not pressing a lot, but would be nice to fix it soon | 09:31 |
dtantsur | mnasiadka: okay, I'll move it higher up my priority list | 09:32 |
openstackgerrit | Riccardo Pittau proposed openstack/tenks master: Fix ansiblelint 208 https://review.opendev.org/749211 | 09:33 |
Qianbiao | hi, is devstack problem fixed | 09:35 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Workaround for setuptools 50.0 https://review.opendev.org/749207 | 09:39 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Stop using --system-site-packages https://review.opendev.org/749081 | 09:40 |
dtantsur | Qianbiao: the workaround has landed. I'm not sure our multinode job has been fixed though | 09:43 |
rpittau | I rechecked a trivial change around 2 hours ago to see how it's going | 09:44 |
rpittau | here should end soon, multinode has not finished yet https://review.opendev.org/748978 | 09:46 |
patchbot | patch 748978 - ironic - [trivial] remove emacs config from devstack script - 1 patch set | 09:46 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Replace oslo's loopingcall with tenacity https://review.opendev.org/749214 | 09:54 |
Qianbiao | dtantsur thanks. | 09:54 |
*** tkajinam has quit IRC | 09:54 | |
*** tkajinam has joined #openstack-ironic | 09:55 | |
Qianbiao | dtantsur Is ironic devstack for virt nodes works for hardware bmc too? | 09:55 |
Qianbiao | I mean it's installed with option "IRONIC_IS_HARDWARE=False" | 09:55 |
Qianbiao | can i just add a hardware node directly | 09:56 |
dtantsur | Qianbiao: ironic should work, but networking may not | 09:59 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: [PoC] Do not use eventlet and oslo.service for serving API https://review.opendev.org/749020 | 09:59 |
Qianbiao | dtantsur I guess so, bmc pxe could not dhcp for now. | 10:00 |
Qianbiao | dtantsur what if i set "IRONIC_IS_HARDWARE=True" and add hardware_info defines, is it ready for use? | 10:01 |
*** antotala has joined #openstack-ironic | 10:08 | |
openstackgerrit | Merged openstack/tenks master: Add skip list to ansible lint https://review.opendev.org/749201 | 10:29 |
*** Nisha_Agarwal has quit IRC | 10:36 | |
*** mgagne has quit IRC | 10:39 | |
mnasiadka | dtantsur: can you also propose bifrost 7.2.2 for Train with the merged fix? | 10:42 |
dtantsur | iurygregory or rpittau or anyone if you have time ^^ | 10:42 |
iurygregory | dtantsur, I can | 10:43 |
openstackgerrit | Aija Jaunteva proposed openstack/sushy master: Add BIOS update apply time and maintenance window https://review.opendev.org/749236 | 10:43 |
iurygregory | [Train and older] Fix several CI problems with this one right? | 10:45 |
*** vmud213 has joined #openstack-ironic | 10:45 | |
dtantsur | iurygregory: yeah | 10:46 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Stop using --system-site-packages https://review.opendev.org/749081 | 10:46 |
iurygregory | dtantsur, done | 10:49 |
dtantsur | thx! | 10:49 |
iurygregory | np | 10:49 |
openstackgerrit | Aija Jaunteva proposed openstack/ironic master: Add Redfish BIOS internface to idrac HW type https://review.opendev.org/749240 | 10:54 |
openstackgerrit | Aija Jaunteva proposed openstack/ironic master: Add Redfish BIOS interface to idrac HW type https://review.opendev.org/749240 | 10:55 |
*** belmoreira has quit IRC | 10:57 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Stop using --system-site-packages https://review.opendev.org/749081 | 11:05 |
*** tonyb has quit IRC | 11:05 | |
dtantsur | devstack seems fixed, we're back to broken multinode.. | 11:06 |
iurygregory | yeah =( | 11:11 |
* iurygregory brb lunch time | 11:11 | |
*** antotala has quit IRC | 11:12 | |
*** Lucas_Gray has quit IRC | 11:14 | |
openstackgerrit | Merged openstack/tenks master: Use victoria tests https://review.opendev.org/749208 | 11:17 |
*** tonyb has joined #openstack-ironic | 11:31 | |
Qianbiao | <dtantsur> devstack still has bug. | 11:38 |
Qianbiao | when setting USE_PYTHON3=True, still fails on ubuntu 18.04 for branch train | 11:38 |
*** Lucas_Gray has joined #openstack-ironic | 11:40 | |
arne_wiebalck | Good morning, ironic! | 11:43 |
*** Qianbiao has quit IRC | 11:46 | |
*** Qianbiao has joined #openstack-ironic | 11:46 | |
*** fmuyassarov has joined #openstack-ironic | 11:58 | |
*** chenhaw has quit IRC | 12:00 | |
*** rh-jelabarre has joined #openstack-ironic | 12:03 | |
dtantsur | Qianbiao: in the ironic plugin or in devstack itself? if the latter, try asking on #openstack-qa | 12:03 |
*** rh-jelabarre has quit IRC | 12:04 | |
*** rh-jelabarre has joined #openstack-ironic | 12:04 | |
Qianbiao | dtantsur devstack | 12:05 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Stop using --system-site-packages https://review.opendev.org/749081 | 12:05 |
dtantsur | folks, this ^^ is getting close but it's so ugly I'm not sure we should merge it :( | 12:05 |
dtantsur | but system-site-packages is also bad.. | 12:05 |
dtantsur | opinions? | 12:05 |
dtantsur | the alternative is https://review.opendev.org/749207 which fixes the setuptools issue via a workaround I don't understand | 12:06 |
patchbot | patch 749207 - bifrost - Workaround for setuptools 50.0 - 2 patch sets | 12:06 |
*** Nisha_Agarwal has joined #openstack-ironic | 12:10 | |
Qianbiao | dtantsur https://github.com/pypa/setuptools/blob/04e3df22df840c6bb244e9b27bc56750c44b7c85/CHANGES.rst | 12:15 |
Qianbiao | maybe changelog will help you | 12:15 |
iurygregory | good afternoon arne_wiebalck =) | 12:16 |
arne_wiebalck | yeah ... dentist this morning :-) | 12:16 |
iurygregory | I hope it was ok =) | 12:17 |
arne_wiebalck | yep | 12:17 |
*** thiagop has joined #openstack-ironic | 12:20 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Adds few of the security dashboard parameters to capabilities https://review.opendev.org/746416 | 12:22 |
rpittau | dtantsur: 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 go | 12:26 |
dtantsur | I've unblocked https://review.opendev.org/#/c/749207/ | 12:27 |
patchbot | patch 749207 - bifrost - Workaround for setuptools 50.0 - 2 patch sets | 12:27 |
*** cdearborn has joined #openstack-ironic | 12:27 | |
dtantsur | I'll probably abandon the system-site-packages patch | 12:27 |
*** Nisha_Agarwal has quit IRC | 12:27 | |
rpittau | ok :/ | 12:28 |
openstackgerrit | QianBiao Ng proposed openstack/ironic master: Feat: add ibmc hardware info support for devstack https://review.opendev.org/748841 | 12:33 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Remove unnecessary overrides of ansible_python_interpreter https://review.opendev.org/749262 | 12:36 |
*** derekh has quit IRC | 12:39 | |
bfournie | hi 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.yml | 12:50 |
patchbot | patch 738042 - bifrost - Use firewalld to open ports on CentOS and RHEL. (MERGED) - 9 patch sets | 12:50 |
bfournie | this is with Centos on Bifrost node | 12:50 |
dtantsur | bfournie: 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-ironic | 12:54 | |
dtantsur | bfournie: firewall-cmd --get-active-zones? | 12:55 |
dtantsur | firewall-cmd --list-all --zone public? | 12:55 |
bfournie | dtantsur: 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 zone | 12:56 |
bfournie | dtantsur: via ./bifrost-cli install --network-interface eno49 --dhcp-pool 192.168.24.5-192.168.24.24 | 12:58 |
dtantsur | bfournie: 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 |
bfournie | dtantsur: I will try it, thanks | 12:59 |
rpittau | ready for the second day of Ironic Victoria Virtual Midcycle Meetup ? -> https://meetpad.opendev.org/Ironic-Victoria-midcycle | 12:59 |
*** fmuyassarov has quit IRC | 13:00 | |
dtantsur | bfournie: hmmm, judging by quick googling, it should be handled by NM | 13:00 |
TheJulia | good morning | 13:00 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/ironic-ui master: [goal] Migrate testing to ubuntu focal https://review.opendev.org/744708 | 13:00 |
iurygregory | good morning TheJulia =) | 13:00 |
bfournie | dtantsur: interesting | 13:01 |
dtantsur | morning TheJulia | 13:01 |
*** Lucas_Gray has quit IRC | 13:02 | |
*** trandles has joined #openstack-ironic | 13:05 | |
janders | bfournie: 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 online | 13:05 |
janders | hi TheJulia o/ | 13:05 |
*** chenhaw has joined #openstack-ironic | 13:05 | |
bfournie | janders: sounds good, thanks! | 13:05 |
trandles | hi janders | 13:06 |
trandles | janders, I hit some firewalld issues with bifrost that dtantsur fixed | 13:07 |
trandles | might be some overlap | 13:07 |
dtantsur | I guess ideally we should create our own zone in bifrost, add our network interface to it and then modify only it | 13:11 |
dtantsur | but I'm not sure I understand the patterns of firewalld well | 13:12 |
* TheJulia needs much more coffeeeeee | 13:12 | |
dtantsur | bfournie: if I prepare a proof-of-concept patch today/tomorrow will you be able to test it? | 13:15 |
dtantsur | I don't have real bare metal at hand | 13:15 |
trandles | I can test since all I have is bare metal | 13:16 |
dtantsur | okay. 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 IRC | 13:17 | |
*** chenhaw has quit IRC | 13:18 | |
janders | dtantsur 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 colleague | 13:20 |
janders | trandles hi! :) and thanks for helping with this! | 13:21 |
janders | it's getting really late here - time for me to knock off | 13:21 |
janders | see you tomorrow Ironic o/ | 13:21 |
trandles | later janders | 13:21 |
*** Goneri has joined #openstack-ironic | 13:27 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Create our own firewalld zone and use it on real bare metal https://review.opendev.org/749285 | 13:33 |
dtantsur | trandles, bfournie, janders ^^^ | 13:33 |
dtantsur | I'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-ironic | 13:34 | |
bfournie | dtantsur: nice, thanks, I will test it out | 13:35 |
*** janders has quit IRC | 13:37 | |
*** janders5 is now known as janders | 13:37 | |
*** chenhaw has joined #openstack-ironic | 13:37 | |
*** tzumainn has joined #openstack-ironic | 13:48 | |
openstackgerrit | Vishal Manchanda proposed openstack/ironic-ui master: [goal] Migrate testing to ubuntu focal https://review.opendev.org/744708 | 13:51 |
JayF | Is the gate fixed? It looks like ironic-whiteboard was never updated for the last outage so I can't tell | 13:52 |
dtantsur | JayF: partly. devstack is back, but we have https://zuul.openstack.org/builds?job_name=ironic-tempest-ipa-wholedisk-direct-tinyipa-multinode | 13:52 |
dtantsur | the IPA CI should be fine, if that's what you're interested in | 13:52 |
JayF | ack, ty, and I should've used that to check my jobs too :D | 13:52 |
openstackgerrit | Christopher Dearborn proposed openstack/sushy master: WIP: Allow monitoring progress of a firmware update https://review.opendev.org/745950 | 13:57 |
JayF | ttps://docs.openstack.org/ironic/latest/install/configdrive.html we document a 64MB max size | 13:59 |
zer0c00l | dtantsur: Quick question on "Link to firmware update's SimpleUpdate asynchronous behavior in the spec: | 14:10 |
zer0c00l | Sorry " The parent patch should be updated with this change. | 14:10 |
dtantsur | yep. git commit --amend && git review :) | 14:11 |
zer0c00l | Should i squash the commits and push a new change? | 14:11 |
zer0c00l | i see okay | 14:11 |
zer0c00l | Thanks. | 14:11 |
dtantsur | merge the changes from the 2nd commit into the 1st and push a new change | 14:11 |
JayF | Switching between Github flow and Gerrit flow can give you whiplash :| | 14:12 |
zer0c00l | yeah, github doesn't like git commit --amend, allows you to squash when you merge the PR | 14:13 |
*** fmuyassarov has joined #openstack-ironic | 14:14 | |
*** fmuyassarov has quit IRC | 14:15 | |
*** Lucas_Gray has joined #openstack-ironic | 14:18 | |
*** sshnaidm has quit IRC | 14:34 | |
*** vmud213 has quit IRC | 14:47 | |
JayF | zer0c00l: fwiw you can --amend with github, you just have to force push your branch | 14:47 |
*** ijw has quit IRC | 14:48 | |
*** vmud213 has joined #openstack-ironic | 14:48 | |
dtantsur | git push --force \o/ | 14:50 |
JayF | We 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 :P | 14:52 |
JayF | never 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-ironic | 14:53 | |
*** sshnaidm has joined #openstack-ironic | 14:53 | |
jroll | and 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 annoying | 14:55 |
*** trandles has quit IRC | 14:55 | |
*** trandles_ has joined #openstack-ironic | 14:55 | |
*** trandles_ is now known as trandles | 14:55 | |
clarkb | jroll: 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|afk | 15:18 | |
dtantsur|afk | going a bit earlier today, see you tomorrow | 15:18 |
Qianbiao | Hello, anyone has experience with ironic devstack. | 15:24 |
Qianbiao | I can not clean baremetal node due to pxe dhcp not reach neutron service. | 15:25 |
Qianbiao | and could not figure out why :( | 15:25 |
Qianbiao | this is my local.conf http://paste.openstack.org/show/797341/, anything else i should config? | 15:27 |
*** chihhsin has quit IRC | 15:28 | |
openstackgerrit | Vishal Manchanda proposed openstack/ironic-ui master: [goal] Migrate testing to ubuntu focal https://review.opendev.org/744708 | 15:32 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/metalsmith master: Explicitly set jobs to ML2/OVS https://review.opendev.org/749346 | 15:35 |
*** antotala has joined #openstack-ironic | 15:38 | |
antotala | Hi 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 |
antotala | ramdisk_error,root_disk_selection,scheduler,validate_interfaces,capabilities,pci_devices | 15:43 |
antotala | is 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 |
JayF | I'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 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Move ironic-standalone-ipa-src to Ubuntu Focal https://review.opendev.org/748213 | 15:49 |
antotala | JayF 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|bbl | 15:50 | |
JayF | ack, then I'm not sure how to help. Sorry! Good luck. | 15:51 |
antotala | Thanks JayF | 15:52 |
JayF | OK, so actually going to try and help | 15:53 |
JayF | I think that needs to be installed on the ramdisk | 15:53 |
JayF | but again not 100% sure | 15:53 |
JayF | For 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 ramdisk | 15:53 |
antotala | Thanks 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 above | 15:57 |
JayF | as 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 guess | 15:57 |
*** zzzeek has quit IRC | 15:59 | |
*** zzzeek has joined #openstack-ironic | 16:00 | |
antotala | Thanks JayF , I am gonna try that | 16:01 |
JayF | good luck | 16:02 |
openstackgerrit | Julia Kreger proposed openstack/metalsmith master: Replace the Centos7 image job with a Centos8 UEFI based job https://review.opendev.org/748315 | 16:02 |
*** Lucas_Gray has quit IRC | 16:04 | |
*** zzzeek has quit IRC | 16:05 | |
*** zzzeek has joined #openstack-ironic | 16:06 | |
*** bnemec has quit IRC | 16:06 | |
*** bnemec has joined #openstack-ironic | 16:06 | |
*** vmud213 has quit IRC | 16:08 | |
*** Qianbiao has quit IRC | 16:08 | |
*** lucasagomes has quit IRC | 16:11 | |
*** dougsz has quit IRC | 16:14 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [WIP] Do not allocate initial space for virtual volumes https://review.opendev.org/749357 | 16:15 |
*** Lucas_Gray has joined #openstack-ironic | 16:25 | |
iurygregory | I've pushed a patch to grenade that will offer the possibility to specify services we wish to skip their upgrade | 16:27 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Replace retrying with tenacity https://review.opendev.org/376574 | 16:34 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Native zuulv3 grenade multinode multitenant https://review.opendev.org/705030 | 16:35 |
*** gyee has joined #openstack-ironic | 16:38 | |
iurygregory | let's see how it goes =) | 16:38 |
* iurygregory fingers crossed that at least I will see the message about skipping nova | 16:39 | |
*** ijw_ has joined #openstack-ironic | 16:42 | |
TheJulia | iurygregory: awesome! | 16:43 |
TheJulia | rpittau: 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 view | 16:44 |
*** ijw__ has joined #openstack-ironic | 16:45 | |
*** ijw has quit IRC | 16:45 | |
rpittau | TheJulia: when migrating ipa jobs to ubuntu focal I stumbled into the issue where we consume all the space available when creating the volumes | 16:45 |
rpittau | TheJulia: https://review.opendev.org/748213 | 16:46 |
patchbot | patch 748213 - ironic-python-agent - Move ironic-standalone-ipa-src to Ubuntu Focal - 8 patch sets | 16:46 |
TheJulia | rpittau: on / ? | 16:46 |
TheJulia | or /opt? | 16:46 |
rpittau | on /opt | 16:46 |
TheJulia | wow | 16:46 |
rpittau | we pass from 35 G to over 70G | 16:46 |
TheJulia | and /opt is variable on the clouds | 16:46 |
TheJulia | funky :\ | 16:46 |
*** sshnaidm|bbl is now known as sshnaidm | 16:46 | |
*** Lucas_Gray has quit IRC | 16:47 | |
rpittau | I 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.txt | 16:47 |
*** antotala has quit IRC | 16:47 | |
rpittau | it might be that by default the allocation is 0 until a certain version, but I couldn't find it documented anywhere | 16:48 |
*** ijw_ has quit IRC | 16:49 | |
*** Lucas_Gray has joined #openstack-ironic | 16:49 | |
*** derekh has quit IRC | 16:51 | |
TheJulia | I think it was doing a thin allocation | 16:57 |
TheJulia | sparse | 16:57 |
*** priteau has quit IRC | 16:59 | |
*** mushrushu has joined #openstack-ironic | 17:01 | |
openstackgerrit | Julia Kreger proposed openstack/metalsmith master: Replace the Centos7 image job with a Centos8 UEFI based job https://review.opendev.org/748315 | 17:01 |
*** sshnaidm is now known as sshnaidm|afk | 17:11 | |
TheJulia | There are many subtle differences in IO behavior there sadly :( | 17:16 |
*** jtomasek has quit IRC | 17:26 | |
*** k_mouza has joined #openstack-ironic | 17:41 | |
*** tonyb has quit IRC | 17:44 | |
*** JamesBenson has quit IRC | 17:44 | |
*** mushrushu has quit IRC | 17:44 | |
*** ddurst has quit IRC | 17:44 | |
*** Lucas_Gray has quit IRC | 17:44 | |
*** mushrushu has joined #openstack-ironic | 17:46 | |
*** tonyb has joined #openstack-ironic | 17:46 | |
*** JamesBenson has joined #openstack-ironic | 17:46 | |
*** ddurst has joined #openstack-ironic | 17:46 | |
JayF | https://review.opendev.org/#/c/748767/ is +2 and Verified, pretty basic bugfix would love to get it reviewed | 17:48 |
patchbot | patch 748767 - ironic-python-agent - Make WSGI server respect listen_* directives - 5 patch sets | 17:48 |
TheJulia | sure! | 17:54 |
TheJulia | stevebaker: looks like metalsmith may be broken :\ | 17:54 |
TheJulia | https://www.irccloud.com/pastebin/s0HtBIM5/ | 17:54 |
JayF | https://review.opendev.org/#/c/747193/ also has 1 +1 and is passing CI ,would love review son it | 17:55 |
patchbot | patch 747193 - ironic-python-agent - If listen_tls is true, enable TLS on wsgi server - 8 patch sets | 17:55 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Add fstab pointer to EFI partition https://review.opendev.org/748260 | 17:56 |
*** k_mouza has quit IRC | 17:56 | |
*** dsneddon has joined #openstack-ironic | 17:58 | |
openstackgerrit | Merged openstack/bifrost master: Move services to internal_ip by default and refactor endpoint creation https://review.opendev.org/748230 | 17:59 |
*** dsneddon has quit IRC | 18:04 | |
openstackgerrit | Merged openstack/bifrost master: Workaround for setuptools 50.0 https://review.opendev.org/749207 | 18:06 |
openstackgerrit | Merged openstack/bifrost master: Update keystone documentation https://review.opendev.org/748392 | 18:06 |
openstackgerrit | Merged openstack/bifrost master: Remove support for the legacy CSV format https://review.opendev.org/747893 | 18:06 |
openstackgerrit | Verification of a change to openstack/bifrost failed: Use packaged iPXE ROMs for openSUSE https://review.opendev.org/747894 | 18:06 |
openstackgerrit | Merged openstack/bifrost master: Remove support for OpenStack MetaData version 2012-08-10 https://review.opendev.org/747900 | 18:06 |
openstackgerrit | Merged openstack/bifrost master: Trivial: remove a reference to update-rc.d https://review.opendev.org/748237 | 18:06 |
* thiagop thinks zuul is working on the double today | 18:07 | |
*** ociuhandu has joined #openstack-ironic | 18:19 | |
TheJulia | bifrost jobs go fairly quick | 18:34 |
TheJulia | ) | 18:34 |
TheJulia | JayF: approverated for the first one :) | 18:35 |
*** ociuhandu has quit IRC | 18:36 | |
trandles | ping TheJulia | 18:52 |
TheJulia | pong | 18:55 |
trandles | TheJulia: thanks, setting up a new IRC client with persistent logging | 18:56 |
trandles | I haven't taken IRC seriously in like a decade...sigh...I've forgotten The Old Ways | 18:56 |
TheJulia | heh | 18:57 |
*** openstackgerrit has quit IRC | 19:21 | |
rpioso | TheJulia: 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 |
TheJulia | rpioso: 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 |
rpioso | TheJulia: Should we send a message to the ML to inform the community? | 19:45 |
*** thiagop has quit IRC | 19:53 | |
TheJulia | rpioso: I suspect that would be good, just to raise awareness | 20:00 |
*** dsneddon has joined #openstack-ironic | 20:03 | |
*** openstackgerrit has joined #openstack-ironic | 20:18 | |
openstackgerrit | Julia Kreger proposed openstack/metalsmith master: Attempt to workaround apparent pip issues https://review.opendev.org/749391 | 20:18 |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Make WSGI server respect listen_* directives https://review.opendev.org/748767 | 20:20 |
TheJulia | I hope that fixes metalsmith since it seems to be presenting like site packages weirdness | 20:21 |
*** Lucas_Gray has joined #openstack-ironic | 20:22 | |
TheJulia | Fyi everyone, I submitted a project update for the upcoming summit. | 20:33 |
TheJulia | Of course, I haven't made slides yet, but figure it is good to get that submitted. | 20:33 |
*** Lucas_Gray has quit IRC | 20:35 | |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Increase memory for tinyipa jobs https://review.opendev.org/747634 | 20:37 |
*** Lucas_Gray has joined #openstack-ironic | 20:42 | |
TheJulia | Reminder: Doodle to discuss kexec - https://doodle.com/poll/ihbbtns36b2rkhfd | 20:49 |
stevebaker | morning | 20:51 |
openstackgerrit | Merged openstack/networking-baremetal master: Add missing keystoneauth1 and oslo.service to requirements https://review.opendev.org/747936 | 20:57 |
stevebaker | TheJulia: 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.txt | 21:00 |
patchbot | patch 749257 - requirements (stable/ussuri) - Blacklist setuptools 50.0.0 - 1 patch set | 21:00 |
JayF | TheJulia: ack, ty | 21:18 |
TheJulia | stevebaker: 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 quietly | 21:23 |
clarkb | also keep in mind there tends to be a chicken and egg issue with restricting setuptools versions | 21:28 |
clarkb | depending on where things break you may need to preinstall setuptools too | 21:29 |
openstackgerrit | Arun S A G proposed openstack/ironic-specs master: Add support for anaconda deploy interface https://review.opendev.org/748503 | 21:36 |
TheJulia | I'd assume it would get pulled in well before this too... and there is also the headache with how requirements processing occurs differently | 21:41 |
*** uzumaki has joined #openstack-ironic | 22:11 | |
openstackgerrit | Christopher Dearborn proposed openstack/sushy master: WIP: Allow monitoring progress of a firmware update https://review.opendev.org/745950 | 22:20 |
*** JamesBenson has quit IRC | 22:27 | |
*** Goneri has quit IRC | 22:37 | |
*** tosky has quit IRC | 22:43 | |
*** JamesBenson has joined #openstack-ironic | 22:58 | |
*** rcernin has joined #openstack-ironic | 23:01 | |
*** hjensas has quit IRC | 23:02 | |
*** JamesBenson has quit IRC | 23:03 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP: Remove token-less agent support https://review.opendev.org/749407 | 23:11 |
openstackgerrit | Julia Kreger proposed openstack/metalsmith master: Use upper constraints on metalsmith installation https://review.opendev.org/749391 | 23:12 |
openstackgerrit | Steve Baker proposed openstack/metalsmith master: Document metalsmith_instances https://review.opendev.org/738131 | 23:14 |
openstackgerrit | Steve Baker proposed openstack/metalsmith master: metalsmith_deployment role switch to metalsmith_instances https://review.opendev.org/737888 | 23:14 |
*** Lucas_Gray has quit IRC | 23:19 | |
*** Lucas_Gray has joined #openstack-ironic | 23:22 | |
uzumaki | hello ironic o/ anybody here? | 23:25 |
uzumaki | a 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-ironic | 23:35 | |
*** JamesBenson has quit IRC | 23:39 | |
*** Lucas_Gray has quit IRC | 23:50 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!