| *** k_mouza has joined #openstack-ironic | 00:09 | |
| *** k_mouza has quit IRC | 00:14 | |
| *** jamesden_ has quit IRC | 00:23 | |
| *** jamesdenton has joined #openstack-ironic | 00:25 | |
| *** chihhsin has joined #openstack-ironic | 00:35 | |
| *** jamesden_ has joined #openstack-ironic | 00:46 | |
| *** jamesdenton has quit IRC | 00:47 | |
| *** tkajinam has quit IRC | 00:59 | |
| *** tkajinam has joined #openstack-ironic | 00:59 | |
| *** gyee has quit IRC | 01:32 | |
| *** yolanda has quit IRC | 02:07 | |
| *** rcernin has quit IRC | 02:09 | |
| *** ijw has quit IRC | 02:17 | |
| *** rcernin has joined #openstack-ironic | 02:18 | |
| *** ijw has joined #openstack-ironic | 02:18 | |
| *** rcernin has quit IRC | 02:18 | |
| *** rcernin has joined #openstack-ironic | 02:19 | |
| *** cdearborn has quit IRC | 02:58 | |
| *** Qianbiao has joined #openstack-ironic | 03:02 | |
| *** zzzeek has quit IRC | 03:17 | |
| *** zzzeek has joined #openstack-ironic | 03:18 | |
| janders | does anyone happen to have working examples of enrolling redfish kit into Ironic? https://docs.openstack.org/ironic/latest/admin/drivers/redfish.html doesn't seem to work: http://paste.openstack.org/show/797445/ and doing it through bifrost playbooks has it's own challenges if the OOB controller is running self-signed SSL... | 03:23 |
|---|---|---|
| janders | to elaborate bit more on the challenges of the playbook approach: enroll playbook doesn't like 'redfish_verify_ca': False (refuses to take the inventory with this included) and I suspect that retro-fitting this option after the fact triggers this issue: http://paste.openstack.org/show/797443/ | 03:26 |
| janders | it's probably something simple but I'm pretty stuck so any pointers would be greatly appreciated :) | 03:26 |
| openstackgerrit | Steve Baker proposed openstack/metalsmith master: metalsmith_deployment role switch to metalsmith_instances https://review.opendev.org/737888 | 03:34 |
| *** rloo has quit IRC | 03:42 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Add missing log for clean failed https://review.opendev.org/749861 | 03:43 |
| *** JamesBenson has quit IRC | 03:48 | |
| openstackgerrit | Christopher Dearborn proposed openstack/sushy master: Allow monitoring progress of a firmware update https://review.opendev.org/745950 | 04:04 |
| openstackgerrit | Christopher Dearborn proposed openstack/ironic master: WIP: Redfish driver firmware update https://review.opendev.org/749619 | 04:04 |
| *** Nisha_Agarwal has joined #openstack-ironic | 04:39 | |
| *** hjensas has joined #openstack-ironic | 05:32 | |
| openstackgerrit | paresh sao proposed openstack/ironic master: OOB one button secure erase for iLO5 based HPE Proliant servers. https://review.opendev.org/743868 | 05:32 |
| arne_wiebalck | Good morning, ironic! | 05:45 |
| arne_wiebalck | janders: for redfish, you mean how to enroll the nodes or what to configure on the controllers? | 05:46 |
| arne_wiebalck | janders: I have done this recently with a first node :) | 05:46 |
| arne_wiebalck | janders: All manually for now, though. But I (hopefully) have some notes on what I did. | 05:47 |
| *** tzumainn has quit IRC | 05:54 | |
| Qianbiao | janders maybe it's due to cmd client version？ | 06:22 |
| Qianbiao | it seems that error is thrown by openstack client, but not conductor | 06:23 |
| Qianbiao | morning, arne_wiebalck | 06:23 |
| arne_wiebalck | Hey Qianbiao o/ | 06:28 |
| arne_wiebalck | janders: sorry, had missed that you pasted the command and the error | 06:28 |
| Qianbiao | Hey o/ | 06:28 |
| *** belmoreira has joined #openstack-ironic | 06:29 | |
| arne_wiebalck | janders: at a first glance, that looks good ... how about omitting the driver_info completely (and try to add it later)? | 06:29 |
| Qianbiao | arne_wiebalck janders i thinks that is openstack client bug. | 06:31 |
| arne_wiebalck | Qianbiao: yeah, looks like it | 06:31 |
| Qianbiao | you may add some log on your local /opt/stack/bifrost/lib64/python3.6/site-packages/openstack/config/loader.py file | 06:31 |
| *** Nisha_Agarwal has quit IRC | 06:36 | |
| *** dsneddon has quit IRC | 06:38 | |
| Qianbiao | arne_wiebalck are u familiar with ironic bm node deploy with nova? Not sure why it did not auto switch network when deploying. | 06:44 |
| Qianbiao | when clean bm, the network is correct, dhcp happens on provisiong network. But when deploy it bind bm to target network(tenant network), not provisiong network. | 06:45 |
| arne_wiebalck | Qianbiao: yes, we deploy with nova ... however: we use a single network (and the whole network part in nova is replaced with a big noop), so I don't think I am able to help :-/ | 06:55 |
| arne_wiebalck | Qianbiao: and on top of this we do not use Ironic's pxe infra but have a downstream patch to connect to our local infra | 06:55 |
| arne_wiebalck | Qianbiao: so, network-wise we are non-standard | 06:56 |
| Qianbiao | arne_wiebalck aha, ok | 06:56 |
| *** jtomasek has joined #openstack-ironic | 06:58 | |
| iurygregory | good morning Ironic, Happy Friday! | 07:06 |
| openstackgerrit | likui proposed openstack/ironic-python-agent master: Remove six https://review.opendev.org/749877 | 07:06 |
| Qianbiao | hi iurygregory | 07:22 |
| *** tosky has joined #openstack-ironic | 07:25 | |
| openstackgerrit | likui proposed openstack/ironic-python-agent master: Remove six/babel/oslo.i18n from lower-constraints.txt https://review.opendev.org/749877 | 07:25 |
| *** mgoddard has joined #openstack-ironic | 07:38 | |
| *** dougsz has joined #openstack-ironic | 07:39 | |
| *** rcernin_ has joined #openstack-ironic | 07:42 | |
| *** rcernin has quit IRC | 07:44 | |
| *** zzzeek has quit IRC | 07:45 | |
| *** zzzeek has joined #openstack-ironic | 07:47 | |
| rpittau | good morning ironic! o/ | 07:48 |
| iurygregory | hey rpittau o/ | 07:51 |
| rpittau | hey iurygregory :) | 07:51 |
| iurygregory | rpittau, the odata.id in redifh will always be "/redfish/v1/....." ? | 07:55 |
| rpittau | iurygregory: normally, yes | 07:55 |
| iurygregory | tks /me blames airship lib to access redfish now | 07:55 |
| *** rcernin_ has quit IRC | 07:56 | |
| openstackgerrit | Arun S A G proposed openstack/ironic-specs master: Add support for anaconda deploy interface https://review.opendev.org/748503 | 07:59 |
| *** lucasagomes has joined #openstack-ironic | 08:06 | |
| openstackgerrit | Aija Jaunteva proposed openstack/sushy master: Add BIOS update apply time and maintenance window https://review.opendev.org/749236 | 08:22 |
| *** dtantsur|afk is now known as dtantsur | 08:24 | |
| dtantsur | morning ironic | 08:26 |
| dtantsur | janders: weird, I'll take a look | 08:26 |
| iurygregory | good morning dtantsur | 08:26 |
| rpittau | hey dtantsur :) | 08:26 |
| *** derekh has joined #openstack-ironic | 08:27 | |
| dtantsur | janders: it's a bug in openstacksdk :( | 08:33 |
| *** dsneddon has joined #openstack-ironic | 08:38 | |
| dtantsur | janders: try https://review.opendev.org/#/c/749903/ | 08:43 |
| patchbot | patch 749903 - openstacksdk - Fix a bogus error in config loader when using seve... - 1 patch set | 08:43 |
| dtantsur | folks, let's pile on sushy reviews, we're in a pressing need of a final release | 08:50 |
| dtantsur | please pay particular attention to features that are ready to go | 08:50 |
| openstackgerrit | Merged openstack/ironic master: Enhance certificate verification for ilo harware type https://review.opendev.org/743490 | 08:51 |
| dtantsur | realistically, it's probably only https://review.opendev.org/#/c/749236/ | 08:54 |
| patchbot | patch 749236 - sushy - Add BIOS update apply time and maintenance window - 2 patch sets | 08:54 |
| iurygregory | yeah | 08:54 |
| dtantsur | I don't think I can grok the TaskMonitor patch so quickly | 08:54 |
| iurygregory | I'm wondering about the https://review.opendev.org/#/c/748927/ | 08:57 |
| patchbot | patch 748927 - sushy - Make message parsing more resilient - 3 patch sets | 08:57 |
| dtantsur | seems contentious? | 08:57 |
| dtantsur | it's a bug fix, so can be backported when finished | 08:58 |
| iurygregory | yeah | 08:58 |
| openstackgerrit | Aija Jaunteva proposed openstack/sushy master: Add BIOS update apply time and maintenance window https://review.opendev.org/749236 | 08:59 |
| openstackgerrit | Merged openstack/bifrost stable/ussuri: Create our own firewalld zone and use it on real bare metal https://review.opendev.org/749760 | 09:02 |
| openstackgerrit | Merged openstack/bifrost bugfix/8.3: Create our own firewalld zone and use it on real bare metal https://review.opendev.org/749759 | 09:08 |
| iurygregory | dtantsur, that problem with 'HTTPBasicAuth' object has no attribute 'get_auth_ref' your patch is merged? | 09:15 |
| dtantsur | nope | 09:15 |
| iurygregory | sad =( | 09:15 |
| iurygregory | about metalsmith I saw stevebaker said there is no problem if we can't merge his patch in the release, so we can move on and release metalsmith? | 09:17 |
| dtantsur | iurygregory: yeah. the releases have been proposed by the release team, so no actions needed from us | 09:20 |
| *** dougsz has quit IRC | 09:22 | |
| iurygregory | dtantsur, oh ok! | 09:23 |
| *** aedc has joined #openstack-ironic | 09:23 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Do not assume that prepare_image is the last command to run https://review.opendev.org/749912 | 09:34 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/ussuri: Do not assume that prepare_image is the last command to run https://review.opendev.org/749913 | 09:38 |
| *** dougsz has joined #openstack-ironic | 09:38 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/ussuri: DNM test ussuri CI https://review.opendev.org/749914 | 09:39 |
| openstackgerrit | Yogesh Ramachandra proposed openstack/ironic master: [WIP] Fix idrac-wsman factory-reset getting updated bios setting values https://review.opendev.org/748696 | 09:39 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add an option to require TLS for agent callback_url https://review.opendev.org/749916 | 10:11 |
| *** Lucas_Gray has joined #openstack-ironic | 10:12 | |
| *** dougsz has quit IRC | 10:14 | |
| *** dougsz has joined #openstack-ironic | 10:30 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 10:33 | |
| Qianbiao | Hello, to provision node through provisioning network, should the node's network interface must be "neutron"? | 10:47 |
| Qianbiao | Current, i am using flat, the node can be cleaned through provisioning network(clean network), but when deploy, it can not load dhcp ip. | 10:48 |
| Nisha_Agarwal | Qianbiao, u can use ur own dhcp server also | 10:55 |
| Nisha_Agarwal | and dhcp_controller in the config file as "None" | 10:55 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Accept and use a TLS certificate from the agent https://review.opendev.org/747136 | 10:55 |
| Nisha_Agarwal | Qianbiao, set the dhcp_controller in the config file as None | 10:56 |
| dtantsur | Qianbiao: to have a separate provisioning network, you need to use the "neutron" network_interface | 10:57 |
| dtantsur | otherwise provisioning happens on the same flat tenant network | 10:57 |
| Qianbiao | <Nisha_Agarwal> i have thought that way. use a standalone dhcp server. | 10:59 |
| Qianbiao | But when we have a big number nodes, it is not easy to maintence | 10:59 |
| janders | arne_wiebalck Qianbiao dtantsur thanks heaps for looking into my issue | 10:59 |
| Qianbiao | dtantsur ok, i see, will try. | 10:59 |
| janders | dtantsur wow you wrote a fix already - legend! will test it now | 11:00 |
| Nisha_Agarwal | Qianbiao, ok. Then you can do as dtantsur said above ^^^^ | 11:00 |
| Nisha_Agarwal | dtantsur, please see this patch https://review.opendev.org/#/c/746416/ when u have time | 11:01 |
| patchbot | patch 746416 - ironic - Adds few of the security dashboard parameters to c... - 12 patch sets | 11:01 |
| *** aedc has quit IRC | 11:04 | |
| *** derekh has quit IRC | 11:05 | |
| *** derekh has joined #openstack-ironic | 11:07 | |
| openstackgerrit | Merged openstack/ironic master: Trivial: fix minor typo on RFC number https://review.opendev.org/749754 | 11:18 |
| *** zzzeek has quit IRC | 11:20 | |
| *** zzzeek has joined #openstack-ironic | 11:21 | |
| Qianbiao | Hi dtantsur, after set node network interface to neutron, I can not add node port manually, last error: http://paste.openstack.org/show/797459/ | 11:22 |
| janders | dtantsur with https://review.opendev.org/#/c/749903/ the node registers correctly (and I had one more issue with my syntax on top of hitting the bug) | 11:22 |
| patchbot | patch 749903 - openstacksdk - Fix a bogus error in config loader when using seve... - 1 patch set | 11:22 |
| Qianbiao | did I miss something, like should add extra info? | 11:22 |
| janders | however I'm hitting futher problems unfortunately, will paste the logs shortly (essentially the node gets stuck in 'enroll') | 11:23 |
| janders | CLI output: http://paste.openstack.org/show/797458/ | 11:23 |
| janders | logs: http://paste.openstack.org/show/797457/ | 11:23 |
| janders | I suspect Ironic can't talk to the redfish interface of the node for some reason but I can't see why yet - I can use these creds and get into the iLO | 11:24 |
| Qianbiao | <janders> | 11:25 |
| Qianbiao | to go on, you should update node status | 11:25 |
| Qianbiao | like "openstack node manage node-name" | 11:25 |
| janders | Qianbiao I tried that, no effect unfortunately | 11:27 |
| janders | http://paste.openstack.org/show/797460/ | 11:27 |
| janders | my first guess would be an issue with redfish details, but I can assemble a working curl call out of what I defined the node with... | 11:27 |
| Qianbiao | the power status is None | 11:28 |
| Qianbiao | it means the redfish driver did not connect to your server maybe? | 11:28 |
| janders | yeah it really looks like it's not getting power info, but I dont see an indication of why | 11:28 |
| janders | yeah I think you are right | 11:28 |
| janders | do you have any ideas how to get more details on why it's not connecting? | 11:29 |
| arne_wiebalck | janders: did you interrupt the manage command? | 11:30 |
| arne_wiebalck | janders: manage will not talk to the node I think | 11:30 |
| arne_wiebalck | janders: hmm ... maybe it verifies sth when it does the state transition? | 11:30 |
| janders | arne_wiebalck - no I don't recall interrupting it. It runs cleanly (exit code 0) but takes no effect. | 11:31 |
| arne_wiebalck | janders: in any case, if the transition fails, I'd expect to see sth in the logs | 11:31 |
| arne_wiebalck | janders: there's a Ctrl-C in your paste, but that is maybe for the reverse search | 11:32 |
| janders | arne_wiebalck yeah that would be the reverse search | 11:32 |
| arne_wiebalck | janders: yes, there is some verification from enroll to manage | 11:33 |
| janders | one thing it does react to is setting/unsetting maintenance | 11:33 |
| janders | that works | 11:33 |
| arne_wiebalck | janders: this is just a flag in the DB I believe | 11:33 |
| arne_wiebalck | janders: if the verification fails, the node will stay in enroll | 11:34 |
| janders | arne_wiebalck I re-tried manage while tailing logs, got the same old error I was hitting before | 11:34 |
| janders | (pasting) | 11:34 |
| janders | http://paste.openstack.org/show/797463/ | 11:35 |
| janders | I've seen this while trying to 1) enroll the node via bifrost and 2) override the ssl_ignore after the fact with the CLI | 11:35 |
| arne_wiebalck | janders: what about "openstack baremetal node validate <IRONIC NODE>" ? | 11:35 |
| janders | trying now | 11:36 |
| arne_wiebalck | janders: this should return sth if there is sth missing | 11:36 |
| janders | http://paste.openstack.org/show/797464/ | 11:36 |
| Qianbiao | <janders> did u check whether redfish url requires /redfish/v1 | 11:37 |
| Qianbiao | the log shows it returns a none json data. | 11:37 |
| janders | Qianbiao good question, I think it does: | 11:37 |
| Qianbiao | I think you may provide a wrong url. | 11:37 |
| janders | http://paste.openstack.org/show/797466/ | 11:38 |
| arne_wiebalck | Qianbiao: I don't think so, I have it configured like this as well. | 11:41 |
| arne_wiebalck | janders: so, the verification is failing (from http://paste.openstack.org/show/797463/) | 11:42 |
| arne_wiebalck | janders: this explains why the node does not move out of enroll | 11:42 |
| janders | arne_wiebalck because of lack of defined ports? | 11:42 |
| arne_wiebalck | janders: I don't think so | 11:43 |
| arne_wiebalck | janders: did you curl to the redfish endpoint to see what the response is? | 11:43 |
| janders | arne_wiebalck I tried a couple generic calls, it seems to work | 11:43 |
| janders | arne_wiebalck should I try something specific? | 11:44 |
| janders | hmm | 11:45 |
| janders | from this: HTTP response for GET https://10.19.109.248/: status code: 200 | 11:45 |
| janders | I wonder if redfish_address should include the context | 11:46 |
| arne_wiebalck | janders: curl -k -X GET http://10.19.109.248/redfish/v1/Systems | 11:46 |
| janders | arne_wiebalck: http://paste.openstack.org/show/797468/ | 11:47 |
| Qianbiao | janders as i said | 11:48 |
| Qianbiao | change your redfish url to xxx/redfish/v1 | 11:48 |
| janders | Qianbiao it took me a while to get it myself - good point, trying now | 11:49 |
| janders | Qianbiao arne_wiebalck there is some change | 11:53 |
| janders | http://paste.openstack.org/show/797469/ | 11:54 |
| janders | still doesnt work but now we get sushy.exceptions.AccessError: HTTP GET https://10.19.109.248/redfish/v1/Systems/1 returned code 401. iLO.0.10.ExtendedInfo: See @Message.ExtendedInfo for more information. | 11:54 |
| janders | I've seen this while trying to navigate around the redfish objects from the web browser | 11:54 |
| janders | {"error":{"code":"iLO.0.10.ExtendedInfo","message":"See @Message.ExtendedInfo for more information.","@Message.ExtendedInfo":[{"MessageId":"Base.1.0.NoValidSession"}]}} | 11:54 |
| janders | ^^^ that's what it results in | 11:54 |
| janders | fix: use curl and explicitly set auth | 11:55 |
| janders | then it works a treat | 11:55 |
| janders | hmmm now what to do with this | 11:55 |
| *** Nisha_Agarwal has quit IRC | 11:56 | |
| Qianbiao | janders seems like sushy version issue. | 11:58 |
| janders | too old? too new? | 11:59 |
| Qianbiao | Not sure. | 12:00 |
| rpittau | janders: can you double-check sushy version installed ? | 12:00 |
| *** Nisha_Agarwal has joined #openstack-ironic | 12:01 | |
| janders | rpittau sushy (3.3.0) sushy-tools (0.12.0) | 12:01 |
| janders | I have to say I am tempted to grab a Dell or a SuperMicro (preferrably Dell) and re-test | 12:02 |
| rpittau | janders: you should, at least to have a second test to compare | 12:02 |
| janders | I dont remember non-HPs kicking me out of redfish objects with 401 while playing around in browser | 12:02 |
| rpittau | versions look good | 12:02 |
| janders | which I think is what is happening to sushy/Ironic now | 12:02 |
| Qianbiao | janders from your log it's pretty like a version incompatible issue. | 12:03 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: [EARLY WIP] Generate a TLS certificate and send it to ironic https://review.opendev.org/749930 | 12:08 |
| *** zbitter is now known as zaneb | 12:10 | |
| arne_wiebalck | janders: on Train, I use python2-sushy-2.0.3-1 | 12:14 |
| janders | allright, it's getting late here so time for me to call it a night | 12:25 |
| janders | have a great weekend everyone! | 12:25 |
| janders | thanks for all your help today, much appreciataed | 12:25 |
| *** Nisha_Agarwal has quit IRC | 12:25 | |
| arne_wiebalck | janders: o/ | 12:36 |
| *** cdearborn has joined #openstack-ironic | 12:50 | |
| rpittau | hey if someone has a moment, I did some cleaning on job names between ipa and ipa-builder https://review.opendev.org/746766 https://review.opendev.org/749723 https://review.opendev.org/749725 | 13:00 |
| patchbot | patch 746766 - ironic-python-agent - Prepare to move ipa-builder jobs - 3 patch sets | 13:00 |
| patchbot | patch 749723 - ironic-python-agent-builder - Use converted ipa jobs - 2 patch sets | 13:00 |
| patchbot | patch 749725 - ironic-python-agent - Remove unused jobs - 2 patch sets | 13:01 |
| openstackgerrit | OpenStack Release Bot proposed openstack/metalsmith stable/victoria: Update .gitreview for stable/victoria https://review.opendev.org/749936 | 13:04 |
| openstackgerrit | OpenStack Release Bot proposed openstack/metalsmith stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria https://review.opendev.org/749937 | 13:04 |
| openstackgerrit | OpenStack Release Bot proposed openstack/metalsmith master: Update master for stable/victoria https://review.opendev.org/749938 | 13:04 |
| openstackgerrit | OpenStack Release Bot proposed openstack/metalsmith master: Add Python3 wallaby unit tests https://review.opendev.org/749939 | 13:04 |
| *** rloo has joined #openstack-ironic | 13:09 | |
| *** e0ne has joined #openstack-ironic | 13:32 | |
| e0ne | could anybody from core team take a look on a simple patch to ironic-ui please? | 13:33 |
| TheJulia | e0ne: sure | 13:40 |
| TheJulia | the goal patch? | 13:40 |
| e0ne | oops, didn't paste the link:( | 13:40 |
| e0ne | here is it https://review.opendev.org/#/c/744708/ | 13:40 |
| patchbot | patch 744708 - ironic-ui - [goal] Migrate testing to ubuntu focal - 16 patch sets | 13:40 |
| e0ne | TheJulia: thank you | 13:40 |
| TheJulia | approved | 13:41 |
| TheJulia | :) | 13:41 |
| dtantsur | good morning TheJulia | 13:41 |
| e0ne | great! we've one more "green" job on horizon gates now :) | 13:41 |
| *** ironic-starter has joined #openstack-ironic | 13:42 | |
| ironic-starter | I am a starter with stand-alone ironic and I have got to the point where I can bootstrap a server with CentOS8 Cloud Image | 13:43 |
| ironic-starter | I have a requirement where I need to set a password for teh cloud user centos | 13:43 |
| ironic-starter | how do I do that | 13:43 |
| ironic-starter | I tried using configdrive but the password is never set in | 13:43 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] Generate a TLS certificate and send it to ironic https://review.opendev.org/749930 | 13:44 |
| *** Goneri has joined #openstack-ironic | 13:45 | |
| ironic-starter | followed this https://docs.openstack.org/ironic/ussuri/install/configdrive.html and created a base64 enconded iso.gz | 13:45 |
| dtantsur | ironic-starter: that's really about configdrive, maybe a problem with the syntax? if you have any way to look at the node's console (via its BMC, for example), you may see what cloud-init says | 13:45 |
| ironic-starter | dtantsur : does not say anything | 13:45 |
| ironic-starter | other than showing that the network interfaces are being configured and centos does not have any authorized ssh keys | 13:46 |
| dtantsur | I'd also recommend to provide a configdrive via the new syntax (also documented there) - smaller chance to make a mistake | 13:46 |
| dtantsur | i.e. where you provide a JSON | 13:46 |
| ironic-starter | dtantsur : you mean " openstack baremetal node deploy $node_identifier \ | 13:47 |
| dtantsur | ironic-starter: a bit of a self-advertisement :) I've written a CLI tool/Python lib that simplifies some of these tasks: https://docs.openstack.org/metalsmith/latest/ | 13:47 |
| ironic-starter | dtantsur : I have seen that project ... :) | 13:48 |
| dtantsur | you can take a look at its code to see how to work with cloud-init user_data: https://opendev.org/openstack/metalsmith/src/branch/master/metalsmith/instance_config.py#L88 | 13:48 |
| ironic-starter | dtantsur : Thanks a lot that helps | 13:49 |
| ironic-starter | I am specifically looking at setting the cloud user's password | 13:49 |
| ironic-starter | would you have any specific inputs for me | 13:49 |
| ironic-starter | for instance when I use this syntax | 13:50 |
| ironic-starter | #cloud-config | 13:50 |
| ironic-starter | with the same cloud image without ironic just using kvm to boot that image it works | 13:50 |
| ironic-starter | I tried to provide the same syntax in user_data and now I think it is incorrect | 13:51 |
| ironic-starter | so looking forward for some guidance | 13:51 |
| dtantsur | I think you should be fine providing {"user_data": "#cloud-config\n<your conf here>"} | 13:51 |
| dtantsur | at least it works for metalsmith, maybe something goes wrong when building an ISO? | 13:51 |
| openstackgerrit | Merged openstack/ironic-ui master: [goal] Migrate testing to ubuntu focal https://review.opendev.org/744708 | 13:53 |
| Qianbiao | Hi dtantsur, may i take you a minute ~ | 13:54 |
| ironic-starter | as per the document I create the following folder | 13:54 |
| ironic-starter | . | 13:54 |
| ironic-starter | 2018-08-27 | 13:54 |
| ironic-starter | and ran | 13:54 |
| ironic-starter | mkisofs -R -V config-2 -o /opt/data/configdrive-8d030b3a-12a7-4f9b-9837-4fed2371b351.iso /root/configdrive/8d030b3a-12a7-4f9b-9837-4fed2371b351 | 13:54 |
| dtantsur | ironic-starter: do you *need* to go down this route? | 13:54 |
| ironic-starter | gzip -c /opt/data/configdrive-8d030b3a-12a7-4f9b-9837-4fed2371b351.iso | base64 > /opt/data/configdrive-8d030b3a-12a7-4f9b-9837-4fed2371b351.iso.gz | 13:54 |
| ironic-starter | I dont | 13:55 |
| dtantsur | providing a JSON leaves much less space for errors | 13:55 |
| ironic-starter | I am lost | 13:55 |
| ironic-starter | hence looking for a guidance | 13:55 |
| ironic-starter | If there is a way that config-drive iso.gz can be generated on the fly that would be best | 13:55 |
| dtantsur | that's literally what I'm talking about | 13:56 |
| ironic-starter | else I would have to pass it on the cli | 13:56 |
| ironic-starter | got your point ... let me test and see | 13:56 |
| dtantsur | see the 2nd example in https://docs.openstack.org/ironic/ussuri/install/configdrive.html#when-used-standalone you can do the same with user_data | 13:56 |
| ironic-starter | dtantsur : let me test that and report back .. hope this works... I just didn't want to put in the user password on the command line ... but let me give it a shot and see | 13:58 |
| dtantsur | ironic-starter: you're not putting a literal password there, if I recall correctly, but a hash of it | 13:58 |
| ironic-starter | cloud-init file which I used with kvm boot had the password in plain-text ... | 13:59 |
| ironic-starter | but when you say hash md5 or sha256 ? | 14:00 |
| dtantsur | ironic-starter: https://cloudinit.readthedocs.io/en/latest/topics/examples.html explains it | 14:00 |
| dtantsur | looks for 'passwd' on that page | 14:00 |
| ironic-starter | ok | 14:01 |
| dtantsur | I don't see an ability to provide a plain text password, only a hash. the comments explain how to generate one. | 14:01 |
| ironic-starter | Thanks :) | 14:02 |
| ironic-starter | let me test now and see | 14:02 |
| * dtantsur wonders if we should document it better | 14:03 | |
| *** uzumaki has joined #openstack-ironic | 14:03 | |
| openstackgerrit | Merged openstack/metalsmith master: Update master for stable/victoria https://review.opendev.org/749938 | 14:04 |
| openstackgerrit | Merged openstack/metalsmith stable/victoria: Update .gitreview for stable/victoria https://review.opendev.org/749936 | 14:04 |
| openstackgerrit | Merged openstack/metalsmith stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria https://review.opendev.org/749937 | 14:04 |
| TheJulia | and it finally occurs to me Monday is a holiday here | 14:06 |
| iurygregory | In Brazil would be holiday also | 14:08 |
| * iurygregory sad | 14:08 | |
| iurygregory | good morning TheJulia =) | 14:08 |
| *** Qianbiao has quit IRC | 14:09 | |
| *** bfournie has joined #openstack-ironic | 14:09 | |
| *** Qianbiao has joined #openstack-ironic | 14:09 | |
| dtantsur | let's decide if we're ready to merge https://review.opendev.org/#/c/749236/ ? | 14:10 |
| patchbot | patch 749236 - sushy - Add BIOS update apply time and maintenance window - 3 patch sets | 14:10 |
| dtantsur | we cannot hold the sushy release forever | 14:11 |
| *** zzzeek has quit IRC | 14:11 | |
| *** uzumaki has quit IRC | 14:12 | |
| iurygregory | dtantsur, I think we can, I was expecting a green run from Dell CI but they are broken atm | 14:12 |
| *** zzzeek has joined #openstack-ironic | 14:12 | |
| dtantsur | sigh | 14:12 |
| iurygregory | neutron problems... | 14:12 |
| dtantsur | this is a new feature, shouldn't be involved unless called explicitly | 14:12 |
| iurygregory | yeah, I'm wondering if they tested downstream or something | 14:13 |
| iurygregory | the code looks good | 14:13 |
| *** jtomasek has quit IRC | 14:15 | |
| * TheJulia looks | 14:16 | |
| dtantsur | unless I'm hallucinating, it is required for their redfish-based BIOS implementation | 14:17 |
| TheJulia | dtantsur: got a link? | 14:17 |
| TheJulia | ugh https://review.opendev.org/#/c/745950/ is a lot | 14:17 |
| patchbot | patch 745950 - sushy - Allow monitoring progress of a firmware update - 15 patch sets | 14:17 |
| dtantsur | aha, https://review.opendev.org/#/c/749240/ | 14:18 |
| patchbot | patch 749240 - ironic - Add Redfish BIOS interface to idrac HW type - 2 patch sets | 14:18 |
| TheJulia | looking | 14:18 |
| dtantsur | the sushy part in question is https://review.opendev.org/#/c/749236/ | 14:18 |
| patchbot | patch 749236 - sushy - Add BIOS update apply time and maintenance window - 3 patch sets | 14:18 |
| TheJulia | yup | 14:18 |
| TheJulia | it is required | 14:18 |
| dtantsur | I honestly gave up on hoping to squeeze the firmware update patch in the release | 14:18 |
| dtantsur | I don't have enough spoons for 700+ loc | 14:19 |
| TheJulia | I do though | 14:19 |
| TheJulia | just need to finish caffination | 14:19 |
| TheJulia | anyone else up for 700 loc? | 14:19 |
| dtantsur | TheJulia: you want to try get it in Victoria or just? | 14:19 |
| dtantsur | the last time I opened it I was worried how invasive it was | 14:20 |
| TheJulia | It is kind of clear to me that dell is trying to push a lot at the last minute... which is not great | 14:20 |
| *** uzumaki has joined #openstack-ironic | 14:20 | |
| TheJulia | if it is invasive, then we really shouldn't at the last minute | 14:20 |
| dtantsur | new arguments, renaming public methods.. | 14:20 |
| TheJulia | if it is "standard" pattern then it shouldn't be too bad | 14:20 |
| TheJulia | oh jeeze | 14:20 |
| dtantsur | dunno, I'll rely on your judgement | 14:20 |
| TheJulia | yeah, no too late in the cycle for renaming methods if there is no compatability | 14:20 |
| dtantsur | just keep in mind that the release deadline was yesterday :) | 14:21 |
| TheJulia | *shrugs* | 14:21 |
| TheJulia | I think the real deadline is actually sunday, at least if I'm remembering what one of the notes said | 14:21 |
| dtantsur | I think it's always Thursday | 14:22 |
| TheJulia | procedural thursday and reality sunday ;) | 14:22 |
| dtantsur | unrelated: if you've ever dreamed of telling me I suck in security, this is your chance: https://review.opendev.org/#/c/749930/2/ironic_python_agent/tls_utils.py | 14:22 |
| patchbot | patch 749930 - ironic-python-agent - [WIP] Generate a TLS certificate and send it to ir... - 2 patch sets | 14:22 |
| TheJulia | heh | 14:22 |
| iurygregory | I'm up for 700 loc | 14:26 |
| TheJulia | Ummm... yeah I'm thinking this is a nope, but it otherwise is in good shape | 14:26 |
| iurygregory | I can start now and finish after dinner probably XD | 14:27 |
| TheJulia | k | 14:27 |
| iurygregory | because in 1hr 30min I need to go see a flat | 14:27 |
| TheJulia | ++ | 14:27 |
| Qianbiao | hello, no means to interup you. | 14:30 |
| Qianbiao | after set node's network interface to neutron, I can not add node port manually, last error: http://paste.openstack.org/show/797459/ | 14:31 |
| Qianbiao | any experience to fix this | 14:31 |
| iurygregory | maybe neutron logs will provide more reasons why it couldnt create the port | 14:34 |
| Qianbiao | <iurygregory> will check now | 14:35 |
| *** ironic-starter has quit IRC | 14:37 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Accept and use a TLS certificate from the agent https://review.opendev.org/747136 | 14:37 |
| *** priteau has joined #openstack-ironic | 14:43 | |
| TheJulia | cdearborn: so https://review.opendev.org/#/c/745950 is missing a release note | 14:43 |
| patchbot | patch 745950 - sushy - Allow monitoring progress of a firmware update - 15 patch sets | 14:43 |
| TheJulia | any chance you can whip one up REALLY quick? | 14:43 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: Install libssl-dev / openssl-devel when building IPA https://review.opendev.org/749975 | 14:43 |
| cdearborn | TheJulia: sure - will do as soon as i get off the phone with rpioso | 14:44 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] Generate a TLS certificate and send it to ironic https://review.opendev.org/749930 | 14:46 |
| Qianbiao | iurygregor no lucky.. | 14:47 |
| iurygregory | Qianbiao, nothing in the logs? | 14:50 |
| Qianbiao | yeah. | 14:50 |
| Qianbiao | I think maybe i miss some option which is required by "neutron" network port. | 14:50 |
| Qianbiao | maybe lldp? | 14:50 |
| Qianbiao | or physical network, i am not sure, did not find a document. | 14:51 |
| TheJulia | cdearborn: ack, thanks | 14:53 |
| *** zzzeek has quit IRC | 14:56 | |
| *** zzzeek has joined #openstack-ironic | 14:59 | |
| TheJulia | erbarr: any luck on figuring out what is going on? | 14:59 |
| erbarr | TheJulia, nope I even used stable/ussuri for neutron and still same problem, i'm trying neutron-lib now | 15:00 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Detail iPXE + LACP troubleshooting information https://review.opendev.org/743787 | 15:01 |
| TheJulia | yeouch | 15:01 |
| *** belmoreira has quit IRC | 15:03 | |
| Qianbiao | what is the service name for neutron api in devstack? | 15:03 |
| TheJulia | q-svc | 15:04 |
| *** uzumaki has quit IRC | 15:05 | |
| *** uzumaki has joined #openstack-ironic | 15:06 | |
| Qianbiao | <TheJulia> thanks. | 15:06 |
| Qianbiao | iurygregory yes nothing. | 15:07 |
| Qianbiao | Is it possible that to provision in a l3 provisioning network with ovs? | 15:07 |
| Qianbiao | or we need some special neutron driver | 15:08 |
| Qianbiao | I find it documented here. https://docs.openstack.org/ironic/rocky/admin/multitenancy.html | 15:08 |
| Qianbiao | at last, it list supported drivers. | 15:08 |
| *** uzumaki has quit IRC | 15:10 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Fix and run the correct functional tests job https://review.opendev.org/749983 | 15:10 |
| dtantsur | sigh ^^^ | 15:10 |
| dtantsur | Qianbiao: you can use an ML2 driver that supports OVS, but this is only for testing on VMs | 15:11 |
| dtantsur | (this includes networking-generic-switch and networking-baremetal) | 15:11 |
| Qianbiao | oh..... | 15:11 |
| Qianbiao | great news.. | 15:12 |
| arne_wiebalck | General bare metal ops question (outside Ironic): how do people manage the boot order changes in UEFI mode? Background: We usually assume hosts *always* boot off the network, which UEFI installations do not do, as they change the boot order at the end. We have a work around (to not run efibootmgr, but create a config, which will preserve the boot order), but we were wondering how this is handled in other | 15:12 |
| arne_wiebalck | deployments ... | 15:12 |
| Qianbiao | dtantsur thanks, It means all i did now has no way to work. | 15:13 |
| dtantsur | generally, separate networks require a suitable ML2 driver to work with your switch | 15:14 |
| dtantsur | otherwise, ironic has no way to put nodes on different networks, and you have to use flat networking | 15:14 |
| Qianbiao | dtantsur i try flat networking. | 15:16 |
| Qianbiao | but it does not support switch networks when deploy | 15:16 |
| Qianbiao | Am i right? | 15:16 |
| openstackgerrit | Christopher Dearborn proposed openstack/sushy master: Allow monitoring progress of a firmware update https://review.opendev.org/745950 | 15:16 |
| Qianbiao | I can put provisioning network on a flat network. | 15:16 |
| Qianbiao | But it did not provide dhcp when deploy. On provide dhcp fro clean. | 15:17 |
| openstackgerrit | Merged openstack/sushy master: Add BIOS update apply time and maintenance window https://review.opendev.org/749236 | 15:17 |
| rpittau | bye everyone, see you in 4 weeks! o/ | 15:18 |
| *** rpittau is now known as rpittau|afk | 15:18 | |
| openstackgerrit | Christopher Dearborn proposed openstack/sushy master: Allow monitoring progress of a firmware update https://review.opendev.org/745950 | 15:20 |
| cdearborn | TheJulia: Added a reno to https://review.opendev.org/745950 | 15:21 |
| patchbot | patch 745950 - sushy - Allow monitoring progress of a firmware update - 17 patch sets | 15:21 |
| cdearborn | TheJulia: Accidentally pushed a change I didn't mean to, so had to undo that | 15:22 |
| cdearborn | TheJulia: should be good to go now | 15:22 |
| cdearborn | TheJulia: how would I go about requesting an FFE for firmware update support in the redfish driver? I have a submitted patch with everything working: https://review.opendev.org/#/c/749619/. Need to do test cases, docs, plus maybe a few more minor changes | 15:25 |
| patchbot | patch 749619 - ironic - WIP: Redfish driver firmware update - 4 patch sets | 15:25 |
| cdearborn | TheJulia - ack! don't know what i did - i just killed that patch. will try to unravel | 15:31 |
| *** jamesden_ is now known as jamesdenton | 15:31 | |
| cdearborn | TheJulia - nevermind - patch is fine - need more coffee | 15:34 |
| dtantsur | I don't think we're in feature freeze yet? | 15:34 |
| openstackgerrit | Merged openstack/ironic master: Add missing log for clean failed https://review.opendev.org/749861 | 15:40 |
| openstackgerrit | Merged openstack/ironic master: Update deploy steps documentation https://review.opendev.org/749729 | 15:40 |
| openstackgerrit | Merged openstack/ironic-lib master: Add Python3 wallaby unit tests https://review.opendev.org/749781 | 15:40 |
| openstackgerrit | Merged openstack/ironic-lib stable/victoria: Update .gitreview for stable/victoria https://review.opendev.org/749779 | 15:40 |
| openstackgerrit | Merged openstack/ironic-lib stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria https://review.opendev.org/749780 | 15:40 |
| *** gyee has joined #openstack-ironic | 15:44 | |
| cdearborn | dtantsur: aok - will try to finish it up - thx! | 15:45 |
| *** Qianbiao has quit IRC | 15:46 | |
| TheJulia | afaik, we are not unless there was a consensus I missed | 16:03 |
| TheJulia | in feature freeze that is | 16:03 |
| cdearborn | TheJulia, thx. Are you trying to cut a sushy release today? | 16:04 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] Generate a TLS certificate and send it to ironic https://review.opendev.org/749930 | 16:05 |
| dtantsur | cdearborn: today or on Monday | 16:05 |
| dtantsur | this the global openstack limitation (non-client library release deadline) | 16:05 |
| cdearborn | dtantsur: ok - thx | 16:05 |
| *** lucasagomes has quit IRC | 16:06 | |
| *** dtantsur is now known as dtantsur|afk | 16:06 | |
| dtantsur|afk | have a good weekend everyone | 16:06 |
| cdearborn | dtantsur: you too! | 16:06 |
| NobodyCam | Good Morning Folks, Glade to be back on IRC oh and Of course: | 16:07 |
| NobodyCam | TGIF! | 16:07 |
| cdearborn | Good morning NobodyCam! Can't wait for the weekend | 16:08 |
| TheJulia | good morning NobodyCam | 16:13 |
| TheJulia | goodnight dtantsur|afk | 16:13 |
| NobodyCam | Good Morning cdearborn and TheJulia | 16:14 |
| TheJulia | clarkb: is there anything fundimentally different between the environmnet variables when passed in the pre and run phases? | 16:16 |
| clarkb | TheJulia: there shouldn't be. They are separate playbooks so if you are setting them within a playbook they don't carry over to the next | 16:16 |
| clarkb | but the behavior of setting them within a playbook should be the same if you set them again in the next playbook | 16:16 |
| TheJulia | That is what I figured | 16:18 |
| TheJulia | then I wonder if neutron broke metalsmith | 16:18 |
| TheJulia | which would be awesome timing | 16:18 |
| *** bnemec is now known as beekneemech | 16:18 | |
| *** dougsz has quit IRC | 16:33 | |
| *** derekh has quit IRC | 16:34 | |
| openstackgerrit | Merged openstack/metalsmith master: Add Python3 wallaby unit tests https://review.opendev.org/749939 | 16:47 |
| *** ijw has quit IRC | 16:51 | |
| *** ijw has joined #openstack-ironic | 16:51 | |
| *** ijw_ has joined #openstack-ironic | 16:54 | |
| *** ijw has quit IRC | 16:57 | |
| TheJulia | sigh, there must be something different | 17:02 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Remove token-less agent support https://review.opendev.org/749407 | 17:09 |
| *** ijw has joined #openstack-ironic | 17:16 | |
| *** ijw_ has quit IRC | 17:20 | |
| *** bdodd_ is now known as bdodd | 17:22 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-inspector master: WIP: Power off before inspection https://review.opendev.org/749845 | 18:03 |
| *** Lucas_Gray has quit IRC | 18:10 | |
| *** zzzeek has quit IRC | 18:19 | |
| *** zzzeek has joined #openstack-ironic | 18:20 | |
| *** ijw has quit IRC | 18:35 | |
| rpioso | TheJulia: I have a couple of concerns about https://review.opendev.org/749236. | 18:52 |
| patchbot | patch 749236 - sushy - Add BIOS update apply time and maintenance window (MERGED) - 3 patch sets | 18:52 |
| rpioso | TheJulia: 1) It hasn't passed the Dell EMC sushy Third-Party CI. | 18:52 |
| rpioso | TheJulia: 2) The co-authors list is incomplete. mraineri (Michael Raineri) should be listed, too. I am not certain the syntax of the two co-authors is correct, because they are missing angle brackets. | 18:52 |
| TheJulia | *sigh* | 18:53 |
| TheJulia | So bottom line I can revert it, but it won't be in Victoria. Full stop. | 18:53 |
| rpioso | TheJulia: Yikes! Well, the first concern may not matter much, because the CI doesn't test BIOS. Could we revert and clean-up the co-authors so Mike gets the recognition/credit he deserves? | 18:54 |
| TheJulia | Propose the patches and I'll approve them. | 18:55 |
| openstackgerrit | Richard G. Pioso proposed openstack/sushy master: Revert "Add BIOS update apply time and maintenance window" https://review.opendev.org/750017 | 18:57 |
| TheJulia | (that way it is not just me doing it on my own) | 18:57 |
| iurygregory | I'm still around so I can +2 | 18:57 |
| iurygregory | =) | 18:57 |
| TheJulia | Woot | 18:58 |
| TheJulia | I think I'm going to go to the store... Trying to fix an inspector bug is making my head hurt. | 18:58 |
| iurygregory | I'm still looking at the 700 lines patch =) | 18:58 |
| TheJulia | That one is not bad really. I had one gripe but I couldn't find usage of it so *shrug* | 18:58 |
| *** qthepirate has joined #openstack-ironic | 18:59 | |
| *** priteau has quit IRC | 19:00 | |
| zer0c00l | I don't think the lookup API docs reflect the 'agent_token' information https://docs.openstack.org/api-ref/baremetal/index.html?expanded=agent-lookup-detail,agent-heartbeat-detail | 19:01 |
| zer0c00l | The example response body of v1/lookup doesn't have agent_token | 19:02 |
| TheJulia | Woot | 19:03 |
| TheJulia | I knew I forgot something! | 19:03 |
| zer0c00l | I was looking up how to heartbeat with the anaconda driver, the code seems to be adding the token but the API docs doesn't reflect that | 19:04 |
| zer0c00l | TheJulia: do you want me to open a docs bug? | 19:04 |
| TheJulia | Eh... I guess. | 19:05 |
| TheJulia | Might be faster just to propose a docs patch :) | 19:05 |
| TheJulia | rpioso: you'll need to strip the commit Id out to reupload | 19:08 |
| TheJulia | Once they are stacked, I can approve them | 19:08 |
| rpioso | TheJulia: Almost done ... | 19:08 |
| TheJulia | cdearborn: looks like there is an issue with your latest sushy patch upload :( | 19:08 |
| cdearborn | TheJulia: checking... | 19:09 |
| bdodd | cdearborn TheJulia Looks like a conflict between the BIOS patch and the f/w update monitoring patch | 19:19 |
| cdearborn | bdodd, TheJulia, y - rebasing... | 19:20 |
| openstackgerrit | Richard G. Pioso proposed openstack/sushy master: Add BIOS update apply time and maintenance window https://review.opendev.org/750020 | 19:22 |
| rpioso | TheJulia: Done | 19:23 |
| rpioso | TheJulia: Thank you! | 19:23 |
| rpioso | cdearborn: You may want to rebase to https://review.opendev.org/750020 | 19:24 |
| patchbot | patch 750020 - sushy - Add BIOS update apply time and maintenance window - 1 patch set | 19:24 |
| *** qthepirate has quit IRC | 19:30 | |
| openstackgerrit | Christopher Dearborn proposed openstack/sushy master: Allow monitoring progress of a firmware update https://review.opendev.org/745950 | 19:33 |
| openstackgerrit | Richard G. Pioso proposed openstack/sushy master: Revert "Add BIOS update apply time and maintenance window" https://review.opendev.org/750017 | 19:35 |
| openstackgerrit | Richard G. Pioso proposed openstack/sushy master: Add BIOS update apply time and maintenance window https://review.opendev.org/750020 | 19:39 |
| TheJulia | rpioso: approved | 19:44 |
| rpioso | TheJulia: Thank you so much! | 19:45 |
| TheJulia | cdearborn: a straight rebase didn't do it | 19:52 |
| TheJulia | zuul status page shows all of the unit tests failing | 19:52 |
| cdearborn | TheJulia, i tried to rebase on top of 750020, but it failed with conflicts and helpfully didn't show me what any of the conflicts were | 19:53 |
| cdearborn | TheJulia, will give it another shot | 19:53 |
| TheJulia | cdearborn: I wonder if you run a fresh unit test run locally if they will pass or not for yourself | 19:53 |
| cdearborn | TheJulia, where do you see the failures? looks like the test haven't been run yet: https://review.opendev.org/#/c/745950/ | 19:56 |
| patchbot | patch 745950 - sushy - Allow monitoring progress of a firmware update - 18 patch sets | 19:56 |
| bdodd | cdearborn After the rebase, you need to re-add the import statement for res_maps (in settings.py) | 19:57 |
| bdodd | (At least I think that is the issue) | 19:58 |
| rpioso | bdodd: Please note the use of @Redfish.SettingsApplyTime in the BIOS changes, both sushy and redfish driver. I expect analogous use of @Redfish.OperationApplyTime will be needed to get your proposed RAID changes to work against iDRACs. | 20:03 |
| openstackgerrit | Christopher Dearborn proposed openstack/sushy master: Allow monitoring progress of a firmware update https://review.opendev.org/745950 | 20:04 |
| cdearborn | bdodd, thanks! I made that change and tox runs clean now | 20:04 |
| bdodd | rpioso Thanks for the heads-up. Will look at it. | 20:04 |
| rpioso | bdodd: You're most welcome! Thanks for assisting with the BIOS stuffs. Very helpful! | 20:05 |
| cdearborn | TheJulia, will hopefully be gooder this time around... | 20:06 |
| bdodd | cdearborn \o/ | 20:06 |
| *** zzzeek has quit IRC | 20:06 | |
| *** zzzeek has joined #openstack-ironic | 20:10 | |
| TheJulia | cdearborn: you'll love this link: https://zuul.opendev.org/t/openstack/status | 20:10 |
| TheJulia | if you click the box for sushy, change set 745950 you'll see unit tests are passing | 20:11 |
| TheJulia | or, to be more precise now, have passed | 20:11 |
| cdearborn | TheJulia, nice! definitely going to bookmark that one!!! | 20:12 |
| rpioso | TheJulia: Will we be meeting on Monday (Labor Day in US)? | 20:13 |
| TheJulia | There should, but I think everyone is taking it off | 20:29 |
| TheJulia | even folks in the EU are taking monday off afaik | 20:29 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-inspector master: Power off before inspection https://review.opendev.org/749845 | 20:32 |
| TheJulia | rpioso: I guess I'll show up and if people want to have a meeting then we'll have it, if people don't show its not a big deal, we'll deffer the agenda for a week I guess | 20:33 |
| TheJulia | err | 20:33 |
| TheJulia | hmm | 20:33 |
| TheJulia | OpenInfra summit cfps for forum is open :\ | 20:33 |
| rpioso | TheJulia: We could defer the subjects you and I discussed late-ish Wednesday for a week. | 20:34 |
| TheJulia | yeah | 20:36 |
| * TheJulia goes off to go to the store since grumbling about the metalsmith ci is not going to fix it | 20:36 | |
| TheJulia | iurygregory: I've left a +2 on cdearborn's patch, +a if you are happy with it | 20:46 |
| iurygregory | TheJulia, I'm happy was waitting for CI to +1 | 20:47 |
| iurygregory | some weird errors before =) | 20:47 |
| iurygregory | I will let a +2 (since the result for the job may take a while - but at least know the unit jobs are green) | 20:48 |
| TheJulia | yeah, I was thinking the same | 20:49 |
| *** Lucas_Gray has joined #openstack-ironic | 21:00 | |
| *** Lucas_Gray has quit IRC | 21:08 | |
| *** Lucas_Gray has joined #openstack-ironic | 21:14 | |
| *** rh-jelabarre has quit IRC | 21:29 | |
| openstackgerrit | Merged openstack/sushy master: Revert "Add BIOS update apply time and maintenance window" https://review.opendev.org/750017 | 21:34 |
| openstackgerrit | Merged openstack/sushy master: Add BIOS update apply time and maintenance window https://review.opendev.org/750020 | 21:34 |
| *** rloo has quit IRC | 21:36 | |
| * cdearborn just got back from a beer run | 21:36 | |
| *** rcernin_ has joined #openstack-ironic | 21:52 | |
| *** hjensas has quit IRC | 22:10 | |
| cdearborn | TheJulia, iurygregory: yay! unit tests are good to go now on https://review.opendev.org/#/c/745950/ | 22:26 |
| patchbot | patch 745950 - sushy - Allow monitoring progress of a firmware update - 19 patch sets | 22:26 |
| iurygregory | cdearborn, +A | 22:27 |
| iurygregory | functional tests are green =) | 22:27 |
| iurygregory | TheJulia, since monday is holiday in the US do you want to cancel the weekly meeting? | 22:28 |
| *** rcernin_ has quit IRC | 22:29 | |
| *** dking has quit IRC | 22:36 | |
| TheJulia | iurygregory: I'd honestly prefer to at least try and have it | 22:44 |
| TheJulia | iurygregory: that way I can at least do annoucements. | 22:45 |
| iurygregory | TheJulia, ++ =) | 22:51 |
| *** tosky has quit IRC | 23:07 | |
| *** Lucas_Gray has quit IRC | 23:12 | |
| openstackgerrit | Julia Kreger proposed openstack/metalsmith master: Rework jobs so failed devstack does not retry the job https://review.opendev.org/749758 | 23:24 |
| TheJulia | cdearborn: rpioso: I took another look at https://review.opendev.org/#/c/748927 and worst comes to worst once CI is back up and it can be ran past it, we can backport it as a butfix I think | 23:25 |
| patchbot | patch 748927 - sushy - Make message parsing more resilient - 3 patch sets | 23:25 |
| TheJulia | just create a bug to cover the MessageKey difference | 23:26 |
| openstackgerrit | Merged openstack/sushy master: Allow monitoring progress of a firmware update https://review.opendev.org/745950 | 23:28 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!