Enable AOF (#255)

* Enable AOF

* Enable AOF in tests via env conf

* Fix use of Env in tests. Run test suite multiple times with different env configurations.

* Comment out panda test

* Update pack version and license in RAMP file

Author: lantiga

opt/Makefile

@@ -134,8 +134,11 @@ ifneq ($(NO_LFS),1)
 endif
 	$(SHOW)set -e ;\
 	cd $(ROOT)/test ;\
+	python3 -m RLTest $(TEST_ARGS) --test basic_tests.py --module $(INSTALL_DIR)/redisai.so ;\
 	python3 -m RLTest $(TEST_ARGS) --test basic_tests.py --module $(INSTALL_DIR)/redisai.so --use-slaves ;\
-	python3 -m RLTest $(TEST_ARGS) --test  double-panda.py --module $(INSTALL_DIR)/redisai.so
+	python3 -m RLTest $(TEST_ARGS) --test basic_tests.py --module $(INSTALL_DIR)/redisai.so --use-aof 
+        # Commented until the MacOS CI is available
+	# python3 -m RLTest $(TEST_ARGS) --test double-panda.py --module $(INSTALL_DIR)/redisai.so
 
 #----------------------------------------------------------------------------------------------
 

ramp.yml

@@ -3,17 +3,18 @@ author: Orobix and RedisLabs
 email: [email protected]
 description: Serving tensors and executing deep learning graphs
 homepage: https://oss.redislabs.com/redisai/
-license: GNU Affero General Public License v3.0
+license: Redis Source Available License v1.0
 command_line_args: ""
 min_redis_version: "5.0.7"
-min_redis_pack_version: "5.4"
+min_redis_pack_version: "5.4.11"
 capabilities:
     - types
     - no_multi_key
     - hash_policy
     - eviction_expiry
     - failover_migrate
     - persistence_rdb
+    - persistence_aof
     - clustering
 exclude_commands:
     - ai.modelset

src/config.h

@@ -22,6 +22,4 @@ typedef enum {
 #define RAI_COPY_RUN_OUTPUT
 #define RAI_PRINT_BACKEND_ERRORS
 
-// #define RAI_OVERRIDE_AOF_CHECK
-
 #endif /* SRC_CONFIG_H_ */

src/redisai.c

@@ -1486,12 +1486,6 @@ int RedisModule_OnLoad(RedisModuleCtx *ctx, RedisModuleString **argv, int argc)
       == REDISMODULE_ERR) return REDISMODULE_ERR;
 
   int flags = RedisModule_GetContextFlags(ctx);
-  if (flags & REDISMODULE_CTX_FLAGS_AOF) {
-    RedisModule_Log(ctx, "warning", "ERR: AOF currently unsupported\r\n");
-#ifndef RAI_OVERRIDE_AOF_CHECK
-    return REDISMODULE_ERR;
-#endif
-  }
 
   RAI_BackendsPath = NULL;