8000 fix(tools): fix docker-compose package required by locmai · Pull Request #5 · khuedoan/homelab · GitHub
[go: up one dir, main page]
Skip to content

fix(tools): fix docker-compose package required - #5

Merged
khuedoan merged 2 commits into
khuedoan:masterfrom
locmai:fix/docker-missing
Dec 5, 2021
Merged

fix(tools): fix docker-compose package required#5
khuedoan merged 2 commits into
khuedoan:masterfrom
locmai:fix/docker-missing

Conversation

@locmai
@locmai locmai commented Dec 5, 2021
Copy link
Copy Markdown
Contributor

On starting PXE server, we encountered the following error:

TASK [pxe-server : Start ephemeral PXE server] *****************************************************************************************************************
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: ModuleNotFoundError: No module named 'requests'
fatal: [localhost]: FAILED! => {
    "changed": false
}

MSG:

Failed to import the required Python library (Docker SDK for Python: docker above 5.0.0 (Python >= 3.6) or docker before 5.0.0 (Python 2.7) or docker-py (Python 2.6)) on lmaiarch's Python /usr/bin/python. Please read the module documentation and install it in the appropriate location. If the required library is installed, but Ansible is using the wrong Python interpreter, please consult the documentation on ansible_python_interpreter, for example via `pip install docker` (Python >= 3.6) or `pip install docker==4.4.4` (Python 2.7) or `pip install docker-py` (Python 2.6). The error was: No module named 'requests'

PLAY RECAP *****************************************************************************************************************************************************
localhost                  : ok=6    changed=5    unreachable=0    failed=1    skipped=0    rescued=0    ignored=0

Solve by adding pip and install docker-compose via pip.

khuedoan
khuedoan reviewed Dec 5, 2021
View reviewed changes
Comment thread tools/Dockerfile Resolved
@khuedoan khuedoan changed the title fix (tools): Fix docker-compose package required fix(tools): fix docker-compose package required Dec 5, 2021
@locmai
locmai requested a review from khuedoan December 5, 2021 14:57
khuedoan
khuedoan approved these changes Dec 5, 2021
View reviewed changes
@khuedoan khuedoan left a comment
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks

@khuedoan
khuedoan merged commit 90ebb96 into khuedoan:master Dec 5, 2021
zanehala added a commit to zanehala/homelab that referenced this pull request Nov 25, 2022
@zanehala
Merge pull request 'Update Helm release hajimari to v2' (khuedoan#5) …
41bb5fa 
…from renovate/hajimari-2.x into master

Reviewed-on: https://git.zanelab.com/ops/homelab/pulls/5
brimdor pushed a commit to brimdor/homelab that referenced this pull request Mar 23, 2024
gitea_admin
Merge pull request 'Update Helm release renovate to v37' (khuedoan#5)…
4f7e3cd 
… from renovate/renovate-37.x into master

Reviewed-on: https://git.eaglepass.io/ops/homelab/pulls/5
HubertusWine pushed a commit to Enraged-Pixels/homelab that referenced this pull request Mar 14, 2025
gitea_admin
Merge pull request 'Update Helm release grafana to v8' (khuedoan#5) f…
0514868 
…rom renovate/grafana-8.x into master

Reviewed-on: https://git.epx.pub/ops/homelab/pulls/5
brimdor pushed a commit to brimdor/homelab that referenced this pull request Apr 14, 2026
Echo
chore(pages-canary): decommission surplus canary — issue khuedoan#5 m…
cf12bee 
…erged
brimdor pushed a commit to brimdor/homelab that referenced this pull request Apr 14, 2026
Echo
chore: decommission pages-canary — issue khuedoan#5 closed, CRUD API …
d8dc994 
…promoted to v2-prod
brimdor pushed a commit to brimdor/homelab that referenced this pull request Apr 23, 2026
Echo Agent
chore(nibble-canary): update image to v2 for QA of issue khuedoan#5 f…
ea39239 
…ixes
brimdor pushed a commit to brimdor/homelab that referenced this pull request Apr 23, 2026
Cadence Agent
chore: update nibble canary+prod image tag to 2026-04-23-v1
6081459 
Issue khuedoan#5: Align AI Service with API Contract
- AI parsing: bracket-depth JSON extraction
- Duplicate React keys: composite ID generation
- Zod schema validation
- Frontend: preference tags re-enabled
brimdor pushed a commit to brimdor/homelab that referenced this pull request Jun 6, 2026
Hermes Backup Bot
feat: enable k8s probes, resource limits, and network policies (refs k…
eec2655 
…huedoan#5)
brimdor pushed a commit to brimdor/homelab that referenced this pull request Jun 6, 2026
gitea_admin
Merge pull request 'feat: enable k8s probes, resource limits, and net…
de5a4de 
…work policies (refs khuedoan#5)' (khuedoan#95) from issue-5-k8s-hardening into master
brimdor pushed a commit to brimdor/homelab that referenced this pull request Jul 29, 2026
Vanguard
deploy(sporecast-canary): roll to main-5d20ae0 (v0.4.1-rc.1, PR khued…
4d78261 
…oan#5 SporeCast Now + allergen fix)
brimdor pushed a commit to brimdor/homelab that referenced this pull request Jul 30, 2026
Vanguard
deploy(sporecast-canary): roll to main-09ab763 (v0.4.1-rc.3, PRs khue…
0ce324e 
…doan#5/khuedoan#6/khuedoan#7 SporeCast Now + allergen fix + Open-Meteo parser fixes)
brimdor pushed a commit to brimdor/homelab that referenced this pull request Jul 30, 2026
vanguard
deploy(tipsbot-canary): roll to v0.1.4-rc.1 (Google OAuth via Cloudfl…
1a14f04 
…are Access)

- Image tag: v0.1.3-rc.1 -> v0.1.4-rc.1 (commit 3b57788)
- Add TIPS_CF_ACCESS: "true" env var (header-based identity extraction)
- Per v0.1.4-rc.1 PR khuedoan#5: internal admin auth replaced with Cloudflare
  Access Cf-Access-User-Email header. SPA login route deleted.
- Canary only. Production promotion requires separate task.
brimdor pushed a commit to brimdor/homelab that referenced this pull request Aug 3, 2026
vanguard
deploy(threads-canary): roll to v1.1.0-rc.1 (BSB Bible App MVP, PR kh…
6f5d98b 
…uedoan#5)
brimdor pushed a commit to brimdor/homelab that referenced this pull request Aug 23, 2026
Vanguard
deploy(scribe-discord): roll to b01f2ca4acf2e1ec7d682d209b260236752dd…
c4a0987 
…f3b for hook verification

Brings in the v0.2.1 ENOENT fix (PR khuedoan#5, scripts/register-commands.ts:5-14)
which wraps process.loadEnvFile() in a try/catch that swallows ENOENT and
re-throws everything else. The chart's pre-install,pre-upgrade hook Job
was previously failing because the ded126e image baked the un-fixed
script and the container has no /app/.env (.dockerignore excludes it).

Without this image bump, the recovery-spec comment-only trigger would
just produce another failed hook Job (as t_a31c457f and t_cdf662e1
already established). The ENOENT-fix spec (spec-register-commands-env-
enoent-fix §5.2) explicitly anticipates this flow: rebuild image from
b01f2ca, push to zot, bump chart's image tag in both controllers.main
and controllers.register (the latter via defaultContainerOptions.image),
push chart, ArgoCD auto-syncs, hook Job succeeds.

Image SHA verified locally: podman build + push to 10.0.20.11:32309/
scribe:b01f2ca4acf2e1ec7d682d209b260236752ddf3b succeeded; manifest
config digest sha256:73d8b37412e8 matches. Local register-commands.js
smoke test against fake token confirms script no longer throws ENOENT
(proceeds to PUT against Discord REST API and returns 401 on bad token,
which is the expected end state for a token-validity check).

Refs t_8c3dda40, t_5358931e, t_3f0d084f, t_a31c457f.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@khuedoan khuedoan khuedoan approved these changes

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

@locmai @khuedoan
0