fix: correct input reference and upload-logs condition in release workflow (3.x)

Two pre-existing bugs in .github/workflows/release.yml:

1. 'Checkout Code One Commit Before' step referenced `inputs.version_tag`
   in both the step name and RELEASE_TARGET_TAG env var. The actual input
   is named `target-tag`. Because the wrong expression always evaluates to
   empty string, the step name appeared as 'Checkout Code One Commit Before '
   (trailing space) in CI, and RELEASE_TARGET_TAG was always empty — meaning
   any re-release triggered with a specific tag would silently check out the
   wrong commit.

   Confirmed in run #129 (https://github.com/scylladb/java-driver/actions/runs/25297315236):
   step 3 is logged as 'Checkout Code One Commit Before ' with no tag value.

2. 'Upload release logs' had `if: failure()` so the artifact was skipped on
   successful runs. Run #129 shows step 9 'Upload release logs' as skipped
   despite the release completing successfully. Release logs are useful as an
   audit trail regardless of outcome (scylla-4.x already uploads
   unconditionally). Remove the condition to align both branches.

Author: nikagra

.github/workflows/release.yml

@@ -33,10 +33,10 @@ jobs:
       - name: Checkout Repository
         uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4.3.1
 
-      - name: Checkout Code One Commit Before ${{ inputs.version_tag }}
+      - name: Checkout Code One Commit Before ${{ inputs.target-tag }}
         if: inputs.target-tag != 'scylla-3.x'
         env:
-          RELEASE_TARGET_TAG: ${{ inputs.version_tag }}
+          RELEASE_TARGET_TAG: ${{ inputs.target-tag }}
         run: make checkout-one-commit-before
 
       - name: Set up Java
@@ -81,7 +81,6 @@ jobs:
         run: make release-dry-run
 
       - name: Upload release logs
-        if: failure()
         uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
         with:
           name: maven-stdout