| *** gyee has quit IRC | 00:00 | |
| *** r-daneel has quit IRC | 00:10 | |
| *** moshele has quit IRC | 00:35 | |
| *** jesusaur has quit IRC | 00:42 | |
| bdodd | TheJulia: Yes, when I updated my master branch, rebased and did git review again it seemed to resolve the merge failure. | 00:46 |
|---|---|---|
| *** ddurst- is now known as ddurst | 01:04 | |
| *** phuongnh has joined #openstack-ironic | 01:09 | |
| *** rmcallis has joined #openstack-ironic | 02:39 | |
| *** rmcall has quit IRC | 02:41 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: Support manipulating iptables filter under IPv6 network https://review.openstack.org/590076 | 02:55 |
| *** rmcall has joined #openstack-ironic | 02:56 | |
| *** rmcallis has quit IRC | 02:57 | |
| *** jesusaur has joined #openstack-ironic | 03:38 | |
| *** e0ne has joined #openstack-ironic | 04:11 | |
| *** e0ne has quit IRC | 04:24 | |
| *** gkadam has joined #openstack-ironic | 04:28 | |
| openstackgerrit | Nguyen Van Trung proposed openstack/ironic master: Update docs to portgroup with creating windows images https://review.openstack.org/585613 | 04:46 |
| *** jesusaur has quit IRC | 04:46 | |
| *** jesusaur has joined #openstack-ironic | 04:48 | |
| *** galaxyblr has joined #openstack-ironic | 04:55 | |
| *** galaxyblr has quit IRC | 05:51 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Reformat instructions related with various OS https://review.openstack.org/590109 | 05:57 |
| *** hamzy_ has quit IRC | 06:29 | |
| *** hamzy_ has joined #openstack-ironic | 06:29 | |
| *** pcaruana has joined #openstack-ironic | 06:38 | |
| *** etingof has quit IRC | 06:42 | |
| *** hshiina has joined #openstack-ironic | 06:56 | |
| *** ajya has joined #openstack-ironic | 07:15 | |
| *** rcernin has quit IRC | 07:16 | |
| openstackgerrit | Aija Jaunteva proposed openstack/sushy master: Add support for loading resources from archive file https://review.openstack.org/589147 | 07:32 |
| *** ajya has quit IRC | 07:38 | |
| *** ajya has joined #openstack-ironic | 07:39 | |
| *** juhak has left #openstack-ironic | 07:43 | |
| *** etingof has joined #openstack-ironic | 07:45 | |
| *** skazi has quit IRC | 07:54 | |
| *** lucasagomes has joined #openstack-ironic | 08:12 | |
| openstackgerrit | Merged openstack/ironic-specs master: Follow-up to owner field specificaiton https://review.openstack.org/588356 | 08:17 |
| *** tssurya has joined #openstack-ironic | 08:19 | |
| *** skazi has joined #openstack-ironic | 08:23 | |
| *** dougsz_ has joined #openstack-ironic | 08:26 | |
| *** ibmko has quit IRC | 08:28 | |
| *** dtantsur|afk is now known as dtantsur | 08:33 | |
| dtantsur | good chilly (sarcasm) morning ironic | 08:33 |
| *** skazi has quit IRC | 08:36 | |
| etingof | good morning, dtantsur o/ | 08:37 |
| etingof | wish today is the pinnacle of this heat madness, it's gonna cool down afterwards | 08:38 |
| dtantsur | yep. well, it's nice and cool today - "just" 32 :D | 08:44 |
| *** priteau has quit IRC | 08:47 | |
| etingof | they promised 36 over here today \o/ | 08:47 |
| *** priteau has joined #openstack-ironic | 08:47 | |
| *** mgoddard has joined #openstack-ironic | 09:00 | |
| dtantsur | ugh, I'm sorry for you :( | 09:11 |
| *** jaganathan has joined #openstack-ironic | 09:28 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: A minor update to documentation of `ilo` hardware type https://review.openstack.org/590203 | 09:33 |
| *** derekh has joined #openstack-ironic | 09:35 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: A minor update to documentation of `ilo` hardware type https://review.openstack.org/590203 | 09:58 |
| *** serlex has joined #openstack-ironic | 09:58 | |
| *** gkadam has quit IRC | 10:13 | |
| *** gkadam has joined #openstack-ironic | 10:13 | |
| * lucasagomes is sorry for dtantsur and etingof | 10:18 | |
| dtantsur | hey lucasagomes, how is it going? chilly at your place? :) | 10:18 |
| lucasagomes | 17C here today (-: | 10:18 |
| dtantsur | hah, nice | 10:18 |
| lucasagomes | no rain which is great too | 10:19 |
| lucasagomes | but lets see, it won't last long I think | 10:19 |
| openstackgerrit | Nguyen Van Trung proposed openstack/ironic master: Fix not exist deploy image within irmc-virtual-media booting https://review.openstack.org/589702 | 10:33 |
| *** hshiina has quit IRC | 10:34 | |
| *** Tomatosoup- has joined #openstack-ironic | 10:42 | |
| Tomatosoup- | hello | 10:43 |
| Tomatosoup- | I'm trying to inspect (fake) baremetal node using ironic-inspector, where by fake i mean that it is a virtual machine with virtualbmc configured | 10:44 |
| Tomatosoup- | using ipmitool I can check the chassis power status of the VM with virtualbmc | 10:45 |
| Tomatosoup- | however `ipmitool power on` is failing by no obvious reason | 10:45 |
| Tomatosoup- | >Set Chassis Power Control to Up/On failed: Command not supported in present state | 10:45 |
| Tomatosoup- | thus, also ironic inspection fails | 10:46 |
| Tomatosoup- | anyone handled such issue, or configured successfully virtualbmc? | 10:46 |
| etingof | Tomatosoup-, this indicates a libvirt error, could you check vbmc log? | 10:49 |
| Tomatosoup- | etingof, where may i find vbmc logs? | 10:51 |
| etingof | Tomatosoup-, no traces in journalctl/syslog? | 10:55 |
| etingof | Tomatosoup-, to enable file logging you need to create an ini file with the name /etc/virtualbmc/virtualbmc.conf containing smth like: [log]\nlogfile=/var/log/virtualbmc/vbmc.log | 10:55 |
| etingof | Tomatosoup-, alternatively, you can try to check out libvirt logs | 10:56 |
| Tomatosoup- | already checked, libvirt logs just contain entries like >2018-08-09 10:55:36.052+0000: shutting down, reason=failed | 10:57 |
| Tomatosoup- | creating vbmc conf file as you suggested | 10:58 |
| Tomatosoup- | etingof, created the file as you suggested, although, after running ipmitool commands it remains empty | 11:03 |
| etingof | Tomatosoup-, vbmc restarted? | 11:03 |
| Tomatosoup- | etingof, that must be it, but how to restart vbmc if it isn't managed by systemd? | 11:05 |
| etingof | Tomatosoup-, just kill all vbmc processes | 11:05 |
| etingof | Tomatosoup-, and then `vbmc start <domain>` | 11:06 |
| Tomatosoup- | etingof, Error: Cannot get interface MTU on 'ironic-pxe': No such device | 11:07 |
| etingof | Tomatosoup-, seems this has smth to do with libvirt domain network configuration... | 11:12 |
| Tomatosoup- | etingof, might be, are there any special requirements on the domain network except from the obvious fact that it must be reachable from the node that we use ipmitool? | 11:13 |
| etingof | Tomatosoup-, this domain can't get up and running. can you start it with virsh? I'd probably look into its network configuration to make sure it refers to existing host network interfaces | 11:16 |
| Tomatosoup- | etingof, ah, i think i know the origin of the error | 11:18 |
| Tomatosoup- | etingof, used default interface name config from openstack-helm | 11:18 |
| Tomatosoup- | etingof, thanks for all the help :) | 11:18 |
| etingof | \o/ | 11:18 |
| *** fultonj has quit IRC | 11:21 | |
| *** fultonj has joined #openstack-ironic | 11:23 | |
| *** phuongnh has quit IRC | 11:28 | |
| *** sambetts|afk has quit IRC | 11:29 | |
| *** sambetts_ has joined #openstack-ironic | 11:31 | |
| *** sambetts_ is now known as sambetts | 11:35 | |
| *** sambetts is now known as sambetts|afk | 11:35 | |
| *** bfournie has quit IRC | 11:38 | |
| *** sambetts|afk has quit IRC | 11:46 | |
| *** sambetts_ has joined #openstack-ironic | 11:47 | |
| TheJulia | good morning | 11:52 |
| *** rh-jelabarre has joined #openstack-ironic | 11:52 | |
| etingof | good morning TheJulia o/ | 11:53 |
| dtantsur | morning TheJulia! do we have releases to do today? /me suspects ALL OF THEM :) | 11:54 |
| TheJulia | dtantsur: I created them all, -1'ed the ironic one for a single patch, and they also asked for cycle hilights... so I'll be doing that as soon as I have coffee | 11:54 |
| dtantsur | nice! | 11:55 |
| TheJulia | dtantsur: well, all minus ironic-staging-drivers | 11:55 |
| dtantsur | yep, I can do that | 11:55 |
| TheJulia | I think we need to go ahead and pull that into our governance this next cycle, just very explicitly place the context around that | 11:55 |
| TheJulia | pas-ha: thoughts on ^^^ | 11:55 |
| pas-ha | no objections from me | 11:55 |
| dtantsur | I have mixed feelings. We need to make sure people don't treat it as official support from us. | 11:56 |
| pas-ha | CAPS BOLD LETTERS | 11:56 |
| pas-ha | DISCLAIMER | 11:56 |
| dtantsur | yeah | 11:56 |
| TheJulia | dtantsur: that would be awesome, since I think my key for signing commits is on one of the hard drives someplace in the back of the bus.... | 11:56 |
| pas-ha | and LOG.warning on import :-) | 11:56 |
| dtantsur | we can set supported=False on them | 11:57 |
| TheJulia | pas-ha: and somehow make those wrapped in <h1></h1> | 11:57 |
| pas-ha | that's definitely | 11:57 |
| pas-ha | re supported = False | 11:57 |
| TheJulia | this sounds like we have a consensus and a plan \o/ | 11:57 |
| TheJulia | Anyway, Coffee time | 11:58 |
| *** sambetts_ has quit IRC | 12:04 | |
| *** sambetts_ has joined #openstack-ironic | 12:04 | |
| * dtantsur releases staging-drivers | 12:12 | |
| *** bfournie has joined #openstack-ironic | 12:12 | |
| *** MattMan has quit IRC | 12:12 | |
| *** MattMan has joined #openstack-ironic | 12:12 | |
| TheJulia | dtantsur: thanks | 12:13 |
| TheJulia | mmmm coffeeeeee | 12:13 |
| TheJulia | a couple quick reviews https://review.openstack.org/#/c/589702/ https://review.openstack.org/590203 | 12:16 |
| patchbot | patch 589702 - ironic - Fix not exist deploy image within irmc-virtual-med... | 12:16 |
| patchbot | patch 590203 - ironic - A minor update to documentation of `ilo` hardware ... | 12:16 |
| dtantsur | TheJulia: I'd prefer the 3rdpary CI passing on the former | 12:22 |
| *** trown|outtypewww is now known as trown | 12:22 | |
| TheJulia | ack, I'm thinking we just release and backport | 12:22 |
| TheJulia | Hate to do that, but c'est la vie | 12:22 |
| dtantsur | TheJulia: yes, both changes are fine to backport. | 12:23 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-specs master: Rework the inspector boot management spec https://review.openstack.org/589230 | 12:27 |
| dtantsur | another take at making ^^^ saner | 12:27 |
| TheJulia | dtantsur: yeah, just wanting to avoid backports at this point but it is unavoidable :) | 12:32 |
| TheJulia | dtantsur: please review https://review.openstack.org/590289 I left out removing classic drivers as I didn't want anything to be perceived as a negative change. | 12:34 |
| patchbot | patch 590289 - releases - Ironic cycle highlights for rocky | 12:34 |
| *** markguz_ has joined #openstack-ironic | 12:34 | |
| *** sambetts_ has quit IRC | 12:36 | |
| *** sambetts_ has joined #openstack-ironic | 12:37 | |
| dtantsur | TheJulia: reviewed | 12:37 |
| TheJulia | dtantsur: gah, ironic-staging-drivers releaese failed | 12:38 |
| *** markguz_ has quit IRC | 12:38 | |
| dtantsur | ugh | 12:39 |
| dtantsur | any link to logs? | 12:39 |
| TheJulia | the instructions have been posted to the mailing list, it appeared in the last couple days | 12:39 |
| dtantsur | lucasagomes: hey, do you still own readthedocs for ironic-staging-drivers? | 12:40 |
| lucasagomes | dtantsur, lemme check but I think so | 12:41 |
| lucasagomes | dtantsur, yup I do | 12:41 |
| dtantsur | lucasagomes: mind adding Julia and me? I'm dtantsur there | 12:42 |
| lucasagomes | dtantsur, not at all! | 12:42 |
| lucasagomes | dtantsur, added. Do you know Julia's username ? | 12:42 |
| lucasagomes | TheJulia, ^ | 12:42 |
| *** oanson has joined #openstack-ironic | 12:42 | |
| TheJulia | I don't have a username there | 12:43 |
| lucasagomes | ah right | 12:43 |
| dtantsur | TheJulia: okay, I'll handle it for now. | 12:43 |
| TheJulia | dtantsur: ack | 12:43 |
| dtantsur | TheJulia: https://review.openstack.org/590295 | 12:45 |
| patchbot | patch 590295 - ironic-staging-drivers - Configure webhook for publishing docs | 12:45 |
| dtantsur | feel free to approve, there's nothing much to review there.. | 12:45 |
| *** ianychoi has quit IRC | 12:49 | |
| dtantsur | TheJulia: re the highlights patch: are deploy steps really exposed to operators? | 12:53 |
| dtantsur | I thought it's all refactoring so far.. well, modulo the deploy_step node's field, which is not overly useful yet. | 12:54 |
| TheJulia | dtantsur: I thought we enabled posting it api for deployment | 12:54 |
| TheJulia | which to me seems exposed | 12:54 |
| *** sambetts_ has quit IRC | 12:54 | |
| Tomatosoup- | etingof, if you are still eager to help, I've adjusted network configs but now other error occur `libvirt: QEMU Driver error : unsupported configuration: append not supported in this QEMU binary` | 12:54 |
| TheJulia | errr, maybe I have old python-ironicclient | 12:55 |
| dtantsur | TheJulia: posting requested deploy steps? I'm not sure. | 12:55 |
| TheJulia | dtantsur: checking | 12:56 |
| Tomatosoup- | might be connected with qemu image i'm creating, not sure tho | 12:56 |
| *** sambetts_ has joined #openstack-ironic | 12:56 | |
| TheJulia | errrrrr, we might not have :\ | 12:56 |
| TheJulia | rutro, we didn't :( | 12:57 |
| * TheJulia sighs | 12:57 | |
| dtantsur | TheJulia: yeah, I think we never got to it | 12:57 |
| TheJulia | We must have gotten pretty far though :\ | 12:58 |
| dtantsur | we have good chances of finishing it in stein, I think | 12:58 |
| TheJulia | very good I think | 12:59 |
| TheJulia | updated the patch | 12:59 |
| TheJulia | and again... ;) | 13:03 |
| etingof | Tomatosoup-, looking at the source, this error might be related to some chardev append property, but I have no idea what that is -- https://github.com/libvirt/libvirt/blob/master/src/qemu/qemu_command.c#L5032 | 13:08 |
| openstackgerrit | OpenStack Release Bot proposed openstack/ironic stable/rocky: Update .gitreview for stable/rocky https://review.openstack.org/590309 | 13:08 |
| openstackgerrit | OpenStack Release Bot proposed openstack/ironic stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/590310 | 13:08 |
| openstackgerrit | OpenStack Release Bot proposed openstack/ironic master: Update reno for stable/rocky https://review.openstack.org/590311 | 13:08 |
| dtantsur | \o/ | 13:09 |
| dtantsur | TheJulia: any plans for stable releases? I'm quite tempted to release queens | 13:09 |
| TheJulia | dtantsur: I think so, I think we need to take a quick look at richard's backport first | 13:10 |
| dtantsur | TheJulia: right. I'm a bit worried it did not pass one of their jobs.. | 13:10 |
| TheJulia | Ahh! was it the job they added? | 13:10 |
| dtantsur | and thanks to the creative tls certificate, I cannot check why it failed | 13:10 |
| dtantsur | no, the non-UEFI one (IIRC) | 13:10 |
| TheJulia | ugh... | 13:10 |
| etingof | Tomatosoup-, possibly something about this option? `-chardev file,append=on|off` | 13:11 |
| *** markguz_ has joined #openstack-ironic | 13:11 | |
| TheJulia | Has anyone reached out to Arkady regarding the certificate? | 13:11 |
| TheJulia | because I think I've raised the issue 3 or more times before | 13:11 |
| * dtantsur too | 13:12 | |
| dtantsur | I recall talking to someone | 13:12 |
| Tomatosoup- | etingof, during qemu image creation, right? | 13:13 |
| openstackgerrit | OpenStack Release Bot proposed openstack/networking-generic-switch stable/rocky: Update .gitreview for stable/rocky https://review.openstack.org/590312 | 13:13 |
| openstackgerrit | OpenStack Release Bot proposed openstack/networking-generic-switch stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/590313 | 13:13 |
| openstackgerrit | OpenStack Release Bot proposed openstack/networking-generic-switch master: Update reno for stable/rocky https://review.openstack.org/590314 | 13:14 |
| openstackgerrit | OpenStack Release Bot proposed openstack/networking-baremetal stable/rocky: Update .gitreview for stable/rocky https://review.openstack.org/590315 | 13:14 |
| openstackgerrit | OpenStack Release Bot proposed openstack/networking-baremetal stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/590316 | 13:14 |
| openstackgerrit | OpenStack Release Bot proposed openstack/networking-baremetal master: Update reno for stable/rocky https://review.openstack.org/590317 | 13:14 |
| openstackgerrit | OpenStack Release Bot proposed openstack/ironic-ui stable/rocky: Update .gitreview for stable/rocky https://review.openstack.org/590318 | 13:14 |
| openstackgerrit | OpenStack Release Bot proposed openstack/ironic-ui stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/590319 | 13:14 |
| openstackgerrit | OpenStack Release Bot proposed openstack/ironic-ui master: Update reno for stable/rocky https://review.openstack.org/590320 | 13:14 |
| openstackgerrit | OpenStack Release Bot proposed openstack/ironic-python-agent stable/rocky: Update .gitreview for stable/rocky https://review.openstack.org/590321 | 13:15 |
| openstackgerrit | OpenStack Release Bot proposed openstack/ironic-python-agent stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/590322 | 13:15 |
| openstackgerrit | OpenStack Release Bot proposed openstack/ironic-python-agent master: Update reno for stable/rocky https://review.openstack.org/590323 | 13:15 |
| TheJulia | all the changes! | 13:15 |
| TheJulia | and there will be more | 13:15 |
| *** sambetts_ has quit IRC | 13:15 | |
| dtantsur | lol, yeah | 13:16 |
| openstackgerrit | OpenStack Release Bot proposed openstack/ironic-inspector stable/rocky: Update .gitreview for stable/rocky https://review.openstack.org/590324 | 13:16 |
| openstackgerrit | OpenStack Release Bot proposed openstack/ironic-inspector stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/590325 | 13:16 |
| TheJulia | And somehow my pxe/ipxe patch is already in merge conflict again.... | 13:16 |
| openstackgerrit | OpenStack Release Bot proposed openstack/ironic-inspector master: Update reno for stable/rocky https://review.openstack.org/590326 | 13:16 |
| TheJulia | :( | 13:16 |
| TheJulia | All, I've removed -2s blocking items from landing to master that were changes. Effectively we're no open for Stein features/items/changes to land to master. | 13:17 |
| TheJulia | s/no/now/ | 13:17 |
| etingof | Tomatosoup-, well, that code snippet I stumbled upon is about the qemu-system command itself, not qemu-img I think... | 13:17 |
| TheJulia | dtantsur: I guess this means next week will be nothing but specs :) | 13:18 |
| *** sambetts_ has joined #openstack-ironic | 13:18 | |
| dtantsur | true | 13:18 |
| openstackgerrit | Julia Kreger proposed openstack/virtualbmc master: Change jobs for Python3 effort https://review.openstack.org/589295 | 13:20 |
| TheJulia | dtantsur: ^^^ removed the test reno to ensure that the job worked | 13:20 |
| openstackgerrit | Merged openstack/ironic master: Follow-up to always link MAC address files https://review.openstack.org/589265 | 13:21 |
| *** markguz_ has quit IRC | 13:21 | |
| dtantsur | TheJulia: I think we're supposed to run all jobs when zuul.d is modified anyway | 13:22 |
| *** sambetts_ has quit IRC | 13:22 | |
| TheJulia | I think the reno stuff limits it to release notes changes. Regardless the destructions indicated that it may be necessary to self test the change. | 13:22 |
| *** sambetts_ has joined #openstack-ironic | 13:23 | |
| dtantsur | okie, we'll know it soon | 13:24 |
| TheJulia | it didn't... | 13:24 |
| TheJulia | but yeah | 13:24 |
| TheJulia | anyway | 13:24 |
| TheJulia | five gazillion things to do next | 13:24 |
| *** jcoufal has joined #openstack-ironic | 13:27 | |
| *** jcoufal_ has joined #openstack-ironic | 13:28 | |
| *** jcoufal has quit IRC | 13:31 | |
| *** sambetts_ has quit IRC | 13:37 | |
| *** sambetts has joined #openstack-ironic | 13:38 | |
| *** jistr is now known as jistr|call | 13:39 | |
| *** baha has joined #openstack-ironic | 13:52 | |
| *** sambetts has quit IRC | 13:55 | |
| *** sambetts_ has joined #openstack-ironic | 13:58 | |
| *** jistr|call is now known as jistr | 13:59 | |
| *** amotoki_ is now known as amotoki | 14:01 | |
| *** markguz_ has joined #openstack-ironic | 14:03 | |
| dtantsur | TheJulia: do you plan on including https://review.openstack.org/#/c/590337/ into the release? | 14:04 |
| patchbot | patch 590337 - ironic-staging-drivers - Clarify status for ironic-staging-drivers | 14:04 |
| TheJulia | no, just wanted to go ahead and get that out of the way since I had the repo in front of me. I think it can merge anytime | 14:04 |
| TheJulia | I really need to stop touching the same lines of code across multiple streams of patches. :\ | 14:05 |
| *** sambetts_ has quit IRC | 14:05 | |
| *** jaypipes has quit IRC | 14:06 | |
| *** sambetts_ has joined #openstack-ironic | 14:07 | |
| *** jaypipes has joined #openstack-ironic | 14:10 | |
| dtantsur | TheJulia: ugh, we somehow ended up using the official theme: https://ironic-staging-drivers.readthedocs.io/en/latest/ | 14:14 |
| dtantsur | time to another fix! | 14:14 |
| TheJulia | dtantsur: propose it to governance ;) | 14:14 |
| TheJulia | lol | 14:14 |
| dtantsur | a bit too late for rocky :) | 14:14 |
| * TheJulia feels slightly lazy, but yeah we need to fix that :( | 14:14 | |
| TheJulia | yeah | 14:14 |
| jroll | morning everyone | 14:15 |
| * jroll still doesn't love the idea of bringing staging drivers into governance, that means we're responsible for them, at that point just bring them into the tree | 14:15 | |
| dtantsur | morning jroll | 14:15 |
| *** ajya has quit IRC | 14:17 | |
| *** ajya has joined #openstack-ironic | 14:17 | |
| jroll | TheJulia: let me know if you need any help with releasing things :) | 14:18 |
| * jroll double checks the guide | 14:18 | |
| TheJulia | we'll have to do the release notes follow-ups once we merge the branch changes | 14:19 |
| TheJulia | some of the steps we have, no longer apply or get done for us of course | 14:19 |
| jroll | TheJulia: did we do the release_mapping patch? | 14:20 |
| TheJulia | jroll: like two weeks ago :) | 14:20 |
| jroll | oh wow, ahead of the game :) | 14:20 |
| *** sambetts_ has quit IRC | 14:21 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: ipxe boot interface - begin the process https://review.openstack.org/583407 | 14:22 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Change BFV tempest job to explicitly use ipxe https://review.openstack.org/583410 | 14:22 |
| TheJulia | jroll: yeah, I just tried to get our ducks in a row so we didn't really have to wait or worry at the last minute | 14:23 |
| TheJulia | It is too much stress to have to deal with that at the point when we have to release | 14:23 |
| TheJulia | NobodyCam: for your python3-ish reviewing pleaseure https://review.openstack.org/#/c/588325/ | 14:24 |
| patchbot | patch 588325 - bifrost - Begin change over to use python3 | 14:24 |
| *** sambetts_ has joined #openstack-ironic | 14:24 | |
| *** gkadam_ has joined #openstack-ironic | 14:26 | |
| jroll | TheJulia: I misread this one yesterday: https://review.openstack.org/#/c/589702/3 | 14:27 |
| patchbot | patch 589702 - ironic - Fix not exist deploy image within irmc-virtual-med... | 14:27 |
| jroll | seems landable, I rechecked | 14:27 |
| TheJulia | if you misread it, on my second patch I misread it too :\ | 14:27 |
| TheJulia | pass, not patch | 14:27 |
| jroll | heh | 14:27 |
| jroll | you had me guiding you incorrectly | 14:27 |
| jroll | oh, I skimmed over openstackgerrit's messages | 14:28 |
| jroll | congrats on the rocky branch cut \o/ | 14:28 |
| TheJulia | Thanks! We should all be really proud. We've done some amazing work this cycle! | 14:29 |
| jroll | ++ | 14:30 |
| *** gkadam has quit IRC | 14:30 | |
| openstackgerrit | Julia Kreger proposed openstack/networking-baremetal master: Changing CI job templates for python3-first https://review.openstack.org/590349 | 14:35 |
| *** sambetts_ has quit IRC | 14:39 | |
| openstackgerrit | Julia Kreger proposed openstack/virtualbmc master: Update name for virtualbmc CI job https://review.openstack.org/589294 | 14:39 |
| dtantsur | broken lower-constraints in ironic-staging-drivers \o/ | 14:39 |
| * dtantsur cries | 14:39 | |
| * TheJulia cries too | 14:40 | |
| *** sambetts_ has joined #openstack-ironic | 14:40 | |
| *** hongbin has joined #openstack-ironic | 14:41 | |
| dtantsur | okay, proposed fixes, will cool down in a park while zuul is working | 14:42 |
| *** rpioso|afk is now known as rpioso | 14:42 | |
| rpioso | Good morning | 14:42 |
| dtantsur | morning rpioso | 14:43 |
| *** dtantsur is now known as dtantsur|brb | 14:43 | |
| rpioso | dtantsur|brb: o/ | 14:43 |
| openstackgerrit | Merged openstack/virtualbmc master: Enable debug logging in tests https://review.openstack.org/588345 | 14:45 |
| *** priteau has quit IRC | 14:52 | |
| mgoddard | hi TheJulia: do you know if bifrost supports UEFI? Looks like it was not finished: https://review.openstack.org/#/c/268808/3 | 14:53 |
| patchbot | patch 268808 - bifrost - Initial support for EFI booting | 14:53 |
| TheJulia | mgoddard: Really trying to head down that path in my sparish time right now, there is a newer where I've been working towards getting UEFI booting VMs for testing, and there are some other dhcp settings. It is not hard to add, it is just hard to test | 14:54 |
| mgoddard | TheJulia: cool, would be a nice addition. Asking for a friend (verdurin) | 14:57 |
| TheJulia | mgoddard: yeah, I actually want to try and get some things like direct http boot running, but I'm unsure | 14:57 |
| TheJulia | it all started with an email from arm folks saying they wanted to use ironic, but in standalone scenarios, and were having some issues. | 14:58 |
| TheJulia | (This has since resulted in a few minor tweaks in the pxe code to make lives easier/better for standalone deployments) | 14:59 |
| verdurin | TheJulia: it worked perfectly with TripleO, once I had a new enough ipxe.efi for my hardware. | 15:01 |
| mgoddard | TheJulia: I have an experiment I've been wanting to try for a while - rip out the service deployment/config from bifrost and replace with kolla-ansible | 15:02 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add conductor_group docs https://review.openstack.org/589277 | 15:02 |
| mgoddard | could probably do that with about 10-20 lines of YAML config | 15:02 |
| mgoddard | keep the node provisioning automation in bifrost | 15:02 |
| TheJulia | verdurin: Yeah, for arm I had to build it :\ But I can't cross-arch build easily with existing tools to build ramdisks and deployment images :( | 15:03 |
| TheJulia | that could be really interesting | 15:03 |
| mgoddard | that's another story, I guess my thinking was it would be one less place to get this working and maintain | 15:03 |
| TheJulia | yeah | 15:05 |
| TheJulia | the goal of bifrost is and has always been to facilitate easy standalone deployment and help ensure ironic doesn't horribly break it | 15:06 |
| mgoddard | it's a noble goal | 15:06 |
| openstackgerrit | Julia Kreger proposed openstack/virtualbmc master: Update name for virtualbmc CI job https://review.openstack.org/589294 | 15:08 |
| TheJulia | indeed | 15:09 |
| TheJulia | also kind of hard to maintain ;) | 15:09 |
| mgoddard | yeah | 15:09 |
| NobodyCam | Good Morning Ioronic'ers | 15:10 |
| mgoddard | wouldn't it be nice if adding neutron support was as simple as putting enable_neutron: true in a config file | 15:10 |
| mgoddard | (this is openstack, perhaps not quite that simple, but kolla gets close) | 15:10 |
| mgoddard | oh also, we could scale up to a multinode control plane | 15:12 |
| *** sambetts_ has quit IRC | 15:13 | |
| NobodyCam | TheJulia: Im confused ... it could be that I've not yet had coffee... in 588325 the run.yaml files use_python3 is exported as false but in install-deps.sh we say # Lets use python3 by default / USE_PYTHON3=${USE_PYTHON3:-true} ??? it very well could be my lack of coffee affecting my ability to think | 15:14 |
| rpioso | NobodyCam: Good morning | 15:16 |
| NobodyCam | Good Morning rpioso :) | 15:16 |
| *** sambetts_ has joined #openstack-ironic | 15:16 | |
| TheJulia | NobodyCam: some of the distros we have jobs for don't support python35 or 36. Or install 3.4 | 15:22 |
| NobodyCam | its the install-deps.sh file that caused me pause. I left it as a comment | 15:23 |
| NobodyCam | I need to actually wakeup with more coffee | 15:23 |
| *** jcoufal_ has quit IRC | 15:24 | |
| TheJulia | coffee +++++ | 15:24 |
| NobodyCam | :) | 15:24 |
| NobodyCam | it got a +1 at least until I can loop back and look at it again :) | 15:25 |
| TheJulia | mgoddard: that is true, would that bring value in your case? I think what really prevents me from adding new ironic feature support directly into the ansible modules is the overhead time of keeping things kind of just in a running state... so if it could be close to that easy... maybe it makes some sense | 15:27 |
| TheJulia | NobodyCam: ack | 15:27 |
| *** dtantsur|brb is now known as dtantsur | 15:29 | |
| *** jcoufal has joined #openstack-ironic | 15:30 | |
| dtantsur | TheJulia: our unsupported message talks about deprecation, it may not be entirely correct: http://logs.openstack.org/37/590337/1/check/ironic-staging-drivers-dsvm-all-drivers/2207ff8/logs/screen-ir-cond.txt.gz?level=WARNING#_Aug_09_14_37_57_605597 | 15:31 |
| TheJulia | oooh! ahh! | 15:32 |
| TheJulia | I guess that makes sense to change | 15:32 |
| dtantsur | TheJulia, pas-ha, need review on https://review.openstack.org/#/c/590352/ and https://review.openstack.org/#/c/590344/ for the release | 15:33 |
| patchbot | patch 590352 - ironic-staging-drivers - Fix lower constraints | 15:33 |
| patchbot | patch 590344 - ironic-staging-drivers - Do not use the official theme from oslosphinx | 15:33 |
| dtantsur | I'm also going to provide a fix for the "async" deprecation, stay tuned | 15:35 |
| *** ajya has quit IRC | 15:35 | |
| *** ajya has joined #openstack-ironic | 15:35 | |
| dtantsur | TheJulia, pas-ha, https://review.openstack.org/590401 | 15:36 |
| patchbot | patch 590401 - ironic-staging-drivers - Do not use deprecated async parameter for passthru | 15:36 |
| dtantsur | vdrok: ^^^ | 15:36 |
| TheJulia | wheeeeee | 15:37 |
| *** priteau has joined #openstack-ironic | 15:37 | |
| mgoddard | TheJulia: I think I need to try it to find out where it falls short, but it could be promising. Limiting the scope of bifrost to tools that build images and interact with the ironic API might help to reduce the maintenance overhead, and allow you to focus on improving that experience | 15:41 |
| TheJulia | mgoddard: worth a try at least | 15:43 |
| prometheanfire | glad lower-constraints are being worked on | 15:45 |
| dtantsur | mgoddard: neutron support is not easy, IIRC ansible modules do not support vif attach/detach | 15:45 |
| dtantsur | one of the reasons I started writing metalsmith (in python) | 15:45 |
| TheJulia | dtantsur: and that is only because I've not had time to get back to them | 15:47 |
| dtantsur | TheJulia: this is also because of proper microversions support in openstacksdk, which only appeared weeks ago :) | 15:47 |
| TheJulia | dtantsur: yeah, it has also become broken a few times silently | 15:47 |
| TheJulia | c'est la vie | 15:48 |
| *** tssurya has quit IRC | 15:48 | |
| dtantsur | yep | 15:48 |
| mgoddard | ah, I see | 15:51 |
| vdrok | dtantsur: heya, yup, ill take a look at the patches to staging today | 15:51 |
| *** ajya has quit IRC | 16:01 | |
| TheJulia | dtantsur: I hvae a question for you on the conductor group patch | 16:01 |
| dtantsur | sure | 16:02 |
| rpioso | What's expected when the BIOS apply_configuration() clean step successfully completes? Should the requested settings been applied to the baremetal server or just recorded and available for application during the next system reset/reboot? | 16:02 |
| dtantsur | rpioso: my gut feeling it should apply the changes and set reboot_required | 16:03 |
| TheJulia | what dtantsur said | 16:03 |
| rpioso | dtantsur, TheJulia: How would an error during the application be captured by the user of the API? | 16:04 |
| TheJulia | I _think_ one of the cases there is a hard need to reboot before exiting... but ideally apply and set reboot required | 16:04 |
| TheJulia | rpioso: CLEAN_ERROR and last_error field | 16:04 |
| TheJulia | I'm going to take a break, bbiab | 16:08 |
| rpioso | dtantsur, TheJulia: I'm not familiar with setting reboot_required. | 16:09 |
| rpioso | Where's that done? | 16:10 |
| dtantsur | rpioso: ah, it's called cleaning_reboot. grep for it, you even have it in iDRAC RAID. | 16:10 |
| *** derekh has quit IRC | 16:14 | |
| jroll | to be clear, reboot_required is a flag in the clean_step decorator | 16:15 |
| jroll | I think | 16:15 |
| rpioso | jroll: I looked there, but didn't see it. More coffee? | 16:16 |
| jroll | maybe that's an IPA thing | 16:16 |
| * rpioso is not as familiar with in-band cleaning | 16:16 | |
| jroll | rpioso: looks like dtantsur is right about cleaning_reboot flag | 16:18 |
| rpioso | dtantsur: I see that in iDRAC RAID. However, I don't understand why it's there. The iDRAC configuration job had a reboot job associated with it. When its _resume_cleaning() is called, the jobs already completed. | 16:18 |
| rpioso | jroll: ^^^ | 16:18 |
| dtantsur | well, it's not a question for me, I guess. I don't know why this RAID support is written this way. | 16:18 |
| jroll | hrm | 16:19 |
| rpioso | dtantsur: I, too. I suppose some more investigation is needed. | 16:19 |
| jroll | rpioso: you mean the raid configuration clean step handles the reboot? | 16:20 |
| rpioso | dtantsur: Anyways, how can the general cleaning mechanism detect an error that occurs when the reboot is performed and provide useful info to the operator about the failure? | 16:20 |
| dtantsur | rpioso: I'm not sure from the top of my head | 16:21 |
| dtantsur | you may have to trigger reboot right from the clean step | 16:21 |
| jroll | yeah, I've no idea where reboot_required went. it looks like now steps trigger the reboot themselves, then set cleaning_reboot to tell the conductor there was a reboot and the agent may not be back yet | 16:22 |
| rpioso | dtantsur: That's what I'm thinking. And not resume cleaning until the reboot is done and the result available to the clean step to report. | 16:22 |
| dtantsur | right | 16:23 |
| rpioso | jroll: Ahhh ... Right! It's to inform the conductor that the agent may not be back, yet. | 16:23 |
| rpioso | jroll: Then it makes sense in iDRAC RAID. | 16:24 |
| jroll | :) | 16:24 |
| dtantsur | aha, that's what it was | 16:25 |
| rpioso | dtantsur, jroll, TheJulia: I'll add my comment to https://review.openstack.org/#/c/582314/ | 16:25 |
| patchbot | patch 582314 - ironic - Add BIOS interface to Redfish hardware type | 16:25 |
| rpioso | And thank you :) | 16:26 |
| dtantsur | np | 16:26 |
| jroll | welcome | 16:26 |
| jroll | I still want to know what happened to reboot_required | 16:26 |
| TheJulia | i think it was always ipa based in returned data | 16:30 |
| TheJulia | but... long long long time ago | 16:31 |
| TheJulia | anyway, break | 16:31 |
| *** gkadam_ has quit IRC | 16:32 | |
| bdodd | rpioso, dtantsur, jroll, TheJulia: Thanks for those comments/discussion. I don't fully understand them all yet, but I recorded them for further study. :-) | 16:34 |
| *** hongbin has quit IRC | 16:37 | |
| rpioso | bdodd: AFAIU, apply_configuration() should reset the server to apply the requested BIOS settings. Seems to me that it should also determine whether or not the application was successful, so it can report the result to the conductor. That would probs require a periodic function to check when the BIOS settings were last updated, if they're consistent with what was requested, etc. to determine whether or not the clean | 16:40 |
| rpioso | step was successful. | 16:40 |
| rpioso | dtantsur: rajinir and I are working on the Dell EMC 3d party testing of the UEFI bug fix backports. | 16:49 |
| etingof | rpioso, bdodd iirc sushy has the `Bios.pending_attributes` property that effectively polls BMC to see if BIOS attrs are applied | 16:50 |
| *** dougsz_ has quit IRC | 16:50 | |
| rpioso | etingof: polls? | 16:50 |
| etingof | rpioso, https://github.com/openstack/sushy/blob/master/sushy/resources/system/bios.py#L74 | 16:51 |
| *** skazi has joined #openstack-ironic | 16:53 | |
| etingof | rpioso, it's all tied up with the Redfish `Settings` object | 16:55 |
| rpioso | etingof: Doesn't that method sample the BMC pending attributes and refresh the cached copy? | 16:56 |
| rpioso | etingof: Check out that method's docstring -- https://github.com/openstack/sushy/blob/master/sushy/resources/system/bios.py#L65-L66 | 16:58 |
| etingof | rpioso, yes, AFAIU, that's the way for the client to figure out when requested BIOS update went through on the BMC side | 16:58 |
| etingof | e.g. that's what the prospective periodic function should probably call and compare with what ironic expects... | 16:59 |
| rpioso | etingof: Exactly! | 16:59 |
| rpioso | At least one of the things. I believe there a timestamp that could be examined. | 17:00 |
| etingof | rpioso, yes, and etag | 17:00 |
| rpioso | etingof: Cool :) | 17:02 |
| *** mgoddard has quit IRC | 17:02 | |
| etingof | rpioso, iirc using etag and/or time is racy e.g. when you have 1+ client tackling BIOS settings more or less at the same time... | 17:03 |
| etingof | rpioso, on the other hand, BMC is not requires to show the whole set of attributes, still it can allow some attributes to be modified | 17:04 |
| etingof | rpioso, in the latter case attributes comparison approach may not work... | 17:04 |
| * etingof feels this part of Redfish is still somewhat fishy | 17:05 | |
| rpioso | etingof: From an ironic conductor perspective, wouldn't multiple clients changing BIOS at the same time be kind of broken? It's risky enough when one client changes BIOS settings :) | 17:06 |
| rpioso | etingof: In other words, practically speaking, aren't those things good enough for our purposes? | 17:07 |
| rpioso | dtantsur: I'm ~100% certain those test failures are false negatives. The backports were soooooo simple. There was a one line conflict at the top of the unit test module. The production module was clean. | 17:09 |
| etingof | rpioso, it is hard to say... | 17:10 |
| etingof | rpioso, for example, could it be racy even with a single client? what if the client times out and retries faster than BMC applies updates... | 17:11 |
| dtantsur | rpioso: is it worth rechecking? | 17:12 |
| rpioso | dtantsur: Not, yet. We're still investigating the root cause. It doesn't appear to be related to the change. | 17:13 |
| dtantsur | TheJulia: wdyt ^^? | 17:14 |
| rpioso | From screen-ir-cond.txt: Error finding the disk or partition device to deploy the image onto: No suitable device was found for deployment - root device hints were not provided and all found block devices are smaller than 4294967296B. | 17:14 |
| dtantsur | mm, does not see disks? curious | 17:14 |
| TheJulia | that flavor is a partition image deploy with what? 40gb requested? | 17:15 |
| TheJulia | that does seem odd... | 17:15 |
| TheJulia | Anyway, rpioso if you guys feel 100% confident it is unrelated, I'm okay with it, but I'd hate for us to have to revert later or find out that something was really broken while you guys are still investigating | 17:16 |
| TheJulia | although... that seems like a fairly distinct error :\ | 17:16 |
| rpioso | dtantsur, TheJulia: And looking at the production code, management.py, its imports and interactions with other, external python code haven't changed. | 17:16 |
| TheJulia | could a task still be running in the bmc from the boot mode being changed? | 17:17 |
| rpioso | TheJulia: The boot mode isn't changed. It's configured to be UEFI for a looooong time. | 17:17 |
| TheJulia | rpioso: ack | 17:17 |
| rpioso | The idrac h/w type doesn't yet support the new management methods for configuring boot mode. | 17:18 |
| rpioso | :( | 17:18 |
| TheJulia | :( | 17:18 |
| openstackgerrit | Merged openstack/ironic stable/rocky: Update .gitreview for stable/rocky https://review.openstack.org/590309 | 17:18 |
| rpioso | What's the deadline +/- for making it into the first release of Rocky? | 17:18 |
| TheJulia | rpioso: we cut 11.1.0 today.... as today is the RC1 deadline. | 17:19 |
| dtantsur | rpioso: we have stable/rocky already, so anything needs backporting. | 17:19 |
| rpioso | Ooops! Please disregard that silly question. | 17:19 |
| * rpioso needs brain food | 17:20 | |
| TheJulia | food++ | 17:20 |
| * TheJulia gets some lunch | 17:21 | |
| rpioso | We need it in a downstream Queens-based distro. The deadline for that is fast approaching. | 17:22 |
| rpioso | The prerequisite for that is the bug fix being available in stable/queens. | 17:23 |
| dtantsur | rpioso: how fast is fast? I think making the CI green is also quite a priority.. | 17:23 |
| rpioso | dtantsur: We're actively working that :) | 17:24 |
| rpioso | I don't know the lead time requirement. | 17:24 |
| *** serlex has quit IRC | 17:24 | |
| rpioso | And the delivery date is an estimate. | 17:24 |
| rpioso | Soon-ish, though. | 17:24 |
| rpioso | dtantsur, TheJulia: We'll dig in more and see how far we get. | 17:27 |
| dtantsur | great | 17:27 |
| TheJulia | rpioso: excellent | 17:27 |
| *** gyee has joined #openstack-ironic | 17:47 | |
| verdurin | TheJulia: I've managed to boot and (sort-of) enrol a node using EFI with Bifrost | 17:57 |
| verdurin | There's something wrong with the ramdisk, so it's stuck in a loop, but Ironic has picked it up | 17:59 |
| *** markguz_ has quit IRC | 18:01 | |
| *** priteau has quit IRC | 18:15 | |
| *** sambetts_ has quit IRC | 18:18 | |
| TheJulia | verdurin: fun :\ | 18:21 |
| TheJulia | verdurin: but also awesome | 18:21 |
| *** dtantsur is now known as dtantsur|afk | 18:24 | |
| dtantsur|afk | o/ | 18:24 |
| *** skazi has quit IRC | 18:26 | |
| *** trown is now known as trown|lunch | 18:26 | |
| TheJulia | goodnight | 18:27 |
| TheJulia | jroll: I put an item on the meeting agenda for in the 21st to discuss ironic-staging-drivers, in the mean time I've -2'ed the patch | 19:06 |
| jroll | TheJulia: thanks | 19:06 |
| TheJulia | no problem :) | 19:08 |
| *** skazi has joined #openstack-ironic | 19:12 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add conductor_group docs https://review.openstack.org/589277 | 19:13 |
| TheJulia | NobodyCam: replied to your questions on the py3 patch | 19:19 |
| NobodyCam | Thank you, I hadn't had coffee when I looked at that :> | 19:20 |
| *** jcoufal_ has joined #openstack-ironic | 19:26 | |
| TheJulia | I've not had any since... and I feel the need for coffee.. | 19:27 |
| *** ptx0 has quit IRC | 19:33 | |
| *** pcaruana has quit IRC | 19:33 | |
| *** jcoufal_ has quit IRC | 19:41 | |
| *** trown|lunch is now known as trown | 19:43 | |
| *** jcoufal_ has joined #openstack-ironic | 19:54 | |
| *** jcoufal_ has quit IRC | 19:55 | |
| *** ptx0_ has joined #openstack-ironic | 20:15 | |
| *** ptx0_ is now known as ptx0 | 20:20 | |
| *** trown is now known as trown|outtypewww | 21:04 | |
| *** jcoufal has quit IRC | 21:17 | |
| *** baha has quit IRC | 21:19 | |
| *** bfournie has quit IRC | 21:26 | |
| *** fultonj has quit IRC | 21:51 | |
| *** rcernin has joined #openstack-ironic | 22:09 | |
| *** jaganathan has quit IRC | 22:35 | |
| *** skazi has quit IRC | 22:44 | |
| *** gyee has quit IRC | 23:06 | |
| *** bfournie has joined #openstack-ironic | 23:13 | |
| *** dlbewley has quit IRC | 23:19 | |
| *** rpioso is now known as rpioso|afk | 23:24 | |
| *** dlbewley has joined #openstack-ironic | 23:45 | |
| *** dlbewley has quit IRC | 23:53 | |
| *** slagle has joined #openstack-ironic | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!