Merge branch 'main' into update-to-quarkus-3.34.1-gradle-wrapper-9.4.1-and-update-dependencies

# Conflicts:
#	gradle/wrapper/gradle-wrapper.properties
#	gradlew

Author: ThoSap

build.gradle.kts

@@ -83,6 +83,7 @@ subprojects {
 
         options.errorprone {
             check("NullAway", CheckSeverity.ERROR)
+            check("RequireExplicitNullMarking", CheckSeverity.ERROR)
             option("NullAway:AnnotatedPackages", "it.aboutbits.postgresql")
             option("NullAway:JSpecifyMode", "true")
         }

gradle/libs.versions.toml

@@ -15,7 +15,7 @@ scram-client = "3.2"
 
 ## Testing ##
 assertj = "3.27.7"
-checkstyle = "13.2.0"
+checkstyle = "13.3.0"
 datafaker = "2.5.4"
 errorProne = "2.48.0"
 errorPronePlugin = "5.1.0"

operator/src/main/java/it/aboutbits/postgresql/PostgreSQLInstanceReadinessCheck.java

@@ -15,9 +15,9 @@
  * configured PostgreSQL instances. Each instance is probed with a lightweight
  * operation, and the aggregated status is exposed.
  */
-@NullMarked
 @Readiness
 @RequiredArgsConstructor
+@NullMarked
 public class PostgreSQLInstanceReadinessCheck implements HealthCheck {
     private final PostgreSQLContextFactory postgreSQLContextFactory;
 

operator/src/main/java/it/aboutbits/postgresql/core/BaseReconciler.java

@@ -14,8 +14,8 @@
 import java.util.Optional;
 import java.util.concurrent.TimeUnit;
 
-@NullMarked
 @Slf4j
+@NullMarked
 public abstract class BaseReconciler<CR extends CustomResource<?, S> & Named, S extends CRStatus> {
     protected abstract S newStatus();
 

operator/src/main/java/it/aboutbits/postgresql/core/CRStatus.java

@@ -15,16 +15,15 @@
  * <p>
  * This object captures the current state of a Custom Resource as observed by the reconciler.
  */
-@NullMarked
 @Getter
 @Setter
 @Accessors(chain = true)
+@NullMarked
 public class CRStatus {
     /**
      * The Custom Resource name (may differ from metadata.name).
      */
-    @Nullable
-    private String name = null;
+    private @Nullable String name = null;
 
     /**
      * Current lifecycle phase of the Bucket.
@@ -35,21 +34,18 @@ public class CRStatus {
     /**
      * Human-readable message providing details about the current state.
      */
-    @Nullable
-    private String message = null;
+    private @Nullable String message = null;
 
     /**
      * Last time the condition was probed/updated.
      */
-    @Nullable
-    private OffsetDateTime lastProbeTime = null;
+    private @Nullable OffsetDateTime lastProbeTime = null;
 
     /**
      * Last time the condition transitioned from one status to another.
      */
-    @Nullable
     @Setter(AccessLevel.NONE)
-    private OffsetDateTime lastPhaseTransitionTime = null;
+    private @Nullable OffsetDateTime lastPhaseTransitionTime = null;
 
     /**
      * Observed resource generation that the controller acted upon.

operator/src/main/java/it/aboutbits/postgresql/core/KubernetesService.java

@@ -8,8 +8,8 @@
 import java.nio.charset.Charset;
 import java.util.Base64;
 
-@NullMarked
 @Singleton
+@NullMarked
 public final class KubernetesService {
     public static final String SECRET_TYPE_BASIC_AUTH = "kubernetes.io/basic-auth";
     public static final String SECRET_DATA_BASIC_AUTH_USERNAME_KEY = "username";

operator/src/main/java/it/aboutbits/postgresql/core/PostgreSQLAuthenticationService.java

@@ -21,9 +21,9 @@
 
 import static it.aboutbits.postgresql.core.infrastructure.persistence.Tables.PG_AUTHID;
 
-@NullMarked
 @Slf4j
 @Singleton
+@NullMarked
 public final class PostgreSQLAuthenticationService {
     private static final String MD5 = "MD5";
     private static final String SHA_256 = "SHA-256";

operator/src/main/java/it/aboutbits/postgresql/core/PostgreSQLContextFactory.java

@@ -11,9 +11,9 @@
 
 import java.util.Properties;
 
-@NullMarked
 @ApplicationScoped
 @RequiredArgsConstructor
+@NullMarked
 public class PostgreSQLContextFactory {
     private static final String POSTGRESQL_AUTHENTICATION_USER_KEY = "user";
     private static final String POSTGRESQL_AUTHENTICATION_PASSWORD_KEY = "password";

operator/src/main/java/it/aboutbits/postgresql/core/Privilege.java

@@ -15,10 +15,10 @@
  * https://www.postgresql.org/docs/current/sql-grant.html
  * </a>
  */
-@NullMarked
 @Getter
 @Accessors(fluent = true)
 @RequiredArgsConstructor
+@NullMarked
 public enum Privilege {
     SELECT(null),
     INSERT(null),
@@ -33,8 +33,7 @@ public enum Privilege {
     USAGE(null),
     MAINTAIN(17);
 
-    @Nullable
-    private final Integer minimumPostgresVersion;
+    private final @Nullable Integer minimumPostgresVersion;
 
     @JsonValue
     public String toValue() {

operator/src/main/java/it/aboutbits/postgresql/core/ReclaimPolicy.java

@@ -4,8 +4,8 @@
 import lombok.RequiredArgsConstructor;
 import org.jspecify.annotations.NullMarked;
 
-@NullMarked
 @RequiredArgsConstructor
+@NullMarked
 public enum ReclaimPolicy {
     RETAIN("Retain"),
     DELETE("Delete");