Skip to content
This repository was archived by the owner on Apr 17, 2026. It is now read-only.

fix workflow deploy command - #4

Merged
maximilien merged 8 commits into
AI4quantum:mainfrom
akihikokuroda:server
Oct 31, 2025
Merged

fix workflow deploy command#4
maximilien merged 8 commits into
AI4quantum:mainfrom
akihikokuroda:server

Conversation

@akihikokuroda

@akihikokuroda akihikokuroda commented Oct 30, 2025 •
edited
Loading

Copy link
Copy Markdown
Member

Fix workflow deploy command

  • --docker and --k8s options are supported.
  • workflow deploy is using "maestro-api" image. It is built by
docker build -t maestro-api -f Dockerfile-deploy .

  • The agents and workflow yaml files are provided to the container

    • Docker: mount the local directory of the host
    • Kubernetes: create a configmap and mount it to the container in the pod

@akihikokuroda
add server
f17be93 
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
@akihikokuroda
add deploy server dockerfile
f4b5c16 
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
@akihikokuroda
mount yaml files
b20b4a1 
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
@akihikokuroda
mound yaml files for docker
99c2edb 
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
@akihikokuroda
fix deploy to kubernetes
2ddb0a9 
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
@akihikokuroda
remove yaml files
006ebcc 
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
@akihikokuroda
akihikokuroda marked this pull request as draft October 30, 2025 12:59
@akihikokuroda
fix unit test failures
ae0d132 
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
@akihikokuroda
fix lint error
d17c8fc 
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
@akihikokuroda
akihikokuroda marked this pull request as ready for review October 30, 2025 14:37
maximilien
maximilien approved these changes Oct 31, 2025
View reviewed changes

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

LGTM

@maximilien
maximilien merged commit 4640300 into AI4quantum:main Oct 31, 2025
11 checks passed
@akihikokuroda

akihikokuroda commented Oct 31, 2025

Copy link
Copy Markdown
Member Author

@maximilien Thanks!

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

Reviewers

@planetf1 planetf1 Awaiting requested review from planetf1
@ajbozarth ajbozarth Awaiting requested review from ajbozarth
@AngeloDanducci AngeloDanducci Awaiting requested review from AngeloDanducci
@psschwei psschwei Awaiting requested review from psschwei
@george-lhj george-lhj Awaiting requested review from george-lhj
1 more reviewer
@maximilien maximilien maximilien approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@akihikokuroda @maximilien