Skip to content

Conversation

@winterhazel
Copy link
Member

Description

This PR adds the cleanup script from #10008 to the 4.20.1.0 upgrade in order to normalize environments that were updated to 4.20 and will not go through the 4.19.2 script.

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)
  • build/CI
  • test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

How Has This Been Tested?

Without the changes in #10008, I created some broken projects by deleting their users. Then, I executed the upgrade script and verified that I was able to list their accounts/users normally.

@codecov
Copy link

codecov bot commented Dec 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 16.03%. Comparing base (ac19379) to head (09d85a7).
Report is 35 commits behind head on 4.20.

Additional details and impacted files
@@            Coverage Diff            @@
##               4.20   #10116   +/-   ##
=========================================
  Coverage     16.03%   16.03%           
- Complexity    12814    12815    +1     
=========================================
  Files          5637     5637           
  Lines        493507   493507           
  Branches      59831    59831           
=========================================
  Hits          79131    79131           
  Misses       405600   405600           
  Partials       8776     8776           
Flag Coverage Δ
uitests 4.02% <ø> (ø)
unittests 16.87% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@DaanHoogland DaanHoogland added this to the 4.21.0 milestone Dec 17, 2024
Copy link
Contributor

@DaanHoogland DaanHoogland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NOTE: This will not be in 4.20 -> 21 , so if 20.1 comes out before 21 we need to reimpleme nt it.

@DaanHoogland DaanHoogland merged commit 2300c1c into apache:4.20 Jan 16, 2025
26 checks passed
DaanHoogland added a commit that referenced this pull request Jan 20, 2025
* 4.20:
  Maintenance mode: Add host to deployment planner avoid list to fix local storage vm migration (#9892)
  Add project-user association normalization script to 4.20.1 upgrade (#10116)
  fix slider component for global settings of the range type (#10187)
  Clean up network permissions on account deletion (#10176)
@Pearl1594 Pearl1594 moved this to Done in ACS 4.20.1 Mar 17, 2025
dhslove pushed a commit to ablecloud-team/ablestack-cloud that referenced this pull request Jun 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants