← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1288681] [NEW] Migration with alter_column function reset nullable parameter for columns

 

Public bug reported:

Migrations 338d7508968c_vpnaas_peer_address_.py and
abc88c33f74f_lb_stats_needs_bigint.py correct columns' length and types,
but nullable=False parameter is lost after its upgrade() :

mysql> explain ipsec_site_connections;
+----------------+-------------------------------------------------------------+------+-----+---------+-------+
| Field          | Type                                                        | Null | Key | Default | Extra |
+----------------+-------------------------------------------------------------+------+-----+---------+-------+
| tenant_id      | varchar(255)                                                | YES  |     | NULL    |       |
| id             | varchar(36)                                                 | NO   | PRI | NULL    |       |
| name           | varchar(255)                                                | YES  |     | NULL    |       |
| description    | varchar(255)                                                | YES  |     | NULL    |       |
| peer_address   | varchar(255)                                                | YES  |     | NULL    |       |
| peer_id        | varchar(255)                                                | NO   |     | NULL    |       |
| route_mode     | varchar(8)                                                  | NO   |     | NULL    |       |
| mtu            | int(11)                                                     | NO   |     | NULL    |       |
| initiator      | enum('bi-directional','response-only')                      | NO   |     | NULL    |       |
| auth_mode      | varchar(16)                                                 | NO   |     | NULL    |       |
| psk            | varchar(255)                                                | NO   |     | NULL    |       |
| dpd_action     | enum('hold','clear','restart','disabled','restart-by-peer') | NO   |     | NULL    |       |
| dpd_interval   | int(11)                                                     | NO   |     | NULL    |       |
| dpd_timeout    | int(11)                                                     | NO   |     | NULL    |       |
| status         | varchar(16)                                                 | NO   |     | NULL    |       |
| admin_state_up | tinyint(1)                                                  | NO   |     | NULL    |       |
| vpnservice_id  | varchar(36)                                                 | NO   | MUL | NULL    |       |
| ipsecpolicy_id | varchar(36)                                                 | NO   | MUL | NULL    |       |
| ikepolicy_id   | varchar(36)                                                 | NO   | MUL | NULL    |       |
+----------------+-------------------------------------------------------------+------+-----+---------+-------+
19 rows in set (0.00 sec)


mysql> explain poolstatisticss;
+--------------------+-------------+------+-----+---------+-------+
| Field              | Type        | Null | Key | Default | Extra |
+--------------------+-------------+------+-----+---------+-------+
| pool_id            | varchar(36) | NO   | PRI | NULL    |       |
| bytes_in           | bigint(20)  | YES  |     | NULL    |       |
| bytes_out          | bigint(20)  | YES  |     | NULL    |       |
| active_connections | bigint(20)  | YES  |     | NULL    |       |
| total_connections  | bigint(20)  | YES  |     | NULL    |       |
+--------------------+-------------+------+-----+---------+-------+

** Affects: neutron
     Importance: Undecided
     Assignee: Ann Kamyshnikova (akamyshnikova)
         Status: New


** Tags: db

** Changed in: neutron
     Assignee: (unassigned) => Ann Kamyshnikova (akamyshnikova)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1288681

Title:
  Migration with alter_column function reset nullable parameter for
  columns

Status in OpenStack Neutron (virtual network service):
  New

Bug description:
  Migrations 338d7508968c_vpnaas_peer_address_.py and
  abc88c33f74f_lb_stats_needs_bigint.py correct columns' length and
  types, but nullable=False parameter is lost after its upgrade() :

  mysql> explain ipsec_site_connections;
  +----------------+-------------------------------------------------------------+------+-----+---------+-------+
  | Field          | Type                                                        | Null | Key | Default | Extra |
  +----------------+-------------------------------------------------------------+------+-----+---------+-------+
  | tenant_id      | varchar(255)                                                | YES  |     | NULL    |       |
  | id             | varchar(36)                                                 | NO   | PRI | NULL    |       |
  | name           | varchar(255)                                                | YES  |     | NULL    |       |
  | description    | varchar(255)                                                | YES  |     | NULL    |       |
  | peer_address   | varchar(255)                                                | YES  |     | NULL    |       |
  | peer_id        | varchar(255)                                                | NO   |     | NULL    |       |
  | route_mode     | varchar(8)                                                  | NO   |     | NULL    |       |
  | mtu            | int(11)                                                     | NO   |     | NULL    |       |
  | initiator      | enum('bi-directional','response-only')                      | NO   |     | NULL    |       |
  | auth_mode      | varchar(16)                                                 | NO   |     | NULL    |       |
  | psk            | varchar(255)                                                | NO   |     | NULL    |       |
  | dpd_action     | enum('hold','clear','restart','disabled','restart-by-peer') | NO   |     | NULL    |       |
  | dpd_interval   | int(11)                                                     | NO   |     | NULL    |       |
  | dpd_timeout    | int(11)                                                     | NO   |     | NULL    |       |
  | status         | varchar(16)                                                 | NO   |     | NULL    |       |
  | admin_state_up | tinyint(1)                                                  | NO   |     | NULL    |       |
  | vpnservice_id  | varchar(36)                                                 | NO   | MUL | NULL    |       |
  | ipsecpolicy_id | varchar(36)                                                 | NO   | MUL | NULL    |       |
  | ikepolicy_id   | varchar(36)                                                 | NO   | MUL | NULL    |       |
  +----------------+-------------------------------------------------------------+------+-----+---------+-------+
  19 rows in set (0.00 sec)

  
  mysql> explain poolstatisticss;
  +--------------------+-------------+------+-----+---------+-------+
  | Field              | Type        | Null | Key | Default | Extra |
  +--------------------+-------------+------+-----+---------+-------+
  | pool_id            | varchar(36) | NO   | PRI | NULL    |       |
  | bytes_in           | bigint(20)  | YES  |     | NULL    |       |
  | bytes_out          | bigint(20)  | YES  |     | NULL    |       |
  | active_connections | bigint(20)  | YES  |     | NULL    |       |
  | total_connections  | bigint(20)  | YES  |     | NULL    |       |
  +--------------------+-------------+------+-----+---------+-------+

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1288681/+subscriptions


Follow ups

References