Skip to content

Develop#17

Merged
vegcom merged 7 commits intomainfrom
develop
Apr 1, 2026
Merged

Develop#17
vegcom merged 7 commits intomainfrom
develop

Conversation

@vegcom
Copy link
Copy Markdown
Owner

@vegcom vegcom commented Apr 1, 2026

No description provided.

June-Cozy and others added 7 commits March 31, 2026 18:54
…step

- Add explicit DNS (1.1.1.1, 1.0.0.1) to x-minion-base anchor
- Minions were failing pip installs due to DNS resolution issues in CI
- Remove frontend network creation step (not used without macvlan overlay)

Contributions by: June
prepare-commit-msg prepends '# branch: name' hint, which broke
validation when using 'git commit -m'. Now we skip # lines to find
the actual subject line.

Contributions by: June
…#14)

## Summary
- Flip pip index order: pypi.org primary, Chinese mirrors as extras
- Extras are checked in parallel so we still get speed benefits
- More reliable for CI and general use since pypi.org is the source of
truth

Contributions by: June
## Summary
- Fix commit-msg hook to skip comment lines (starting with #) when
finding the subject line
- Resolves issue where prepare-commit-msg's '# branch: name' hint broke
validation when using 'git commit -m'

## Test plan
- [ ] Run `git commit -m "test: message"` and verify hook validates
correctly
- [ ] Run `git commit` (editor mode) and verify hint comment is skipped
…step (#15)

## Summary

- Add explicit DNS configuration (1.1.1.1, 1.0.0.1) to the x-minion-base
anchor in docker-compose.yaml
- Minions were failing pip installs during CI due to DNS resolution
issues inside containers
- Remove the frontend network creation step from test-states.yml (not
used without macvlan overlay)

## Test plan

- [ ] Verify CI workflow passes with the DNS configuration
- [ ] Confirm pip installs work correctly in minion containers
- [ ] Check that removing the frontend network step does not break any
tests
@vegcom vegcom merged commit 1479c92 into main Apr 1, 2026
2 of 4 checks passed
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