PostgreSQL version decreased in CentOs7

[Alexander Kislitsky]

Current version of the PostgreSQL is 9.2: postgresql-server-9.2.14-1.el7_1.x86_64
On CentOs6 we had 9.3 version. It is critical to have the same DB version in CentOs7.

In the PostgreSQL yum repo 9.3 and 9.4 versions are available:

http://yum.postgresql.org/repopackages.php

[Roman Vyalov (r0mikiam)]

Why PostgreSQL version 9.3 is blocker for you ?

[Dmitry Teselkin (teselkin-d)]

Don't think it's critical unless there are features in 9.3 that are missed in 9.2 but needed for us.
Upgrade from mos7 to mos8 likely to be done in a way of data migration, so we could eliminate the difference between databases.

[Dmitry Teselkin (teselkin-d)]

Alexander, please provide any reason why it's critical.

[Artem Silenkov (asilenkov)]

Postgresql should be rebuild with 9.3. It is critical, downgrade is hardly possible in case of master node upgrades.

[Roman Vyalov (r0mikiam)]

FYI related bug: https://bugs.launchpad.net/fuel/+bug/1311030

[Fuel Devops McRobotson (fuel-devops-robot)]

Related fix proposed to branch: master
Change author: Artem Silenkov <email address hidden>
Review: https://review.fuel-infra.org/14470

[Sergii Golovatiuk (sgolovatiuk)]

As far as I know we do not use functionality of 9.3. Also there won't be upgrade from CentOs 6 master nodes to CentOs 7. The original plan is to dump configuration, create a new master node on CentOS 7 and load configuration.

[Artem Silenkov (asilenkov)]

Still. It's not polite to downgrade software version in new product release version without very strong reasons. Let's rebuild.

[Alexander Kislitsky (akislitsky)]

We are using JSON DB data type [1]. It is provided by PostgreSQL version >= 9.3. Also JSON is required for fix [2].

[1] https://github.com/openstack/fuel-web/blob/a95a0bc965c11b0d412a00c4cb088888b919e054/nailgun/nailgun/db/sqlalchemy/models/master_node_settings.py#L20
[2] https://bugs.launchpad.net/fuel/+bug/1482399

[Dmitry Teselkin (teselkin-d)]

I wonder how it [1] works now with 9.2 on CentOS7 master node.

JSON was added in postgresql 9.2 [2] and in 9.3 was improved [3]

[1] https://github.com/openstack/fuel-web/blob/a95a0bc965c11b0d412a00c4cb088888b919e054/nailgun/nailgun/db/sqlalchemy/models/master_node_settings.py#L20
[2] https://wiki.postgresql.org/wiki/What's_new_in_PostgreSQL_9.2#JSON_datatype
[3] https://wiki.postgresql.org/wiki/What's_new_in_PostgreSQL_9.3#JSON:_Additional_functionality

[Artem Silenkov (asilenkov)]

Let me remind that 9.2 was no tested, only 9.3 we have in Fuel 7.0 and fuel 6.1. We could bring a lot of regressions with version downgrade. I insist on repackaging to 9.3 at least.

[Ihor Kalnytskyi (ikalnytskyi)]

Folks,

First of all we don't use JSONB [1], we use JSON [2] type. As Dmitry T. has pointed, the JSON type was introduced in PostgreSQL 9.2. That means, I see no reason why we need to upgrade PostgreSQL. Let's be as close as possible to distributive package, and do not maintain our own one.

[1] http://docs.sqlalchemy.org/en/latest/dialects/postgresql.html#sqlalchemy.dialects.postgresql.JSONB
[2] http://docs.sqlalchemy.org/en/latest/dialects/postgresql.html#sqlalchemy.dialects.postgresql.JSON

I believe we should move it to Invalid.

[Alexander Kislitsky (akislitsky)]

Igor, operations with JSON introduced in PostgreSQL 9.2 definitely are not enough. There are no indexing and searching by JSON value content. As Artem S. pointed, 9.3 was in the Fuel 7.0 and Fuel 6.1 and was tested. Do we really want to introduce unexpected regression by introducing not tested old version of the DB? Is this risk is acceptable only for saving provided by default package?

I want to point, that current stable version of PostgeSQL is 9.4.

[Fuel Devops McRobotson (fuel-devops-robot)]

Reviewed: https://review.fuel-infra.org/14470
Submitter: Mateusz Matuszkowiak <email address hidden>
Branch: master

Commit: 7d41d8d5bd55e8e7705184751c9535f793267ce9
Author: Artem Silenkov <email address hidden>
Date: Fri Dec 11 10:41:12 2015

postgresql added for centos7 liberty 8.0

* packages/centos7/postgresql

Change-Id: I2afb1a4a62541286f7b895593aef55205c4ea7e2
Related-Bug: #1523544

[Fuel Devops McRobotson (fuel-devops-robot)]

Related fix proposed to branch: 8.0
Change author: Artem Silenkov <email address hidden>
Review: https://review.fuel-infra.org/14623

[Fuel Devops McRobotson (fuel-devops-robot)]

Reviewed: https://review.fuel-infra.org/14623
Submitter: Pkgs Jenkins <email address hidden>
Branch: 8.0

Commit: 70dbe36a5778352097f1c4c3a465b36a5a51567c
Author: Artem Silenkov <email address hidden>
Date: Fri Dec 11 19:51:11 2015

postgresql-server-9.3 packaged for centos7 liberty 8.0

* sources are taken from http://koji.fedoraproject.org/koji/packageinfo?packageID=655

Change-Id: I7580a400797f63bd642ff46e868e25f6c4a31187
Related-Bug: #1523544

[Artem Silenkov (asilenkov)]

Please reply if any objections. Package removal is manual procedure.

[Alexander Zatserklyany (zatserklyany)]

[root@nailgun ~]# find / -type f -name postgresql*.rpm
/var/www/nailgun/liberty-8.0/mos-centos/x86_64/Packages/postgresql-upgrade-9.3.10-1.el7~mos2.x86_64.rpm
/var/www/nailgun/liberty-8.0/mos-centos/x86_64/Packages/postgresql-9.3.10-1.el7~mos2.x86_64.rpm
/var/www/nailgun/liberty-8.0/mos-centos/x86_64/Packages/postgresql-plperl-9.3.10-1.el7~mos2.x86_64.rpm
/var/www/nailgun/liberty-8.0/mos-centos/x86_64/Packages/postgresql-libs-9.3.10-1.el7~mos2.x86_64.rpm
/var/www/nailgun/liberty-8.0/mos-centos/x86_64/Packages/postgresql-test-9.3.10-1.el7~mos2.x86_64.rpm
/var/www/nailgun/liberty-8.0/mos-centos/x86_64/Packages/postgresql-plpython-9.3.10-1.el7~mos2.x86_64.rpm
/var/www/nailgun/liberty-8.0/mos-centos/x86_64/Packages/postgresql-devel-9.3.10-1.el7~mos2.x86_64.rpm
/var/www/nailgun/liberty-8.0/mos-centos/x86_64/Packages/postgresql-pltcl-9.3.10-1.el7~mos2.x86_64.rpm
/var/www/nailgun/liberty-8.0/mos-centos/x86_64/Packages/postgresql-docs-9.3.10-1.el7~mos2.x86_64.rpm
/var/www/nailgun/liberty-8.0/mos-centos/x86_64/Packages/postgresql-server-9.3.10-1.el7~mos2.x86_64.rpm
/var/www/nailgun/liberty-8.0/mos-centos/x86_64/Packages/postgresql-contrib-9.3.10-1.el7~mos2.x86_64.rpm
[root@nailgun ~]# cat /etc/fuel/version.yaml
VERSION:
  feature_groups:
    - mirantis
  production: "docker"
  release: "8.0"
  api: "1.0"
  build_number: "461"