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

DM-44884: Remove DP0.1 from TAP_SCHEMA build & DataLink #238

Merged
merged 1 commit into from
Sep 27, 2024

Conversation

stvoutsin
Copy link
Contributor

@stvoutsin stvoutsin commented Jul 6, 2024

PR Summary:
Removes DP0.1 from the TAP_SCHEMA build for idfprod and removes all associated DataLink support.

Jira issue:
https://rubinobs.atlassian.net/browse/DM-44884

Description:
The current TAP_SCHEMA image expects to find the DP0.1 catalog & tables. While Qserv for idfprod still has this dataset, we plan to remove it imminently, so removing it from TAP_SCHEMA first allows the Qserv team to take the actual data down later at their discretion without worrying about producing a non-valid service. I've removed them from the build-all script, but I haven't removed the actual yaml metadata files, in case they are needed in the future.

Update: Changed to use ticket as branch name & modified datalink directory to remove dp01 there as well. I've removed the columns-minimal.yaml as well, let me know if this is incorrect.

@JeremyMcCormick JeremyMcCormick changed the title Remove dp01 from TAP_SCHEMA build & datalink DM-44884: Remove dp01 from TAP_SCHEMA build & datalink Jul 15, 2024
@JeremyMcCormick JeremyMcCormick requested a review from gpdf July 29, 2024 15:13
@stvoutsin stvoutsin force-pushed the tickets/DM-44884 branch 2 times, most recently from 2a56696 to 776e100 Compare August 19, 2024 15:57
@gpdf
Copy link
Collaborator

gpdf commented Sep 7, 2024

I have updated the PR to reflect the initial removal of DP0.1 from idfdev and idfint, done on #250 . That PR avoid current validation errors due to the previous deletion of the DP0.1 data from the backing Qserv for dev/int.

@gpdf
Copy link
Collaborator

gpdf commented Sep 7, 2024

Approval of this PR will be contingent on approval of DM-46163, "Remove DP0.1 catalogs from public RSP".

@gpdf gpdf changed the title DM-44884: Remove dp01 from TAP_SCHEMA build & datalink DM-44884: Remove DP0.1 from TAP_SCHEMA build & DataLink Sep 7, 2024
Copy link
Collaborator

@gpdf gpdf left a comment

Choose a reason for hiding this comment

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

@frossie has approved the removal of DP0.1 from idf-prod, and the DP0 delegates have long since been warned that this was coming. See https://rubinobs.atlassian.net/browse/DM-46163 .

This change is approved for deployment.

Preserves the schema in the archive subdirectory for reference.
Removes all DataLink configuration associated with DP0.1.
@JeremyMcCormick JeremyMcCormick merged commit 41840ef into main Sep 27, 2024
9 checks passed
@JeremyMcCormick JeremyMcCormick deleted the tickets/DM-44884 branch September 27, 2024 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants