| Chapi | Done, finally! | 00:19 |
|---|---|---|
| Chapi | https://review.opendev.org/c/openstack/ironic/+/813543 | 00:19 |
| Chapi | Thank you so much <3 | 00:19 |
| stevebaker | Chapi: Hey, I'd recommend you read this commit message guide and then update the commit message https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure | 00:24 |
| Chapi | Thank you! | 00:24 |
| opendevreview | Luis Alfredo Chaparro Gomez proposed openstack/ironic master: Add flask as a dependency and basic setup. https://review.opendev.org/c/openstack/ironic/+/813546 | 00:51 |
| Chapi | Done, but the other files now don't show up. Sorry, I am not the best with git. | 00:52 |
| stevebaker | Chapi: thats ok, I would fix this with an interactive rebase, but you might want to learn about that a bit later https://riptutorial.com/git/example/3282/interactive-rebase | 01:06 |
| stevebaker | Chapi: If you're fine with some profanity, I'd recommend https://ohshitgit.com/ | 01:06 |
| Chapi | Thank you so much Steve! | 01:06 |
| stevebaker | Chapi: But in this case I'd recommend something like: | 01:07 |
| stevebaker | git reset HEAD~1 | 01:07 |
| stevebaker | git add ironic/api/app.py | 01:07 |
| stevebaker | git commit --amend | 01:07 |
| stevebaker | (then fix the commit message | 01:07 |
| stevebaker | ) | 01:07 |
| stevebaker | you should then only have one commit to git review | 01:08 |
| stevebaker | and this one can be abandoned, button on the top right https://review.opendev.org/c/openstack/ironic/+/813546/1 | 01:09 |
| opendevreview | David Vallee Delisle proposed openstack/ironic-lib stable/wallaby: json_rpc: list_opts() should return tuple https://review.opendev.org/c/openstack/ironic-lib/+/813570 | 02:42 |
| opendevreview | David Vallee Delisle proposed openstack/ironic-lib stable/wallaby: json_rpc: setup.cfg is missing entrypoint for json_rpc's list_opts https://review.opendev.org/c/openstack/ironic-lib/+/813571 | 02:43 |
| opendevreview | David Vallee Delisle proposed openstack/ironic-lib stable/wallaby: json_rpc: setup.cfg is missing entrypoint for json_rpc's list_opts https://review.opendev.org/c/openstack/ironic-lib/+/813571 | 02:43 |
| iurygregory | good morning Ironic o/ | 05:54 |
| janders | hey iurygregory o/ | 06:06 |
| iurygregory | hey janders o/ | 06:09 |
| arne_wiebalck | Good morning iurygregory janders and Ironic! | 06:21 |
| iurygregory | morning arne_wiebalck o/ | 06:21 |
| arne_wiebalck | iurygregory: I am blindly re-checking a small patch I have ... is there still a known issue with CI before I send it on another round? | 07:03 |
| iurygregory | arne_wiebalck, do you have the link for the patch? I can check ( we had to land a few changes in metalsmith and inspector since we are default to uefi now) | 07:04 |
| arne_wiebalck | iurygregory: https://review.opendev.org/c/openstack/ironic-python-agent/+/813125 | 07:04 |
| arne_wiebalck | iurygregory: right, it is probably this then ... are these merged already? | 07:05 |
| iurygregory | arne_wiebalck, let me double check 1min | 07:05 |
| arne_wiebalck | iurygregory: thanks! | 07:06 |
| iurygregory | ok the metalsmith job should be ok since the patch merged | 07:07 |
| iurygregory | arne_wiebalck, go ahead and send another recheck it should be fine now | 07:08 |
| arne_wiebalck | iurygregory: thanks, will do! | 07:09 |
| rpittau | good morning ironic! o/ | 07:12 |
| janders | hey arne_wiebalck and rpittau o/ | 07:18 |
| rpittau | hey janders :) | 07:19 |
| iurygregory | morning rpittau o/ | 07:30 |
| rpittau | hey iurygregory :) | 07:30 |
| dtantsur | morning ironic | 08:03 |
| janders | hey dtantsur o/ | 08:03 |
| iurygregory | morning dtantsur o/ | 08:24 |
| iurygregory | arne_wiebalck, metalsmith-integration-ipa-src success :D | 08:25 |
| pas-ha[m] | stevebaker: re matrix + element - I just added the irc bridge for OFTC, seeing it as @oftc-irc:matrix.org in my "contacts" in element app, and thru chatting with that I can control the IRC, e.g "!join #openstack-ironic" | 08:33 |
| stevebak[m] | pas-ha: thanks, I figured it out. I wonder if instructions would be easy enough for an outreachy | 08:36 |
| pas-ha[m] | that's a good question, and mostly depends on if that person had any exposure to IRC at all (most probably not :-/) | 08:37 |
| pas-ha[m] | for outreachy I would indeed say the web version is the most convenient (still need to understand the IRC a bit though) | 08:38 |
| dtantsur | nice re IRC bridge! | 08:42 |
| * dtantsur should play with matrix finally | 08:42 | |
| stevebak[m] | Oh I can attend the second SIG slot if I wake in time | 08:42 |
| iurygregory | dtantsur, which pill will you choose? =P | 08:53 |
| dtantsur | sleeping? :D | 09:07 |
| iurygregory | it's a good one, but I don't think Morpheus has this one :D | 09:08 |
| dtantsur | wut, his name is literally Morpheus, he's supposed to! | 09:09 |
| iurygregory | you can choose between uncertain future (red pill) or beautiful prison (blue) :D | 09:09 |
| iurygregory | dtantsur, yeah I agree :D | 09:09 |
| dtantsur | given that the current state is closer to an uncertain prison, both options sound viable | 09:09 |
| * dtantsur is grumpy old man | 09:10 | |
| arne_wiebalck | iurygregory: uh, nice, ta | 09:16 |
| iurygregory | yw | 09:23 |
| opendevreview | Merged openstack/ironic master: Retool devstack plugin to use pxe loaders configuration https://review.opendev.org/c/openstack/ironic/+/812188 | 09:29 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: node_periodics: encapsulate the interface class check https://review.opendev.org/c/openstack/ironic/+/813458 | 09:31 |
| opendevreview | Merged openstack/ironic-inspector master: CI: Make inspector configuration UEFI aware https://review.opendev.org/c/openstack/ironic-inspector/+/812137 | 10:31 |
| opendevreview | Merged openstack/ironic-python-agent master: Assert EFI part UUID is not None before editing fstab https://review.opendev.org/c/openstack/ironic-python-agent/+/813125 | 10:58 |
| viks__ | hi, when i try to bringup centos 8 baremetal instance, the instance stuck in boot stage with `Probing EDD (edd=off to disable)...`. How do i resolve this? or how do i debug this? | 11:38 |
| janders | see you tomorrow Ironic o/ | 11:46 |
| dtantsur | viks__: it could be a kernel crash, you probably don't see the cause because the kernel is not set to output to the console | 11:56 |
| dtantsur | you may want to play with the `console` argument in your kernel_append_params | 11:57 |
| viks__ | dtantsur: `[pxe]/kernel_append_params` this is only for pxe boot right? i.e.these parameters gets passed to only during deploy image setup right? i'm getting the error when actual instance creation | 12:00 |
| dtantsur | right, yeah. maybe tweak the image for debugging? | 12:01 |
| viks__ | dtantsur: tweak the image? i tried mounting it via guestmount to modify grub, but centos 8 image is not allowing me to mount with "--rw" ... it allows me to mount it only with "--ro" | 12:04 |
| dtantsur | viks__: this is weird, I definitely used virt-builder on centos images: https://github.com/dtantsur/config/blob/master/virt-install.sh#L39-L46 | 12:06 |
| dtantsur | you may also try diskimage-builder, I *think* it has a way of modifying grub params | 12:06 |
| viks__ | dtantsur: ok.. thanks.. have not tried `virt-builder` before... will try it out... i'm using images from https://cloud.centos.org/centos/8/x86_64/images/ .. so if i use disk-image-builder, i'm not sure how exactly to create the images as present in the given link..... so will try modifying the existing using 'virt-builder' | 12:11 |
| Inwerejosic1 | Hello @TheJulia and @stevebaker | 12:35 |
| dtantsur | Hi Inwerejosic1, I think TheJulia hasn't woken up yet, you may need to wait a bit. | 12:35 |
| dtantsur | (or use email) | 12:35 |
| Inwerejosic1 | Okay @dtantsur, I will try that and also wait for her to wake up | 12:36 |
| Inwerejosic1 | dtsantsur, are you also an intending intern like me? | 12:38 |
| dtantsur | Inwerejosic1: nope, I'm a community member :) | 12:38 |
| inwerejosic | Thank you, Its a pleasure to finally have a chat with a community member, This means a lot.. | 12:39 |
| Inwerejosic1 | I hope to become fruitful part of this community | 12:42 |
| Inwerejosic1 | Through this internship and progression going forward | 12:42 |
| dtantsur | nice, welcome! | 12:44 |
| *** thelounge94 is now known as redrobot | 13:02 | |
| *** redrobot is now known as thelounge94 | 13:04 | |
| *** thelounge94 is now known as redrobot | 13:04 | |
| iurygregory | Hey everyone, after removing some of the topics we discussed during our weekly meeting from the PTG schedule, this is the current proposal I have https://paste.opendev.org/show/809919/ | 13:10 |
| TheJulia | viks__: sounds like console=ttyS0 is present on your PXE command line get it to write to the screen, you need to likely remove that. or add console=tty0 | 13:13 |
| TheJulia | viks__: probing EDD is exactly the only thing that gets logged before the kernel switches to the configured console afaik | 13:14 |
| TheJulia | arne_wiebalck: fyi, I had a call with a hardware vendor get pushed to the same time as the sig start for EU friendly timezones. I'm not expecting it to run more than 15-20 minutes | 13:15 |
| viks__ | TheJulia: ok... i think i have set that `console=ttyS1,115200n8`... will change it and see... But i do not get this error on centos 7 though... not sure why is it happening on cent 8.. | 13:16 |
| arne_wiebalck | TheJulia: ok, thanks for letting me know, join whenever you can :) | 13:17 |
| TheJulia | viks__: yeah, something chanted between the two, I'm not exactly what | 13:17 |
| viks__ | TheJulia: ok.. will check removing it if cent8 comes up fine | 13:18 |
| dtantsur | viks__: at the very least, there are driver removals in CentOS 8 that people complained about | 13:20 |
| viks__ | dtantsur: ok | 13:20 |
| TheJulia | Yeah... we have some friends of this community which were not exactly thrilled by the removal of some drivers.... | 13:21 |
| inwerejosic | hi @TheJulia, I am a prospective Outreachy intern, looking to work with you in Openstack project #4 | 13:24 |
| TheJulia | Greetings inwerejosic | 13:24 |
| inwerejosic | I am glad to be here. @TheJulia, please how can make the best of this opportunity? | 13:27 |
| TheJulia | inwerejosic: Generally we recommend applicants to reviews the contribution guidelines and process documentation for the larger OpenStack community. https://docs.openstack.org/infra/manual/developers.html | 13:28 |
| TheJulia | And then review https://docs.openstack.org/ironic/latest/contributor/contributing.html | 13:28 |
| TheJulia | And then maybe look through https://storyboard.openstack.org looking for an low hanging fruit or minor bug in the ironic project in order to try and fix and practice the workflow of working with Gerrit. | 13:29 |
| inwerejosic | Thank you so much, I will head over there now and get cracking... Thanks for you guidance. | 13:31 |
| arne_wiebalck | For those interested, Bare metal SIG meeting in 2mins: https://etherpad.opendev.org/p/bare-metal-sig | 13:58 |
| dtantsur | folks don't hear me, do they? | 14:14 |
| dtantsur | arne_wiebalck, ^^^\ | 14:15 |
| *** pmannidi is now known as pmannidi|AFK | 14:21 | |
| Kathy | HI @TheJulia,I am Kathy i am glad to be here and would like to be your team on the project Openstack Project 4. | 15:14 |
| arne_wiebalck | thanks everyone for joining the SIG meeting, quite interested discussion once more | 15:20 |
| Kathy | Hi @stevebak glad to be here to contribute to Openstack Project4 | 15:20 |
| arne_wiebalck | reminder: there is another SIG session later today at 6pm UTC, all details on https://etherpad.opendev.org/p/bare-metal-sig | 15:20 |
| TheJulia | Kathy: Hi, So Just for your context, Steve Baker is located in UTC+12. | 15:21 |
| TheJulia | Kathy: How can we help? | 15:21 |
| Kathy | Okay thank you just felt i should say hello to him too.I will like to know how i can progress in my contribution for Openstack Project 4 .Thank you | 15:28 |
| TheJulia | Kathy: Okay, thanks! | 15:28 |
| rpittau | bye, good night! o/ | 16:19 |
| jeny_sadadia | Hello, I am Jeny Sadadia from India. I am an applicant of Outreachy 2021 December internship. I am excited to contribute in 'Auto-Documenting REST API classes for Ironic' project. As I am new to open source, could you please guide me on who I can start with contributions ? | 17:09 |
| NobodyCam | so there's a thing today? | 17:42 |
| TheJulia | Yes, Baremetal SIG operators feedback session. Express wants/dreams/needs/desires/issues/successes with Ironic | 17:43 |
| NobodyCam | :) | 17:44 |
| NobodyCam | at 11? | 17:44 |
| TheJulia | yes | 17:45 |
| NobodyCam | :) | 17:45 |
| TheJulia | You can always just listen in | 17:45 |
| TheJulia | NobodyCam: https://meet.google.com/mzh-tjpi-vrr | 18:02 |
| stevebaker | inwerejosic: hello, welcome! | 18:05 |
| Inwerejosic1 | Thank you very much Sir, @stevebaker | 18:07 |
| *** stevebak[m] is now known as stevebaker[m] | 18:11 | |
| arne_wiebalck | NobodyCam: https://bugs.launchpad.net/nova/+bug/1884217 | 18:14 |
| arne_wiebalck | NobodyCam: if you chime in on this one ^^, we may be able to get it up the priority list :) | 18:59 |
| arne_wiebalck | bye everyone, see you tomorrow o/ | 18:59 |
| NobodyCam | Night arne_wiebalck | 19:02 |
| lmcgann | Im working on handling failures and retries in the Keylime driver, and the keylime package has code that would be useful for this. However, because Keylime isnt in pypi the operator has to install Keylime from the source repo. Is it ok to import Keylime in modules then or woud this break dependencies and unit tests in Ironic? | 19:33 |
| TheJulia | lmcgann: not in pypi?! | 19:49 |
| lmcgann | not yet. Theyve been reworking their packaging lately and havent gotten around to it yet | 19:51 |
| JayF | It's borderline ridiculous for us to even plugin to a service that is not available on pypi yet :| that's a pretty basic requirement. | 20:00 |
| JayF | We can't tell the average operator to go install a package from github... | 20:01 |
| ade_lee | TheJulia, stevebaker -- hey - just fyi - after making changes to fips tests as requested, the jobs are not passing. https://review.opendev.org/c/openstack/ironic/+/797739 | 20:17 |
| ade_lee | not sure why yet .. | 20:18 |
| TheJulia | ade_lee: I can try and look tomorrow | 20:37 |
| ade_lee | TheJulia, thanks! | 20:37 |
| lmcgann | Would adding a repo to requirements.txt in the format git+https://github.com/keylime/keylime.git be sufficient or no? | 20:40 |
| TheJulia | lmcgann: it wouldn't be, because realistically we also have an upper-constraints.txt which is governed community wide | 20:41 |
| TheJulia | and that would break packaging folks trying to consume/ship ironic | 20:41 |
| lmcgann | ok. I've relayed the message to the Keylime people, hopefully this gets fixed soon | 20:45 |
| lmcgann | For what its worth, the reason Keylime hasnt been available on pypi is because tpm2-tools that the agent depends on cannot be installed with pypi, and keylime has had its own special installer script in source. But those tpm-tools are irrelevant to an ironic driver | 20:59 |
| janders | good morning Ironic o/ | 22:06 |
| Chapi | Hello everyone. | 22:31 |
| Chapi | Could anyone help me to fix this issue with my contribution? | 22:31 |
| Chapi | https://zuul.opendev.org/t/openstack/build/8d36626ae679468ba2836ecfeb3bc64c | 22:31 |
| Chapi | I don't understand what it means. | 22:31 |
| stevebaker[m] | Chapi: The first log file which is useful to check is job-output.txt in https://zuul.opendev.org/t/openstack/build/8d36626ae679468ba2836ecfeb3bc64c/logs | 23:31 |
| stevebaker[m] | Chapi: The failure will be near the end, but not at the end (because it logs the log collection and cleanup tasks) https://zuul.opendev.org/t/openstack/build/8d36626ae679468ba2836ecfeb3bc64c/log/job-output.txt#32297 | 23:33 |
| stevebaker[m] | Chapi: Since this is a test failure, testr_results.html presents the failure nicely https://320f233b0916ba880ac7-d272393bb0af55a2c09c8548e5e61a42.ssl.cf1.rackcdn.com/813546/1/check/ironic-inspector-tempest/8d36626/testr_results.html | 23:34 |
| stevebaker[m] | Chapi: It looks like introspection timed out, which is probably a temporary environmental failure or a broken CI job. You can check controller/logs/screen-i*.txt for any errors which might be related to your change to confirm. If you think the failure is unrelated to your change you can run all of the jobs again by leaving "recheck" as a gerrit comment | 23:37 |
| TheJulia | Chapi: so, first thing first... You want to ensure unit tests and pep8 tests pass | 23:41 |
| TheJulia | `tox -epy3,pep8` locally can help a lot there | 23:41 |
| TheJulia | Chapi: You'll also want to review https://docs.opendev.org/opendev/infra-manual/latest/developers.html#committing-changes | 23:42 |
| TheJulia | Chapi: For what its worth, If unit tests pass, generally it is a good sign the change is good but we want to run integration/cross service tests as well, so first test locally, then in CI services will be stood up and tested in the integration tests. | 23:44 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!