yahoo-eng-team team mailing list archive

[Nikhil Komawar <nik.komawar@xxxxxxxxx>]

Public bug reported:

We have removed s3 references in patch
https://review.openstack.org/#/c/348507/

However there are a few incorrect references of s3 in the below
mentioned migration scripts.

We have a project policy to not modify the migration scripts unless a
significant impact exists.
http://docs.openstack.org/developer/glance/contributing/minor-code-
changes.html#database-migration-scripts

So, this bug has been opened to gather feedback / opinions on the need
for this change.

https://github.com/openstack/glance/blob/d0f4316752d313bffeafd2be0283201813e1e9a2/glance/db/sqlalchemy/migrate_repo/versions/015_quote_swift_credentials.py#L157

https://github.com/openstack/glance/blob/d0f4316752d313bffeafd2be0283201813e1e9a2/glance/db/sqlalchemy/migrate_repo/versions/017_quote_encrypted_swift_credentials.py#L219

** Affects: glance
     Importance: Low
         Status: New

** Changed in: glance
   Importance: Undecided => Low

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

Title:
  incorrect references of s3 in migration scripts

Status in Glance:
  New

Bug description:
  We have removed s3 references in patch
  https://review.openstack.org/#/c/348507/

  However there are a few incorrect references of s3 in the below
  mentioned migration scripts.

  We have a project policy to not modify the migration scripts unless a
  significant impact exists.
  http://docs.openstack.org/developer/glance/contributing/minor-code-
  changes.html#database-migration-scripts

  So, this bug has been opened to gather feedback / opinions on the need
  for this change.

  https://github.com/openstack/glance/blob/d0f4316752d313bffeafd2be0283201813e1e9a2/glance/db/sqlalchemy/migrate_repo/versions/015_quote_swift_credentials.py#L157

  https://github.com/openstack/glance/blob/d0f4316752d313bffeafd2be0283201813e1e9a2/glance/db/sqlalchemy/migrate_repo/versions/017_quote_encrypted_swift_credentials.py#L219

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