Skip to content

Commit

Permalink
Rename RofV2 support flag to BigStore version 2 support in module met…
Browse files Browse the repository at this point in the history
…adata and command line options
  • Loading branch information
eranhd committed Nov 21, 2024
1 parent 3cabc81 commit 05ebdf7
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
6 changes: 3 additions & 3 deletions RAMP/module_metadata.py
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@
MIN_REDIS_VERSION = "4.0"
MIN_REDIS_PACK_VERSION = "5.2"
COMPATIBLE_REDIS_VERSION = "7.4"
ROFV2_SUPPORT = False
BIGSTORE_VERSION_2_SUPPORT = False
RAMP_FORMAT_VERSION = 1
CONFIG_COMMAND = ""
OVERIDE_COMMAND = [] # type: List[Dict[str, str]]
Expand Down Expand Up @@ -58,7 +58,7 @@
"min_redis_pack_version",
"min_redis_version",
"compatible_redis_version",
"rofv2_support",
"bigstore_version_2_support",
"module_file",
"module_name",
"os",
Expand Down Expand Up @@ -125,7 +125,7 @@ def create_default_metadata(module_path):
"min_redis_version": MIN_REDIS_VERSION,
"min_redis_pack_version": MIN_REDIS_PACK_VERSION,
"compatible_redis_version": COMPATIBLE_REDIS_VERSION,
"rofv2_support": ROFV2_SUPPORT,
"bigstore_version_2_support": BIGSTORE_VERSION_2_SUPPORT,
"sha256": sha256_checksum(module_path),
"commands": MODULE_COMMANDS,
"ramp_format_version": RAMP_FORMAT_VERSION,
Expand Down
2 changes: 1 addition & 1 deletion RAMP/ramp.py
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ def unpack(bundle):
@click.option('--redis-min-version', '-r', 'min_redis_version', default=None, help='redis minimum version')
@click.option('--redis-pack-min-version', '-R', 'min_redis_pack_version', default=None, help='redis pack minimum version')
@click.option('--redis-compatible-version', '-cr', 'compatible_redis_version', default=None, help='redis compatible version')
@click.option('--rofv2-support', 'rofv2_support', default=False, help='This module supports RofV2')
@click.option('--bigstore-version-2-support', 'bigstore_version_2_support', default=False, help='This module supports RofV2')
@click.option('--config-command', '-cc', default=None, help='command used to configure module args at runtime')
@click.option('--os', '-O', default=None, help='build target OS (Darwin/Linux)')
@click.option('--capabilities', '-C', callback=comma_seperated_to_list, help='comma separated list of module capabilities')
Expand Down
6 changes: 3 additions & 3 deletions test.py
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ def test_bundle_from_cmd():
min_redis_version = "4.6"
min_redis_pack_version = "5.0"
compatible_redis_version = "7.2"
rofv2_support = True
bigstore_version_2_support = True
display_name = "test_module"
capability_name = "Test & Module"
module_name = "module_test"
Expand All @@ -124,7 +124,7 @@ def test_bundle_from_cmd():
'-d', display_name, '-b', capability_name, '-n', module_name,
'-h', homepage, '-l', _license, '-c', command_line_args,
'-r', min_redis_version, '-R', min_redis_pack_version, '-cr', compatible_redis_version,
'--rofv2-support', rofv2_support,
'--bigstore-version-2-support', bigstore_version_2_support,
'-C', ','.join([cap['name'] for cap in MODULE_CAPABILITIES]),
'-o', BUNDLE_ZIP_FILE, '-cc', CONFIG_COMMAND, '-E', 'graph.bulk',
'-E', 'graph.BULK',
Expand Down Expand Up @@ -154,7 +154,7 @@ def test_bundle_from_cmd():
assert metadata["min_redis_version"] == min_redis_version
assert metadata["min_redis_pack_version"] == min_redis_pack_version
assert metadata["compatible_redis_version"] == compatible_redis_version
assert metadata["rofv2_support"] == rofv2_support
assert metadata["bigstore_version_2_support"] == bigstore_version_2_support
assert metadata["config_command"] == CONFIG_COMMAND
assert metadata["sha256"] == sha256_checksum(MODULE_FILE_PATH)
assert len(metadata["capabilities"]) == len(MODULE_CAPABILITIES)
Expand Down

0 comments on commit 05ebdf7

Please sign in to comment.