Issue #1993: Add version field to monitor action - copr

copr / copr

#1993 Add version field to monitor action

Closed: Fixed 2 years ago by praiskup. Opened 2 years ago by iucar.

#1953 fixed #1932 nicely, but I noticed I'm missing an important field that is present in the web monitor/detailed view: the version of the package. It should be inexpensive to add, and it allows us to detect version mismatches across chroots.

Metadata Update from @praiskup:
- Issue assigned to praiskup
- Issue tagged with: API, RFE

2 years ago

praiskup commented 2 years ago

I agree, thanks for the suggestion.

praiskup commented 2 years ago

The only problem I see is that there's a misleading "dist" tag:

    {
        "build_id": 2846764,
        "chroot": "fedora-35-x86_64",
        "name": "R-CRAN-RClone",
        "pkg_version": "1.0.3-1.fc34",
        "state": "succeeded"
    }

The thing is that we store the "NVR" to Build, not BuildChroot ... so only one BuildChroot among all assigned to Build wins, and the NVR (including dist tag) is used. But the same
happens for WebUI monitor (and I think there used to be an issue related to bus messages..).