Skip to content

chore: PoC STT - fix dockerfile, add license - #1058

Draft
gpaoloni wants to merge 43 commits into
masterfrom
gian_speech-to-text-poc
Draft

chore: PoC STT - fix dockerfile, add license#1058
gpaoloni wants to merge 43 commits into
masterfrom
gian_speech-to-text-poc

Conversation

@gpaoloni

Copy link
Copy Markdown
Collaborator

Description

Checklist

  • Corresponding issue has been opened
  • New tests added
  • Feature flags / configuration added

Other Related Issues

None

Verification steps

AFTER YOU MERGE

  1. Cut a release tag using the GitHub workflow. Wait for it to complete and notify in the #aselo-deploys Slack channel.
  2. Comment on the ticket with the release tag version AND any additional instructions required to configure an environment to test the changes.
  3. Only then move the ticket into the QA column in JIRA

You are responsible for ensuring the above steps are completed. If you move a ticket into QA without advising what version to test, the QA team will assume the latest tag has the changes. If it does not, the following confusion is on you! :-P

gpaoloni and others added 30 commits April 16, 2026 18:06
…proxy-service endpoints and CLI commands)

Agent-Logs-Url: https://github.com/techmatters/hrm/sessions/a704a02f-24b7-4a75-bc18-0c7c565bc660

Co-authored-by: gpaoloni <15805319+gpaoloni@users.noreply.github.com>
… safety, path injection)

Agent-Logs-Url: https://github.com/techmatters/hrm/sessions/a704a02f-24b7-4a75-bc18-0c7c565bc660

Co-authored-by: gpaoloni <15805319+gpaoloni@users.noreply.github.com>
@gpaoloni

Copy link
Copy Markdown
Collaborator Author

@copilot In a new branch, branched from this one (and PR should be open against this one), please edit .github/workflows/hrm-speech-to-text-ecs-deploy.yml deploy action so that it:

  • Asks for 3 radio buttons / checkboxes, one for each service (proxy, limina and pyannote).
  • If the corresponding checkbox is not checked, the container build is skipped, and instead the task definition resuses the latest/live tagged one (already present in ECR repo).
  • By default:
    • proxy is checked.
    • limina and pyannote are unchecked.

Copilot AI commented Apr 28, 2026

Copy link
Copy Markdown
Contributor

@copilot In a new branch, branched from this one (and PR should be open against this one), please edit .github/workflows/hrm-speech-to-text-ecs-deploy.yml deploy action so that it:

  • Asks for 3 rad...

Done in commit d05746f. The workflow now has three boolean checkboxes in the manual dispatch form:

  • build_proxy — checked by default (true)
  • build_limina — unchecked by default (false)
  • build_pyannote — unchecked by default (false)

When a checkbox is unchecked, all build-related steps for that service are skipped (Private AI registry login, licence injection, HUGGINGFACE_TOKEN fetch, and the Docker build/push itself). The Generate output step still runs unconditionally and emits the latest-tagged ECR URI for that service, so the ECS task definition update always references a valid image.

The same three boolean inputs were added to workflow_call as well (with the same defaults) so callers can opt-in to rebuilding specific services programmatically.

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