You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/06/15 00:38:11 UTC
[GitHub] [pulsar] cckellogg opened a new issue #10923: Potentially breaking broker config change in version 2.8
cckellogg opened a new issue #10923:
URL: https://github.com/apache/pulsar/issues/10923
**Describe the bug**
The default behavior changed for force deleting a namespace has changed from 2.7 -> 2.8.
https://github.com/apache/pulsar/pull/9819
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] eolivelli commented on issue #10923: Potentially breaking broker config change in version 2.8
Posted by GitBox <gi...@apache.org>.
eolivelli commented on issue #10923:
URL: https://github.com/apache/pulsar/issues/10923#issuecomment-1004368614
I believe that is it safer to let this be 'disabled' and we can add something in the release notes.
We are now at 2.10, so if we change the default we will break compatibility with 2.8 and 2.9 users.
We did something wrong in breaking compatibility, we should not do it twice.
@hsaputra you could start a thread in dev@ mailing list
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] hsaputra commented on issue #10923: Potentially breaking broker config change in version 2.8
Posted by GitBox <gi...@apache.org>.
hsaputra commented on issue #10923:
URL: https://github.com/apache/pulsar/issues/10923#issuecomment-1004289536
Ping for this issue. It was added to support:
[Issue 9790] Support disabled the tenants/namespaces force deletion in the broker.conf
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] codelipenghui commented on issue #10923: Potentially breaking broker config change in version 2.8
Posted by GitBox <gi...@apache.org>.
codelipenghui commented on issue #10923:
URL: https://github.com/apache/pulsar/issues/10923#issuecomment-861197750
@cckellogg We should disable the namespace force deleting by default, I can add this one in the release note.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] hsaputra commented on issue #10923: Potentially breaking broker config change in version 2.8
Posted by GitBox <gi...@apache.org>.
hsaputra commented on issue #10923:
URL: https://github.com/apache/pulsar/issues/10923#issuecomment-1005199001
@cckellogg - we should just close this issue for now. It is already exist to 2 versions. Thanks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] cckellogg commented on issue #10923: Potentially breaking broker config change in version 2.8
Posted by GitBox <gi...@apache.org>.
cckellogg commented on issue #10923:
URL: https://github.com/apache/pulsar/issues/10923#issuecomment-861688121
I think the default settings should be `true` to maintain backwards compatibility.
```
forceDeleteTenantAllowed=true
forceDeleteNamespaceAllowed=true
```
The current settings are:
```
# Allow you to delete a tenant forcefully.
forceDeleteTenantAllowed=false
# Allow you to delete a namespace forcefully.
forceDeleteNamespaceAllowed=false
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] eolivelli commented on issue #10923: Potentially breaking broker config change in version 2.8
Posted by GitBox <gi...@apache.org>.
eolivelli commented on issue #10923:
URL: https://github.com/apache/pulsar/issues/10923#issuecomment-861191201
@codelipenghui @cdbartholomew PTAL
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] codelipenghui commented on issue #10923: Potentially breaking broker config change in version 2.8
Posted by GitBox <gi...@apache.org>.
codelipenghui commented on issue #10923:
URL: https://github.com/apache/pulsar/issues/10923#issuecomment-1058889375
The issue had no activity for 30 days, mark with Stale label.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org