ddanielr commented on PR #5383:
URL: https://github.com/apache/accumulo/pull/5383#issuecomment-2766668022

   Tested running this with a 4.0 running cluster, shutting it down and then 
running `./accumulo zoo-zap -prepare-for-upgrade`.
   
   Found that zooZap needs an update to the opts param check to allow upgrade 
to be a valid option. Otherwise it just prints a usage statement.
   
   Updated lines 115-119: 
   ```
       if (!opts.zapManager && !opts.zapTservers && !opts.zapCompactors && 
!opts.zapScanServers
           && !opts.upgrade) {
         new JCommander(opts).usage();
         return;
       }
   ```
   
   After fixing that, I could get zoozap to run but kept getting an exception 
message when attempting to create the upgrade node.
   ```
   Thread 'zoo-zap' died.
   java.lang.IllegalStateException: Error creating 
/accumulo/4c7c8947-b70f-48bd-827a-bc609060f441/upgrade_prepped node in 
zookeeper. Check for any issues and retry.
        at org.apache.accumulo.server.util.ZooZap.zap(ZooZap.java:181)
        at org.apache.accumulo.server.util.ZooZap.execute(ZooZap.java:107)
        at org.apache.accumulo.start.Main.lambda$execKeyword$0(Main.java:81)
        at java.base/java.lang.Thread.run(Thread.java:840)
   Caused by: org.apache.zookeeper.KeeperException$NoNodeException: 
KeeperErrorCode = NoNode for 
/accumulo/4c7c8947-b70f-48bd-827a-bc609060f441/upgrade_prepped
        at org.apache.zookeeper.KeeperException.create(KeeperException.java:117)
        at org.apache.zookeeper.KeeperException.create(KeeperException.java:53)
        at org.apache.zookeeper.ZooKeeper.create(ZooKeeper.java:1347)
        at 
org.apache.accumulo.core.zookeeper.ZooSession.create(ZooSession.java:273)
        at 
org.apache.accumulo.core.fate.zookeeper.ZooReaderWriter.lambda$putPersistentData$1(ZooReaderWriter.java:92)
   ```
   
   Using zkCLI.sh, I can see that `upgrade_prepped` doesn't exist and the 
accumulo instance path does.
   ```
   [zk: localhost:2181(CONNECTED) 15] ls 
/accumulo/4c7c8947-b70f-48bd-827a-bc609060f441
   [compactions, compactors, config, dead, fate, gc, hdfs_reservations, 
managers, monitor, namespaces, next_file, recovery, root_tablet, sservers, 
table_locks, tables, tservers, users, wals]
   ```
   
   Current acl
   ```
   [zk: localhost:2181(CONNECTED) 16] getAcl 
/accumulo/4c7c8947-b70f-48bd-827a-bc609060f441
   'digest,'accumulo:x
   : cdrwa
   'world,'anyone
   : r
   ```
   
   
   
   


-- 
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-unsubscr...@accumulo.apache.org

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

Reply via email to