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

ApolloAdmin删除重建namespace报错app namespaces already exist. #4999

Closed
3 tasks done
apa7 opened this issue Oct 18, 2023 · 4 comments
Closed
3 tasks done

ApolloAdmin删除重建namespace报错app namespaces already exist. #4999

apa7 opened this issue Oct 18, 2023 · 4 comments

Comments

@apa7
Copy link

apa7 commented Oct 18, 2023

ApolloAdmin删除重建namespace报错app namespaces already exist.

描述bug

复现
在管理
1.新建一个namespace: demo.properties
2.删除这个namespace
3.重建namespace: demo.properties
报错app namespaces already exist.

原因:
删除是逻辑删除,会标记IsDeleted = 1。而在namespace新建的时候,查询条件没有带IsDeleted = 0

期望
1.应该允许重建,查询的时候查询未逻辑删除的。
2.重建后,应该允许再次删除

额外的细节和日志

image
image

@apa7
Copy link
Author

apa7 commented Oct 18, 2023

我们为什么会有这个诉求?
有些同学新建namespace时,public/private选错了,还有在废弃namespace之后又想恢复...
总之种种原因,有删除重新建一个同名namespace的需要。
希望开发者帮忙处理一下这个问题,感谢!

@zihan524
Copy link

我的2.1没有这个问题呢。

@apa7
Copy link
Author

apa7 commented Oct 19, 2023

我的2.1没有这个问题呢。

thx,2.1确实OK。
我们考虑1.9升级2.1。

@apa7 apa7 closed this as completed Oct 19, 2023
@nobodyiam
Copy link
Member

管理员工具可以删除 app namespace

image

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

No branches or pull requests

3 participants