| thomasb06 | gtema: Hello. Did you have a chance to give a try to my script? I should walk the full directory tree, but I tried it only on one file in same directory as the script... | 06:53 |
|---|---|---|
| thomasb06 | Maybe I should add a working directory change somewhere | 06:54 |
| gtema | thomasb06 - we do not want single huge change, but rather lot of small ones | 07:14 |
| thomasb06 | gtema: what I can do is change all the files at the same time, but commit them one by one? | 07:23 |
| gtema | yes, this can be done. Pretty much same as before - one service at a time | 07:24 |
| thomasb06 | Great | 07:24 |
| thomasb06 | gtema: my script finds only these 25 files: https://x0.at/vx5Y.py | 08:22 |
| thomasb06 | is it ok for you? | 08:22 |
| gtema | yupp, those are precisely as expected | 08:23 |
| thomasb06 | cool, I start commiting | 08:23 |
| gtema | thks | 08:23 |
| thomasb06 | the command `git review` returns `error: failed to push some refs` | 08:28 |
| thomasb06 | maybe I need to rebase? | 08:28 |
| gtema | Maybe. There should be bit more details | 08:29 |
| thomasb06 | The full error message is: https://x0.at/UUhP.txt | 08:32 |
| gtema | are you confirming upload of multiple commits? if not this is clear why it fails | 08:33 |
| thomasb06 | To the question `Do you really want to submit the above commits?`, I say `yes`. Should I say no? | 08:35 |
| thomasb06 | If I say `no`, it's aborting | 08:35 |
| gtema | hm, "yes" is exactly that, or "git review -y" | 08:36 |
| gtema | maybe try adding "-v"? | 08:36 |
| thomasb06 | after a `git reset HEAD~3` it worked, but I haven't a clue what I did: https://review.opendev.org/c/openstack/openstacksdk/+/814369 | 08:43 |
| frickler | thomasb06: "! [remote rejected] HEAD -> refs/for/feature/r1 (no new changes)" indicates that you are trying so submit an empty commit | 08:43 |
| gtema | yeah, I also noticed that and was suspecting it | 08:43 |
| thomasb06 | frickler: indeed, my script generates a new _proxy_new.py, but in this case, the diff was null... | 08:46 |
| thomasb06 | thanks though | 08:46 |
| thomasb06 | message/v2: https://review.opendev.org/c/openstack/openstacksdk/+/814370 | 08:46 |
| thomasb06 | accelerator/v2: https://review.opendev.org/c/openstack/openstacksdk/+/814372 | 08:48 |
| thomasb06 | going for my meal, I'll do the rest this afternoon. Meanwhile, let's see what Zuul says... | 08:48 |
| gtema | ok | 08:49 |
| *** ministry is now known as __ministry | 08:56 | |
| *** mnasiadka_ is now known as mnasiadka | 10:10 | |
| thomasb06 | baremetal/v1: https://review.opendev.org/c/openstack/openstacksdk/+/814395 | 10:16 |
| thomasb06 | identity/v2: https://review.opendev.org/c/openstack/openstacksdk/+/814396 | 10:18 |
| thomasb06 | identity/v3: https://review.opendev.org/c/openstack/openstacksdk/+/814398 | 10:19 |
| thomasb06 | key_manager/v1: https://review.opendev.org/c/openstack/openstacksdk/+/814399 | 10:21 |
| thomasb06 | Arg, `:rtype:` is also an entry... | 10:24 |
| gtema | stephenfin, diablo_rojo_phone joining the meeting? | 16:03 |
| opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch server backup to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/814332 | 16:04 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!