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

Adding columns for resource, PI, and person organization ID for cloud realm #1936

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

eiffel777
Copy link
Contributor

This adds the Resource, PI, and Person organization ID columns for the cloud realm. 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.

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
@@ -24,7 +24,7 @@ public function execute()

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
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 5 commits December 4, 2024 09:43
…umns and remove resource_organization_id column from cloud realm
Co-authored-by: Aaron Weeden <31246768+aaronweeden@users.noreply.github.com>
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.

2 participants