Skip to content

feat: support deployment variables as inputs#10

Merged
andrey-canon merged 1 commit intomainfrom
and/support_inputs
Jun 25, 2025
Merged

feat: support deployment variables as inputs#10
andrey-canon merged 1 commit intomainfrom
and/support_inputs

Conversation

@andrey-canon
Copy link
Collaborator

Description

This adds support for input variables. It allows using either the predefined environment configuration or custom values provided through inputs.

@andrey-canon andrey-canon requested a review from johanseto June 24, 2025 22:35
Copy link

@johanseto johanseto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :shipit:

required: false
AWS_CLOUDFRONT_DISTRIBUTION_ID:
type: string
required: false
Copy link

@johanseto johanseto Jun 25, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the bucket and aws_cloudfront are attached to https://github.com/nelc/actions-hub/pull/10/files#diff-c80649a9b4831cd14c8699c5fdbe0cb4bb85eea9e979ecb99158b1bdd3b26b95R130-R132

AWS credentials(secrets), but in my opinion is ok to keep them in vars

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry I don't understand your comment and the link redirects me to this comment

Copy link

@johanseto johanseto Jun 25, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, missed click... this only works with Cloufront using the same credentials.

aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY_ID }}
aws-secret-access-key: ${{ secrets.AWS_SECRET_ACCESS_KEY }}
aws-region: ${{ secrets.AWS_DEFAULT_REGION }}

Copy link
Collaborator Author

@andrey-canon andrey-canon Jun 25, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes you're right this only allows changing the cloudfront configuration in the same aws account, it's not possible to use a different aws account

@andrey-canon andrey-canon merged commit cdbdfe4 into main Jun 25, 2025
2 checks passed
@andrey-canon andrey-canon deleted the and/support_inputs branch June 25, 2025 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants