| iurygregory | no worries | 00:01 |
|---|---|---|
| iurygregory | yes, you will need to implement some functions in node.py and equivalent ones in _proxy.py and write some tests (unit tests and functional tests I think) | 00:06 |
| iurygregory | for example you can look at how node.py handles traits (look at the function and check the commit that added so you can have a better idea on how things would look like) | 00:07 |
| *** paras333 has joined #openstack-ironic | 00:22 | |
| *** paras333 has quit IRC | 00:27 | |
| kajalsah07 | Thanks, I was indeed looking for some similar commits/functions. I will look at how traits is implemented. | 00:37 |
| *** spotz has quit IRC | 01:32 | |
| *** rh-jelabarre has quit IRC | 01:37 | |
| *** rcernin has quit IRC | 02:30 | |
| *** k_mouza has joined #openstack-ironic | 02:48 | |
| *** k_mouza has quit IRC | 02:52 | |
| *** rcernin has joined #openstack-ironic | 02:57 | |
| *** rcernin has quit IRC | 03:01 | |
| *** rcernin has joined #openstack-ironic | 03:25 | |
| *** rcernin has quit IRC | 03:29 | |
| *** tkajinam is now known as tkajinam|lunch | 03:35 | |
| *** pmannidi has joined #openstack-ironic | 03:47 | |
| *** rcernin has joined #openstack-ironic | 03:47 | |
| *** pmannidi has quit IRC | 03:52 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Fix idrac-wsman BIOS factory_reset result finding https://review.opendev.org/c/openstack/ironic/+/748696 | 03:53 |
| *** pmannidi has joined #openstack-ironic | 04:15 | |
| *** pmannidi has quit IRC | 04:20 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Fix idrac-wsman BIOS factory_reset result finding https://review.opendev.org/c/openstack/ironic/+/748696 | 04:46 |
| *** k_mouza has joined #openstack-ironic | 04:48 | |
| *** k_mouza has quit IRC | 04:53 | |
| *** tzumainn has quit IRC | 05:36 | |
| *** k_mouza has joined #openstack-ironic | 05:52 | |
| *** k_mouza has quit IRC | 05:56 | |
| *** k_mouza has joined #openstack-ironic | 05:59 | |
| *** uzumaki has quit IRC | 06:02 | |
| *** k_mouza has quit IRC | 06:03 | |
| *** uzumaki has joined #openstack-ironic | 06:35 | |
| *** moshiur has joined #openstack-ironic | 06:36 | |
| *** tosky has joined #openstack-ironic | 06:45 | |
| *** tkajinam|lunch is now known as tkajinam | 06:46 | |
| arne_wiebalck | Good morning, ironic! | 06:49 |
| *** ociuhandu has joined #openstack-ironic | 07:01 | |
| *** rpittau|afk is now known as rpittau | 07:01 | |
| rpittau | good morning ironic! o/ | 07:01 |
| *** uzumaki has quit IRC | 07:07 | |
| *** uzumaki has joined #openstack-ironic | 07:08 | |
| *** ociuhandu has quit IRC | 07:13 | |
| moshiur | Good morning, rpittau: my other PR in DIB got merged today. Thanks to everyone. can you please help to merge my PR in IPA builder. https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/778726 | 07:13 |
| *** ociuhandu has joined #openstack-ironic | 07:14 | |
| rpittau | moshiur: great news! The patch in ipa-builder should now automatically run gate ci jobs and merge if pass | 07:14 |
| moshiur | ok thanks rpittau. I will wait. | 07:18 |
| *** ociuhandu has quit IRC | 07:19 | |
| *** ociuhandu has joined #openstack-ironic | 07:19 | |
| *** ociuhandu has quit IRC | 07:19 | |
| *** ociuhandu has joined #openstack-ironic | 07:21 | |
| *** ociuhandu has quit IRC | 07:25 | |
| *** ociuhandu has joined #openstack-ironic | 07:26 | |
| janders | good morning arne_wiebalck rpittau moshiur and Ironic o/ | 07:29 |
| arne_wiebalck | hey janders ricolin moshiur o/ | 07:33 |
| openstackgerrit | Merged openstack/ironic master: Fix typo in security docs around is_admin rule https://review.opendev.org/c/openstack/ironic/+/783963 | 07:52 |
| rpittau | hey janders :) | 07:53 |
| arne_wiebalck | s/ricolin /ricolin rpittau/ :) | 07:54 |
| ricolin | arne_wiebalck, :) | 07:54 |
| rpittau | :D | 07:54 |
| rpittau | hey arne_wiebalck :) | 07:55 |
| *** ociuhandu has quit IRC | 07:59 | |
| *** ociuhandu has joined #openstack-ironic | 07:59 | |
| *** dougsz has joined #openstack-ironic | 08:01 | |
| *** lucasagomes has joined #openstack-ironic | 08:03 | |
| *** ociuhandu has quit IRC | 08:04 | |
| *** ociuhandu has joined #openstack-ironic | 08:04 | |
| *** rcernin has quit IRC | 08:13 | |
| *** ociuhandu has quit IRC | 08:27 | |
| *** ociuhandu has joined #openstack-ironic | 08:28 | |
| *** derekh has joined #openstack-ironic | 08:33 | |
| *** ociuhandu has quit IRC | 08:33 | |
| *** ociuhandu has joined #openstack-ironic | 08:33 | |
| *** rcernin has joined #openstack-ironic | 08:43 | |
| *** kamlesh_chauvhan has joined #openstack-ironic | 08:50 | |
| *** kamlesh_chauvhan has quit IRC | 08:52 | |
| *** rcernin has quit IRC | 08:55 | |
| *** ociuhandu has quit IRC | 09:07 | |
| *** ociuhandu has joined #openstack-ironic | 09:08 | |
| cenne | goodmorning everyone | 09:08 |
| *** ameya49 has joined #openstack-ironic | 09:10 | |
| *** ociuhandu has quit IRC | 09:13 | |
| *** rcernin has joined #openstack-ironic | 09:14 | |
| *** rcernin has quit IRC | 09:18 | |
| *** abhishekyadav has joined #openstack-ironic | 09:22 | |
| ameya49 | dtantsur|afk : Good Morning. I'm testing your patch : https://review.opendev.org/c/openstack/ironic/+/771493 (secure boot management) and have following observations : | 09:29 |
| ameya49 | 1. Enabling secure boot : After deploying node via devstack (ironic), node gets deployed and gets into "active" state. Also Secure boot sets as "Enabled" on iDRAC, but not able to see OS login screen on iDRAC console. Message displayed : UEFI0073 : Unable to boot because of the Secure Boot policy. | 09:29 |
| ameya49 | Query : Any pre-requisites/configuration needed for enabling secure boot? | 09:29 |
| ameya49 | 2.Disabling secure boot : Observed that the disabling secure boot triggers while "undeploy" of node. But its been observed that iDRAC creates a pending job to set secure boot parameter as "Disabled" but do not execute it. iDRACs get powered off. On the next reboot, the pending job gets executed and sets parameter of secure boot as "Disabled". | 09:29 |
| ameya49 | Query : Is this expected? | 09:29 |
| ameya49 | Please let me know if any more details needed | 09:29 |
| *** abhishekyadav has quit IRC | 09:30 | |
| *** sshnaidm|afk is now known as sshnaidm | 09:31 | |
| *** dtantsur|afk is now known as dtantsur | 09:41 | |
| dtantsur | ameya49: morning! 2. Yes. 1. You need a secure boot compatible user image (not cirros or anything like that). | 09:42 |
| rpittau | dtantsur, TheJulia, I'd like to ask a release of sushy for victoria and ussuri to get a couple of fixes in, wdyt? | 09:42 |
| dtantsur | no objections | 09:42 |
| rpittau | cool! | 09:42 |
| *** rcernin has joined #openstack-ironic | 09:46 | |
| *** rcernin has quit IRC | 09:46 | |
| *** rcernin has joined #openstack-ironic | 09:47 | |
| *** uzumaki has quit IRC | 09:47 | |
| rpittau | oh and train... | 09:49 |
| dtantsur | and once the suse fix lands, we need to release IPA-builder | 09:49 |
| rpittau | yeah | 09:49 |
| *** ociuhandu has joined #openstack-ironic | 09:57 | |
| *** ociuhandu_ has joined #openstack-ironic | 09:59 | |
| *** ociuhandu has quit IRC | 10:02 | |
| *** ociuhandu_ has quit IRC | 10:03 | |
| *** ameya49 has quit IRC | 10:23 | |
| arne_wiebalck | bye everyone o/ | 10:26 |
| *** ameya49 has joined #openstack-ironic | 10:30 | |
| *** ociuhandu has joined #openstack-ironic | 10:34 | |
| ameya49 | Thanks dtantsur. Will check on the compatible user image part. | 10:34 |
| *** ameya49 has quit IRC | 10:34 | |
| *** ameya49 has joined #openstack-ironic | 10:36 | |
| *** ameya49 has quit IRC | 10:38 | |
| *** ociuhandu has quit IRC | 10:39 | |
| iurygregory | good morning Ironic o/ | 10:44 |
| ajya | Hi, there is a patch failing grenade job, it looks because of "No Image found for cirros-0.5.1-x86_64-uec". Any ideas? The patch - https://review.opendev.org/c/openstack/ironic/+/748696 | 10:48 |
| dtantsur | I hope it's not broken CI.. | 10:48 |
| *** k_mouza has joined #openstack-ironic | 10:54 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Do not use pregenerated tokens with a pre-built ISO https://review.opendev.org/c/openstack/ironic/+/785372 | 11:00 |
| *** sshnaidm has quit IRC | 11:01 | |
| *** sshnaidm has joined #openstack-ironic | 11:04 | |
| *** k_mouza has quit IRC | 11:08 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Do not use pregenerated tokens with a pre-built ISO https://review.opendev.org/c/openstack/ironic/+/785372 | 11:18 |
| *** rh-jelabarre has joined #openstack-ironic | 11:24 | |
| ajya | can't find any errors for image creation, between recent failures there was one success, will retry.. | 11:25 |
| *** paras333_ has joined #openstack-ironic | 11:26 | |
| *** k_mouza has joined #openstack-ironic | 11:26 | |
| openstackgerrit | Merged openstack/ironic-python-agent-builder master: Fix: IPA image building with OpenSuse. https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/778726 | 11:28 |
| iurygregory | seems like CI config is using cirros 0.5.2 | 11:34 |
| ajya | iurygregory: smth that changed recently? | 11:36 |
| iurygregory | trying to confirm things... probably it's a config from infra that updated the cirros image | 11:37 |
| *** sshnaidm has quit IRC | 11:37 | |
| iurygregory | and we default to their values I think | 11:37 |
| iurygregory | I'm checking now | 11:37 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Wipe agent tokens on inspection start and abort https://review.opendev.org/c/openstack/ironic/+/785399 | 11:43 |
| dtantsur | agent tokens strike again ^^ :) | 11:43 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Do not use pregenerated tokens with a pre-built ISO https://review.opendev.org/c/openstack/ironic/+/785372 | 11:43 |
| moshiur | Thanks dtantsur, rpittau and JayF for review and merge the PR. | 11:44 |
| dtantsur | np | 11:44 |
| openstackgerrit | Bob Fournier proposed openstack/ironic stable/ussuri: Refactor vendor detection and add Redfish implementation https://review.opendev.org/c/openstack/ironic/+/784452 | 11:46 |
| *** sshnaidm has joined #openstack-ironic | 11:50 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Do not use pregenerated tokens with a pre-built ISO https://review.opendev.org/c/openstack/ironic/+/785372 | 11:53 |
| *** zaneb has joined #openstack-ironic | 11:57 | |
| *** zaneb has quit IRC | 11:59 | |
| *** ociuhandu has joined #openstack-ironic | 12:02 | |
| *** ociuhandu has quit IRC | 12:02 | |
| openstackgerrit | Aija Jauntēva proposed x/sushy-oem-idrac stable/victoria: Remove lower constraints https://review.opendev.org/c/x/sushy-oem-idrac/+/785402 | 12:19 |
| openstackgerrit | Merged openstack/ironic-python-agent master: Capture the early logging https://review.opendev.org/c/openstack/ironic-python-agent/+/783888 | 12:23 |
| *** k_mouza has quit IRC | 12:24 | |
| *** k_mouza has joined #openstack-ironic | 12:25 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent stable/wallaby: Capture the early logging https://review.opendev.org/c/openstack/ironic-python-agent/+/785377 | 12:28 |
| rpittau | dtantsur, TheJulia, I requested sushy (train,ussuri,victoria) and ipa-builder releases | 12:28 |
| dtantsur | k thanks | 12:30 |
| *** samcat116 has joined #openstack-ironic | 12:38 | |
| *** k_mouza has quit IRC | 12:46 | |
| *** k_mouza has joined #openstack-ironic | 12:47 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Fix fast track with redfish-virtual-media https://review.opendev.org/c/openstack/ironic/+/785407 | 12:52 |
| *** rcernin_ has joined #openstack-ironic | 12:58 | |
| *** zaneb has joined #openstack-ironic | 12:59 | |
| *** rcernin has quit IRC | 12:59 | |
| *** lmcgann has joined #openstack-ironic | 13:04 | |
| *** frigo has joined #openstack-ironic | 13:06 | |
| frigo | Hello Ironic! Above makes me think that booting servers with virtual media take a while.. it can take 10 minutes from the boot loader screen to the start of the system (Centos8 minimal iso is ~350MB) | 13:08 |
| *** samcat116 has quit IRC | 13:08 | |
| TheJulia | frigo: bandwidth limitations plus BMCs tend to have limited CPU capacity since it becomes an out of band device and is not using in-band networking to boot | 13:16 |
| frigo | I get that. Still... | 13:16 |
| frigo | a "production ready tinyipa" would be nice | 13:17 |
| frigo | or some magic to load the kernel from the iso and the ramdisk from the network... or to split the ramdisk in 2 | 13:17 |
| *** Goneri has joined #openstack-ironic | 13:18 | |
| dtantsur | we're going towards something like ^^ in the openshift world | 13:20 |
| frigo | really? you have more info? | 13:22 |
| TheJulia | I suspect that has the greatest possibility, I don't expect we would really be able to maintain tinyipa as something "production-ready", although we could likely strip much more out of our ramdisks | 13:22 |
| TheJulia | rpittau: thanks | 13:23 |
| dtantsur | frigo: openshift stuff is coreos specific, probably won't help you | 13:23 |
| openstackgerrit | Kafilat Adeleke proposed openstack/ironic master: Aliases for a few unfortunately named state transitions https://review.opendev.org/c/openstack/ironic/+/784898 | 13:31 |
| *** rcernin_ has quit IRC | 13:42 | |
| *** samcat116 has joined #openstack-ironic | 13:44 | |
| dtantsur | ror rebooting system: Reboot script failed with exit code 1. stdout: . stderr: System has not been booted with systemd as init system (PID 1) | 13:46 |
| dtantsur | Fair enough, didn't we have a fallback for that? | 13:46 |
| cenne | dtantsur: I've reading through the files in openstack sdk to gaet an idea of the codebase. | 13:47 |
| cenne | A quick quesn when you find time. In openstack/proxy.py at line 519 (in _list docstring) is it really supposed to be delete? Feels like a typo. Just making sure. | 13:47 |
| dtantsur | cenne: it's a typo indeed, good catch | 13:48 |
| dtantsur | bonus point for fixing it (in a separate patch) :) | 13:48 |
| cenne | alright! | 13:49 |
| *** tzumainn has joined #openstack-ironic | 13:51 | |
| openstackgerrit | Kafilat Adeleke proposed openstack/ironic master: Aliases for a few unfortunately named state transitions https://review.opendev.org/c/openstack/ironic/+/784898 | 13:56 |
| *** k_mouza has quit IRC | 14:01 | |
| *** k_mouza has joined #openstack-ironic | 14:02 | |
| *** k_mouza has quit IRC | 14:02 | |
| *** k_mouza has joined #openstack-ironic | 14:02 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic stable/ussuri: Refactor vendor detection and add Redfish implementation https://review.opendev.org/c/openstack/ironic/+/784452 | 14:07 |
| rpioso | Good morning, ironic :) | 14:35 |
| rpioso | Is there an update on our plan to release 17.0.1? | 14:37 |
| *** moshiur has quit IRC | 14:39 | |
| TheJulia | I cut it as indicated I would last night based on the state of the branch | 14:46 |
| TheJulia | or at least requested it | 14:46 |
| dtantsur | I see the tag in git | 14:47 |
| TheJulia | I was about to say, it looks like it had been merged and thus released | 14:47 |
| rpioso | dtantsur, TheJulia: Thank you | 14:47 |
| cenne | when making a small change, are we supposed to upload it and then edit with more commits later, or make a bunch of commits and then 'git review' | 14:47 |
| dtantsur | cenne: each commit should be self-contained and independent | 14:48 |
| dtantsur | locally you can start a branch of master for each independent change and submit them separately | 14:48 |
| TheJulia | cenne: And uploaded to gerrit using `git review`, and you can download/edit/reupload using that tool. | 14:48 |
| cenne | so a fix for they typo for example. should i git review it right away? | 14:49 |
| TheJulia | cenne: the key aspect to all of this is the commit message corresponds to a change. Gerrit tracks these with a `Change-Id` value in the commit message. | 14:49 |
| TheJulia | cenne: if it is not related to another change your making, I don't see why not | 14:50 |
| dtantsur | it's even better, yes | 14:51 |
| openstackgerrit | Aija Jauntēva proposed openstack/ironic master: Update to not use deprecated `get_task_monitor`. https://review.opendev.org/c/openstack/ironic/+/785429 | 14:55 |
| *** spotz has joined #openstack-ironic | 14:58 | |
| cenne | okay something happened, could you guys review this? https://review.opendev.org/c/openstack/openstacksdk/+/785431 | 15:13 |
| *** k_mouza_ has joined #openstack-ironic | 15:13 | |
| cenne | It's not much. Hope I didn't mess up the process.. | 15:13 |
| rpittau | cenne: that looks ok, just no need to be connected to that task | 15:16 |
| *** k_mouza has quit IRC | 15:17 | |
| cenne | oh okay. so .. should i ammend and resubmit? | 15:17 |
| cenne | *amend | 15:17 |
| rpittau | cenne: yeah | 15:17 |
| rpittau | just remove story and task entries | 15:18 |
| *** diurnalist has joined #openstack-ironic | 15:22 | |
| cenne | done. but it says rejected when I try to git review. | 15:22 |
| rpittau | mmmm you should amend just the commit msg, so use git commit --amend, and remove the 2 lines, nothing more | 15:24 |
| dtantsur | cenne: if you removed the Change-Id line, copy it from the gerrit UI | 15:26 |
| dtantsur | it's required for gerrit | 15:26 |
| dtantsur | in other news, I think bifrost centos 8 is broken :-/ | 15:28 |
| dtantsur | example https://zuul.opendev.org/t/openstack/build/0494e8b8b5d74a658f0d6802f77fe463/log/job-output.txt | 15:28 |
| cenne | yes i removed the task and story, left the changeid intact | 15:28 |
| cenne | here's the error https://paste.centos.org/view/e7ec93d3 | 15:29 |
| cenne | here's my git show https://paste.centos.org/view/98e8ffd5 | 15:29 |
| dtantsur | eeeeehhhmm | 15:29 |
| cenne | (sorry paste.openstack.org wasn't opening for me) | 15:29 |
| dtantsur | try again maybe? | 15:29 |
| *** bburns has quit IRC | 15:30 | |
| dtantsur | honestly, I'm a bit puzzles by it | 15:30 |
| cenne | same error :( | 15:31 |
| dtantsur | cenne: you have just this one commit on top of git master, right? | 15:31 |
| JayF | cenne: that is usually caused by permissions issues locally | 15:31 |
| kajalsah07 | Hi all, when I try to enter `git review -s` I get the following error: | 15:31 |
| kajalsah07 | Please make sure you have the correct access rights | 15:31 |
| kajalsah07 | and the repository exists. | 15:31 |
| kajalsah07 | We don't know where your gerrit is. Please manually create a remote | 15:31 |
| kajalsah07 | named 'gerrit' and try again. | 15:31 |
| kajalsah07 | Could not connect to gerrit at ssh://kajalsah@review.opendev.org:29418/openstack/openstacksdk.git | 15:31 |
| kajalsah07 | Is it the problem of my SSH key setup or can there be any other problem? I confirmed that SSH keys are correctly setup. | 15:31 |
| JayF | cenne: check to ensure the entire git repo including the .git/ dir is owned by the user running the git command | 15:32 |
| JayF | kajalsah07: please utilize paste.openstack.org or some other paste service. I also suggest posing this question in a channel better suited to openstacksdk or opendev discussion. | 15:32 |
| dtantsur | kajalsah07: curious which distribution you use. On F33 you need to generate SSH keys of a newer type. | 15:32 |
| cenne | kajalsah07 did you add your username to gitreview.username | 15:33 |
| kajalsah07 | How do I check the distribution? | 15:33 |
| kajalsah07 | Yes I added my username | 15:33 |
| dtantsur | kajalsah07: which linux are you using? | 15:33 |
| JayF | kajalsah07: a little birdie just told me you're suppoesd to be here :D I hope I didn't make a bad first impression, if you need anything please ask | 15:33 |
| dtantsur | EXCUSE ME! OWLS ARE LARGE BIRDS!! | 15:34 |
| dtantsur | :D | 15:34 |
| rpittau | heh, mid-size :P | 15:34 |
| kajalsah07 | I'm using Ubuntu 18.04 | 15:34 |
| JayF | dtantsur: is an owlet? | 15:35 |
| dtantsur | okay, so it's not a key type probably | 15:35 |
| rpittau | kajalsah07: I suggest to use 20.04 , it's the current version supporte by our CI | 15:35 |
| dtantsur | JayF: welll.. you got me :) | 15:35 |
| cenne | dtatsur: yes it's the only commit i made. I'll try pulling master and rebasing?? | 15:36 |
| cenne | seems that might complicate things.. | 15:36 |
| JayF | cenne: check the permissions like I suggested, `ls -la * .*` in the git repo dir, make sure all the files are owned by the user you're logged in as | 15:37 |
| dtantsur | this ^^^ | 15:37 |
| cenne | ok. | 15:37 |
| kajalsah07 | Is there anything else I can try to make `git review -s` work before moving to Ubuntu 20.04? It'll take me some time to upgrade Ubuntu. | 15:38 |
| dtantsur | kajalsah07: can you try something like `ssh -v kajalsah@review.opendev.org:29418`, maybe the extended output will give some clue? | 15:38 |
| kajalsah07 | This is the output | 15:41 |
| kajalsah07 | OpenSSH_7.6p1 Ubuntu-4ubuntu0.3, OpenSSL 1.0.2n 7 Dec 2017 | 15:41 |
| kajalsah07 | debug1: Reading configuration data /home/kajalPC/.ssh/config | 15:41 |
| kajalsah07 | debug1: Reading configuration data /etc/ssh/ssh_config | 15:41 |
| kajalsah07 | debug1: /etc/ssh/ssh_config line 19: Applying options for * | 15:41 |
| kajalsah07 | ssh: Could not resolve hostname review.opendev.org:29418: Name or service not known | 15:41 |
| dtantsur | kajalsah07: sorry `ssh -v kajalsah@review.opendev.org -p 29418` | 15:42 |
| *** bburns has joined #openstack-ironic | 15:44 | |
| cenne | JayF: yes , it's all just me | 15:45 |
| cenne | i didnt change anything locally after the first commit | 15:45 |
| cenne | except the commit message | 15:46 |
| cenne | and patchset comment in the review webui | 15:46 |
| openstackgerrit | Riccardo Pittau proposed openstack/bifrost master: Use explicit path for which https://review.opendev.org/c/openstack/bifrost/+/785437 | 15:47 |
| rpittau | dtantsur: this ^ should fix the issue with bifrost centos8 | 15:47 |
| dtantsur | OMG | 15:48 |
| dtantsur | why | 15:48 |
| dtantsur | WHY | 15:48 |
| rpittau | I would love to have an answer :/ | 15:48 |
| dtantsur | cenne: hmm. this starts going outside of our expertise, could you join the #opendev channel and ask tehre? | 15:48 |
| cenne | it says missing tree something. could it be that the commit id wants the same story,topic? | 15:49 |
| dtantsur | rpittau: I wonder if we should also talk to #opendev.. or if it's our problem? | 15:49 |
| dtantsur | cenne: git has no knowledge of topics or stories | 15:50 |
| dtantsur | these are purely gerrit features, the topic is usually set from your branch name | 15:50 |
| rpittau | dtantsur: mmmm yeah, that could be indeed an issue for others, I did not went too deep into the root cause, but the alias seems to be a problem when using it in scripts | 15:51 |
| cenne | yes, i thought gerrit might be messing up. since it says remote unpack failed | 15:51 |
| dtantsur | cenne: maybe try `git gc`? | 15:52 |
| cenne | done same | 15:54 |
| rpittau | cenne: you could also try to go back to master, download your patch and switch to that using 'git review -d 785431', and work on that one | 15:54 |
| dtantsur | under "go back to master" rpittau probably means "clone the repository again" | 15:54 |
| rpittau | no, I mean, just git checkout master :) | 15:54 |
| dtantsur | I see at least one report of a similar problem https://bugs.launchpad.net/git-review/+bug/1332549 | 15:54 |
| openstack | Launchpad bug 1332549 in git-review "git review sometimes causes unpacker error for Missing tree" [Undecided,New] | 15:54 |
| dtantsur | rpittau: that is unlikely to help | 15:55 |
| cenne | what is this! | 15:55 |
| cenne | messge from @openstack? | 15:56 |
| dtantsur | @openstack is a bot | 15:56 |
| dtantsur | it handles some things, like helpfully showing the error title | 15:56 |
| dtantsur | it reacts to anyone posting a link to certain known services | 15:56 |
| JayF | wow that's a brutal bug | 15:56 |
| cenne | that oh ok.. | 15:56 |
| JayF | bad luck to get hit with that as a new contributor | 15:56 |
| dtantsur | cenne: I asked on #opendev, they're asking which version of git do you have | 15:57 |
| dtantsur | could you join that channel for further investigation? | 15:57 |
| *** rpittau is now known as rpittau|bbl | 15:57 | |
| cenne | git version 2.25.1 | 15:57 |
| *** frigo has quit IRC | 16:04 | |
| *** lucasagomes has quit IRC | 16:07 | |
| kajalsah07 | dtantsur: Output is | 16:12 |
| kajalsah07 | debug1: connect to address 104.130.246.32 port 29418: Connection timed out | 16:12 |
| kajalsah07 | debug1: Connecting to review.opendev.org [2001:4800:7819:103:be76:4eff:fe04:9229] port 29418. | 16:12 |
| kajalsah07 | debug1: connect to address 2001:4800:7819:103:be76:4eff:fe04:9229 port 29418: Network is unreachable | 16:12 |
| kajalsah07 | ssh: connect to host review.opendev.org port 29418: Network is unreachable | 16:12 |
| dtantsur | kajalsah07: can you even about review.opendev.org in a browser? if yes, can there be some sort of firewall that blocks port 29418? | 16:13 |
| *** ociuhandu has joined #openstack-ironic | 16:15 | |
| kajalsah07 | Yes, I can open them on browser. Looks like my ISP is blocking this port. I will try with another connection. | 16:15 |
| openstackgerrit | Kafilat Adeleke proposed openstack/ironic master: Aliases for a few unfortunately named state transitions https://review.opendev.org/c/openstack/ironic/+/784898 | 16:16 |
| openstackgerrit | Merged openstack/ironic master: Fix idrac-wsman BIOS factory_reset result finding https://review.opendev.org/c/openstack/ironic/+/748696 | 16:20 |
| *** ociuhandu has quit IRC | 16:21 | |
| *** samcat116 has quit IRC | 16:24 | |
| *** samcat116 has joined #openstack-ironic | 16:24 | |
| cenne | I remember reading something about an https connection too, on that page. (for when that port is blocked) | 16:30 |
| *** Dhuldev_Valekar has joined #openstack-ironic | 16:31 | |
| dtantsur | yep, there should be something | 16:32 |
| kajalsah07 | dtantsur: Is there a way to use some other port for ssh? I am not able to use the other mobile network for ssh | 16:33 |
| *** dougsz has quit IRC | 16:33 | |
| kajalsah07 | cenne: Which page? | 16:33 |
| cenne | https://docs.opendev.org/opendev/infra-manual/latest/developers.html#accessing-gerrit-over-https | 16:33 |
| cenne | this ^ | 16:33 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Fix fast track with redfish-virtual-media https://review.opendev.org/c/openstack/ironic/+/785407 | 16:34 |
| dtantsur | TheJulia: circling back to ^^^ | 16:34 |
| dtantsur | at some point I decided that it worked.. but it did not, which is very painful with a much slower CoreOS ramdisk | 16:34 |
| cenne | kajalsah07: try that. have linked to the section. "accessing-gerrit-over-https" | 16:35 |
| kajalsah07 | This works. Thanks a lot! | 16:42 |
| dtantsur | cenne: now that you've recovered your ability to use git, could you remove duplicating change-id (and the dot) from the commit message? | 16:42 |
| dtantsur | actually, you ended up with two change ids and thus two changes | 16:43 |
| *** sshnaidm has quit IRC | 16:43 | |
| dtantsur | you need to abandon one of them and update the other | 16:43 |
| cenne | oh.. | 16:43 |
| dtantsur | change ID is how gerrit understands which change a commit belongs to | 16:43 |
| *** sshnaidm has joined #openstack-ironic | 16:44 | |
| cenne | hmm. i wonder why i got another one .. I didn't put it there myself. I think it appeared after my last try. | 16:44 |
| *** kafilat has joined #openstack-ironic | 16:46 | |
| TheJulia | dtantsur: could you just be hitting the timeout? | 16:48 |
| dtantsur | TheJulia: re virtual media? no, it's purely a logic problem | 16:48 |
| dtantsur | quite the opposite: nobody has noticed this issue so far because our ramdisks are reasonably fast | 16:48 |
| dtantsur | the CoreOS one is quite slow to start, so any reboot really feels | 16:49 |
| * cenne which one to remove ... | 16:54 | |
| * cenne hate's such choices | 16:55 | |
| JayF | the one with the lower number is the *real* change, the later one is just a clone! /s | 16:55 |
| cenne | ouch! but i like clones.. they're as much real as the ~~real~~ original.. | 16:56 |
| JayF | then I think you made you choice :D | 16:56 |
| kajalsah07 | Hi, I have put a draft of my progress on opendev to review for the story https://storyboard.openstack.org/#!/story/2008193. My first patch here. Please review :) | 16:57 |
| JayF | Oh, this'll be nice! I didn't realize that was the project you all were working on. | 16:57 |
| dtantsur | kajalsah07, cenne, it's end of day for me, I'll try to get to your changes early tomorrow | 17:03 |
| JayF | What's your expected working hours kajalsah07 cenne? More europe-y or more US-y or somewhere else? | 17:05 |
| kajalsah07 | Sure. Thanks for the help today with setting up ssh/https. | 17:05 |
| *** derekh has quit IRC | 17:05 | |
| kajalsah07 | I'm from India. So it aligns with europe zone as I start generally from afternoon :) | 17:07 |
| JayF | Alright, that works :) I'm in US PST, usually work from about 1500-2400 UTC | 17:08 |
| JayF | so if you all ever need something in the mornings, just ask | 17:08 |
| *** sshnaidm is now known as sshnaidm|afk | 17:10 | |
| *** benj_ has quit IRC | 17:14 | |
| kajalsah07 | JayF: Thanks. I'll surely ask if I get stuck somewhere. | 17:21 |
| cenne | ok dtantsur: | 17:24 |
| cenne | ftr i've removed the older one. the newer one is up for review. | 17:25 |
| dtantsur | k thx! | 17:28 |
| *** dtantsur is now known as dtantsur|afk | 17:29 | |
| dtantsur|afk | talk to you tomorrow | 17:29 |
| *** Dhuldev_Valekar has quit IRC | 17:43 | |
| *** rpittau|bbl is now known as rpittau | 17:44 | |
| *** k_mouza_ has quit IRC | 17:46 | |
| *** diurnalist has quit IRC | 17:57 | |
| rpittau | mmm bifrost upgrade job may be broken because of an interesting chicken'n'egg issue | 18:07 |
| rpittau | soooo I guess we'll have to fix wallaby first and go the other way around, or even victoria.... | 18:16 |
| rpittau | let's try with wallaby first | 18:16 |
| openstackgerrit | Riccardo Pittau proposed openstack/bifrost stable/wallaby: Use explicit path for which https://review.opendev.org/c/openstack/bifrost/+/785454 | 18:18 |
| TheJulia | dtantsur|afk: w/r/t fasttrack, ack | 18:23 |
| *** benj_ has joined #openstack-ironic | 18:33 | |
| *** kafilat has quit IRC | 18:55 | |
| *** dking has joined #openstack-ironic | 19:04 | |
| openstackgerrit | Kafilat Adeleke proposed openstack/ironic master: Aliases for a few unfortunately named state transitions https://review.opendev.org/c/openstack/ironic/+/784898 | 19:06 |
| *** bburns has quit IRC | 19:14 | |
| *** bburns has joined #openstack-ironic | 19:14 | |
| *** paras333_ has quit IRC | 19:18 | |
| *** paras333 has joined #openstack-ironic | 19:19 | |
| *** paras333 has quit IRC | 19:23 | |
| openstackgerrit | Riccardo Pittau proposed openstack/bifrost stable/victoria: Use explicit path for which https://review.opendev.org/c/openstack/bifrost/+/785501 | 19:23 |
| rpittau | wallaby was not enough..... | 19:23 |
| TheJulia | the world is not enough | 19:23 |
| *** ociuhandu has joined #openstack-ironic | 19:29 | |
| mnaser | now that we've got supermicro gear under control, it's time to enter the idrac fun | 19:40 |
| mnaser | https://docs.openstack.org/ironic/ussuri/admin/drivers/idrac.html#pxe-reset-with-factory-reset-bios-clean-step -- does this unfortunately mean that if i use UEFI, i'm not able to actually do factory reset on idrac? | 19:40 |
| rpioso | mnaser: No, it does not mean you cannot perform a factory reset of BIOS settings if you are using UEFI. | 19:42 |
| mnaser | rpioso: factory reset .. then set to uefi all in one clean task? | 19:44 |
| rpioso | mnaser: Rather, it means you will lose your desired NIC PXE enabled configuration when that is performed if that configuration is different from the factory default. | 19:45 |
| rpioso | mnaser: In UEFI boot mode, BIOS settings are used to configure NIC PXE boot. | 19:46 |
| *** k_mouza has joined #openstack-ironic | 19:46 | |
| * mnaser slowly re-reads to get a grasp of that | 19:47 | |
| rpioso | mnaser: AFAIU, by default, recent generations of Dell EMC servers ship with UEFI boot mode enabled from the factory. | 19:48 |
| mnaser | yes, that's been my experience too as well, so default should bring me back to UEFI, but will need to much around the pxe interface i guess | 19:48 |
| rpioso | mnaser: Yes, you will need to muck with the PXE interface configuration if it differs from the factory default. | 19:49 |
| *** samcat116 has quit IRC | 19:51 | |
| *** k_mouza has quit IRC | 19:51 | |
| rpioso | mnaser: That *could* be done by following factory_reset with apply_configuration, but I believe a PXE boot of the IPA ramdisk is unconditionally done between clean steps, which will, well, fail :-( | 19:57 |
| rpioso | dtantsur, TheJulia: ^^^ | 19:59 |
| TheJulia | uhh | 20:00 |
| TheJulia | so dtantsur's change to allow oob only steps shouldn't try to invoke that between steps | 20:00 |
| TheJulia | at least, that is how I percieve it | 20:00 |
| rpioso | TheJulia: At least it shouldn't be mandatory :-) Optional? | 20:01 |
| TheJulia | I think it is required to express that you only want to do oob things | 20:02 |
| *** rcernin_ has joined #openstack-ironic | 20:06 | |
| *** ociuhandu has quit IRC | 20:09 | |
| rpittau | TheJulia: the fix for bifrost centos8 needs to happen in reverse order, starting with victoria stable branch, because of the upgrade job, so we need this https://review.opendev.org/c/openstack/bifrost/+/785501 first | 20:17 |
| TheJulia | rpittau: approved | 20:20 |
| rpittau | thanks! | 20:20 |
| *** akahat has quit IRC | 20:24 | |
| *** tosky_ has joined #openstack-ironic | 20:25 | |
| *** janders has quit IRC | 20:26 | |
| *** janders8 has joined #openstack-ironic | 20:26 | |
| *** rcernin_ has quit IRC | 20:26 | |
| TheJulia | rpittau: w/r/t ci job cleanup, on https://etherpad.opendev.org/p/ironic-xena-ptg line 212. Do we need to discuss grenade job changes that are expected? | 20:26 |
| *** trident has quit IRC | 20:26 | |
| *** tosky has quit IRC | 20:26 | |
| *** trident has joined #openstack-ironic | 20:28 | |
| *** rcernin_ has joined #openstack-ironic | 20:28 | |
| *** tosky_ is now known as tosky | 20:28 | |
| mnaser | rpioso: ok so my systems do default UEFI boot mode enabled, so that settles that | 20:29 |
| *** dsneddon has joined #openstack-ironic | 20:29 | |
| rpittau | TheJulia: any change to the grenade jobs I think should be discussed, yeah | 20:29 |
| rpittau | my concern is on old stable branches, if we want to keep them "alive" removing grenade is probably not enough | 20:29 |
| TheJulia | my point is they are expected to be removed from service... like train's job can likely be nuked from high orbit | 20:31 |
| TheJulia | well | 20:32 |
| TheJulia | already done on train it seems | 20:32 |
| *** rcernin_ has quit IRC | 20:32 | |
| TheJulia | Which also means we did it on train a while back, which means ussuri is soon() | 20:32 |
| rpittau | oh yeah | 20:33 |
| rpittau | ok, yes, next will be ussuri | 20:33 |
| *** akahat has joined #openstack-ironic | 20:37 | |
| rpioso | TheJulia: We could loop dtantsur|afk into this conversation about OOB things and PXE boot of IPA. I believe you understand the issue :-) | 20:39 |
| TheJulia | rpioso: I would expect he will respond tomorrow. | 20:41 |
| rpioso | TheJulia: +1 | 20:41 |
| rpioso | mnaser: Hope that helped. | 20:42 |
| iurygregory | TheJulia, not sure if you remember the chat we had 2 months ago about being able to validate if the Redfish BMC has support for vmedia... I've created and RFE based on the discussions https://storyboard.openstack.org/#!/story/2008809 not sure if is 100% correct =) | 20:43 |
| *** rcernin_ has joined #openstack-ironic | 20:43 | |
| TheJulia | iurygregory: what would this periodic task do? | 20:44 |
| *** diurnalist has joined #openstack-ironic | 20:45 | |
| mnaser | rpioso: that was helpful! but now it looks like ironic will always try to pxe boot for clean even if the clean steps doesnt actually need to do that | 20:47 |
| openstackgerrit | Merged openstack/bifrost stable/victoria: Use explicit path for which https://review.opendev.org/c/openstack/bifrost/+/785501 | 20:47 |
| mnaser | but progress | 20:47 |
| TheJulia | mnaser: what version of ironic? | 20:47 |
| mnaser | TheJulia: ussuri in this case | 20:47 |
| rpioso | mnaser: Ack | 20:48 |
| TheJulia | mnaser: yeah, the most recent work landed in wallaby | 20:48 |
| mnaser | `[{"interface": "bios", "step": "factory_reset"}]` => tries to boot | 20:48 |
| iurygregory | TheJulia, I think the idea of periodic task is that it would set node.fault (In my mind it may not be necessary, maybe it covers the case where people will change the information of the node?) | 20:49 |
| TheJulia | iurygregory: so here is my worry. We already have a ton of periodics | 20:49 |
| TheJulia | For every one we add, another worker gets consumed | 20:50 |
| TheJulia | more database queries/locking | 20:50 |
| *** rcernin_ has quit IRC | 20:50 | |
| TheJulia | So we have to be moderately careful about them | 20:50 |
| TheJulia | And need to have clearly defined what/why/when | 20:50 |
| *** rcernin has joined #openstack-ironic | 20:50 | |
| iurygregory | I was a bit worried of adding a periodic task for this hehe | 20:51 |
| iurygregory | in my mind an API call would be easier... | 20:51 |
| iurygregory | but it would be synchronous | 20:51 |
| iurygregory | to me this would probably fit like an extension of /validate (at least in my mind) | 20:52 |
| rpioso | mnaser, TheJulia: I expect even more challenging would be [{"interface": "bios", "step": "factory_reset"}, {{"interface": "bios", "step": "apply_configuration", ...}], because PXE boot of IPA would fail between the two steps if the needed bare metal system NIC PXE configuration does not the match the factory default. | 20:52 |
| mnaser | rpioso: but dont these get applied out of band directly via idrac-wsman or so? | 20:53 |
| TheJulia | mnaser: I think he is thinking current state code usage | 20:53 |
| TheJulia | and I *think* they should be | 20:54 |
| TheJulia | but... | 20:54 |
| mnaser | wouldnt they happen in 'one clean' | 20:54 |
| mnaser | unless there is a reboot post factory_reset then that's a fail =) | 20:54 |
| TheJulia | they should | 20:54 |
| mnaser | so having one clean run which does a factory reset THEN apply should be ok, but doing it as two seperate steps = probable fail | 20:55 |
| rpioso | mnaser: AFAIU, ironic cleaning has unconditionally PXE booted the IPA ramdisk before each cleaning step, even for OOB steps. | 20:55 |
| TheJulia | mnaser: I rpioso is talking about before the capability was added | 20:56 |
| mnaser | ah yes =) | 20:56 |
| TheJulia | err, I think | 20:56 |
| mnaser | so if thats the case i would need to do a clean factory reset, then fix via vendor interface | 20:56 |
| mnaser | for me, for now | 20:56 |
| TheJulia | yes | 20:56 |
| * rpioso hasn't been closely following the change dtantsur|afk has been working/worked on, although he's keenly interested in it :) | 20:57 | |
| rpioso | mnaser: That would work :) | 20:57 |
| mnaser | and then set the actual values i want with a followup clean_step i guess, or just keep using vendor interface for the whole thing | 20:57 |
| rpioso | mnaser: +1 | 20:59 |
| TheJulia | iurygregory: how long would it be for ironic to find a value be acceptable? | 21:00 |
| TheJulia | iurygregory: and you can't say instantly | 21:00 |
| *** samcat116 has joined #openstack-ironic | 21:01 | |
| iurygregory | I can run the script I was using to access redfish bmc's and provide some info | 21:01 |
| TheJulia | iurygregory: so... I'd prefer an information gathering periodic and maybe we adapt power sync | 21:03 |
| TheJulia | https://etherpad.opendev.org/p/ironic-xena-ptg-database-activity has some ideas of maybe taking a similar approach for drivers | 21:03 |
| TheJulia | like a master periodic for driver that would then know the conditions in which methods had to be called | 21:04 |
| openstackgerrit | Richard G. Pioso proposed openstack/ironic stable/wallaby: Fix idrac-wsman BIOS factory_reset result finding https://review.opendev.org/c/openstack/ironic/+/785513 | 21:06 |
| *** ociuhandu has joined #openstack-ironic | 21:07 | |
| rpioso | TheJulia: How far back should I back port ^^^? | 21:08 |
| TheJulia | rpioso: train would be nice :) | 21:09 |
| iurygregory | TheJulia, it took 23s... | 21:09 |
| TheJulia | iurygregory: okay, but how fast do we demand the data to be in palce after the record is created in ironic? | 21:09 |
| * TheJulia needs to go to the autoparts store... | 21:10 | |
| rpioso | TheJulia: Nevermind ... It can only be backported to stable/ussuri, which is when the WS-Man BIOS interface was introduced. | 21:10 |
| iurygregory | well the use case in metal3 is that we want to find asap if the BMC has the support before trying to continue the deployment | 21:10 |
| TheJulia | rpioso: ok | 21:11 |
| iurygregory | they had a more complicated wish, check if FW version of the node is also compatible with virtual media... | 21:11 |
| TheJulia | so why not launch data collection after create? | 21:11 |
| *** ociuhandu has quit IRC | 21:12 | |
| TheJulia | iurygregory: this is sounding like a ptg topic... could we do an apac friendly time for the discussion? | 21:12 |
| iurygregory | totally | 21:12 |
| iurygregory | I will on the PTG till thursday | 21:12 |
| iurygregory | since they moved my flight to 23... | 21:12 |
| iurygregory | rescheduled | 21:13 |
| TheJulia | iurygregory: part of me is thinking the outcome of the discussion on database activity and periodics might guide us | 21:13 |
| iurygregory | ++ | 21:13 |
| iurygregory | Im looking at the etherpad | 21:13 |
| TheJulia | okay | 21:13 |
| TheJulia | I've got time windows posted and estimated times. APAC stuffs are alrgely open for feedback/follow-up | 21:14 |
| TheJulia | I'm going to run into town, I'll be back later | 21:15 |
| iurygregory | Monday - 00:00 UTC (Really, Tuesday UTC) would make sense I think | 21:17 |
| rpittau | goodnight all! o/ | 21:17 |
| *** rpittau is now known as rpittau|afk | 21:17 | |
| TheJulia | iurygregory: ack | 21:18 |
| *** sshnaidm|afk is now known as sshnaidm|off | 21:24 | |
| *** lmcgann has quit IRC | 21:30 | |
| *** rcernin has quit IRC | 21:31 | |
| *** rcernin has joined #openstack-ironic | 21:56 | |
| *** rcernin has quit IRC | 22:02 | |
| *** diurnalist has quit IRC | 22:08 | |
| *** samcat116 has quit IRC | 22:09 | |
| *** rcernin has joined #openstack-ironic | 22:11 | |
| *** k_mouza has joined #openstack-ironic | 23:02 | |
| *** k_mouza has quit IRC | 23:07 | |
| *** tosky has quit IRC | 23:38 | |
| *** Goneri has quit IRC | 23:59 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!