Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-63670

Confusing warning about PostgreSQL version

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Open (View Workflow)
    • Priority: Minor
    • Resolution: Unresolved
    • Component/s: pipeline-maven-plugin
    • Labels:
      None
    • Environment:
      Jenkins 2.249.1
      org.jenkins-ci.main:maven-plugin:3.7
    • Similar Issues:

      Description

      I followed the "Using a PostgreSQL database" guide on https://plugins.jenkins.io/pipeline-maven/ to replace my current H2 database. It seems that everything is set up properly, but still I get a warning

      Non tested PostgreSQL version 12.2. MySQL Server version 5.7+ or Amazon Aurora MySQL 5.6+ or MariaDB 10.2+ or PostgreSQL 10+ is required

      I indeed have PostgreSQL version 12.2, so why is Jenkins complaining and telling me PostgreSQL 10+ is required? Isn't 12.2 more than 10?

      > psql --version
      psql (PostgreSQL) 12.2
      

       

      Here a screenshot of my current configuration:

        Attachments

          Activity

          Hide
          dokaspar Dominik Kaspar added a comment -

          It looks like the versions 10 and 11 are hard-coded and version 12 is simply missing:
          https://github.com/jenkinsci/pipeline-maven-plugin/blob/master/jenkins-plugin/src/main/java/org/jenkinsci/plugins/pipeline/maven/GlobalPipelineMavenConfig.java#L559

          switch (metaData.getDatabaseMajorVersion()) {
              case 11:
              case 10:
                  // OK
                  break;
              default:
                  return FormValidation.warning("Non tested PostgreSQL version " + metaData.getDatabaseProductVersion() + ". " + databaseRequirement);
          }

           

          Show
          dokaspar Dominik Kaspar added a comment - It looks like the versions 10 and 11 are hard-coded and version 12 is simply missing: https://github.com/jenkinsci/pipeline-maven-plugin/blob/master/jenkins-plugin/src/main/java/org/jenkinsci/plugins/pipeline/maven/GlobalPipelineMavenConfig.java#L559 switch (metaData.getDatabaseMajorVersion()) { case 11: case 10: // OK break; default: return FormValidation.warning("Non tested PostgreSQL version " + metaData.getDatabaseProductVersion() + ". " + databaseRequirement); }  
          Hide
          aheritier Arnaud Héritier added a comment -

          Hi Dominik Kaspar

          Thanks for reporting the issue.

          Effectively we probably didn't test it with PostgreSQL 12 and thus it was referenced as a valid version.

          I will try to do some tests ASAP and update the plugin and also the doc.

          The message is saying correctly it's not tested (not that it's not supported or not working) but we could highlight it the doc.

          Potentially maybe it could be an info message instead of a warning but not sure.

          In general DB upgrades are safe but ... who knows

           

          Show
          aheritier Arnaud Héritier added a comment - Hi Dominik Kaspar Thanks for reporting the issue. Effectively we probably didn't test it with PostgreSQL 12 and thus it was referenced as a valid version. I will try to do some tests ASAP and update the plugin and also the doc. The message is saying correctly it's not tested (not that it's not supported or not working) but we could highlight it the doc. Potentially maybe it could be an info message instead of a warning but not sure. In general DB upgrades are safe but ... who knows  
          Show
          ianw Ian Williams added a comment - Probably unrelated coincidwnce .. AWS Aurora PostgreSQL versions vanish from the mega-cloud for days, leaving customers in the dark    
          Hide
          jansohn Robin Jansohn added a comment -

          Arnaud Héritier any update on those postgres12 tests? We are also in the process of upgrading our postgres instances to 12 or even 13...

          Show
          jansohn Robin Jansohn added a comment - Arnaud Héritier any update on those postgres12 tests? We are also in the process of upgrading our postgres instances to 12 or even 13...
          Hide
          aheritier Arnaud Héritier added a comment -

          no sorry Robin JansohnI didn't have the time to do some tests.

          I will try to do some this month (but not promised - I have already a lot of things to do on my personal time)

          Show
          aheritier Arnaud Héritier added a comment - no sorry Robin Jansohn I didn't have the time to do some tests. I will try to do some this month (but not promised - I have already a lot of things to do on my personal time)
          Hide
          hadjiyski Robert Hadjiyski added a comment -

          Hi Arnaud Héritier, same here:

          Non tested PostgreSQL version 13.1. MySQL Server version 5.7+ or Amazon Aurora MySQL 5.6+ or MariaDB 10.2+ or PostgreSQL 10+ is required
          could you please check. Thx.

          Show
          hadjiyski Robert Hadjiyski added a comment - Hi Arnaud Héritier , same here: Non tested PostgreSQL version 13.1. MySQL Server version 5.7+ or Amazon Aurora MySQL 5.6+ or MariaDB 10.2+ or PostgreSQL 10+ is required could you please check. Thx.

            People

            Assignee:
            aheritier Arnaud Héritier
            Reporter:
            dokaspar Dominik Kaspar
            Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated: