Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[WIP - do not merge!!!] Fix something #979

Closed
wants to merge 10 commits into from
Closed

[WIP - do not merge!!!] Fix something #979

wants to merge 10 commits into from

Conversation

christoph-zededa
Copy link
Contributor

No description provided.

@christoph-zededa
Copy link
Contributor Author

see #976 (comment)

@christoph-zededa
Copy link
Contributor Author 

(ns: pillar) linuxkit-525400123456:/# ps auxww | grep dhcpcd
 2685 root      0:00 dhcpcd: eth1 [ip4] [ip6]
 2699 root      0:00 dhcpcd: eth0 [ip4] [ip6]
 4963 root      0:00 grep dhcpcd
(ns: pillar) linuxkit-525400123456:/# dhcpcd -U -4
no such user dhcpcd
dhcpcd is not running

there seems to be at least one false warning

@christoph-zededa christoph-zededa force-pushed the dhcpcd_error_fix branch 3 times, most recently from f459203 to c0f81f0 Compare June 7, 2024 18:11
@milan-zededa
Copy link
Contributor

Are you seeing the same issue also when running the test locally or is it reproducible only on buildjet runners?

@christoph-zededa
Copy link
Contributor Author

Are you seeing the same issue also when running the test locally or is it reproducible only on buildjet runners?

I gave up trying to run it locally after it failed somewhere different.

@christoph-zededa
github: test with newest EVE
d64e19e 
Signed-off-by: Christoph Ostarek <christoph@zededa.com>
@christoph-zededa christoph-zededa changed the title [WIP - do not merge!!!] Dhcpcd error fix [WIP - do not merge!!!] Fix something Jun 10, 2024
@christoph-zededa christoph-zededa force-pushed the dhcpcd_error_fix branch 2 times, most recently from 1fb27f8 to e36d711 Compare June 11, 2024 09:14
@christoph-zededa
app_local_info: send PURGING/RESTARTING cmd
85e1fea 
but expect HALTING state

Signed-off-by: Christoph Ostarek <christoph@zededa.com>
@christoph-zededa
pod: change PURGING state
c7da9fb 
PURGING state does not exist anymore, use HALTING instead

Signed-off-by: Christoph Ostarek <christoph@zededa.com>
@christoph-zededa christoph-zededa force-pushed the dhcpcd_error_fix branch 3 times, most recently from 867f5c4 to 7d67911 Compare June 12, 2024 13:12
@christoph-zededa christoph-zededa force-pushed the dhcpcd_error_fix branch 2 times, most recently from 34a5dc0 to c8a4b20 Compare June 13, 2024 11:23
@christoph-zededa christoph-zededa force-pushed the dhcpcd_error_fix branch 2 times, most recently from 974d6c1 to 061305a Compare June 13, 2024 15:52
@milan-zededa
Copy link
Contributor

Closing this. The attempt to "Fix something" apparently failed :)

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

@uncleDecart uncleDecart Awaiting requested review from uncleDecart uncleDecart is a code owner

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
@christoph-zededa @milan-zededa