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

[stable-4.7] Better choice of distribution when multiple per repo (#4007) #4088

Merged
merged 1 commit into from
Aug 4, 2023

Conversation

himdel
Copy link
Collaborator

@himdel himdel commented Aug 4, 2023

Backports #4007 to 4.7 (AAH-2586)


when a repository has multiple distributions, UI will just use the first candidate that leads to inbound-* distros being used instead of, say, published

change the logic to first search for distribution where the name matches the repo name (and is still associated with the right repo), and only then fall back to the first candidate, but sorted so it will be the oldest distribution

Issue: AAH-2586

  • add repositoryBasePath(name, pulp_href?): Promise<string>, replace RepositoriesUtils.distributionByRepoName
  • use repositoryBasePath instead of manual or findDistroBasePathByRepo
  • use helper in even more places
  • rm unused api methods
  • types cleanup, merge ansible repository and repository
  • merge Repositories into AnsibleRepositoryAPI
  • fix imports, lint. sort-exports, tests

(cherry picked from commit 367295f)

when a repository has multiple distributions, UI will just use the first candidate
that leads to `inbound-*` distros being used instead of, say, `published`

change the logic to first search for distribution where the name matches the repo name (and is still associated with the right repo),
and only then fall back to the first candidate, but sorted so it will be the oldest distribution

Issue: AAH-2586

* add `repositoryBasePath(name, pulp_href?): Promise<string>`, replace `RepositoriesUtils.distributionByRepoName`
* use repositoryBasePath instead of manual or findDistroBasePathByRepo
* use helper in even more places
* rm unused api methods
* types cleanup, merge ansible repository and repository
* merge Repositories into AnsibleRepositoryAPI
* fix imports, lint. sort-exports, tests

(cherry picked from commit 367295f)
@himdel himdel marked this pull request as ready for review August 4, 2023 22:24
@himdel himdel merged commit 3103c88 into ansible:stable-4.7 Aug 4, 2023
@himdel himdel deleted the 47-4007 branch August 4, 2023 22:49
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.

1 participant