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.
          Hide
          jpara29 PJ added a comment -

          Hello Arnaud Héritier

          Could you please let me know if these tests are completed and are good to go for PostGreSQL 12+ ? We are planning to upgrade to higher Postgresql version (from 11.3) and wanted to know if moving to 12/13 would help or not.

          Show
          jpara29 PJ added a comment - Hello Arnaud Héritier Could you please let me know if these tests are completed and are good to go for PostGreSQL 12+ ? We are planning to upgrade to higher Postgresql version (from 11.3) and wanted to know if moving to 12/13 would help or not.
          Hide
          aheritier Arnaud Héritier added a comment -

          Sadly it is still on my too long todo list.

          What I would like is to automate these tests to be sure that we cover a good number of DBs/Versions ...

          Show
          aheritier Arnaud Héritier added a comment - Sadly it is still on my too long todo list. What I would like is to automate these tests to be sure that we cover a good number of DBs/Versions ...

            People

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

              Dates

              Created:
              Updated: