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

[Bug][Gitlab] Data scopes can have the same name. #7490

Open
2 of 3 tasks
antoniomuso opened this issue May 20, 2024 · 7 comments
Open
2 of 3 tasks

[Bug][Gitlab] Data scopes can have the same name. #7490

antoniomuso opened this issue May 20, 2024 · 7 comments
Assignees
Labels
pr-type/bug-fix This PR fixes a bug severity/p2 This bug doesn’t affect the functionality or isn’t evident type/bug This issue is a bug
Milestone

Comments

@antoniomuso
Copy link
Contributor

Search before asking

  • I had searched in the issues and found no similar issues.

What happened

When you have two repositories in two different groups but with the same name, for example: firstgroup/website and secondgroup/website. The name assigned to the scopes config is the same, and you cannot distinguish one from the other.

What do you expect to happen

I expect that the group name will be added to the data scope ex

How to reproduce

You can reproduce by adding to repositories with the same name but in different GitLab groups.

Anything else

No response

Version

v0.21.0

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@antoniomuso antoniomuso added the type/bug This issue is a bug label May 20, 2024
Copy link

dosubot bot commented May 20, 2024

To address the issue of having two repositories in different GitLab groups with the same name resulting in identical data scope names, a detailed investigation is necessary. Please provide the specific steps you followed when adding the repositories and configuring the data scopes. Additionally, sharing any error messages or logs you encountered during this process would be immensely helpful. This information is crucial for accurately reproducing the issue and working towards a resolution.

Details

About Dosu
This response is meant to be useful and save you time. It is not meant to be a precise solution, but rather a starting point for your own research.

Help me be more useful! Please leave a 👍 if this is helpful and 👎 if it is irrelevant.

To continue the conversation, mention @dosu.

@dosubot dosubot bot added pr-type/bug-fix This PR fixes a bug severity/p1 This bug affects functionality or significantly affect ux labels May 20, 2024
@Startrekzky
Copy link
Contributor

Hi @antoniomuso , I think this one has been fixed in the latest v1.0.0-beta8 by adding the namespace before the repo name in the data scope configuration dialogue. Please give it a go.

@antoniomuso
Copy link
Contributor Author

antoniomuso commented May 21, 2024

Startrekzky I tried to upgrade, but I don't see this fix on the v1.0.0-beta8

@Startrekzky
Copy link
Contributor

Hi @antoniomuso , I tried in the v1.0.0-beta8. I didn't have repos with the same names, but I can confirm when selecting data scope, the group name is displayed.
image

Not sure if I have understood your question correctly.

@antoniomuso
Copy link
Contributor Author

antoniomuso commented May 21, 2024

@Startrekzky I mean, when you add the data scope to the project, if you have two data scopes with the same name, I don't know which data scope I am adding.
image

@Startrekzky
Copy link
Contributor

Startrekzky commented May 21, 2024

@antoniomuso I see, the group name is not displayed in the project connection's data scope list. Will fix it later.

@klesh
Copy link
Contributor

klesh commented May 22, 2024

We've identified a potential issue with the config-ui component's reliance on the optional field scope.fullName within the API response. This inconsistency can affect not only Gitlab but also potentially other integrations.

Currently, the behavior of scope.fullName depends on the implementation of individual plugins, leading to potential errors. We're exploring solutions to address this:

  1. Enforce fullName in Plugins: We could require all plugins to include fullName in the scope entity. This ensures consistency but requires modifying a significant number of plugins and maintaining compatibility.
  2. Modify scopeHelper to Inject fullName or Response Format: We could modify the scope helper to inject value for fullName or redesign the entire response format.

Both options require careful consideration and thorough testing. We're actively evaluating the best approach and will keep you updated.

In the meantime, if you have any questions or suggestions, feel free to comment below!

@klesh klesh added this to the v1.1 milestone May 22, 2024
@klesh klesh added severity/p2 This bug doesn’t affect the functionality or isn’t evident and removed severity/p1 This bug affects functionality or significantly affect ux labels May 22, 2024
@antoniomuso antoniomuso changed the title [Bug][Gitlab] Data scopes can have the same name! [Bug][Gitlab] Data scopes can have the same name. May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-type/bug-fix This PR fixes a bug severity/p2 This bug doesn’t affect the functionality or isn’t evident type/bug This issue is a bug
Projects
None yet
Development

No branches or pull requests

4 participants