Add gitea release handling

[Mai-Lapyst]

This PR adds the ability to discover releases in gitea as "branches" to be build seperatly.

It also adds an task for pipelines ("publishGiteaAssets") which works a little like copyArtifacts but uploads artifacts as an asset to the current building release.

The PR also contains an (toggleable) feature to "map" artifacts originally created via copyArtifacts to gitea release assets.

Relevant issues:

• JENKINS-67790 - Upload built artifacts

---

• Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
• Ensure that the pull request title represents the desired changelog entry
• Please describe what you did
• Link to relevant issues in GitHub or Jira
• Link to relevant pull requests, esp. upstream and downstream changes
• Ensure you have provided tests - that demonstrates feature works or fixes the issue

[lafriks]

Please fix lint errors

[Mai-Lapyst]

Please fix lint errors

Should be fixed now

[lafriks]

Is this still a WIP or finished? (asking because there is quite some TODO left in code)

[Mai-Lapyst]

Is this still a WIP or finished? (asking because there is quite some TODO left in code)

It's finished. I left the TODO's mostly there to note things for future additions or to match the rest of the file (in case of MockGiteaConnection).

[lafriks]

I would say draft releases should be skipped and removed from this as in the future it's planned that gitea won't issue webhooks for draft releases (go-gitea/gitea#21594)

[Mai-Lapyst]

I would say draft releases should be skipped and removed from this as in the future it's planned that gitea won't issue webhooks for draft releases (go-gitea/gitea#21594)

Yeah makes sense; will update the PR in the coming days.

[RokkuCode]

any news on this when it will merged? this change would help us in handling releases with gitea.

[RokkuCode]

@lafriks are there blockers why this pr is not merged yet?

[Mai-Lapyst]

Rebased pr to current master

[justusbunsi]

@Mai-Lapyst @lafriks As mentioned in #67, my apologies for not responding earlier. I received the "review requested" notification back then and simply missed acting upon it after reading it. Reviewing and testing your changes now.

[justusbunsi]

@Mai-Lapyst Thank you for this great contribution.

It looks like the current implementation does not fully take the "prerelease" flag into account. Instead, scanning the repository triggers all releases anyway. I've suggested changes to fixes that behavior.

And there is something off with publishGiteaAssets. I am not sure what exactly. Do you have an idea?

[Mai-Lapyst]

Added a additional patch since the generation of the url for fetching releases was still not correct; but this time bc gitea interprets draft=true as a way to ONLY include drafts in the response. Should be fixed now by only specifing draft=false when no drafts should be fetched. Same goes for the pre-release flag.

[AdamRGrey]

@justusbunsi not to annoy, but plz pretty plz

(i'm looking forward to this feature)

[justusbunsi]

@justusbunsi not to annoy, but plz pretty plz

(i'm looking forward to this feature)

I'll review until the end of week. My daily schedule did not allow rewiewing earlier. Should've commented earlier. 🙂

[justusbunsi]

@Mai-Lapyst, thanks for the deep dive regarding asset uploads from local machine. It is working for me now, too. 👍

I've noticed one last detail that should be added in this PR: Commit status informing about the build state of a release - like it's done for tags. It requires a change in GiteaNotifier.java which is unchanged and I don't know how to suggest the changes in unchanged files via GitHub UI. So here are my suggestions for this missing part. As soon as release builds send their status to Gitea: This PR is going to LGTM 😃

diff --git a/src/main/java/org/jenkinsci/plugin/gitea/GiteaNotifier.java b/src/main/java/org/jenkinsci/plugin/gitea/GiteaNotifier.java
index 23ee4a3..eb45482 100644
--- a/src/main/java/org/jenkinsci/plugin/gitea/GiteaNotifier.java
+++ b/src/main/java/org/jenkinsci/plugin/gitea/GiteaNotifier.java
@@ -132,6 +132,11 @@ public class GiteaNotifier {
                     status.getState().name(), status.getDescription());
             hash = ((TagSCMRevision) revision).getHash();
             statusContext += "tag";
+        } else if (revision instanceof ReleaseSCMRevision) {
+            listener.getLogger().format("[Gitea] Notifying release build status: %s %s%n",
+                    status.getState().name(), status.getDescription());
+            hash = ((ReleaseSCMRevision) revision).getHash();
+            statusContext += "release";
         } else {
             return;
         }
@@ -237,6 +242,10 @@ public class GiteaNotifier {
                             LOGGER.log(Level.INFO, "Notifying tag pending build {0}", job.getFullName());
                             statusContext += "tag";
                             hash = ((TagSCMRevision) revision).getHash();
+                        } else if (revision instanceof ReleaseSCMRevision) {
+                            LOGGER.log(Level.INFO, "Notifying release pending build {0}", job.getFullName());
+                            statusContext += "release";
+                            hash = ((ReleaseSCMRevision) revision).getHash();
                         } else {
                             return;
                         }

[Mai-Lapyst]

@justusbunsi Have added the requested changes; should work now :)

[justusbunsi]

🎉