-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Cannot edit/remove subproject when duplicate subprojects are defined. #6369
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
Comments
Found a failure on our side that I believe led to the multiple sub-project relationship creations https://jenkins.o-ran-sc.org/view/rtdv3-global/job/rtdv3-global-merge-master/230/console |
Looking into our implementation now. |
Reviewed this and confirmed Aricg's findings |
This was already fixed in #6275, but the fix isn't deployed yet |
@Aricg I'll need to delete the duplicate subproject relatioship manually for you. Which one is it? |
I just tested this and I think the fix just deployed worked. Please, let us know if it's still there. Thanks! |
@stsewd There are several listed here https://readthedocs.org/projects/o-ran-sc-doc/ |
$ cat /tmp/foo | sort | uniq -c |
@Aricg ok, I think I have deleted all duplicated subprojects. Let me know if I missed one |
Details
sub-project relationship created with v3 api.
Cannot remove subproject when duplicate subprojects are defined.
https://readthedocs.org/dashboard/o-ran-sc-doc/subprojects/o-ran-sc-ric-app-mc/edit/
returns:
Fail. Check back in a bit!
Expected Result
If a subproject listing already exists, refuse to create it a second time.
If duplicates do exist, allow user to remove them.
Actual Result
Subproject listing is duplicated many times https://imgur.com/a/BgYClJa
Note
The docs are now setup with a domain, docs.o-ran-sc.org and that may effect this. unsure.
My code actually checks if the subproject relationship exists, and tries to do the right thing.
So not sure how this happend.
Here is the code with the subproject check before subproject creation.
https://gist.github.com/Aricg/95812e76e8611a16ffde5c3f490813e2
The text was updated successfully, but these errors were encountered: