Skip to content

KEH-2566: Copilot Lambda pointed at v1.0.0 - #86

Merged
hadiqur merged 3 commits into
mainfrom
KEH-2566-Copilot-Lambda-Deployment-Bug
Aug 20, 2026
Merged

KEH-2566: Copilot Lambda pointed at v1.0.0#86
hadiqur merged 3 commits into
mainfrom
KEH-2566-Copilot-Lambda-Deployment-Bug

Conversation

@hadiqur

@hadiqur hadiqur commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

What

This ticket addresses a bug where deployments for the Lambda would point it towards a stale version, causing it to error on it's weekly run. This ticket also ensures that the container_ver variable is not supplied a default value, to prevent a stale version deployment slipping under the radar.

Testing

Have any new tests been added as part of this issue? If not, try to explain why test coverage is not needed here.

  • Yes
  • No
    Please write a brief description of why test coverage is not necessary here.
  • Not as part of this ticket. (Could be done at a later point)

Documentation

Has any new documentation been written as part of this issue? We should try to keep documentation up to date
as new code is added, rather than leaving it for the future.

  • Yes
  • No
    Please write a brief description of why documentation is not necessary here.
  • Not as part of this ticket. (Could be done at a later point)

Related issues

N/A

How to review

N/A

@hadiqur hadiqur self-assigned this Aug 20, 2026
@hadiqur hadiqur added the bug Something isn't working label Aug 20, 2026
@github-actions

github-actions Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

⚠️MegaLinter analysis: Success with warnings

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ ACTION zizmor 3 0 0 1.23s
⚠️ BASH bash-exec 6 4 0 0.02s
✅ BASH shellcheck 6 0 0 0.09s
✅ BASH shfmt 6 0 0 0.02s
✅ CSHARP csharpier 1 0 0 0.56s
✅ DOCKERFILE hadolint 1 0 0 0.06s
✅ JSON jsonlint 1 0 0 0.12s
✅ JSON prettier 1 0 0 1.25s
✅ JSON v8r 1 0 0 2.92s
✅ MARKDOWN markdownlint 6 0 0 1.02s
✅ REPOSITORY checkov yes no no 31.38s
✅ REPOSITORY dustilock yes no no 0.01s
✅ REPOSITORY gitleaks yes no no 16.88s
✅ REPOSITORY grype yes no no 86.64s
✅ REPOSITORY kingfisher yes no no 13.4s
✅ REPOSITORY osv-scanner yes no no 0.58s
✅ REPOSITORY secretlint yes no no 1.33s
✅ REPOSITORY syft yes no no 4.83s
✅ REPOSITORY trivy yes no no 13.67s
✅ REPOSITORY trivy-sbom yes no no 0.59s
✅ TERRAFORM terraform-fmt 6 0 0 0.54s
✅ YAML prettier 9 0 0 0.54s
✅ YAML v8r 9 0 0 10.05s
✅ YAML yamllint 9 0 0 0.68s

Detailed Issues

⚠️ BASH / bash-exec - 4 errors
Results of bash-exec linter (version 5.3.3)
See documentation on https://megalinter.io/9.5.0/descriptors/bash_bash_exec/
-----------------------------------------------

❌ [ERROR] concourse/scripts/assume_role.sh
    Error: File:[concourse/scripts/assume_role.sh] is not executable

✅ [SUCCESS] concourse/scripts/build_image.sh
✅ [SUCCESS] concourse/scripts/set_pipeline.sh
❌ [ERROR] concourse/scripts/terraform_infra.sh
    Error: File:[concourse/scripts/terraform_infra.sh] is not executable

❌ [ERROR] shell_scripts/md_fix.sh
    Error: File:[shell_scripts/md_fix.sh] is not executable

❌ [ERROR] shell_scripts/md_lint.sh
    Error: File:[shell_scripts/md_lint.sh] is not executable

Notices

📣 MegaLinter 9.5.0 is out! Discover the new features and security recommendations in the release announcement. (Skip this info by defining SECURITY_SUGGESTIONS: false)

See detailed reports in MegaLinter artifacts

Your project could benefit from a custom flavor, which would allow you to run only the linters you need, and thus improve runtime performances. (Skip this info by defining FLAVOR_SUGGESTIONS: false)

  • Documentation: Custom Flavors
  • Command: npx mega-linter-runner@9.5.0 --custom-flavor-setup --custom-flavor-linters ACTION_ZIZMOR,BASH_EXEC,BASH_SHELLCHECK,BASH_SHFMT,CSHARP_CSHARPIER,DOCKERFILE_HADOLINT,JSON_JSONLINT,JSON_V8R,JSON_PRETTIER,MARKDOWN_MARKDOWNLINT,REPOSITORY_CHECKOV,REPOSITORY_DUSTILOCK,REPOSITORY_GITLEAKS,REPOSITORY_GRYPE,REPOSITORY_OSV_SCANNER,REPOSITORY_SECRETLINT,REPOSITORY_SYFT,REPOSITORY_TRIVY,REPOSITORY_TRIVY_SBOM,REPOSITORY_KINGFISHER,TERRAFORM_TERRAFORM_FMT,YAML_PRETTIER,YAML_YAMLLINT,YAML_V8R

MegaLinter is graciously provided by OX Security
Show us your support by starring ⭐ the repository

@hadiqur
hadiqur requested a review from a team August 20, 2026 09:52
@TotalDwarf03

Copy link
Copy Markdown
Contributor

Make sure to address the Zizmor issues

@delterr delterr 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.

Looks good

@hadiqur
hadiqur merged commit bb2852c into main Aug 20, 2026
5 checks passed
@hadiqur
hadiqur deleted the KEH-2566-Copilot-Lambda-Deployment-Bug branch August 20, 2026 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants