Move pigz to done in arch migrators

[jakirkham]

Fixes regro/cf-scripts#1888

[beckermr]

Remember you have to edit the pr info files like last time.

[jakirkham]

Sorry Matt. I'm not following. Should there be PRs tracked if the bot errored?

[beckermr]

No but the pre pr migration errors are in those files.

[jakirkham]

Ok here are the files I see, which of these should we update?

$ git grep -l 'pigz'
all_feedstocks.json
graph.json
node_attrs/6/6/3/3/3/xopen.json
node_attrs/b/a/3/f/f/makeself.json
node_attrs/d/2/5/f/0/pigz.json
ranked_hubs_authorities.json
status/migration_json/aarch64andppc64leaddition.json
status/migration_json/armosxaddition.json
status/migration_svg/aarch64andppc64leaddition.svg
status/migration_svg/armosxaddition.svg

[beckermr]

That grep doesn't search the file names, only their contents. Look for the file names with the package name and you'll find the right spot.

[jakirkham]

Ah ok! That makes more sense 😅 Thanks Matt! 🙏

Now I see

$ git ls-files '*pigz*'
node_attrs/d/2/5/f/0/pigz.json
pr_info/d/2/5/f/0/pigz.json
version_pr_info/d/2/5/f/0/pigz.json
versions/d/2/5/f/0/pigz.json

Take it we want pr_info/d/2/5/f/0/pigz.json, right? Or am I still missing something?

[jakirkham]

How does that look?

Also the pr_json/* blobs referenced appear to already be removed. Guessing that is ok? Or do we need to recover those and modify them somehow?

[beckermr]

Look down at the bottom for the pre pr migrator status. Clear that for aarch too.

[jakirkham]

Meaning this? It appears empty. Or should the line be deleted altogether?

$ git grep 'pre_pr_migrator_status' -- pr_info/d/2/5/f/0/pigz.json
pr_info/d/2/5/f/0/pigz.json: "pre_pr_migrator_status": {},

[beckermr]

Ohhh weird. Let's merge and see what happens.

[jakirkham]

Sounds good. Thanks again Matt! 🙏

Happy to follow up as needed 🙂

[jakirkham]

Looks like the bot added pigz to awaiting PRs for both migrators in commit: 819cc3f

That's a good start. Let's see if it figures out that both are done already