Revert "remote: fix manifest add --annotation"

[Luap99]

This reverts commit 32f54a8.

pkg/bindings is supported outside of podman and we have to keep it stable.

Does this PR introduce a user-facing change?

None

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Luap99

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [Luap99]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[Luap99]

We have to reopen #15952 once this is merged

[vrothberg]

I'd argue that fixing the issues is probably better than reverting but given we cut v4.3, let's better play it safe and revert to prevent releasing the changes.

LGTM

[mheon]

/lgtm
/hold

[edsantiago]

Oh, how nice, the api-change label bot is triggering.

[vrothberg]

Oh, how nice, the api-change label bot is triggering.

I changed the label color to red. That'll hopefully catch my attention next time.

[edsantiago]

I disagree pretty strongly about that: there is way, way too much red in the labels already, I firmly believe that another red is just going to be ignored even more. I deliberately set the color to an orange-yellow because that's still a danger signal to human eyes, and it's a color that is not otherwise used in the labels.

I will not undo your change, though. I am just pointing out a human-factors problem, will leave for others to discuss and fix.

[vrothberg]

Apologies, Ed. I didn't know the yellow/orange was deliberate. To me/on my screen, it looks yellow which does not indicate any danger to me.

[edsantiago]

No apology needed - I clearly chose poor colors. (I am not a UI expert). Someone with better UI skills should look into this.

[rhatdan]

/hold cancel