| hjensas | sean-k-mooney: are you using rhel 8.2, or did some other distro also pull in the changes from 2.81 ? | 00:02 |
|---|---|---|
| hjensas | sean-k-mooney: and you have a dual-stack network with ipv4 and ipv6? | 00:03 |
| sean-k-mooney | hjensas: it happens in dnsmasq-2.79-11.el8 which is available in Centos 8 | 00:07 |
| sean-k-mooney | hjensas: so i was using the centos-8 images from docker hub for kolla | 00:07 |
| sean-k-mooney | it has 2.79 | 00:07 |
| hjensas | sean-k-mooney: oh, ok I had missed that CentOS caught up already. | 00:08 |
| sean-k-mooney | its currently only a single ip stack | 00:08 |
| sean-k-mooney | but im guessing we have issue with how neutron specifics the extra pxe otptions too | 00:08 |
| *** rloo has quit IRC | 00:11 | |
| hjensas | sean-k-mooney: no aware of any issues with the options. If you do a port show on the port used for provisioning, fixed_ips only have one address? | 00:12 |
| sean-k-mooney | http://paste.openstack.org/show/793148/ | 00:13 |
| sean-k-mooney | so yes | 00:13 |
| sean-k-mooney | its after 1 here and i dont want to wake peopel by booting my server but i can try and reproduce it tomorrow and capture more info | 00:15 |
| sean-k-mooney | hjensas: im not 100% certin this is the issue but im highly suspicios based on how its failing | 00:15 |
| hjensas | sean-k-mooney: the bug is related to neutron writing two dhcp-host records in the config when both ipv4 and ipv6 is used, dual-stack, dnsmasq iterates over the list of host records and uses the first match. | 00:16 |
| sean-k-mooney | hjensas: yep i know but it would appear that neutron is either incorrectly writing the info in this case too or there is a related failure | 00:17 |
| sean-k-mooney | http://paste.openstack.org/show/793150/ | 00:17 |
| sean-k-mooney | if i look at the neutron dhcp agaent log it repatly say that it completed configuring dhcp for the interface | 00:19 |
| sean-k-mooney | but it still does not work | 00:19 |
| sean-k-mooney | anyway im going to call it a night o/ | 00:19 |
| hjensas | sean-k-mooney: ok, feel free to ping me again tomorrow. | 00:19 |
| sean-k-mooney | sure ill let you know what i find if anything http://paste.openstack.org/show/793152/ is the dhcp agent log for the interval when it was trying to boot too | 00:21 |
| *** Qianbiao has joined #openstack-ironic | 01:37 | |
| *** xinliang has joined #openstack-ironic | 01:39 | |
| *** ociuhandu has joined #openstack-ironic | 01:57 | |
| *** ociuhandu has quit IRC | 02:02 | |
| *** xinliang has quit IRC | 02:03 | |
| *** tzumainn has quit IRC | 02:22 | |
| *** mkrai has joined #openstack-ironic | 02:50 | |
| *** rh-jelabarre has quit IRC | 04:02 | |
| *** openstack has joined #openstack-ironic | 04:22 | |
| *** ChanServ sets mode: +o openstack | 04:22 | |
| *** uzumaki has quit IRC | 04:29 | |
| *** gyee has quit IRC | 04:40 | |
| *** jdandrea has quit IRC | 04:48 | |
| *** cp- has quit IRC | 04:57 | |
| *** ysandeep|away is now known as ysandeep | 05:51 | |
| *** ociuhandu has joined #openstack-ironic | 06:01 | |
| *** ociuhandu has quit IRC | 06:09 | |
| *** ysandeep is now known as ysandeep|away | 06:12 | |
| *** ysandeep|away has quit IRC | 06:29 | |
| *** ociuhandu has joined #openstack-ironic | 06:39 | |
| arne_wiebalck | Good morning, ironic! | 06:51 |
| *** ociuhandu has quit IRC | 07:00 | |
| *** yolanda has joined #openstack-ironic | 07:14 | |
| iurygregory | good morning arne_wiebalck and Ironic o/ | 07:19 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Stop aborting the process on periodic task failures https://review.opendev.org/725688 | 07:24 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Stop running a periodic task for the noop PXE filter https://review.opendev.org/725770 | 07:27 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Stop running a periodic task for the noop PXE filter https://review.opendev.org/725770 | 07:31 |
| *** rpittau|afk is now known as rpittau | 07:31 | |
| rpittau | good morning ironic! o/ | 07:31 |
| arne_wiebalck | gm iurygregory and rpittau o/ | 07:31 |
| rpittau | hey arne_wiebalck :) | 07:32 |
| iurygregory | rpittau, o/ | 07:33 |
| rpittau | hey iurygregory :) | 07:35 |
| *** gokhani has joined #openstack-ironic | 07:35 | |
| iurygregory | rpittau, wdyt of disabling smoke tests in the post-upgrade? for ironic-inspector-grenade | 07:39 |
| *** dtantsur|afk is now known as dtantsur | 07:40 | |
| dtantsur | morning ironic | 07:40 |
| iurygregory | morning dtantsur | 07:40 |
| *** xinliang has joined #openstack-ironic | 07:44 | |
| dtantsur | folks, can I get some reviews on https://review.opendev.org/725530 and https://review.opendev.org/725239 please? | 07:44 |
| patchbot | patch 725530 - ironic - redfish: split reboot into power off followed by p... - 1 patch set | 07:44 |
| patchbot | patch 725239 - ironic - redfish: handle hardware that is unable to set per... - 6 patch sets | 07:44 |
| dtantsur | the latter fixes an unpleasant issue that Julia highlighted on the ML | 07:44 |
| rpittau | iurygregory: mmm I would worry a bit of missing something :) is this to reduce the timeout values and general running time ? | 07:46 |
| *** openstackstatus has quit IRC | 07:46 | |
| *** openstack has joined #openstack-ironic | 07:49 | |
| *** ChanServ sets mode: +o openstack | 07:49 | |
| rpittau | in that case we can remove them from any ironic-inspector job, I guess ? | 07:49 |
| iurygregory | we only run smoke in grenade I think | 07:49 |
| iurygregory | =) | 07:49 |
| dtantsur | yes | 07:50 |
| iurygregory | so to make grenade working again should I remove from pre-upgrade and pos-upgrade or just pos-upgrade? | 07:53 |
| iurygregory | I have the patch to switch to run our tests but it seems I'm missing some configs there to have it working | 07:53 |
| dtantsur | I'd remove both probably | 07:53 |
| dtantsur | getting to a stable CI is the topmost priority now, getting to it without disabling grenade completely is the 2nd top | 07:54 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Add ipa-builder to grenade and disable smoke tests https://review.opendev.org/725514 | 07:55 |
| dtantsur | iurygregory: don't we need *some* tempest_test_regex? | 07:57 |
| dtantsur | also let's capture the reasoning in the commit message | 07:57 |
| *** dmellado has quit IRC | 07:58 | |
| iurygregory | dtantsur, I'm re adding the tests in https://review.opendev.org/#/c/723646/ | 07:59 |
| patchbot | patch 723646 - ironic-inspector - Update grenade job - 8 patch sets | 07:59 |
| iurygregory | I'm checking what configs are missing since there are some errors https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_b3d/723646/8/check/ironic-inspector-grenade/b3d8d7d/controller/logs/screen-ironic-inspector.txt | 08:01 |
| iurygregory | still looking to see what it can be | 08:01 |
| gokhani | hi dtantsur , I continue to deal with ironic and solve my problems. when I look at neutron dhcp logs, it seems it gets ip but on tftp side ı can't see any traffic. my ironic node gets ip 10.1.45.100 and my tftp server is on different ip like 10.1.20.3:69 . when I manually test tftp from 10.1.48.x to 10.1.20 it connect tftp but gives timeout error | 08:01 |
| gokhani | when getting files. I don't know this is expected behavior or not ? | 08:01 |
| dtantsur | gokhani: it means that TFTP is not reachable from the subnet. you need to make sure your nodes can reach your tftp server. | 08:02 |
| iurygregory | oh yeah it will probably fail since I have tox_envlist and no regex to run | 08:03 |
| dtantsur | iurygregory: yeah, I think you need to have some tests from the start | 08:03 |
| iurygregory | yeah | 08:04 |
| *** dmellado has joined #openstack-ironic | 08:04 | |
| *** gokhani43 has joined #openstack-ironic | 08:04 | |
| gokhani43 | dtantsur, ok thanks now time to solve tftp issues. | 08:06 |
| *** gokhani has quit IRC | 08:08 | |
| *** lucasagomes has joined #openstack-ironic | 08:12 | |
| *** mkrai has quit IRC | 08:16 | |
| *** alexmcleod has joined #openstack-ironic | 08:19 | |
| *** ociuhandu has quit IRC | 08:20 | |
| iurygregory | dtantsur, standalone for ironic-inspector-grenade should be True? I have the feeling it shouldn't | 08:23 |
| iurygregory | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_b3d/723646/8/check/ironic-inspector-grenade/b3d8d7d/controller/logs/etc/ironic-inspector/inspector_conf.txt | 08:23 |
| dtantsur | iurygregory: I think most jobs run in standalone mode now | 08:23 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: redfish: split reboot into power off followed by power on https://review.opendev.org/725530 | 08:24 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: redfish: handle hardware that is unable to set persistent boot https://review.opendev.org/725239 | 08:24 |
| dtantsur | rpittau: updated ^^ | 08:24 |
| iurygregory | the meaning for standalone in the config is that we are running in another node no? The standalone in the DEFAULT section - This determines whether ironic-inspector services are intended to be deployed separately. | 08:25 |
| iurygregory | but everything is in the same node lol | 08:25 |
| dtantsur | iurygregory: standalone only means if the services are split or not | 08:26 |
| iurygregory | yeah | 08:26 |
| iurygregory | True means that they are, but we are installing in the same machine | 08:27 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Silence debug logging from oslo_messaging._drivers https://review.opendev.org/725786 | 08:30 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Silence debug logging from oslo_messaging https://review.opendev.org/725786 | 08:31 |
| dtantsur | iurygregory: the other way around, standalone means we do NOT split | 08:31 |
| iurygregory | dtantsur, ok gotcha | 08:31 |
| *** gokhani43 has quit IRC | 08:33 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Silence debug messages from oslo_messaging https://review.opendev.org/725789 | 08:34 |
| *** ociuhandu has joined #openstack-ironic | 08:34 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [WIP] Fix grub2 pxe job with native bionic ovmf https://review.opendev.org/716889 | 08:35 |
| dtantsur | mgoddard: we didn't release IPA-builder, did we? | 08:38 |
| dtantsur | could you add it to your queue? (I can do it if you're out of time) | 08:39 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: [WIP] Use ubuntu focal for ipa src https://review.opendev.org/725794 | 08:44 |
| gudrutis2 | Hello ironic 0/ | 08:45 |
| arne_wiebalck | dtantsur: in order to share the release work load (which may even increase soon), do we already have a documented procedure of the general workflow and all the required steps? | 08:45 |
| dtantsur | arne_wiebalck: kind of: https://docs.openstack.org/ironic/latest/contributor/releasing.html | 08:45 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] Build tinyipa on focal https://review.opendev.org/725799 | 08:49 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: [WIP] Use ubuntu focal for ipa src https://review.opendev.org/725794 | 08:49 |
| *** mkrai has joined #openstack-ironic | 08:50 | |
| arne_wiebalck | dtantsur: Thanks! For additional volunteers it might be an idea to have some "pair-releasing" sessions to spread knowledge and experience how to go through this process. I think mgoddard picked that up since 2 or 3 releases, but there may be others which are willing to help and this may come in handy if we move to a different release cadence. | 08:51 |
| arne_wiebalck | dtantsur: unless you think there are enough hands already :) | 08:51 |
| dtantsur | if/when we implement the new release model, we'll likely need more hands indeed | 09:00 |
| dtantsur | ++ to a session, but you need to ask mgoddard now :) | 09:00 |
| openstackgerrit | yuanliu proposed openstack/ironic master: If the [conductor]XXX_timeout is less than 0，disable periodic task https://review.opendev.org/723795 | 09:00 |
| *** strigazi has joined #openstack-ironic | 09:00 | |
| *** gokhani has joined #openstack-ironic | 09:01 | |
| mgoddard | dtantsur: busy today, can probably get to it tomorrow | 09:01 |
| rpittau | dtantsur, arne_wiebalck, I'm quite interested also in helping on the releases :) | 09:01 |
| mgoddard | happy to talk through releasing with anyone who wants to know | 09:02 |
| dtantsur | mgoddard: nvm, I'll guide one of the volunteers today | 09:02 |
| mgoddard | k, thanks | 09:02 |
| dtantsur | okay, so rpittau or arne_wiebalck, who's up today? :) | 09:02 |
| rpittau | :D | 09:02 |
| rpittau | I can "make" some time now | 09:03 |
| arne_wiebalck | go ahead rpittau, I take the next tour ;) | 09:04 |
| dtantsur | okay, I'll do the tour right here anyway :) | 09:04 |
| dtantsur | rpittau: step #1: check what applies from https://docs.openstack.org/ironic/latest/contributor/releasing.html#things-to-do-before-releasing | 09:05 |
| dtantsur | nothing applies to IPA-builder right now, but this document will grow more things over time | 09:05 |
| dtantsur | if the project you're releasing has its own releasing document (IPA-builder doesn't), check it as well | 09:05 |
| arne_wiebalck | dtantsur: where would one find that? | 09:05 |
| dtantsur | arne_wiebalck: in the contributor guide of a project. I think only ironic currently has a comprehensive releasing guide though. | 09:06 |
| arne_wiebalck | +1 | 09:06 |
| rpittau | arne_wiebalck: https://docs.openstack.org/ironic/latest/contributor/releasing.html | 09:06 |
| rpittau | that's specific for ironic | 09:06 |
| dtantsur | rpittau: step #2: check for open patches that are close to merging OR are release critical | 09:07 |
| dtantsur | from step #1 the only thing that applies to IPA-builder is checking that the release notes are in a sane state: https://docs.openstack.org/releasenotes/ironic-python-agent-builder/ | 09:08 |
| dtantsur | especially watch out of rendering issues or completely unreadable wording | 09:08 |
| * dtantsur waits for rpittau to confirm finishing steps 1 and 2 | 09:08 | |
| rpittau | ok, I don't see any patches we need to wait for | 09:10 |
| dtantsur | release notes looking good? | 09:11 |
| rpittau | yeah | 09:11 |
| dtantsur | rpittau: okay. now clone the openstack/releases repo. This is where deliverables are tracked and all the automation resides. | 09:12 |
| rpittau | done :P | 09:12 |
| dtantsur | arne_wiebalck: you can follow along, just don't submit your patch :) | 09:13 |
| arne_wiebalck | +1 | 09:13 |
| dtantsur | rpittau, arne_wiebalck, in the 'deliverables' directory you can see yaml files for each deliverable (i.e. subproject) grouped by release cycles. | 09:13 |
| *** mkarpiarz has joined #openstack-ironic | 09:14 | |
| rpittau | ok | 09:14 |
| dtantsur | the _independent directory contains deliverables that are not bound to (official) cycles, IPA-builder is one of them | 09:14 |
| dtantsur | (you can find its file to get familiar with the format) | 09:14 |
| rpittau | I have that open | 09:14 |
| dtantsur | feel free to close it, there are scripts to update these files :-P | 09:15 |
| rpittau | ok :D | 09:15 |
| rpittau | I was getting worried about that hash | 09:15 |
| dtantsur | now, use this script to check the changes you're about to release: ./tools/list_unreleased_changes.sh master openstack/ironic-python-agent-builder | 09:15 |
| dtantsur | the first argument is a branch, not a release series (i.e. master or stable/train, not ussuri or train) | 09:15 |
| dtantsur | the second is obvious :) | 09:15 |
| dtantsur | from this listing and the release note you need to decide on whether the next version will be major, minor or patch | 09:16 |
| rpittau | interesting | 09:16 |
| rpittau | looks like major to me :) | 09:17 |
| dtantsur | it is, because of python 3 | 09:17 |
| dtantsur | now, get familiar with the script to update deliverable files: $ tox -e venv -- new-release -h | 09:17 |
| dtantsur | it's pretty obvious, just note that "series" is a code name (train, ussuri), not a branch | 09:18 |
| dtantsur | type "bugfix" corresponds to a patch release, "feature" - to a minor one | 09:18 |
| rpittau | ok | 09:18 |
| dtantsur | the --stable-branch argument is used for branching in the end of a cycle. independent projects are not branched this way though. | 09:19 |
| dtantsur | rpittau: so, step #4: use this script to update the deliverable file, commit the change and propose for review | 09:19 |
| rpittau | ok, makes sense | 09:19 |
| dtantsur | since you're not a liaison, one of us (Julia, Mark or myself) will need to +1 it | 09:20 |
| rpittau | newbie-proofed :) | 09:20 |
| dtantsur | if you want to double-check the changes before 'git review', use $ tox -e list-changes | 09:20 |
| dtantsur | `tox -e validate` does some sanity-checks, but if you use the scripts, you should be fine | 09:20 |
| arne_wiebalck | rpittau: post the new-release command you ran in the end please | 09:22 |
| * arne_wiebalck is taking notes | 09:22 | |
| rpittau | dtantsur: do we have any particular naming convention for the topic ? | 09:23 |
| rpittau | arne_wiebalck: sure :) | 09:23 |
| dtantsur | rpittau: not really, just mention the deliverable name, the new version and the branch (not in this case) | 09:23 |
| rpittau | ok | 09:23 |
| dtantsur | "Release ironic 1.2.3 for Train" is what I do | 09:23 |
| rpittau | wow the list-changes command is long | 09:24 |
| dtantsur | it is :) | 09:24 |
| dtantsur | it's designed for the release team to review the requests, so it contains all potentially useful information | 09:24 |
| dtantsur | btw the IRC channel is #openstack-release, all release liaisons should be present there | 09:25 |
| rpittau | I see | 09:25 |
| rpittau | oh yeah, I'm there already | 09:25 |
| * rpittau waiting for validate to end.... | 09:26 | |
| dtantsur | after the release is approved by the release team, it's handled automatically by zuul | 09:27 |
| dtantsur | once it's done, check https://docs.openstack.org/ironic/latest/contributor/releasing.html#things-to-do-after-releasing for any actions | 09:27 |
| rpittau | arne_wiebalck, this is the command I used: tox -e venv -- new-release _independent ironic-python-agent-builder major -v | 09:28 |
| arne_wiebalck | yep | 09:28 |
| arne_wiebalck | thanks | 09:28 |
| *** k_mouza has joined #openstack-ironic | 09:29 | |
| arne_wiebalck | dtantsur: the validate usually succeeds? IOW, what to do if that fails? | 09:30 |
| rpittau | aaaand sent | 09:30 |
| *** gokhani has quit IRC | 09:32 | |
| * arne_wiebalck is still validating ... | 09:32 | |
| rpittau | yeah, that took a while :) | 09:32 |
| dtantsur | arne_wiebalck: I guess it will tell you what is wrong? if unclear, ask #openstack-release | 09:32 |
| arne_wiebalck | +1 | 09:33 |
| *** gokhani has joined #openstack-ironic | 09:33 | |
| dtantsur | thanks rpittau! now I've +1ed it, and it will receive PTL-Approved+1 from Zuul | 09:33 |
| rpittau | \o/ | 09:33 |
| dtantsur | then we just wait (ping #openstack-release if in hurry) | 09:33 |
| dtantsur | the crash course is over, thank you for coming :) | 09:35 |
| rpittau | thanks :) | 09:35 |
| arne_wiebalck | thanks, dtantsur ! | 09:35 |
| * arne_wiebalck is still validating ... | 09:36 | |
| dtantsur | bus_factor += 2 | 09:36 |
| dtantsur | FYI: you can request several releases in one patch. just run new-release several times and submit the result. | 09:37 |
| rpittau | so you can specify the point of release ? | 09:37 |
| dtantsur | rpittau: you mean, the specific commit? I don't think so, you need to update the file manually in this case. | 09:39 |
| rpittau | yes, ok | 09:39 |
| dtantsur | in most cases you're expected to release the tip of a branch | 09:39 |
| arne_wiebalck | "validate: commands succeeded" | 09:40 |
| rpittau | that's a good sign :D | 09:40 |
| dtantsur | iurygregory: where are we with https://review.opendev.org/#/c/725514/ ? | 09:42 |
| patchbot | patch 725514 - ironic-inspector - Add ipa-builder to grenade and disable smoke tests - 2 patch sets | 09:42 |
| * rpittau goes foraging, ttyl | 09:44 | |
| *** rpittau is now known as rpittau|bbl | 09:44 | |
| arne_wiebalck | dtantsur: should we add a section with instructions between "Things to do before releasing" and "Things to do after releasing"? | 09:45 |
| iurygregory | dtantsur, rpittau|bbl gave the idea to split, but I think this isn't working =) (the idea was have one adding ipa-builder and the other with timeouts / other tests) | 09:45 |
| arne_wiebalck | dtantsur: Sth like "Releasing" :) | 09:45 |
| dtantsur | arne_wiebalck: I'd expect it to be covered somewhere in the release team docs.. | 09:45 |
| arne_wiebalck | dtantsur: oh, ok | 09:45 |
| dtantsur | iurygregory: let's try adding a regex and updating the commit message (see my comments) | 09:45 |
| arne_wiebalck | dtantsur: better not duplicate then | 09:45 |
| iurygregory | dtantsur, acl | 09:46 |
| iurygregory | ack* | 09:46 |
| *** kbaegis has quit IRC | 09:47 | |
| dtantsur | does anyone know if there's been any progress with stable/queens? | 09:48 |
| rpittau|bbl | The idea of splitting was proposed to focus on fixing the grenade job before any other change, not just to add ipa-builder | 09:48 |
| rpittau|bbl | dtantsur: I was looking at it, there is a patch with my tests | 09:49 |
| iurygregory | oh XD | 09:49 |
| dtantsur | rpittau|bbl: link handy? | 09:49 |
| rpittau|bbl | dtantsur: https://review.opendev.org/#/c/724277/ | 09:51 |
| patchbot | patch 724277 - ironic (stable/queens) - [DNM] test CI - 5 patch sets | 09:51 |
| rpittau|bbl | iurygregory: I probably didn't express myself very well about that :/ | 09:51 |
| iurygregory | rpittau|bbl, no worries =) | 09:51 |
| dtantsur | I'll poke at it while you're bbl | 09:52 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Fix grenade job https://review.opendev.org/725514 | 09:53 |
| gokhani | dtantsur , I realized that if tftp server and tftp client are on different subnet, tftp is not working and on client side it gets timeout. In my environment tftp server and ironic services are on openstack management network and the node which ı try to boot with ironic is getting ip from different subnet which is flat provider network. do I have | 09:59 |
| gokhani | to run tftp server from flat provider network subnet or ı don't know how ironic works :( | 09:59 |
| dtantsur | gokhani: I think you need to run the TFTP server on the provide network OR make sure UDP is routable from it to the management network | 10:00 |
| *** Qianbiao has quit IRC | 10:00 | |
| dtantsur | (this problem is not ironic specific; in fact, ironic doesn't manage the TFTP server) | 10:00 |
| *** Qianbiao has joined #openstack-ironic | 10:01 | |
| gokhani | dtantsur, ok I see. I installed ironic with openstack ansible project and unfortunately there are missing things :( firstly need to solve tftp issue. thanks again :) | 10:08 |
| *** mkrai has quit IRC | 10:09 | |
| *** mkrai has joined #openstack-ironic | 10:10 | |
| dtantsur | unfortunately, I'm not familiar with how openstack-ansible installs ironic, so I cannot say if something is missed there or just an environment/documentation problem | 10:10 |
| *** kbaegis has joined #openstack-ironic | 10:10 | |
| *** xinliang has quit IRC | 10:24 | |
| dtantsur | arne_wiebalck: do you have a minute for two trivial CI patches https://review.opendev.org/#/c/724320/ and https://review.opendev.org/#/c/724323/ | 10:24 |
| patchbot | patch 724320 - ironic - CI: repair the SNMP job and make it voting. - 1 patch set | 10:24 |
| patchbot | patch 724323 - ironic - CI: exclude the inspection abort tests from the in... - 2 patch sets | 10:24 |
| openstackgerrit | Mahendra Kamble proposed openstack/ironic master: DRAC : idrac-redfish inspect updates pxe port https://review.opendev.org/693328 | 10:26 |
| arne_wiebalck | dtantsur: sure | 10:39 |
| *** derekh has joined #openstack-ironic | 10:46 | |
| *** livelace has joined #openstack-ironic | 11:01 | |
| iurygregory | dtantsur, {0} ironic_tempest_plugin.tests.scenario.test_introspection_basic.InspectorBasicTest.test_baremetal_introspection [690.335735s] ... ok =) | 11:02 |
| iurygregory | green \o/ | 11:03 |
| dtantsur | \o/ | 11:03 |
| *** priteau has joined #openstack-ironic | 11:03 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-specs master: Proposal for a new release model https://review.opendev.org/725547 | 11:04 |
| dtantsur | TheJulia and everyone: finished typing words ^^ | 11:04 |
| iurygregory | dtantsur, test_baremetal_introspection will be enough for us or I should keep trying to run grenade with Inspector regex in https://review.opendev.org/#/c/725514 ? wdyt? | 11:08 |
| patchbot | patch 725514 - ironic-inspector - Fix grenade job - 3 patch sets | 11:08 |
| iurygregory | https://review.opendev.org/#/c/723646 ops | 11:08 |
| patchbot | patch 723646 - ironic-inspector - Update grenade job - 8 patch sets | 11:08 |
| iurygregory | in this one ^ | 11:08 |
| *** vkmc has quit IRC | 11:09 | |
| *** uzumaki has joined #openstack-ironic | 11:11 | |
| dtantsur | iurygregory: it's enough IMO | 11:12 |
| iurygregory | cool o/ | 11:13 |
| *** ociuhandu has quit IRC | 11:13 | |
| uzumaki | good morning ironic! o/ | 11:13 |
| dtantsur | morning uzumaki | 11:13 |
| iurygregory | morning uzumaki o/ | 11:13 |
| uzumaki | how's it going? dtantsur iurygregory | 11:13 |
| dtantsur | a very slow day | 11:13 |
| uzumaki | oh, any particular reason? | 11:14 |
| *** ociuhandu has joined #openstack-ironic | 11:14 | |
| dtantsur | nothing particular, although I did wake up somewhat earlier than I'd like to | 11:14 |
| iurygregory | uzumaki, small problems =) | 11:14 |
| uzumaki | oh, yes, that always slows things dow for me as well | 11:14 |
| iurygregory | but doing fine | 11:14 |
| uzumaki | iurygregory, I'm sure you're much bigger than those small problems xD | 11:14 |
| iurygregory | not when the problem is your hand XD | 11:15 |
| uzumaki | lol xD | 11:15 |
| uzumaki | What'd you do to your hand? Writing too much code? Carpel Tunnel? | 11:15 |
| iurygregory | I cut myself yesterday in the hand while preparing the dinner XD | 11:15 |
| uzumaki | iurygregory, gosh, that's nasty! | 11:16 |
| iurygregory | yeah XD | 11:16 |
| uzumaki | I like how you're taking it all very bravely though ;) | 11:16 |
| iurygregory | hehehe | 11:17 |
| dtantsur | these vegetables can be quite fierce in a battle | 11:17 |
| iurygregory | I was cutting meat XD | 11:18 |
| uzumaki | dtantsur, indeed. Who knows, what kinda heroic war awaits iurygregory ? These skills might come in handy | 11:18 |
| * iurygregory note: buy meat in slices... | 11:18 | |
| dtantsur | :D | 11:18 |
| dtantsur | well, meat tends to resist when you're trying to eat it | 11:19 |
| iurygregory | hehehe | 11:19 |
| uzumaki | a feat of nature.. | 11:19 |
| iurygregory | living ones normally... | 11:19 |
| *** ociuhandu has quit IRC | 11:19 | |
| *** mkrai has quit IRC | 11:30 | |
| *** kbaegis has quit IRC | 11:33 | |
| *** kbaegis has joined #openstack-ironic | 11:34 | |
| *** kashifkhan has joined #openstack-ironic | 11:34 | |
| *** ociuhandu has joined #openstack-ironic | 11:36 | |
| iurygregory | we can get rid off https://opendev.org/openstack/ironic/src/branch/master/ironic/conf/default.py#L155-L163 https://opendev.org/openstack/ironic/src/branch/master/ironic/conf/cinder.py#L20-L27 | 11:36 |
| iurygregory | right? | 11:36 |
| kashifkhan | Hi everyone, I have a question, when I go inside the ironic container and print the version of ironic-conduction I get 13.1.0.dev127, but I do not see any release version 13.1.0 in ironic github, how can I know which version of release this corresponds to, I need to know the commit/release hash | 11:37 |
| kashifkhan | *ironic-conductor | 11:37 |
| *** ociuhandu has quit IRC | 11:41 | |
| iurygregory | I think 13 is Train | 11:42 |
| *** kbaegis has quit IRC | 11:51 | |
| *** rh-jelabarre has joined #openstack-ironic | 12:02 | |
| *** rh-jelabarre has quit IRC | 12:02 | |
| *** rh-jelabarre has joined #openstack-ironic | 12:03 | |
| *** CoDeAmRo has joined #openstack-ironic | 12:05 | |
| *** livelace has quit IRC | 12:13 | |
| *** rpittau|bbl is now known as rpittau | 12:18 | |
| rpittau | hi kashifkhan, welcome! That looks like a dev release between train and ussuri, where does the container come from ? | 12:19 |
| *** CoDeAmRo has quit IRC | 12:22 | |
| *** CoDeAmRo has joined #openstack-ironic | 12:23 | |
| kashifkhan | hi rpittau, thanks for the reply. We were using this in metal3 until yesterday | 12:24 |
| *** ociuhandu has joined #openstack-ironic | 12:25 | |
| kashifkhan | i think it was coming from master current | 12:26 |
| rpittau | kashifkhan: that's interesting, I know those containers very well :) that's not coming from master though, that's because the packages were coming from an outdated source | 12:27 |
| rpittau | kashifkhan: the packages come from the tripleo repos, that was not up-to-date with centos8 packages until recently | 12:29 |
| kashifkhan | we actually changed it and updated it yesterday but things seem to be broken for us, so we are reverting back to the one that worked and the version there was 13.1.0.dev127 | 12:30 |
| openstackgerrit | Merged openstack/ironic master: CI: repair the SNMP job and make it voting. https://review.opendev.org/724320 | 12:30 |
| rpittau | I see the current version is 15.1.0 though, so that's what should be installed now in recent containers | 12:31 |
| kashifkhan | yes that we can see now, but it is not working for us, nodes always stay stuck in inspecting | 12:31 |
| rpittau | kashifkhan: that's a bug that was reported and is being worked on | 12:32 |
| Qianbiao | kashifkhan: what image are u using? i remember metal3 has a official image in quay.io which updated everyday? | 12:33 |
| kashifkhan | yeah that I know, so the answer of my initial question would be there is no such commit upstream in github which would correspond to 13.1.0.dev127 ??? | 12:33 |
| kashifkhan | yes that image from quay | 12:33 |
| jroll | .dev127 means that it is 127 commits after 13.1.0, but not tagged since 13.1.0 | 12:33 |
| rpittau | kashifkhan: my suggestion is to use old containers if you can't work around the issue, although you could using the workaround that was proposed by maelk, waiting for ironic to be up before starting inspector | 12:34 |
| Qianbiao | why not just start a new container. | 12:34 |
| *** tkajinam has joined #openstack-ironic | 12:34 | |
| kashifkhan | even with mael's workaround we cannot get rid of the problem | 12:35 |
| kashifkhan | we need to know the commit/release version since in another setup we are trying to build it from the source | 12:36 |
| openstackgerrit | Merged openstack/ironic master: CI: exclude the inspection abort tests from the inspector-redfish-vmedia job https://review.opendev.org/724323 | 12:37 |
| kashifkhan | @jroll, is there a 13.1.0 release version upstream? | 12:37 |
| rpittau | kashifkhan: you might want to use train then | 12:38 |
| Qianbiao | <kashifkhan> dev127 does not refer to a obviously commit | 12:38 |
| Qianbiao | it's 127th commit in 13.1.0 branch | 12:38 |
| *** Goneri has joined #openstack-ironic | 12:38 | |
| kashifkhan | <rpittau>, I am a bit ignorant in what do you mean by train. Would you please elaborate or a link where can I find it? | 12:39 |
| rpittau | kashifkhan: the stable/train branch | 12:39 |
| rpittau | kashifkhan: please consider that the issue is *not* on ironic side though | 12:40 |
| rpittau | kashifkhan: the bug seems to be on ironic-inspector | 12:41 |
| jroll | kashifkhan: oh, you're right, I don't see one. weird | 12:42 |
| kashifkhan | <rpittau> Thanks, I am looking into the branch now. Lets see if I can find it. Regarding the issue, I understand the issue is is inspector. doesn't ironic and ironic-inspector use the same image? | 12:42 |
| rpittau | kashifkhan: no, they use separate images | 12:42 |
| jroll | might be 127th commit after 13.0.3, with version tooling magic that knows the next release will be 13.1.0 | 12:43 |
| rpittau | jroll: that is probably it, got me confused as well :) | 12:43 |
| rpittau | kashifkhan: I think you might be good with 14.0.0 | 12:43 |
| kashifkhan | ok, thanks all of you, we will try with 14.0.0 then | 12:44 |
| rpittau | kashifkhan: but I suggest to pair that with ironic-inspector stable/train or up to 10.0.0 | 12:44 |
| kashifkhan | rpittau: ok thanks | 12:45 |
| kashifkhan | rpittau: do you mean to say the but is there after 10.0.0 version in the inspector?? | 12:49 |
| kashifkhan | *bug | 12:49 |
| *** vkmc__ has joined #openstack-ironic | 12:50 | |
| rpittau | kashifkhan: correct | 12:52 |
| *** vkmc__ is now known as vkmc | 12:52 | |
| *** rloo has joined #openstack-ironic | 12:56 | |
| dtantsur | TheJulia, arne_wiebalck, rpittau, JFYI Jacob has added a SuperCloud section to the whitepaper | 12:59 |
| TheJulia | good morning | 13:00 |
| TheJulia | awesome dtantsur | 13:00 |
| iurygregory | wow SuperCloud sounds cool | 13:00 |
| iurygregory | good morning TheJulia | 13:00 |
| dtantsur | morning TheJulia | 13:00 |
| *** cp- has joined #openstack-ironic | 13:01 | |
| TheJulia | iurygregory: re deprecation removal, seems reasonable | 13:01 |
| iurygregory | TheJulia, cool =) | 13:01 |
| iurygregory | tks! | 13:02 |
| TheJulia | rpittau: 15.1.0 wut? | 13:02 |
| rpittau | TheJulia: the current rdo package version :) | 13:03 |
| iurygregory | O.o | 13:03 |
| TheJulia | wtf | 13:03 |
| dtantsur | 15.1.0 goes after 15.0.0, all correct | 13:03 |
| TheJulia | yeah, but we've not dropped a semver tag have we? | 13:03 |
| dtantsur | that's pbr behavior, it's been always like that | 13:03 |
| dtantsur | TheJulia: it's done automatically in the auto-generated commit | 13:03 |
| rpittau | that's how they build them | 13:03 |
| TheJulia | so pbr should be signaling 15.1 yet | 13:03 |
| rpittau | TheJulia: we also have 13.1.0 and it doesn't exist | 13:04 |
| dtantsur | TheJulia, rpittau, https://opendev.org/openstack/ironic/commit/2afbd77c12d90925f24555fa4e36e28b2bc6e6a6 | 13:04 |
| dtantsur | note SemVer:feature | 13:04 |
| TheJulia | yup | 13:04 |
| rpittau | yeah | 13:04 |
| TheJulia | that would do it then | 13:04 |
| dtantsur | it's done so that the version on master do not end up smaller than on a stable branch | 13:04 |
| dtantsur | it's caused problems for TripleO people in the past | 13:04 |
| dtantsur | (yum/dnf won't upgrade packages to a smaller version) | 13:05 |
| TheJulia | what exactly is this metal3 problem people are having, I'm seemingly missing some sort of context | 13:05 |
| rpittau | TheJulia: the current version of inspector complains if it starts before ironic is up and smiling | 13:06 |
| dtantsur | I have a fix for that | 13:06 |
| TheJulia | That seems like something fixable | 13:06 |
| TheJulia | \o/ | 13:06 |
| dtantsur | https://review.opendev.org/725688 | 13:06 |
| patchbot | patch 725688 - ironic-inspector - Stop aborting the process on periodic task failures - 2 patch sets | 13:06 |
| dtantsur | waits for grenade to be fixed first | 13:06 |
| TheJulia | how bad is the grenade break? | 13:07 |
| iurygregory | it's on gate the fix | 13:07 |
| dtantsur | pretty bad, but iurygregory handled it | 13:07 |
| TheJulia | iurygregory: thanks | 13:07 |
| TheJulia | <3 | 13:07 |
| rpittau | and then hopefully we'll have the new inspector package ready soon | 13:08 |
| iurygregory | np o/ | 13:08 |
| dtantsur | TheJulia: now you can check my redfish patches and go back to bed ;) | 13:08 |
| iurygregory | I love the random failures in Ilya's patches XD | 13:08 |
| dtantsur | and the new release model spec if you're in a mood for bad prose | 13:08 |
| TheJulia | dtantsur: sadly I have to give a talk later this morning at an internal team conference | 13:08 |
| TheJulia | not our team, but another team | 13:08 |
| dtantsur | good luck then! | 13:09 |
| iurygregory | good luck TheJulia | 13:09 |
| TheJulia | Wheeeeee :( https://usercontent.irccloud-cdn.com/file/CCnPV2pZ/firstprivatemessageoftheday.png | 13:11 |
| *** cdearborn has joined #openstack-ironic | 13:11 | |
| *** CoDeAmRo has quit IRC | 13:12 | |
| iurygregory | WUT?! | 13:12 |
| TheJulia | iknowrite! | 13:12 |
| iurygregory | please tell me you didn't receive this | 13:12 |
| TheJulia | Oh, I received it minutes ago | 13:12 |
| iurygregory | OH GOD | 13:12 |
| iurygregory | massive facepalm | 13:13 |
| *** akahat|ruck is now known as akahat | 13:13 | |
| rpittau | well, he's honest at least :) | 13:14 |
| TheJulia | this is true | 13:14 |
| *** CoDeAmRo has joined #openstack-ironic | 13:16 | |
| rpittau | dtantsur: I'm finally reading the new release model proposal, one thing that you mention is that it might requires to switch to independent model, but since there still will be named releases I don't understand the reason :/ | 13:19 |
| TheJulia | which was originally floated by ttx | 13:20 |
| TheJulia | openstack consumption | 13:20 |
| rpittau | newbie-releaser question :) | 13:20 |
| dtantsur | rpittau: it's a purely political aspect: whether we'll be allowed to use non-standard stable branches in the current model | 13:21 |
| rpittau | dtantsur: ok, I see, thanks | 13:22 |
| dtantsur | TheJulia: oh mother of dragons (re the private message) | 13:22 |
| dtantsur | rpittau: do be clear: I don't insist we should change, but people might | 13:22 |
| rpittau | no, I get that | 13:23 |
| TheJulia | I guess the conundrum that I kind of want ensure gets... fixed, at least fixed in my mind is the fact that we are viewed by some as only consumable at openstack releases, and this would make that a position that cannot be defended no matter how we label branches | 13:26 |
| *** gokhani has quit IRC | 13:26 | |
| *** CoDeAmRo has quit IRC | 13:30 | |
| *** CoDeAmRo has joined #openstack-ironic | 13:31 | |
| *** livelace has joined #openstack-ironic | 13:34 | |
| openstackgerrit | Merged openstack/ironic master: Monkey patch original current_thread _active https://review.opendev.org/725356 | 13:37 |
| openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Fix grenade job https://review.opendev.org/725514 | 13:37 |
| dtantsur | ofc | 13:38 |
| rpittau | ^^ we probably want to increaese timeout anyway | 13:38 |
| *** tzumainn has joined #openstack-ironic | 13:42 | |
| iurygregory | rpittau, yeah | 13:44 |
| rpittau | although | 13:44 |
| rpittau | hold on a sec | 13:44 |
| iurygregory | I just love that some times it works | 13:45 |
| iurygregory | oh god we are still creating 7 Virtual BM... | 13:46 |
| * iurygregory facepalm | 13:46 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: Add variable to set inspection timeout in tempest tests https://review.opendev.org/725855 | 13:46 |
| rpittau | this ^^ might help | 13:46 |
| iurygregory | ++ | 13:47 |
| iurygregory | if it fails again I will update increasing the timeout and reducing the number of VM's for tests | 13:47 |
| iurygregory | rpittau, this way we will have chicken egg problem if we can't merge grenade XD | 13:48 |
| openstackgerrit | Corey Bryant proposed openstack/ironic stable/ussuri: Monkey patch original current_thread _active https://review.opendev.org/725857 | 13:48 |
| rpittau | iurygregory: yeah, I was thinking to exclude grenade just for that to merge if we're forced to increase the timeout for introspection | 13:49 |
| iurygregory | rpittau, makes sense to me | 13:50 |
| rpittau | the problem seems to be the introspection time and that can't be changed currently :/ | 13:51 |
| iurygregory | yup | 13:51 |
| iurygregory | we are using the default value | 13:51 |
| rpittau | iurygregory: reducing the number of vms might help | 13:51 |
| iurygregory | rpittau, yeah, should I update now or just pray for a good cloud provider? | 13:51 |
| rpittau | iurygregory: I've never been much of a believer, let's do it now :) | 13:53 |
| iurygregory | rpittau, yeah | 13:53 |
| iurygregory | maybe if we had a cthulhu cloud | 13:54 |
| iurygregory | 3 or 2? and also wondering if it would be good to add the LIBVIRT_POOL var | 13:54 |
| TheJulia | Can we create a lab someplace and call it cthulhu cloud!??!?!?!?!?!?!?!?! | 13:55 |
| rpittau | iurygregory: let's go for 3 for now | 13:55 |
| rpittau | TheJulia: a tentacled, undersea, ancient cloud ? | 13:55 |
| iurygregory | we can XD | 13:55 |
| TheJulia | Maybe the Church of Flying Spaghetti Monster needs some neighbors? | 13:55 |
| iurygregory | ++ | 13:56 |
| *** kashifkhan has quit IRC | 13:57 | |
| rpittau | dtantsur went for the angry recheck :D | 13:57 |
| dtantsur | :D | 13:57 |
| *** mkrai has joined #openstack-ironic | 13:57 | |
| TheJulia | I must know what animated gif represents an angry recheck | 13:58 |
| rpittau | lol | 13:58 |
| *** jdandrea has joined #openstack-ironic | 13:59 | |
| * TheJulia really needs a caffeine IV this morning | 13:59 | |
| *** livelace has quit IRC | 13:59 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Fix grenade job https://review.opendev.org/725514 | 13:59 |
| iurygregory | boom | 14:00 |
| *** livelace has joined #openstack-ironic | 14:01 | |
| *** CoDeAmRo has quit IRC | 14:04 | |
| iurygregory | dtantsur, rpittau ^ sorry =) | 14:05 |
| TheJulia | so apparently newer ipmitool, at least downstream, is blowing up with newer hpe firmware versions | 14:06 |
| TheJulia | so bfournie is going to test and possibly propose dropping the present level of retries upstream | 14:07 |
| rpittau | TheJulia: newer ipmitool ? they released a new version? | 14:07 |
| rpittau | oh downstream, nvm | 14:07 |
| TheJulia | crypto ciphres stuff, similar headache to why cern runs a custom ipmitool downstream | 14:08 |
| iurygregory | ouch =( | 14:08 |
| rpittau | well to be able to use tinycore 11.x I had to patch ipmitool :) | 14:09 |
| iurygregory | hehe | 14:11 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] Build tinyipa on focal https://review.opendev.org/725799 | 14:12 |
| *** ociuhandu has quit IRC | 14:15 | |
| *** ociuhandu has joined #openstack-ironic | 14:16 | |
| *** ociuhandu has quit IRC | 14:20 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Make much more configuration options as reloadable https://review.opendev.org/725867 | 14:22 |
| dtantsur | This was surprisingly exhausting ^^^ | 14:22 |
| dtantsur | also an example of a patch with a release note longer than the patch :D | 14:22 |
| *** jhesketh has quit IRC | 14:22 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Mark more configuration options as reloadable https://review.opendev.org/725867 | 14:23 |
| dtantsur | rpittau: if you have a minute today: https://review.opendev.org/#/c/725218/ | 14:23 |
| patchbot | patch 725218 - metalsmith - Get rid of testtools and fixtures - 1 patch set | 14:23 |
| *** ociuhandu has joined #openstack-ironic | 14:24 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Temporary add per-file-ignores to flake8 tests https://review.opendev.org/723886 | 14:25 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Fix 3rd party driver mocks and related tests https://review.opendev.org/724080 | 14:25 |
| iurygregory | I love the random sqlalchemy failures #no | 14:35 |
| dtantsur | yeaaah | 14:35 |
| TheJulia | *sigh* | 14:35 |
| dtantsur | they've been happening from time to time since I joined | 14:35 |
| iurygregory | tox-cover failed -> locally works -> recheck -> py37 fails WTF?! | 14:36 |
| TheJulia | mirrors | 14:36 |
| * dtantsur is wondering how to make periodic tasks parameters reloadable | 14:36 | |
| TheJulia | dtantsur: any way to make the periodic task exit and "restart" ? | 14:36 |
| TheJulia | like a additional wrapping decorator maybe? | 14:36 |
| dtantsur | I doubt that, but we can extend futurist | 14:37 |
| dtantsur | unfortunately, such solution won't be backportable.. | 14:37 |
| TheJulia | no, but enabling to change options while running really is not either | 14:37 |
| TheJulia | I guess I'm curious what problem is trying to be solved | 14:37 |
| iurygregory | they started showing to me when I did the rebase for Ilya's patches | 14:38 |
| dtantsur | TheJulia: mm? I don't see a problem with backporting something that adds mutable=True, honestly I was going to.. | 14:38 |
| TheJulia | dtantsur: I guess I'm thinking like a purist | 14:39 |
| TheJulia | likely not an issue there even though ability to reload specific options becomes features imho | 14:39 |
| dtantsur | heh, wasn't it you who proposed allowing low-impact feature backports if they may improve the ops life substantially? ;) | 14:40 |
| TheJulia | it it totally makes sense, at least depending on the problem being solved | 14:40 |
| dtantsur | I even included this idea in my release model spec | 14:40 |
| TheJulia | dtantsur: and the tc! :) | 14:40 |
| dtantsur | right :) | 14:40 |
| TheJulia | so yeah | 14:40 |
| TheJulia | makes sense, as long as the risk is not high. I think it still kind of depends on the actual options and why | 14:40 |
| TheJulia | but generally shouldn't be an issue, actually thinking about it | 14:40 |
| TheJulia | the periodic timings are the hard ones though. | 14:41 |
| dtantsur | yep, I haven't touched them for now | 14:41 |
| dtantsur | I probably won't, it seems a can of worms | 14:41 |
| dtantsur | as to "why", I've been meaning to do this work item for a long time as part of the overall "make ironic more manageable at scale" topic | 14:42 |
| dtantsur | no specific reason, although it was mentioned on a call with field folks the other day | 14:43 |
| TheJulia | dtantsur: ahh | 14:44 |
| TheJulia | more manageable at scale may be a good topic for the PTG as well | 14:44 |
| * TheJulia puts two topics in for the PTG | 14:46 | |
| TheJulia | err 3 | 14:46 |
| *** jhesketh has joined #openstack-ironic | 14:51 | |
| iurygregory | I thought we were using bionic in bifrost already O.o | 14:56 |
| iurygregory | ok the patch is from Aug 26 2019 XD | 14:58 |
| rpittau | aren't we?!? | 14:58 |
| rpittau | ah ok | 14:58 |
| iurygregory | I forgot to check when it was submitted | 14:58 |
| TheJulia | no heart attacks for at least the next 45 minutes or so | 15:02 |
| iurygregory | sorry TheJulia =) | 15:03 |
| * TheJulia awaits her turn to give a short-ish talk | 15:04 | |
| iurygregory | dtantsur, hey re- using the new collections for bifrost do we have a story with details etc so I can have a better idea? =) | 15:04 |
| TheJulia | new collections? | 15:04 |
| dtantsur | iurygregory: unfortunately, no | 15:05 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add timeout and retries to JSON RPC client https://review.opendev.org/725883 | 15:05 |
| dtantsur | TheJulia: OpenStack Ansible Collection | 15:05 |
| TheJulia | ahh | 15:05 |
| dtantsur | TheJulia: re IPAM integration topic, I wonder if we should delay merging network_data until we figure it out | 15:06 |
| TheJulia | we'll always have standalone users | 15:07 |
| TheJulia | and I think the interfaces are open-enough for logical compatability | 15:07 |
| dtantsur | I mean, if we land node.network_data and then realize we need node.ip_address or something.. | 15:07 |
| dtantsur | will be suboptimal | 15:07 |
| clarkb | jroll: Qianbiao: `pbr freeze` will give you a git sha too. We had those in our versions before pep440 but that pep removed our ability to do that in the versions directly | 15:07 |
| * dtantsur -> tea, brb | 15:07 | |
| TheJulia | dtantsur: oh god no, that was not what I meant | 15:08 |
| TheJulia | we shouldn't be an ipam | 15:08 |
| iurygregory | would be using this https://github.com/openstack/ansible-collections-openstack/tree/1849f8bd4750f1d5f40b95811a934b1547a1bde4/plugins/modules ? | 15:08 |
| TheJulia | we should integrate somehow | 15:08 |
| TheJulia | with things that are not neutron | 15:08 |
| iurygregory | or I'm going in the wrong path? | 15:08 |
| Qianbiao | <clarkb> thanks | 15:09 |
| *** Qianbiao is now known as qianbiao|afk | 15:10 | |
| *** qianbiao|afk has quit IRC | 15:11 | |
| *** ociuhandu has quit IRC | 15:13 | |
| *** ociuhandu has joined #openstack-ironic | 15:14 | |
| *** mkrai has quit IRC | 15:14 | |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Add global_request_id into the base client methods https://review.opendev.org/725661 | 15:15 |
| *** ociuhandu has quit IRC | 15:16 | |
| *** ociuhandu has joined #openstack-ironic | 15:16 | |
| *** alexmcleod has quit IRC | 15:24 | |
| dtantsur | TheJulia: ah, got it! yeah, makes sense to me. | 15:27 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Remove [conductor]api_url https://review.opendev.org/724797 | 15:33 |
| iurygregory | dtantsur, the new collections is https://github.com/openstack/ansible-collections-openstack/tree/1849f8bd4750f1d5f40b95811a934b1547a1bde4/plugins/modules ? or I'm going in the wrong path? | 15:42 |
| dtantsur | iurygregory: yep, that's it. but note that the modules will be renamed eventually | 15:42 |
| iurygregory | ouch | 15:43 |
| iurygregory | XD | 15:43 |
| iurygregory | so we should wait before trying to use in bifrost | 15:44 |
| iurygregory | ? | 15:44 |
| iurygregory | or try to use the new ones and when they are renamed we just push "small" fixes? | 15:44 |
| * iurygregory not sure if the impact would be small =) | 15:45 | |
| dtantsur | iurygregory: we should probably wait a bit more, yeah | 15:47 |
| iurygregory | dtantsur, ack | 15:47 |
| iurygregory | I thought it was something that we start working on =) | 15:47 |
| iurygregory | we can probably have a topic to track in Victoria | 15:48 |
| dtantsur | we should migrate away from testtools eventually.. things like https://github.com/testing-cabal/testtools/issues/235 are annoying | 15:50 |
| *** ociuhandu has quit IRC | 15:50 | |
| *** cdearborn has quit IRC | 15:51 | |
| dtantsur | and https://github.com/testing-cabal/testtools/issues/275 and probably more | 15:51 |
| *** ociuhandu has joined #openstack-ironic | 15:51 | |
| iurygregory | ouch | 15:52 |
| dtantsur | and https://github.com/testing-cabal/testtools/issues/144 | 15:52 |
| dtantsur | I think I've hit all 3 at some point | 15:52 |
| dtantsur | the SystemExit one was annoying on metalsmith | 15:52 |
| *** gyee has joined #openstack-ironic | 15:53 | |
| clarkb | fwiw a big motivation for testtools back in ~2012 was being able to use cleanups under python2.6 (and I think even python2.7 lacked them in stdlib?) | 15:54 |
| clarkb | python3 has cleanups in unittest so should be fine there | 15:54 |
| dtantsur | yeah, it was very handy on python 2 | 15:54 |
| dtantsur | starting another ML thread.. | 15:55 |
| clarkb | and the big reason cleanups are important are they run even if setup fails | 15:55 |
| clarkb | which means you can cleanup fixtures and related resources even if the next fixture setup fails in setup | 15:55 |
| *** gokhani has joined #openstack-ironic | 15:56 | |
| dtantsur | yep, addCleanup is a huge helper | 15:56 |
| clarkb | this was a huge issue back in the day with unittests leaking all over /tmp and elsewhere when we didn't have single use test nodes | 15:56 |
| *** ociuhandu has quit IRC | 15:56 | |
| clarkb | but also developer machines tend to not be single use so its good hygiene anyway | 15:56 |
| clarkb | (I just want to call out that any conversion should ideally not drop using cleanups) | 15:56 |
| dtantsur | the standard unittest seems to support all we need in 3.6+ | 15:59 |
| dtantsur | clarkb: https://docs.python.org/3.6/library/unittest.html#unittest.TestCase.addCleanup | 16:01 |
| *** ociuhandu has joined #openstack-ironic | 16:01 | |
| clarkb | excellent | 16:01 |
| dtantsur | Tired and leaving earlier today. Have a good night! | 16:11 |
| *** dtantsur is now known as dtantsur|afk | 16:11 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [Trivial] Fix awkward mocking in redfish boot tests https://review.opendev.org/725918 | 16:11 |
| *** ociuhandu has quit IRC | 16:13 | |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Add global_request_id into the base client methods https://review.opendev.org/725661 | 16:13 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Allow to pass global request id in node methods https://review.opendev.org/725919 | 16:13 |
| *** ociuhandu has joined #openstack-ironic | 16:14 | |
| *** ociuhandu has quit IRC | 16:19 | |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Allow to pass global request id in node methods https://review.opendev.org/725919 | 16:19 |
| openstackgerrit | Merged openstack/metalsmith master: Get rid of testtools and fixtures https://review.opendev.org/725218 | 16:21 |
| *** gokhani has quit IRC | 16:23 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Enforce autospec in some unit tests modules https://review.opendev.org/725922 | 16:23 |
| rpittau | ^ interesting typos here | 16:23 |
| rpittau | more stuff to come.... tomorrow :) | 16:23 |
| rpittau | bye everyone! o/ | 16:24 |
| *** rpittau is now known as rpittau|afk | 16:24 | |
| *** priteau has quit IRC | 16:25 | |
| *** alexmcleod has joined #openstack-ironic | 16:33 | |
| *** ociuhandu has joined #openstack-ironic | 16:33 | |
| openstackgerrit | Riccardo Pittau proposed openstack/metalsmith master: Upgrade flake8-import-order version to 0.17.1 https://review.opendev.org/721658 | 16:34 |
| openstackgerrit | Riccardo Pittau proposed openstack/metalsmith master: Upgrade flake8-import-order version to 0.17.1 https://review.opendev.org/721658 | 16:34 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Allow to pass global request id in node methods https://review.opendev.org/725919 | 16:42 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Allow to pass global request id in port and volume https://review.opendev.org/725930 | 16:42 |
| *** alexmcleod has quit IRC | 16:54 | |
| *** lucasagomes has quit IRC | 16:58 | |
| *** jdandrea has quit IRC | 17:04 | |
| *** jdandrea has joined #openstack-ironic | 17:06 | |
| openstackgerrit | Verification of a change to openstack/python-ironic-inspector-client failed: Convert job to dib https://review.opendev.org/718698 | 17:07 |
| *** k_mouza has quit IRC | 17:10 | |
| openstackgerrit | Merged openstack/ironic master: Use tempest_plugins Zuul CI role https://review.opendev.org/725649 | 17:13 |
| rpioso | Good morning, ironic | 17:16 |
| rpioso | TheJulia: Is the by_path root device hint supported? | 17:18 |
| rpioso | TheJulia: I see it in the IPA source, but not in the doc. | 17:18 |
| *** irclogbot_2 has quit IRC | 17:20 | |
| *** irclogbot_1 has joined #openstack-ironic | 17:23 | |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Allow to pass global request id for remaining objects https://review.opendev.org/725941 | 17:25 |
| *** derekh has quit IRC | 17:26 | |
| *** ociuhandu has quit IRC | 17:26 | |
| *** sshnaidm is now known as sshnaidm|afk | 17:26 | |
| *** ociuhandu has joined #openstack-ironic | 17:38 | |
| *** ociuhandu has quit IRC | 17:43 | |
| *** livelace has quit IRC | 17:44 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/ironic-tempest-plugin master: Add stable/ussuri job on master gate https://review.opendev.org/725728 | 17:47 |
| uzumaki | morning rpioso ! o/ | 17:47 |
| rpioso | uzumaki: :-) | 17:47 |
| openstackgerrit | Ghanshyam Mann proposed openstack/ironic-tempest-plugin master: Add stable/ussuri job on master gate https://review.opendev.org/725728 | 17:48 |
| arne_wiebalck | bye everyone o/ | 17:51 |
| uzumaki | adios arne_wiebalck \o | 17:51 |
| *** hamzy_ has quit IRC | 17:52 | |
| openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Fix grenade job https://review.opendev.org/725514 | 17:52 |
| rpioso | arne_wiebalck: Good nite | 17:53 |
| *** hamzy has joined #openstack-ironic | 17:54 | |
| *** aedc_ has joined #openstack-ironic | 17:55 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-specs master: Propose iso booting https://review.opendev.org/725949 | 17:55 |
| *** aedc has quit IRC | 17:58 | |
| *** aedc_ has quit IRC | 18:01 | |
| *** ociuhandu has joined #openstack-ironic | 18:02 | |
| *** livelace has joined #openstack-ironic | 18:03 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic master: Limit the number of ipmitool retries https://review.opendev.org/725954 | 18:04 |
| openstackgerrit | Bob Fournier proposed openstack/ironic master: Limit the number of ipmitool retries https://review.opendev.org/725954 | 18:18 |
| sean-k-mooney | hjensas: i think i have ruled out my ironic issues being related to your dnsmasq bug by the way | 18:25 |
| hjensas | sean-k-mooney: good. I just saw your post on the LP bug. I think iPXE does not support classless-static-routes. | 18:26 |
| sean-k-mooney | im currently redeploying my openstack with my api network on the same vlan/subnet as my ipmi network and hopefully that will resolve teh issue | 18:26 |
| hjensas | sean-k-mooney: https://github.com/ipxe/ipxe/pull/104 | 18:27 |
| sean-k-mooney | so inpector will be using 192.168.3.50-99 for its dhcpu range neutron will use 192.168.3.150-199 and my bmcs are using 192.168.3.101-.104 with the api runing on 192.168.3.1 | 18:28 |
| sean-k-mooney | hjensas: yep that would be the issue | 18:28 |
| sean-k-mooney | anyway since im redploying with everthing in the same subnet it should work now | 18:29 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: redfish: handle hardware that is unable to set persistent boot https://review.opendev.org/725239 | 18:31 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: redfish: split reboot into power off followed by power on https://review.opendev.org/725530 | 18:34 |
| *** hamzy_ has joined #openstack-ironic | 18:39 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-specs master: Propose iso booting https://review.opendev.org/725949 | 18:39 |
| *** hamzy has quit IRC | 18:42 | |
| *** ociuhandu has quit IRC | 18:48 | |
| TheJulia | hmm... where is markguz when I have a question for him :( | 18:50 |
| iurygregory | email =) | 18:51 |
| * iurygregory prays for Zuul so it will merge https://review.opendev.org/#/c/725514/ | 18:53 | |
| patchbot | patch 725514 - ironic-inspector - Fix grenade job - 4 patch sets | 18:53 |
| *** gokhani has joined #openstack-ironic | 19:09 | |
| sean-k-mooney | hjensas: so nope that did not solve the issue. it looks like its actully an issue with ipxe.... i have just disabeld ipxe in ironic and gone back to plain old pxe and its looks like its working | 19:17 |
| TheJulia | sean-k-mooney: uefi? legacy boot? | 19:18 |
| sean-k-mooney | legacy boot | 19:18 |
| sean-k-mooney | TheJulia: there are a couple of fourm posts that re similar https://forum.ipxe.org/showthread.php?tid=21729 | 19:19 |
| TheJulia | yeah, the routing code is... interesting in ipxe | 19:22 |
| TheJulia | it doesn't surprise me, tbh | 19:22 |
| sean-k-mooney | its failing even without the need for routing | 19:23 |
| sean-k-mooney | *staic routes | 19:23 |
| sean-k-mooney | but on the plus side ironic just finished deploying the server successfully | 19:23 |
| sean-k-mooney | so ill just stick with plain old pxe | 19:24 |
| TheJulia | there is similar behavior with some of the ipv6 behavior | 19:24 |
| TheJulia | it is maddening | 19:25 |
| *** k_mouza has joined #openstack-ironic | 19:25 | |
| sean-k-mooney | ya.. i just spent the better part of 2 days tring to figure this out | 19:25 |
| sean-k-mooney | the first issue was the service im testing was the only one out of the 4 where i swaped the nic cables so that was a thing | 19:26 |
| sean-k-mooney | once i got passed that i hit the ipxe issue | 19:26 |
| TheJulia | Deja vu to when I was last banging my head against it with ipv6 | 19:26 |
| sean-k-mooney | what was really confusing is inspection worked the whole time | 19:27 |
| sean-k-mooney | but that is because it was not using ironic for dhcp | 19:27 |
| sean-k-mooney | *neutron | 19:27 |
| sean-k-mooney | but in anycase now my home openstack can boot vms and ironic servers which is awsome | 19:28 |
| sean-k-mooney | its been about 2 years since it could last do that and this time im not even using bifrost for the ironic bit its fully integrated with nova | 19:29 |
| TheJulia | fun | 19:29 |
| sean-k-mooney | i need to go do some sriov work in the next few weeks and im tired fo manually reinstalling these servers every time i have to do that | 19:30 |
| sean-k-mooney | ironic defaults to local boot now too right? | 19:30 |
| TheJulia | as of ussuri yes | 19:31 |
| TheJulia | or master branch | 19:31 |
| sean-k-mooney | yes im being dumb and runing master | 19:31 |
| sean-k-mooney | im sure it will be fine | 19:31 |
| * TheJulia still has not thrown a party for that patch merging | 19:31 | |
| TheJulia | Yeah, it should be perfectly fine | 19:32 |
| *** gokhani has quit IRC | 19:33 | |
| sean-k-mooney | it normally is to be fair i just need to make sure to do the right things when im upgrading | 19:33 |
| *** yolanda has quit IRC | 19:33 | |
| sean-k-mooney | e.g. rememeber its ussuri iss | 19:34 |
| sean-k-mooney | *ish | 19:34 |
| *** gokhani has joined #openstack-ironic | 19:36 | |
| *** gokhani has quit IRC | 19:47 | |
| *** ociuhandu has joined #openstack-ironic | 19:53 | |
| *** k_mouza has quit IRC | 19:57 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Move ipmi logging to a separate option https://review.opendev.org/717323 | 20:09 |
| *** rloo has quit IRC | 20:10 | |
| *** ociuhandu has quit IRC | 20:13 | |
| *** ociuhandu has joined #openstack-ironic | 20:46 | |
| *** ociuhandu has quit IRC | 20:54 | |
| *** Goneri has quit IRC | 21:05 | |
| *** rloo has joined #openstack-ironic | 21:20 | |
| *** threestrands has joined #openstack-ironic | 22:20 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: redfish: split reboot into power off followed by power on https://review.opendev.org/725530 | 22:54 |
| *** rh-jelabarre has quit IRC | 23:07 | |
| *** livelace has quit IRC | 23:07 | |
| *** ociuhandu has joined #openstack-ironic | 23:09 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Fix configdrive encoding error https://review.opendev.org/726023 | 23:14 |
| *** ociuhandu has quit IRC | 23:15 | |
| TheJulia | I'm fairly sure we will need to backport that | 23:29 |
| TheJulia | I have no real idea how markguz hit it, but its legit | 23:29 |
| TheJulia | well, he likely has store in swift turned on | 23:54 |
| TheJulia | and we're likely not using it elsewhere | 23:55 |
| openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Fix grenade job https://review.opendev.org/725514 | 23:56 |
| *** dking has quit IRC | 23:57 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
