[bug] Git import imports all available domains, but locks only one of them

[manageiq]

[bug] Git import imports all available domains, but locks only one of them

[gmcculloug]

@mkanoor Can you review this issue and provide feedback? Thinking this is something @pkomanek could work on.

[mkanoor]

@paulslaby @gmcculloug
Typically our recommendation has been 1 domain per git repository.
I think in your case you have 2 domains in your repository.
Since the underlying code searches for multiple domain, when it encounters them it imports it in.
We can restrict the git importer to only import a single domain and reject if the domain has multiples.
@pkomanek can you please add a check to restrict git importer to a single domain.

[paulslaby]

Thanks for reaction. Sounds like a good pattern. I'm going to redesign my use case.
I hope, you'll make it behave more expected. But I understand, it's not priority.

[CCoupel]

Hi, i need to give my intention on this behaviour.
My customer manage its appliance with more than 3 domains in a single GIT repo.
It will be usefull if, as for file import, we can choose which domain we want to use instead of rejecting all unless we will not import all the domains. This way, we have a coherent working maner on importing from file and importing from GIT.

[miq-bot]

This issue has been automatically marked as stale because it has not been updated for at least 6 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

Thank you for all your contributions!

[JPrause]

@paulslaby is this still a valid issue? If yes, let me know and I'll remove the stale label. If not can you close. If there's no update by next week, I'll be closing this issue.

[paulslaby]

Probably the problem in the code is still valid, but it is not pain for me any more, because I followed the mkanoor's advice to have only 1 domain per repository.

[gmcculloug]

Note: multi-domain import is now prevented. See #124