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

Ensure resourcetypes table is correctly populated. #1108

Merged
merged 1 commit into from
Oct 15, 2019

Conversation

jpwhite4
Copy link
Member

  • Revert the change to the primary key value for the 'UNK' resource
    type. This is now the same value (0) as previous s/w versions.
  • Ensure the 'UNK' resource is added to the various resource type tables
    in the jobs_staging pipeline.
  • Enuse that the 'UNK' rows are added on upgrade path too.
  • Revert the addition of the ingest-resource-types pipeline when
    shredding jobs. This was added in Updated to add resource_types and resource_type_realms #1006 possibly as a work around for
    the fact that the staging pipeline had not been modified appropriately.
  • Add a test to make sure this does not happen again.

- Revert the change to the primary key value for the 'UNK' resource
  type. This is now the same value (0) as previous s/w versions.
- Ensure the 'UNK' resource is added to the various resource type tables
  in the jobs_staging pipeline.
- Enuse that the 'UNK' rows are added on upgrade path too.
- Revert the addition of the ingest-resource-types pipeline when
  shredding jobs. This was added in ubccr#1006 possibly as a work around for
  the fact that the staging pipeline had not been modified appropriately.
- Add a test to make sure this does not happen again.
@jpwhite4 jpwhite4 added this to the 8.5.0 milestone Oct 15, 2019
@jpwhite4 jpwhite4 added the bug Bugfixes label Oct 15, 2019
@jpwhite4 jpwhite4 merged commit 801337d into ubccr:xdmod8.5 Oct 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bugfixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants