Comment 26 for bug 1939723

Hey Nikhil,

Thanks for performing the verifications in such a detailed manner.

I happened to be looking at this SRU/bug and had some observations:

> the patch verification testing details are in https://pastebin.canonical.com/p/2J6C9tx4np/

Please always keep the verification steps/logging (if needed) in the bug
(as comments or attachments, as appropriate), for documentation purposes.

Even though pastebins are useful, they might be removed, and in this case,
canonical's pastebin isn't available externally (ubuntu's pastebin is),
which restricts the availability to other people/community members.

> I have tested this fix on focal/ussuri [...]
> Since the patch is exactly the same for the cloud archive, I am marking that done as well.

Although this is ultimately up to the Cloud Archive's release team, I would like to suggest the Cloud Archive to be separately verified too.

The reason is, there is more involved in the verification than just the source code change alone.
Specially in the case of the Cloud Archive, there are different (more recent) package versions that may be pulled and are exercised as part of the verification steps (e.g., python dependencies or other libraries).
Thus in practice, the actually verified code is different, even if the patch is the same.

Thanks!
Mauricio