diff --git a/doc/configuration.rst b/doc/configuration.rst index 907f72f4..0ae95638 100644 --- a/doc/configuration.rst +++ b/doc/configuration.rst @@ -1641,13 +1641,13 @@ OSBuild Composer for building images * ``tenant_id`` -- Azure tenant ID to upload the image to * ``subscription_id`` -- Azure subscription ID to upload the image to * ``resource_group`` -- Azure resource group to upload the image to - * ``location`` -- Azure location to upload the image to + * ``location`` -- Azure location of the resource group (optional) * ``image_name`` -- Image name of the uploaded Azure image (optional) * **GCP upload options** -- upload to Google Cloud Platform. * ``region`` -- GCP region to upload the image to - * ``bucket`` -- GCP bucket to upload the image to + * ``bucket`` -- GCP bucket to upload the image to (optional) * ``share_with_accounts`` -- list of GCP accounts to share the image with * ``image_name`` -- Image name of the uploaded GCP image (optional) diff --git a/pungi/checks.py b/pungi/checks.py index c4b39613..274e4e9b 100644 --- a/pungi/checks.py +++ b/pungi/checks.py @@ -1229,7 +1229,10 @@ def make_schema(): "ostree_ref": {"type": "string"}, "ostree_parent": {"type": "string"}, "upload_options": { - "oneOf": [ + # this should be really 'oneOf', but the minimal + # required properties in AWSEC2 and GCP options + # overlap. + "anyOf": [ # AWSEC2UploadOptions { "type": "object", @@ -1268,7 +1271,6 @@ def make_schema(): "tenant_id", "subscription_id", "resource_group", - "location", ], "properties": { "tenant_id": {"type": "string"}, @@ -1284,7 +1286,7 @@ def make_schema(): { "type": "object", "additionalProperties": False, - "required": ["region", "bucket"], + "required": ["region"], "properties": { "region": {"type": "string"}, "bucket": {"type": "string"},