You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2021/09/09 13:38:56 UTC

[GitHub] [accumulo] dlmarion opened a new pull request #2266: Reduce usage of VolumeManagerImpl.get, use ServerContext.getVolumeManager instead

dlmarion opened a new pull request #2266:
URL: https://github.com/apache/accumulo/pull/2266


   Closes #666


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] dlmarion commented on pull request #2266: Reduce usage of VolumeManagerImpl.get, use ServerContext.getVolumeManager instead

Posted by GitBox <gi...@apache.org>.
dlmarion commented on pull request #2266:
URL: https://github.com/apache/accumulo/pull/2266#issuecomment-916209567


   Let me check....


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] dlmarion commented on pull request #2266: Reduce usage of VolumeManagerImpl.get, use ServerContext.getVolumeManager instead

Posted by GitBox <gi...@apache.org>.
dlmarion commented on pull request #2266:
URL: https://github.com/apache/accumulo/pull/2266#issuecomment-916199485


   No, I did not do that yet. Good find though.


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] dlmarion commented on pull request #2266: Reduce usage of VolumeManagerImpl.get, use ServerContext.getVolumeManager instead

Posted by GitBox <gi...@apache.org>.
dlmarion commented on pull request #2266:
URL: https://github.com/apache/accumulo/pull/2266#issuecomment-916226081


   So, I did the following:
   
   ```
   ~/Software/accumulo-2.1.0-SNAPSHOT/bin$ ./accumulo org.apache.accumulo.server.util.ChangeSecret --old secret --new secret2
   2021-09-09T15:48:30,014 [conf.SiteConfiguration] INFO : Found Accumulo configuration on classpath at /home/dlmari2/Software/accumulo-2.1.0-SNAPSHOT/conf/accumulo.properties
   old zookeeper password: 
   new zookeeper password: 
   Usage: org.apache.accumulo.server.util.ChangeSecret [options]
     Options:
       -auths, --auths
         the authorizations to use when reading or writing
         Default: <empty string>
       -c, --config-file
         Read the given client config file. If omitted, the classpath will be 
         searched for file named accumulo-client.properties
       -h, -?, --help, -help
   
       --password
         connection password (can be specified as '<password>', 
         'pass:<password>', 'file:<local file containing the password>' or 
         'env:<variable containing the pass>')
       --trace
         turn on distributed tracing
         Default: false
       -u, --user
         Connection user
       -o
         Overrides property in accumulo-client.properties. Expected format: -o 
         <key>=<value> 
         Default: []
   
   Can only specify option --old once.
   ```
   
   So, I guess I don't know how to run it. So I tried it w/out args and it did not fail.
   
   ```
   ~/Software/accumulo-2.1.0-SNAPSHOT/bin$ ./accumulo org.apache.accumulo.server.util.ChangeSecret
   2021-09-09T15:49:29,824 [conf.SiteConfiguration] INFO : Found Accumulo configuration on classpath at /home/dlmari2/Software/accumulo-2.1.0-SNAPSHOT/conf/accumulo.properties
   old zookeeper password: 
   new zookeeper password: 
   ~/Software/accumulo-2.1.0-SNAPSHOT/bin$ 
   ```


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] dlmarion merged pull request #2266: Reduce usage of VolumeManagerImpl.get, use ServerContext.getVolumeManager instead

Posted by GitBox <gi...@apache.org>.
dlmarion merged pull request #2266:
URL: https://github.com/apache/accumulo/pull/2266


   


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] dlmarion commented on pull request #2266: Reduce usage of VolumeManagerImpl.get, use ServerContext.getVolumeManager instead

Posted by GitBox <gi...@apache.org>.
dlmarion commented on pull request #2266:
URL: https://github.com/apache/accumulo/pull/2266#issuecomment-916230283


   I'm not sure what password it's changing. But I noticed that using the old and new password in the shell worked. For example, both worked:
   ```
   ./bin/accumulo shell -u root -p secret
   ./bin/accumulo shell -u root -p secret2
   ```
   
   Also, the following worked too:
   ```
   ./bin/accumulo shell -u root -p secret3
   ./bin/accumulo shell -u root -p foo
   ```
   
   
   This doesn't make sense to me. I am going to init a new instance on `main` and try.


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] Manno15 commented on pull request #2266: Reduce usage of VolumeManagerImpl.get, use ServerContext.getVolumeManager instead

Posted by GitBox <gi...@apache.org>.
Manno15 commented on pull request #2266:
URL: https://github.com/apache/accumulo/pull/2266#issuecomment-916148161


   @dlmarion Were you able to run the ChangeSecret utility successfully? It is erroring out for me but it's unrelated to this change. Happens on main as well. 


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] DomGarguilo commented on pull request #2266: Reduce usage of VolumeManagerImpl.get, use ServerContext.getVolumeManager instead

Posted by GitBox <gi...@apache.org>.
DomGarguilo commented on pull request #2266:
URL: https://github.com/apache/accumulo/pull/2266#issuecomment-916248401


   > It looks like you can log into the shell with whatever password you want, and it fails later when you try to do something.
   
   This reminded me of #1868. Not too sure if they are related but seem similar to me.


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] Manno15 commented on pull request #2266: Reduce usage of VolumeManagerImpl.get, use ServerContext.getVolumeManager instead

Posted by GitBox <gi...@apache.org>.
Manno15 commented on pull request #2266:
URL: https://github.com/apache/accumulo/pull/2266#issuecomment-916227958


   Interesting. That is what I did but I get a zookeeper KeeperException. 


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] Manno15 commented on pull request #2266: Reduce usage of VolumeManagerImpl.get, use ServerContext.getVolumeManager instead

Posted by GitBox <gi...@apache.org>.
Manno15 commented on pull request #2266:
URL: https://github.com/apache/accumulo/pull/2266#issuecomment-916265422


   It doesn't appear we do any authentication until we try to exec a command. 


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] Manno15 commented on pull request #2266: Reduce usage of VolumeManagerImpl.get, use ServerContext.getVolumeManager instead

Posted by GitBox <gi...@apache.org>.
Manno15 commented on pull request #2266:
URL: https://github.com/apache/accumulo/pull/2266#issuecomment-916205784


   > No, I did not do that yet. Good find though. Are you going to write a new issue for the ChangeSecret failure?
   
   If you can confirm that it happens on your end too then I will. It is possible I am not running it correctly.


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] dlmarion edited a comment on pull request #2266: Reduce usage of VolumeManagerImpl.get, use ServerContext.getVolumeManager instead

Posted by GitBox <gi...@apache.org>.
dlmarion edited a comment on pull request #2266:
URL: https://github.com/apache/accumulo/pull/2266#issuecomment-916199485


   No, I did not do that yet. Good find though. Are you going to write a new issue for the ChangeSecret failure?


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] dlmarion commented on pull request #2266: Reduce usage of VolumeManagerImpl.get, use ServerContext.getVolumeManager instead

Posted by GitBox <gi...@apache.org>.
dlmarion commented on pull request #2266:
URL: https://github.com/apache/accumulo/pull/2266#issuecomment-916238269


   Ok, I think I see what's happening now. It looks like you can log into the shell with whatever password you want, and it fails later when you try to do something. However, some things work (list tables):
   
   ```
   ~/Software/accumulo-2.1.0-SNAPSHOT/bin$ ./accumulo shell -u root -p foo
   Loading configuration from /home/dlmari2/Software/accumulo-2.1.0-SNAPSHOT/conf/accumulo-client.properties
   
   Shell - Apache Accumulo Interactive Shell
   - 
   - version: 2.1.0-SNAPSHOT
   - instance name: password-test
   - instance id: a467172e-f040-420e-acb9-3cc0ec9dc02f
   - 
   - type 'help' for a list of available commands
   - 
   WARNING: An illegal reflective access operation has occurred
   WARNING: Illegal reflective access by org.apache.hadoop.security.authentication.util.KerberosUtil (file:/home/dlmari2/Software/hadoop-3.2.1/share/hadoop/client/hadoop-client-api-3.2.1.jar) to method sun.security.krb5.Config.getInstance()
   WARNING: Please consider reporting this to the maintainers of org.apache.hadoop.security.authentication.util.KerberosUtil
   WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
   WARNING: All illegal access operations will be denied in a future release
   root@password-test> tables
   accumulo.metadata
   accumulo.replication
   accumulo.root
   trace
   root@password-test> scan -t accumulo.root
   2021-09-09T16:05:33,102 [shell.Shell] ERROR: Could not load the specified formatter. Using the DefaultFormatter
   2021-09-09T16:05:33,137 [shell.Shell] ERROR: org.apache.accumulo.core.client.AccumuloSecurityException: Error BAD_CREDENTIALS for user root - Username or Password is Invalid
   ```
   Maybe this changed with 2.0, I'm more familiar with bad passwords failing to let you in the shell at all.


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org