Skip to content

Adding upgrade pipeline for Person and PI Organization for cloud realm#1936

Closed
eiffel777 wants to merge 14 commits intoubccr:mainfrom
eiffel777:add-resource-pi-organization-columns-cloud
Closed

Adding upgrade pipeline for Person and PI Organization for cloud realm#1936
eiffel777 wants to merge 14 commits intoubccr:mainfrom
eiffel777:add-resource-pi-organization-columns-cloud

Conversation

@eiffel777
Copy link
Copy Markdown
Contributor

@eiffel777 eiffel777 commented Oct 28, 2024

This adds the migration pipeline for the columns added in #1958. The cloud-state-pipeline etl pipeline has been added to the migration to re-aggregate the cloud realm so these columns are added and the columns are populated. These columns are used in the xsede module at this point in time.

The XDMoD version number also needed to be updated to 11.0.1 in various places.

Tests performed

Tested in docker.

Checklist:

  • The pull request description is suitable for a Changelog entry
  • The milestone is set correctly on the pull request
  • The appropriate labels have been added to the pull request

@eiffel777 eiffel777 added the Category:Cloud Cloud Realm label Oct 28, 2024
@eiffel777 eiffel777 added this to the 11.0.1 milestone Oct 28, 2024
@eiffel777 eiffel777 self-assigned this Oct 28, 2024
if ($mysql_helper->tableExists('modw_cloud.event')) {
Utilities::runEtlPipeline(
['cloud-migration-10-5-0_11-0-0'],
['cloud-migration-10-5-0_11-0-0','cloud-state-pipeline'],
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should this be changed to an 11.0.0 -> 11.0.1 migration? We can't be modifying the 10.5 to 11.0 migration anymore since the 11.0.0 is released

@eiffel777 eiffel777 requested a review from jpwhite4 December 3, 2024 19:29
eiffel777 and others added 6 commits December 4, 2024 09:43
@eiffel777 eiffel777 changed the title Adding columns for resource, PI, and person organization ID for cloud realm Adding upgrade pipeline for PR #1958 Jan 7, 2025
@eiffel777 eiffel777 changed the title Adding upgrade pipeline for PR #1958 Adding upgrade pipeline for Person and PI Organization for cloud realm Jan 7, 2025
@aaronweeden
Copy link
Copy Markdown
Contributor

It looks like this should be based on xdmod11.0 branch instead of main.

@eiffel777 eiffel777 changed the base branch from main to xdmod11.0 January 29, 2025 18:02
@eiffel777 eiffel777 changed the base branch from xdmod11.0 to main January 29, 2025 18:03
@eiffel777
Copy link
Copy Markdown
Contributor Author

This is replaced by #1966

@eiffel777 eiffel777 closed this Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Category:Cloud Cloud Realm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants