| opendevreview | Vanou Ishii proposed openstack/ironic stable/yoga: Fix iRMC driver to use certification file in HTTPS https://review.opendev.org/c/openstack/ironic/+/852797 | 00:11 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/metalsmith stable/train: Ensure tox.ini is ASCII https://review.opendev.org/c/openstack/metalsmith/+/829837 | 01:25 |
| arne_wiebalck | Good morning, Ironic! | 06:38 |
| jssfr | Good Morning, Ironic and arne! | 06:45 |
| rpittau | good morning ironic! o/ | 07:42 |
| dtantsur | arozman: there is a way to pass kernel params per node, I think BMO even uses a bit of it already | 08:33 |
| dtantsur | arozman: https://github.com/metal3-io/baremetal-operator/pull/1109/files#diff-f05fc4a0c9679815bc34bcef100c2a8d525688f2804a1128c5b9e1e575b76c29R604 | 08:34 |
| adam-rozman | dtantsur: Isn't this only for node images? Anyways I have switched to a solution that does not require kernel parameter or a condition as JayF mentioned. | 09:12 |
| dtantsur | adam-rozman: nope, it's only for IPA images, we don't have a way to provide kernel parameters for instance images. | 09:13 |
| dtantsur | good to know, I must have missed that in the scrollback | 09:13 |
| adam-rozman | It works nicely for me at the moment althogugh I have only tested this with virtual machines: https://github.com/Nordix/ironic-python-agent/commit/42d7fd5ad33b36f98e9bbfedd262a3bdd95b3b1e | 09:17 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector bugfix/11.0: Update .gitreview for bugfix/11.0 https://review.opendev.org/c/openstack/ironic-inspector/+/853609 | 09:28 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent bugfix/9.0: Update .gitreview for bugfix/9.0 https://review.opendev.org/c/openstack/ironic-python-agent/+/853610 | 09:31 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/yoga: Fix iRMC driver to use certification file in HTTPS https://review.opendev.org/c/openstack/ironic/+/852797 | 09:55 |
| opendevreview | Vishal Manchanda proposed openstack/ironic-ui master: Migrate to AngularJS v1.8.2 https://review.opendev.org/c/openstack/ironic-ui/+/852702 | 10:07 |
| opendevreview | kamlesh chauvhan proposed openstack/ironic-tempest-plugin master: Add iDRAC Redfish firmware update test https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/853621 | 10:43 |
| opendevreview | Ebbex proposed openstack/bifrost master: make sure we quote extra_kernel_options https://review.opendev.org/c/openstack/bifrost/+/852877 | 10:51 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic bugfix/21.0: Update .gitreview for bugfix/21.0 https://review.opendev.org/c/openstack/ironic/+/853624 | 10:52 |
| opendevreview | Merged openstack/ironic-inspector bugfix/11.0: Update .gitreview for bugfix/11.0 https://review.opendev.org/c/openstack/ironic-inspector/+/853609 | 11:14 |
| opendevreview | Merged openstack/ironic-python-agent bugfix/9.0: Update .gitreview for bugfix/9.0 https://review.opendev.org/c/openstack/ironic-python-agent/+/853610 | 11:14 |
| opendevreview | Merged openstack/ironic bugfix/21.0: Update .gitreview for bugfix/21.0 https://review.opendev.org/c/openstack/ironic/+/853624 | 11:26 |
| iurygregory | good morning Ironic | 11:28 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/yoga: Fix iRMC driver to use certification file in HTTPS https://review.opendev.org/c/openstack/ironic/+/852797 | 12:11 |
| TheJulia | plus side of over sleeping... sleep() | 14:06 |
| TheJulia | good morning | 14:15 |
| rpittau | good morning TheJulia :) | 14:17 |
| iurygregory | good morning TheJulia =) | 14:39 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add docs for VMware deployment https://review.opendev.org/c/openstack/ironic/+/853549 | 14:49 |
| JayF | dtantsur: adam-rozman: Whoa, TIL. That's pretty sweet, I could've used that a time or three in the past. | 15:09 |
| opendevreview | Julia Kreger proposed openstack/ironic master: CI: Changes to support Anaconda CI jobs https://review.opendev.org/c/openstack/ironic/+/849587 | 15:26 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: Initial tempest test idea anaconda deploy https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/835917 | 15:28 |
| TheJulia | adam-rozman: if you want to propose https://github.com/Nordix/ironic-python-agent/commit/42d7fd5ad33b36f98e9bbfedd262a3bdd95b3b1e to openstack/ironic-python-agent, I suspect we could likely accept it without much hassle | 15:30 |
| TheJulia | fwiw, injection of host kernel args seems more viable with grubby | 15:33 |
| TheJulia | but not everyone uses bls and it can explode of course | 15:33 |
| adam-rozman | TheJulia: Thanks that would be very nice of you, I will create a proposal as soon as I can. | 15:37 |
| TheJulia | adam-rozman: okay, cool | 15:40 |
| opendevreview | Julia Kreger proposed openstack/ironic master: CI: Changes to support Anaconda CI jobs https://review.opendev.org/c/openstack/ironic/+/849587 | 15:46 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: Initial tempest test idea anaconda deploy https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/835917 | 15:46 |
| rpittau | good night! o/ | 15:53 |
| iurygregory | gn rpittau o/ | 16:02 |
| opendevreview | Julia Kreger proposed openstack/ironic master: CI: Changes to support Anaconda CI jobs https://review.opendev.org/c/openstack/ironic/+/849587 | 16:34 |
| opendevreview | Vishal Manchanda proposed openstack/ironic-ui master: Migrate to AngularJS v1.8.2 https://review.opendev.org/c/openstack/ironic-ui/+/852702 | 17:41 |
| iurygregory | =O | 17:51 |
| iurygregory | \o/ | 17:51 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: Initial tempest test idea anaconda deploy https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/835917 | 18:02 |
| opendevreview | Mallikarjuna Reddy Kolagatla proposed openstack/ironic master: Enables event subscription methods for ilo and ilo5 hardware types https://review.opendev.org/c/openstack/ironic/+/853690 | 18:14 |
| iurygregory | ironic-ui is green! | 18:25 |
| iurygregory | https://review.opendev.org/c/openstack/ironic/+/853690 sounds interesting... I tested the eventsubscriptions in HP machines and it was working... not sure if is really required O.o | 18:29 |
| iurygregory | will try to take a look at it | 18:29 |
| iurygregory | TheJulia, ironic-ui is green <O> https://review.opendev.org/c/openstack/ironic-ui/+/852702 | 18:40 |
| *** rcastillo|rover_ is now known as rcastillo|rover | 19:16 | |
| TheJulia | woot | 20:01 |
| iurygregory | yes! | 20:07 |
| TheJulia | hjensas: so I think your srbac patch has found us a bug | 20:13 |
| TheJulia | and I *think* I've got a fix, but I'll test it after lunch | 20:13 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM: Fix indicator possible failure https://review.opendev.org/c/openstack/ironic/+/853706 | 21:41 |
| TheJulia | stevebaker[m]: hjensas; ^ That pesky indicators api.... | 21:44 |
| TheJulia | that works on get_one | 21:44 |
| TheJulia | but not put... :( | 21:44 |
| TheJulia | but since i've seen it happen multiple times, and it seems like it is totally possible to occur... seems like we should carry a patch... dunno | 21:44 |
| JayF | Why would that kind of syntax sugar belong in our API rather than, e.g. the client? | 21:51 |
| JayF | I don't neccessarily disagree, this is just the first time I've seen someone give up on teaching proper API usage and support the more-obvious-but-wrong API usage that users want :) | 21:52 |
| opendevreview | Julia Kreger proposed openstack/ironic master: CI: Changes to support Anaconda CI jobs https://review.opendev.org/c/openstack/ironic/+/849587 | 22:05 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: Initial tempest test idea anaconda deploy https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/835917 | 22:31 |
| TheJulia | JayF: because I spotted harald working on tempest tests with a / in between the values, and I've seen it before, and the code expects it to be @ | 23:09 |
| TheJulia | I think it was go client code too the last time I saw it | 23:09 |
| TheJulia | dunno, I could go either way I guess | 23:10 |
| *** rcastillo|rover is now known as rcastillo | 23:13 | |
| JayF | I am probably an extremely good person to go digging issues in the golang client, fwiw | 23:25 |
| JayF | so if you want to track that down you can likely consider me nerd-sniped | 23:25 |
| TheJulia | that was a long time ago if memory serves, it just seems like @ was a bad idea for an api | 23:27 |
| TheJulia | it happens | 23:28 |
| JayF | ack, yeah I understand | 23:28 |
| JayF | might be interesting to put some of that context in a release note if it moves forward | 23:28 |
| JayF | future-us might appreciate remembering the "Why?" after a long time passes | 23:28 |
| JayF | e.g. right now I find myself wondering why we did it that way in the first place, and if there's something we're missing. Likely doesn't matter as long as both ways always work tho. | 23:30 |
| TheJulia | yeah.... I don't remember why at this point :( | 23:50 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!