Skip to content

feat | LAY-917 Added an action to create a new tag. Added scripts for…#30

Merged
Robert-H-Leonard merged 3 commits into
mainfrom
feat/LAY-917-action-tag
Aug 27, 2025
Merged

feat | LAY-917 Added an action to create a new tag. Added scripts for…#30
Robert-H-Leonard merged 3 commits into
mainfrom
feat/LAY-917-action-tag

Conversation

@Lzok

@Lzok Lzok commented Aug 26, 2025

Copy link
Copy Markdown
Collaborator

… release. Added guideline for release process

… release. Added guideline for release process
@Lzok Lzok force-pushed the feat/LAY-917-action-tag branch from 79aa5fa to 483ae09 Compare August 26, 2025 12:07

@Robert-H-Leonard Robert-H-Leonard left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I setup the AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY in the production env and gave you permissions to add the other 3 we need (CLOUDFRONT_DISTRIBUTION_ID, CLOUDFRONT_DOMAIN, S3_BUCKET_NAME).

https://github.com/LayerLens/atlas-python/settings/environments/8110884380/edit

Comment thread .github/workflows/create-tag.yaml Outdated
jobs:
check-branch:
runs-on: ubuntu-latest
environment: development

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This should run on the production env for the release branch (our production branch)

Comment on lines +6 to +17
on:
workflow_dispatch:
inputs:
dry_run:
description: "Run in dry-run mode (show what would be done without actually creating/pushing the tag)"
required: false
type: boolean
default: true
confirm_release:
description: "Type 'YES' to confirm you want to create and push the release tag"
required: true
type: string

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We should configure the enviornment in this section so we don't specify it at each step.

The two env we have are development and production. The production env should be used on the release branch.

Comment thread .github/workflows/create-tag.yaml Outdated
create-release-tag:
runs-on: ubuntu-latest
needs: check-branch
environment: development

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

See comment above on not specifying the env on each step.

Comment thread .github/workflows/publish-to-aws.yaml Outdated
jobs:
validate:
runs-on: ubuntu-latest
environment: development

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We need this on the production env as well

@Robert-H-Leonard Robert-H-Leonard merged commit 1957189 into main Aug 27, 2025
7 checks passed
@m-peko m-peko deleted the feat/LAY-917-action-tag branch February 24, 2026 08:18
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