Sunday, 2020-07-26

*** Lucas_Gray has joined #openstack-ironic00:01
*** stevebaker has quit IRC00:11
*** stevebaker has joined #openstack-ironic00:11
*** Goneri has joined #openstack-ironic00:36
*** Goneri has quit IRC02:11
*** diurnalist has joined #openstack-ironic03:47
*** diurnalist has quit IRC03:52
*** Lucas_Gray has quit IRC04:05
*** abdysn has joined #openstack-ironic05:04
*** sshnaidm|off is now known as sshnaidm07:54
*** yaawang has quit IRC08:08
*** yaawang has joined #openstack-ironic08:09
*** uzumaki has quit IRC08:27
*** uzumaki has joined #openstack-ironic08:28
*** uzumaki has quit IRC08:35
*** uzumaki has joined #openstack-ironic08:46
*** uzumaki has quit IRC12:03
*** riuzen has joined #openstack-ironic12:04
*** riuzen has quit IRC12:05
*** yolanda has quit IRC12:35
*** yolanda has joined #openstack-ironic12:35
*** Lucas_Gray has joined #openstack-ironic13:21
*** abdysn has quit IRC14:31
*** Goneri has joined #openstack-ironic14:32
*** Lucas_Gray has quit IRC15:35
*** alexmcleod has quit IRC19:33
TheJuliahmm, I guess we never added indicator support to the api client20:01
*** stevebaker has quit IRC20:23
*** stevebaker has joined #openstack-ironic20:32
*** mkowalski has joined #openstack-ironic20:52
*** jj123 has quit IRC20:58
*** rcernin has joined #openstack-ironic22:46
*** janders has joined #openstack-ironic22:49
*** tkajinam has joined #openstack-ironic22:51
jandersgood morning Ironic o/23:07
stevebakerjanders: morning!23:08
jandershi stevebaker23:19
jandersI just realised I forgot adding the Ansible Collections symlink in bifrost to .gitignore23:19
jandersdo changes to .gitignore go through the review process just like other patches, or is this different in some way?23:19
stevebakerjanders: yes, they're just normal changes23:20
jandersthanks stevebaker I will submit this  now23:20
jandersdo I submit it as a new change or would this be another revision of the change that introduced the issue (but is now merged)?23:25
stevebakerjanders: It will have to be a follow-up new commit if the first change has merged, otherwise I'd say add it to the active review23:28
jandersthe change has merged indeed23:30
jandershow to indicate the fact it's a follow-up commit in the new change?23:31
janderssame change-id?23:31
openstackgerritJulia Kreger proposed openstack/ironic master: WIP: kexec  https://review.opendev.org/68897723:33
openstackgerritJulia Kreger proposed openstack/ironic master: Allow Active nodes to heartbeat  https://review.opendev.org/71185223:33
openstackgerritJulia Kreger proposed openstack/ironic master: wip: kexec to ramdisk  https://review.opendev.org/71185323:33
TheJuliadtantsur|afk: conundrum for tomorrow: https://review.opendev.org/#/c/711853/3/ironic/drivers/modules/agent.py23:34
patchbotpatch 711853 - ironic - wip: kexec to ramdisk - 3 patch sets23:34
TheJuliaif we were to put in kexec, where would we? would it be it's own step (i kind of like this idea, or a default disabled step or a step that could be leveraged other ways... #questions #lotsofquestions23:34
TheJuliajanders: if you wants some python-ironicclient low hanging fruit, I realized today that we never added support for the indicators endpoint on the rest API23:35
jandersnoted! thanks TheJulia :)23:46
stevebakerjanders: no, it'll need a new change-id, but you could reference the previous change-id or review URL in the commit comment. *If* this gets backported to a stable branch then we can merge them to a single commit and use the original change-id23:52
jandersgreat, thank you stevebaker23:57

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!