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

Upgrade validate-refs to work with new AOSS registry #621

Open
jordanpadams opened this issue Mar 30, 2023 · 9 comments
Open

Upgrade validate-refs to work with new AOSS registry #621

jordanpadams opened this issue Mar 30, 2023 · 9 comments
Assignees

Comments

@jordanpadams
Copy link
Member

jordanpadams commented Mar 30, 2023

💡 Description

Blocked by NASA-PDS/registry#350

@github-project-automation github-project-automation bot moved this to Release Backlog in B14.0 Mar 30, 2023
@jordanpadams jordanpadams added B14.1 and removed B14.0 labels Aug 17, 2023
@jordanpadams jordanpadams removed this from B14.0 Sep 22, 2023
@github-project-automation github-project-automation bot moved this to Release Backlog in B14.1 Sep 22, 2023
@github-project-automation github-project-automation bot moved this to Release Backlog in EN Portfolio Backlog Nov 20, 2023
@jordanpadams jordanpadams added enhancement New feature or request sprint-backlog labels Nov 5, 2024
@jordanpadams jordanpadams changed the title Re-test validate-refs using populated multi-tenant registry Upgrade validate-refs to work with new AOSS registry Nov 5, 2024
@github-project-automation github-project-automation bot moved this to Sprint Backlog in B15.1 Nov 5, 2024
@jordanpadams jordanpadams moved this from Sprint Backlog to In Progress in B15.1 Nov 5, 2024
@jordanpadams
Copy link
Member Author

moving this to future sprint since this is blocked by NASA-PDS/registry#350

@jordanpadams jordanpadams moved this from In Progress to Sprint Backlog in B15.1 Nov 25, 2024
@jordanpadams
Copy link
Member Author

jordanpadams commented Jan 14, 2025

Per separate email discussions on NASA-PDS/registry#350, this may be unblocked. Validate will need to be updated to support multiple indexes.

Is there a way to perform this operation in the OpenSearch Java API in order to get the known registry indices?

GET /_cat/indices?format=json&pretty=true

If not, we will need a config file that is included in the package that we can pull from GitHub whenever validate-refs executed.

@al-niessner
Copy link
Contributor

al-niessner commented Jan 14, 2025

@jordanpadams

As per separate email: comma, semi-colon, or both?

Also, I think NASA-PDS/registry#350 was about comma, semi-colon, or a virtual index that resolved to all of our indices. I do not remember it being about read permissions although that could have distracted us.

@jordanpadams
Copy link
Member Author

jordanpadams commented Jan 14, 2025

@al-niessner NASA-PDS/registry#350 was pertaining to enabling this for other users. that being said, an alias index seems to make a lot of sense for something like this.

@tloubrieu-jpl
Copy link
Member

@jordanpadams @al-niessner , the registry-api is using a configuration containing the list of prefixes of the existing indexes (e.g. en,geo,atm,...).

If we had a virtual index aggregating them all I guess we would use it as well but this is not blocking.

@al-niessner
Copy link
Contributor

@jordanpadams @tloubrieu-jpl

Sigh - around the loop one more time. Yes, that is correct Java SDK2 uses list of indices. However, validate does not because it uses the connection XML file instead of "knowning" the right answer a priori as registry does. In the connection XML file, do you want to use comma, semi-colon, or both for the list? If a separated list in a string attribute is unappealing, then do you want to do a virtual node that does it for us (the "knowing" a priori for everyone that keeps everyone in sync rather than registry knowing a different answer than validate)?

@jordanpadams
Copy link
Member Author

@al-niessner this information will not be supplied by the user in their connection XML file because that file is a re-use of the same config from harvest/registr-mgr, which is index-specific. So we will need to maintain this list of available indices in validate similar to registry-api. I would recommend we keep this in a separate config file so we can eventually code up a means of automatically updating that file by grabbing the latest from GitHub or elsewhere. For now, let's just hard code it in a config file.

@al-niessner
Copy link
Contributor

@jordanpadams

Okay, but comma, semi-colon, or both?

@jordanpadams
Copy link
Member Author

@al-niessner It is something we will maintain, so whatever you prefer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ToDo
Status: ToDo
Status: Release Backlog
Development

No branches or pull requests

4 participants