Skip to content

release_dir not set, release artifacts not uploaded #121

Closed

Description

@doebrowsk

if: steps.read-type.outputs.output-value == 'orchestrator' || steps.read-type.outputs.output-value == 'ca-gateway' || steps.read-type.outputs.output-value == 'pam' || steps.read-type.outputs.output-value == 'anyca-plugin'

The above line is the conditional for reading and setting the release_dir environment variable, which is used to determine later on if a release artifact should be created and uploaded.

The list of types is not exhaustive, and too restrictive currently:

  • orchestrator
  • ca-gateway
  • pam
  • anyca-plugin

Additional integration_type values will need to be supported:

The setting of the release_dir value should not be conditional; if it is set, it should be loaded into the build environment.

Conditional checking of uploading or creating a release should happen explicitly where that needs to be evaluated, and not as a side effect of not setting this required value.

Activity

  1. doebrowsk commented on Jun 24, 2025

    @doebrowsk
    ContributorAuthor

    The release_dir field is now always pulled from the integration-manifest.json as of release 3.3.1, currently tagged as v3. However, this now causes issues for integrations that do not define (or need to define) that field. The action that pulls these properties from the json file needs to be updated to handle missing properties, such as allowing them to be missing unless marked required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions