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

Test cross-index ingestion and resolution #2272

Closed
8 tasks done
fridex opened this issue Jan 20, 2022 · 7 comments
Closed
8 tasks done

Test cross-index ingestion and resolution #2272

fridex opened this issue Jan 20, 2022 · 7 comments
Assignees
Labels
kind/feature Categorizes issue or PR as related to a new feature. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Milestone

Comments

@fridex
Copy link
Contributor

fridex commented Jan 20, 2022

Is your feature request related to a problem? Please describe.

As Thoth, I would like to make sure pulp-pypi-sync-job operates well (start with stage environment) and registers new indexes available on the Pulp instance.

As Thoth, I would like to make sure packages published on the Operate First Pulp instance are solved by the background data aggregation logic and information is shown on Grafana dashboards.

As a user of Thoth, I would like to make sure that the resolution logic respects strict index configuration and labeled requests to the resolution process, and the resolution process correctly resolves software packages respecting the index configuration.

As a user of Thoth, I would like to see a listing of Python package indexes available in Thoth (including the ones registered on the Operate First Pulp instance) in thamos indexes listing.

Describe the solution you'd like

  • make sure pulp-pypi-sync-job is running in stage environment (@harshad16 Set the pulp sync job in stage cluster #2271)
  • make sure pulp-pypi-sync-job is registering indexes availble on Pulp
  • make sure graph-refresh triggers ingestion of packages that are published on the Operate First Pulp instance
  • make sure packages hosted on the Operate First Pulp instance are solved and synced into the database
  • make sure Grafana dashboards show registered Operate First Pulp indexes
  • make sure thamos advise respects labeled requests, using opf-pulp-indexes=solely and opf-pulp-indexes=disabled
  • make sure thamos advise respects strict index configuration
  • make sure thamos indexes shows Operate First Pulp indexes
  • write an integration test that makes sure cross-index resolution works well - moved to Test cross-index resolution integration-tests#240
@fridex fridex added the kind/feature Categorizes issue or PR as related to a new feature. label Jan 20, 2022
@sesheta sesheta added the needs-triage Indicates an issue or PR lacks a `triage/...` label and requires one. label Jan 20, 2022
@fridex
Copy link
Contributor Author

fridex commented Jan 20, 2022

/triage accepted
/priority critical-urgent

@sesheta sesheta added triage/accepted Indicates an issue or PR is ready to be actively worked on. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. and removed needs-triage Indicates an issue or PR lacks a `triage/...` label and requires one. labels Jan 20, 2022
@goern
Copy link
Member

goern commented Jan 20, 2022

/milestone 2022.01.31

@sesheta sesheta added this to the 2022.01.31 milestone Jan 20, 2022
@Gregory-Pereira
Copy link
Member

Gregory-Pereira commented Feb 8, 2022

/assign @Gregory-Pereira

@fridex
Copy link
Contributor Author

fridex commented Feb 9, 2022

@fridex
Copy link
Contributor Author

fridex commented Feb 14, 2022

/assign

@fridex
Copy link
Contributor Author

fridex commented Feb 14, 2022

All the features were tested. The cross-index resolution feature seems to be operational in stage environment and should be operational with the new Thoth release. I will close this and we can create integration tests supporting cross-index resolution based on thoth-station/integration-tests#240

/close

@sesheta
Copy link
Member

sesheta commented Feb 14, 2022

@fridex: Closing this issue.

In response to this:

All the features were tested. The cross-index resolution feature seems to be operational in stage environment and should be operational with the new Thoth release. I will close this and we can create integration tests supporting cross-index resolution based on thoth-station/integration-tests#240

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@sesheta sesheta closed this as completed Feb 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Projects
None yet
Development

No branches or pull requests

4 participants