diff options
author | Ed Tanous <edtanous@google.com> | 2022-09-27 05:53:36 +0300 |
---|---|---|
committer | Ed Tanous <ed@tanous.net> | 2023-02-08 02:15:15 +0300 |
commit | 5b5574ace856c7f787528ab8f5e5352a90f2a6cb (patch) | |
tree | aedf147dd8d61ce7f201526181525bc4ad212e6a /scripts | |
parent | 353163e99b8394e98adae000c72abc0578ec1459 (diff) | |
download | bmcweb-5b5574ace856c7f787528ab8f5e5352a90f2a6cb.tar.xz |
Fix OemAccountService schema
This schema as-written broke a bunch of Redfish specification rules for
schemas, naming and other things. This commit starts to fix them.
It can be used as a starting point for fixing the existing schemas.
This working through testing.
Tested: OpenBMCAccountService_v1.xml now passes the redfish odata schema
validator.
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Id991794344b8b7d36af3c44bba7214bf025deec8
Diffstat (limited to 'scripts')
-rwxr-xr-x | scripts/update_schemas.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/scripts/update_schemas.py b/scripts/update_schemas.py index edc583abe4..7d6e7680a2 100755 --- a/scripts/update_schemas.py +++ b/scripts/update_schemas.py @@ -194,12 +194,12 @@ class SchemaVersion: # Remove the old files -skip_prefixes = "Oem" +skip_prefixes = ["Oem", "OpenBMC"] if os.path.exists(schema_path): files = [ os.path.join(schema_path, f) for f in os.listdir(schema_path) - if not f.startswith(skip_prefixes) + if not any([f.startswith(prefix) for prefix in skip_prefixes]) ] for f in files: os.remove(f) @@ -207,7 +207,7 @@ if os.path.exists(json_schema_path): files = [ os.path.join(json_schema_path, f) for f in os.listdir(json_schema_path) - if not f.startswith(skip_prefixes) + if not any([f.startswith(prefix) for prefix in skip_prefixes]) ] for f in files: if os.path.isfile(f): @@ -392,7 +392,7 @@ if os.path.exists(schema_path): files = [ os.path.join(schema_path, f) for f in os.listdir(schema_path) - if not f.startswith(skip_prefixes) + if not any([f.startswith(prefix) for prefix in skip_prefixes]) ] for filename in files: # filename will include the absolute path |