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