Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade Prod Superset instance to version 4.0.1 #34

Closed
himeshr opened this issue Jul 18, 2024 · 8 comments
Closed

Upgrade Prod Superset instance to version 4.0.1 #34

himeshr opened this issue Jul 18, 2024 · 8 comments
Assignees

Comments

@himeshr
Copy link
Contributor

himeshr commented Jul 18, 2024

Acceptance Criteria

  • Upgrade our current prod superset to version 4.0.1. https://github.com/apache/superset/releases/tag/4.0.1
    • The metadata is currently stored on the ec2 instance itself, therefore, we'll need to ensure that we take a manual backup before we start with the upgradation work.
  • Also, setup automatic storage backups for the prod superset ec2 instance for recovery purposes
  • Configure the system to use required features
    • TAGGING_SYSTEM
    • Drill-By
    • Full CSV export
      Discuss with implementation team and update this section.

Fixing issue with existing report after upgradation

  1. Analyse report.
  2. If report broken then Categorise into type of issues .
  3. Document recovery steps for each issue.
  4. Apply fix in all broken reports.

Validation

  • All existing reports should not be broken, make changes in reports via sql commands or so to fix large scale modifications required

Tech details

  • Create a replica of prod superset ec2 instance
  • Upgrade to latest 4.1 version of superset
  • No Containers switch
  • No DB upgrade
  • Configure the system to use all new features
    • TAGGING_SYSTEM
    • Drill-By
    • Full CSV export
  • Compare the old and new systems and give it out for UAT to implementation team
  • After successful UAT, perform the similar upgradation on Prod Superset
  • Bring down the temp infra which is no longer required (Routes and EC2 instances)

Reference doc

https://preset.io/blog/apache-superset-4-0-release-notes/
https://preset.io/blog/superset-3-0-release-notes/

@github-project-automation github-project-automation bot moved this to New Issues in Avni Product Jul 18, 2024
@vedfordev vedfordev self-assigned this Jul 23, 2024
@vedfordev vedfordev moved this from New Issues to In Progress in Avni Product Jul 24, 2024
@vedfordev
Copy link
Contributor

vedfordev commented Jul 24, 2024

Created Documentation For Update File

Issues and Workaround Sheet

@vedfordev
Copy link
Contributor

vedfordev commented Jul 25, 2024

blocked by apache/superset#28020

Moving card to on hold

@vedfordev
Copy link
Contributor

move card to qa ready. implementation team will do sanity test. @himeshr @pkundu

@vedfordev vedfordev moved this from In Progress to QA Ready in Avni Product Aug 22, 2024
@himeshr
Copy link
Contributor Author

himeshr commented Oct 8, 2024

@dinesh2096 dinesh2096 moved this from QA Ready to In QA in Avni Product Oct 9, 2024
@dinesh2096
Copy link

QA Notes :

Charts

  • Mandatory field should be displayed - In the old env in aggregrate we showing the mandarory field but in the new env we didnt
  • Jinja Template is missing in the reports - In the old report we have the jinja added in the report but in the new env it is misisng
  • Under data we missed Include time - Under the data in the old env we have the option named as include time but missing in the new env
  • Total is missing in the result - In the charts under result the count is not displayed

Users :

  • Getting unexpected error - when we change the role for the user the report breaks

@dinesh2096 dinesh2096 moved this from In QA to QA Failed in Avni Product Oct 9, 2024
@dinesh2096 dinesh2096 self-assigned this Oct 9, 2024
@vedfordev
Copy link
Contributor

QA Notes :

Charts

  • Mandatory field should be displayed - In the old env in aggregrate we showing the mandarory field but in the new env we didnt
  • Jinja Template is missing in the reports - In the old report we have the jinja added in the report but in the new env it is misisng
  • Under data we missed Include time - Under the data in the old env we have the option named as include time but missing in the new env
  • Total is missing in the result - In the charts under result the count is not displayed

Users :

  • Getting unexpected error - when we change the role for the user the report breaks

last one was the issue that resolved. others are mostly ui issue that we can't resolve.

@vedfordev vedfordev moved this from QA Failed to In QA in Avni Product Oct 10, 2024
@dinesh2096
Copy link

dinesh2096 commented Oct 12, 2024

  • when we login to the user we are getting a warning message i have tested in diff org and diff user (Unable to see recent)

Click here to watch video

  • When we have role added to the user we are getting the error

Click here to watch video

@dinesh2096 dinesh2096 moved this from In QA to QA Failed in Avni Product Oct 12, 2024
@himeshr
Copy link
Contributor Author

himeshr commented Oct 29, 2024

@vedfordev Include Dinesh, me and a representative from Implementation team tech leads, do discuss the issues reported by Dinesh and the solutions found / issues disregarded.

After this, we'll determine the future course of action related to migration of superset.
We might have to present this to rest of the product team and get go ahead before asking for prod migration date from Implementation team.

vedfordev added a commit that referenced this issue Nov 27, 2024
vedfordev added a commit that referenced this issue Nov 27, 2024
himeshr added a commit that referenced this issue Nov 27, 2024
@vedfordev vedfordev moved this from In Progress to QA Ready in Avni Product Nov 27, 2024
@himeshr himeshr moved this from QA Ready to Further Action Required in Avni Product Dec 4, 2024
@vedfordev vedfordev moved this from Further Action Required to In Progress in Avni Product Dec 4, 2024
@vedfordev vedfordev moved this from In Progress to Code Review Ready in Avni Product Dec 4, 2024
vedfordev added a commit that referenced this issue Dec 4, 2024
@himeshr himeshr moved this from Code Review Ready to Done in Avni Product Dec 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

4 participants