*** artom__ is now known as artom | 03:28 | |
opendevreview | Jan Hartkopf proposed openstack/openstacksdk master: fix creation of protected image https://review.opendev.org/c/openstack/openstacksdk/+/820926 | 10:02 |
---|---|---|
thomasb06 | gtema: do you see anything wrong in the `register_machine` function? There seem to be an error of block quoting: https://zuul.opendev.org/t/openstack/build/d412329810e941cabf4b7c4794cb5781 | 12:38 |
gtema | looking | 12:40 |
thomasb06 | The error is triggered during the Coe compilation while the Baremetal one went well | 12:43 |
gtema | mostly you will need to "rebase" on top of parent change every time you update previous commit | 12:50 |
gtema | alternatively (preferred) you break chain by rebasing each change on top of base branch | 12:51 |
thomasb06 | Wow... How can I do this now? Move head backward to the Coe commit and pull? | 12:53 |
gtema | in the UI | 12:53 |
gtema | go to rebase and select " Rebase on top of the feature/r1 branch (breaks relation chain) " | 12:54 |
gtema | and this for every change you created | 12:54 |
thomasb06 | In the Coe commit? Let me check | 12:54 |
thomasb06 | Found. This is it? | 12:55 |
thomasb06 | Before, I need to 'recheck' maybe? | 12:56 |
gtema | no | 12:56 |
gtema | this will trigger it automatically | 12:56 |
thomasb06 | Ah, you did it. This is why it didn't work, there was an error message | 12:56 |
thomasb06 | For the next ones, I'll know. Thanks | 12:57 |
gtema | I rebased still on the latest version of the previous change (updated underlaying ver) | 12:57 |
gtema | but as I meant - best is to break dependency | 12:57 |
thomasb06 | Understood | 13:01 |
opendevreview | Pierre Riteau proposed openstack/ansible-collections-openstack master: Add support for project parent https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825108 | 13:21 |
thomasb06 | gtema: same for the Compute service. So I click on "Rebase on top of the feature/r1 branch (breaks relation chain)"? | 14:17 |
gtema | on every chain, otherwise you will always need to rebase them once you update bottom change | 14:17 |
thomasb06 | alright | 14:19 |
opendevreview | Ivan ROGER proposed openstack/ansible-collections-openstack master: Fix identity user lookup with a domain https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825117 | 14:58 |
stephenfin | diablo_rojo_phone: are you guys meeting? Is there anyone to meet? | 17:33 |
diablo_rojo_phone | Artem and I are here. | 17:34 |
diablo_rojo_phone | No students though | 17:34 |
diablo_rojo_phone | Did I miss something? | 17:34 |
stephenfin | yeah, they're both sick | 17:35 |
diablo_rojo_phone | Oh whoops. | 17:38 |
diablo_rojo_phone | Just Artem and I chatting then lol | 17:38 |
diablo_rojo_phone | How's your wrist? | 17:38 |
diablo_rojo_phone | Ah I see the email now. | 17:42 |
stephenfin | diablo_rojo_phone: All good. Things are staying where they are, so as long as that continues I'm all good :) | 18:34 |
opendevreview | Thiago Paiva Brito proposed openstack/openstacksdk master: Add update capabilities to Snapshots https://review.opendev.org/c/openstack/openstacksdk/+/778760 | 20:39 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!