[superset] 11/11: fix: migrate_roles (#16098)

[vi...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

villebro pushed a commit to branch 1.3
in repository https://gitbox.apache.org/repos/asf/superset.git

commit 6a850be59699a6809ab392b30fb2ac0fccbf5259
Author: Beto Dealmeida <ro...@dealmeida.net>
AuthorDate: Fri Aug 6 10:05:19 2021 -0700

    fix: migrate_roles (#16098)
    
    (cherry picked from commit 28c383af68d92bdbfe69fe8c90f8f6a769985b2b)
---
 superset/migrations/versions/f6196627326f_update_chart_permissions.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/superset/migrations/versions/f6196627326f_update_chart_permissions.py b/superset/migrations/versions/f6196627326f_update_chart_permissions.py
index f9a0904..9a24138 100644
--- a/superset/migrations/versions/f6196627326f_update_chart_permissions.py
+++ b/superset/migrations/versions/f6196627326f_update_chart_permissions.py
@@ -38,6 +38,7 @@ from superset.migrations.shared.security_converge import (
 revision = "f6196627326f"
 down_revision = "143b6f2815da"
 
+NEW_PVMS = {"Chart": ("can_read",)}
 PVM_MAP = {
     Pvm("Chart", "can_get_data"): (Pvm("Chart", "can_read"),),
     Pvm("Chart", "can_post_data"): (Pvm("Chart", "can_read"),),
@@ -49,6 +50,7 @@ def upgrade():
     session = Session(bind=bind)
 
     # Add the new permissions on the migration itself
+    add_pvms(session, NEW_PVMS)
     migrate_roles(session, PVM_MAP)
     try:
         session.commit()