| *** ricolin_ is now known as ricolin | 05:05 | |
| cenne | JayF re: git history. thanks that's a neat trick. 😀 | 05:58 |
|---|---|---|
| cenne | Good morning ironic! | 05:58 |
| iurygregory | good morning cenne and Ironic o/ | 06:18 |
| cenne | Hey iurygregory o/ | 06:19 |
| opendevreview | kamlesh chauvhan proposed openstack/ironic master: Upgrade oslo.db version https://review.opendev.org/c/openstack/ironic/+/796811 | 06:29 |
| arne_wiebalck | Good morning cenne iurygregory and Ironic! | 06:34 |
| iurygregory | hey arne_wiebalck o/ | 06:39 |
| cenne | good morning arne_wiebalck | 07:01 |
| *** rpittau|afk is now known as rpittau | 07:06 | |
| rpittau | good morning ironic! o/ | 07:06 |
| iurygregory | hey rpittau o/ | 07:06 |
| rpittau | janders: re cleaning enhancements update, thank you! | 07:06 |
| rpittau | hey iurygregory :) | 07:06 |
| Noam | hi | 07:11 |
| Noam | is there some why for ironic baremetal nodes to have multiple vif's? | 07:11 |
| Noam | there is the default one used for pxe/provsioning | 07:12 |
| Noam | is there a way to add another one lets say for external network | 07:12 |
| cenne | Yay. Tests running green. | 07:31 |
| cenne | The | 07:31 |
| cenne | TheJulia: , JayF: Thanks. indeit was the fake objects thingy. Had to | 07:33 |
| cenne | TheJulia: , JayF: Thanks! Found it. It was the fake objects thingy after all. Had to modify unit test's db/utils and add the two feilds in the fake node. That's the one that is saved to the db and then read back. | 07:38 |
| cenne | I was following an earlier commit but thought i could do with not with modifying any test files, to first see if at least old tests work. But looks like modifying this file is required. Thanks for your help! | 07:43 |
| cenne | Thanks everyone 😊 | 07:48 |
| iurygregory | Noam, when creating a node you can have multiple ports and each port can be assigned to a vif | 08:01 |
| iurygregory | https://docs.openstack.org/api-ref/baremetal/#ports-ports | 08:01 |
| iurygregory | https://docs.openstack.org/api-ref/baremetal/#vifs-virtual-interfaces-of-nodes | 08:01 |
| Noam | i have trouble assigned the the baremetal port to vif, do you have example commands ? | 08:05 |
| iurygregory | https://docs.openstack.org/python-ironicclient/latest/cli/osc/v1/index.html#baremetal-node-vif-attach | 08:07 |
| * dtantsur rubs eyes | 08:08 | |
| rpittau | good morning dtantsur :) | 08:08 |
| iurygregory | I think he is not awake yet... | 08:08 |
| iurygregory | =) | 08:08 |
| iurygregory | cenne, congrats on getting the tests green \o/ | 08:08 |
| Noam | http://paste.openstack.org/show/806854/ | 08:09 |
| dtantsur | thank you for helping cenne while I was out! | 08:09 |
| Noam | i have created the following baremetal port 9c1b7834-c5f1-4c66-b5eb-5fe03c583cba | 08:09 |
| dtantsur | now, may I pretend I forgot how to open slack? | 08:09 |
| iurygregory | dtantsur, totally fair =) | 08:09 |
| cenne | good morning dtantsur ! | 08:11 |
| dtantsur | o/ | 08:11 |
| iurygregory | Noam, I don't remember if we can update ports directly.. it will depend on the state of the node "All Ports must be associated to a Node when created. This association can be changed, though the request may be rejected if either the current or destination Node are in a transitive state (e.g., in the process of deploying) or are in a state that would be non-deterministically affected by such a change (e.g., there | 08:11 |
| iurygregory | is an active user instance on the Node)." | 08:11 |
| dtantsur | it will take me some time to go through all emails, slack messages and pings, so if you have anything urgent for me, please let me know | 08:12 |
| janders | good morning cenne iurygregory arne_wiebalck rpittau Noam dtantsur and Ironic o/ | 08:15 |
| iurygregory | hey janders o/ | 08:15 |
| janders | (hope I haven't missed anyone :) ) | 08:15 |
| Noam | iurygrego baremetal deployed with server create command and is running | 08:16 |
| cenne | good morning janders o/ | 08:17 |
| opendevreview | cenne proposed openstack/ironic master: [WIP] Add `boot_mode` and `secure_boot` to node object https://review.opendev.org/c/openstack/ironic/+/797055 | 08:35 |
| Noam | iurygregory Unable to attach VIF a2789a63-e77f-4125-8ff9-58cc73fe73a7, not enough free physical ports. (HTTP 400) | 08:52 |
| Noam | http://paste.openstack.org/show/806855/ | 08:52 |
| arne_wiebalck | dtantsur: If you have a moment, I may need to pick your brain for an TLS related issue I see: http://paste.openstack.org/show/806832/ (the attempt to get the clean steps from a node fail, only to succeed on the retry). I can reproduce this every time. | 09:19 |
| dtantsur | sure, although I don't have any ideas other than clock skew | 09:19 |
| arne_wiebalck | the clock skew is the same on the retry, no? | 09:20 |
| arne_wiebalck | (the retry happens in the same second) | 09:21 |
| dtantsur | hmm, I assumed a retry happens later | 09:21 |
| dtantsur | then you need to somehow (?) hack agent_client to output more details about the error | 09:21 |
| arne_wiebalck | I have the logs from the IPA side if that helps? | 09:22 |
| dtantsur | I doubt so. Does it have the exact cause of the certificate failure? | 09:22 |
| arne_wiebalck | ssl.SSLError: [SSL: TLSV1_ALERT_UNKNOWN_CA] tlsv1 alert unknown ca | 09:23 |
| arne_wiebalck | it seems like the IPA is not happy with its self-signed cert, but is ok with it on a retry | 09:26 |
| dtantsur | this.. should be impossible? | 09:29 |
| * dtantsur is lost | 09:29 | |
| iurygregory | I will tell "computers were a mistake" | 09:30 |
| dtantsur | ++ | 09:30 |
| dtantsur | my only assumption would be that ironic tries to access IPA before it stored the certificate information | 09:30 |
| dtantsur | which should be impossible, because this is also when we store the IPA endpoint URL | 09:31 |
| arne_wiebalck | I was thinking this as well, so I delayed the initial request by 5 seconds. | 09:32 |
| arne_wiebalck | What happens is that the request happens 5 seconds later, fails, and the retry works. | 09:32 |
| dtantsur | Oo | 09:33 |
| dtantsur | oO | 09:33 |
| iurygregory | O.o | 09:33 |
| iurygregory | WOW | 09:33 |
| dtantsur | I can only suggest inserting logging everywhere and trying to trace the problem | 09:33 |
| arne_wiebalck | ok, will do and let you know :) | 09:33 |
| arne_wiebalck | thanks! | 09:33 |
| opendevreview | Aija Jauntēva proposed openstack/ironic master: Redfish: Skip non-RAID controllers for storage https://review.opendev.org/c/openstack/ironic/+/796592 | 09:44 |
| opendevreview | Aija Jauntēva proposed openstack/ironic master: Fix Redfish RAID interface_type physical disk hint https://review.opendev.org/c/openstack/ironic/+/795505 | 10:37 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy master: Protect Connector against empty auth object https://review.opendev.org/c/openstack/sushy/+/797435 | 11:52 |
| dtantsur | this is a confusing bug, I hope I fixed it correctly ^^ | 11:52 |
| janders | see you tomorrow Ironic o/ | 12:05 |
| *** osmanlicilegi is now known as Guest187 | 12:25 | |
| *** ChanServ changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-discuss mailing list." | 12:29 | |
| rpittau | dtantsur, TheJulia, should we ask for a new ironic release? | 13:11 |
| TheJulia | for 18.0.1? | 13:16 |
| TheJulia | wallaby? | 13:17 |
| TheJulia | both | 13:17 |
| dtantsur | if wellaby, can we merge https://review.opendev.org/c/openstack/ironic/+/796638 please? | 13:20 |
| dtantsur | or bugfix? | 13:20 |
| TheJulia | https://review.opendev.org/c/openstack/ironic/+/797212 also needs to be merged | 13:21 |
| dtantsur | also good morning TheJulia | 13:21 |
| TheJulia | good morning! | 13:21 |
| dtantsur | I can check if after the current meeting | 13:22 |
| iurygregory | looking now | 13:22 |
| dtantsur | even better :) | 13:22 |
| TheJulia | awesome | 13:23 |
| TheJulia | Anyone seen random post_failures like https://zuul.opendev.org/t/openstack/build/d3159791bcca4ac89aceddbec6c43611 | 13:25 |
| rpittau | wallaby + master, I wanted to double-check if we needed to merge something first | 13:26 |
| dtantsur | s/master/bugfix/ | 13:26 |
| dtantsur | we release master 3x a cycle | 13:26 |
| iurygregory | TheJulia, looking at the build history something like that happened in stable/victoria... | 13:26 |
| dtantsur | (I may be nitpicking, but would like to avoid confusion) | 13:26 |
| TheJulia | dtantsur: +1 on bugfix | 13:27 |
| rpittau | mmm 18.0.1 should be master ? | 13:27 |
| TheJulia | no | 13:27 |
| TheJulia | bugfix/18.0 | 13:27 |
| dtantsur | rpittau: no, bugfix/18.0 | 13:27 |
| dtantsur | right :) | 13:27 |
| dtantsur | I think releases from bugfix branches are still done by the release team manually | 13:27 |
| TheJulia | iurygregory: Yeah, I've seen a couple random ones and it just doesn't add up to me | 13:27 |
| iurygregory | yeah, wondering if is related to a cloud provider or something... | 13:28 |
| * iurygregory checks the cloud in the post failures | 13:28 | |
| iurygregory | interesting that both from ironic-standalone are in ovh | 13:29 |
| iurygregory | region BHS1 | 13:29 |
| rpittau | argh yeah I totally confused myself with the versions | 13:31 |
| rpittau | alright let's wait for the patches to merge | 13:31 |
| rpittau | TheJulia: that error comes from https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/fetch-subunit-output/tasks/main.yaml#L66 | 13:37 |
| TheJulia | iurygregory: I had another one fail yesterday for no reason, no errors anywhere in the logs. Kind of frustrating :( | 13:37 |
| TheJulia | rpittau: good find! | 13:37 |
| rpittau | maybe a bug there, not sure | 13:38 |
| TheJulia | not sure, but I'm of the opinion that post_failures shouldn't be held against CI jobs | 13:39 |
| rpittau | heh not really no | 13:39 |
| iurygregory | TheJulia, do you have the link handy for the other failure you saw? | 13:42 |
| TheJulia | Not off hand | 13:42 |
| * TheJulia looks | 13:42 | |
| iurygregory | https://review.opendev.org/c/openstack/ironic/+/792275 ? | 13:43 |
| iurygregory | this one ? | 13:43 |
| TheJulia | yes | 13:43 |
| TheJulia | well, one with the subunit entry | 13:43 |
| iurygregory | yeah same error but different cloud provider | 13:43 |
| iurygregory | not sure if we should say yay or cry... | 13:44 |
| rpittau | the problem is the stestr output, or the lack of it | 13:45 |
| rpittau | her https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/fetch-subunit-output/tasks/main.yaml#L68 it takes as assumed that the stestr command is in the first line | 13:47 |
| rpittau | but in that case it seems it's in the second line | 13:47 |
| rpittau | so the result is " last --subunit" which fails | 13:48 |
| iurygregory | maybe a different version is installed and something changed? | 13:48 |
| TheJulia | that could be... | 13:49 |
| rpittau | maybe | 13:49 |
| TheJulia | 3.2.0 released april 21st, 2021 | 13:49 |
| rpittau | the error could be in the script used to determine the stestr executable path https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/fetch-subunit-output/files/find-testr.sh | 13:52 |
| TheJulia | rpittau: raising in #opendev | 13:53 |
| rpittau | ack | 13:53 |
| iurygregory | I will skip some meetings now I have headache, going to take some medicine and relax a little bit (I hope to be back later) | 13:57 |
| dtantsur | ugh, I hope you get better soon! | 13:57 |
| iurygregory | tks | 13:58 |
| rpittau | iurygregory: take care! | 14:02 |
| arne_wiebalck | iurygregory: ouch, have a good rest! | 14:06 |
| ajya | get well soon, iurygregory | 14:13 |
| ajya | What's with LOST CI results? I have started seeing them lately, don't recall having them before. | 14:13 |
| TheJulia | They have occassionally happened int he past and it is typically connectivity | 14:14 |
| ajya | ok, recheck it is, but seeing this lately a bit too often | 14:23 |
| TheJulia | ajya: make note of it and if we can find a few different cases, hopefully the opendev folks can figure out what is going on | 14:24 |
| ajya | ok | 14:26 |
| ygk_12345 | hi all | 14:36 |
| ygk_12345 | can anyone join the to good resources on ironic networking for the bare metals and if ironic supports tenant or provider networks ? | 14:37 |
| * TheJulia sighs | 14:48 | |
| opendevreview | Merged openstack/ironic master: Rename ilo_boot_iso -> boot_iso https://review.opendev.org/c/openstack/ironic/+/796289 | 14:49 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP Scoped RBAC Devstack Plugin support https://review.opendev.org/c/openstack/ironic/+/778957 | 14:51 |
| opendevreview | Merged openstack/ironic stable/wallaby: Handle non-key-value params in [inspector]extra_kernel_params https://review.opendev.org/c/openstack/ironic/+/796638 | 15:01 |
| opendevreview | Merged openstack/ironic stable/wallaby: Fix node detail instance_uuid request handling https://review.opendev.org/c/openstack/ironic/+/797212 | 15:01 |
| dtantsur | rpittau: ready ^^ | 15:07 |
| TheJulia | \o/ | 15:10 |
| rpittau | great | 15:18 |
| rpittau | do we want to release 18.0.1 too or we leave that to the release team ? | 15:25 |
| rpittau | wallaby -> https://review.opendev.org/c/openstack/releases/+/797499 | 15:25 |
| *** ajya is now known as ajya|afk | 15:27 | |
| dtantsur | rpittau: we need to explicitly ask them, releases from bugfix branches could not be handled automatically the last time I checked | 15:30 |
| JayF | eiddccidrhjirdldvbgjuilucuruivgrcjlkltvkltnd | 15:32 |
| JayF | Whoops, sorry | 15:32 |
| cenne | hehe. | 15:32 |
| JayF | Assuming it's not a backportable change to get configdrive and docs for anaconda into wallaby? | 15:32 |
| JayF | Just double-checking before I do the backport work for downstream | 15:32 |
| opendevreview | Merged openstack/ironic stable/victoria: Fix node detail instance_uuid request handling https://review.opendev.org/c/openstack/ironic/+/797253 | 15:33 |
| JayF | nevermind, I just learned configdrive for anaconda hadn't merged yet; I'll have to get that in :| | 15:34 |
| rpittau | dtantsur: gotcha | 15:36 |
| dtantsur | JayF: docs are always backportable if they apply to the corresponding branch | 15:36 |
| JayF | yeah so I'll backport that, configdrive will have to get done | 15:36 |
| JayF | ty | 15:36 |
| opendevreview | Jay Faulkner proposed openstack/ironic stable/wallaby: Add documentation for anaconda deploy interface https://review.opendev.org/c/openstack/ironic/+/797482 | 15:38 |
| JayF | TheJulia: thanks, that was quick! | 16:14 |
| TheJulia | doc :) | 16:15 |
| opendevreview | Merged openstack/ironic stable/wallaby: Add documentation for anaconda deploy interface https://review.opendev.org/c/openstack/ironic/+/797482 | 16:24 |
| rpittau | dtantsur, TheJulia, talked on the release channel and apparently we never did a release from a bugfix branch | 16:25 |
| TheJulia | Ummm... I'm fairly sure we have | 16:25 |
| TheJulia | but, I'm feeling under the weather at the moment | 16:25 |
| TheJulia | hmmm | 16:28 |
| TheJulia | 17.0.2 looks like it was off the branch. at least looking at the gitk display | 16:30 |
| dtantsur | we definitely have | 16:30 |
| TheJulia | same with 17.0.1 | 16:30 |
| TheJulia | yeah, we have. | 16:30 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Allow ramdisk_image_download_source in instance_info for ramdisk deploy https://review.opendev.org/c/openstack/ironic/+/797508 | 16:51 |
| dtantsur | rpittau: FYI ^^^ | 16:51 |
| rpittau | dtantsur: ack | 16:52 |
| TheJulia | Do people want to do a review jam today? I'm feeling really low energy | 16:58 |
| dtantsur | same | 16:58 |
| JayF | I'm OK to skip if everyone else is skipping it | 16:59 |
| TheJulia | I'm low energy enough that making some tempest progress is all that is keeping me from going and crawling up in bed | 16:59 |
| rpittau | I was having a look at the patches, maybe we don't have to do a review jam for them, just need to assess some comments | 16:59 |
| TheJulia | if anyone has questions for me, just ping me | 16:59 |
| rpittau | ok to skip for me | 16:59 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Nicer error message when a deploy step fails https://review.opendev.org/c/openstack/ironic/+/797512 | 17:00 |
| rpittau | good night! o/ | 17:11 |
| *** rpittau is now known as rpittau|afk | 17:11 | |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Fix ramdisk boot option handling https://review.opendev.org/c/openstack/ironic/+/797517 | 17:12 |
| dtantsur | see you tomorrow folks | 17:15 |
| cenne | good night | 17:19 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: Secure RBAC support https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/797521 | 17:28 |
| arne_wiebalck | bye everyone o/ | 17:37 |
| opendevreview | cenne proposed openstack/ironic master: [WIP] Add `boot_mode` and `secure_boot` to node object and expose in api https://review.opendev.org/c/openstack/ironic/+/797055 | 17:42 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: Secure RBAC support https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/797521 | 17:47 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP Scoped RBAC Devstack Plugin support https://review.opendev.org/c/openstack/ironic/+/778957 | 17:56 |
| * TheJulia crosses her fingers | 17:56 | |
| *** iurygregory_ is now known as iurygregory | 18:39 | |
| TheJulia | gmann: do you happen to be around? | 21:12 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!