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

ffi: fix makefile for file-allocate #240

Merged
merged 2 commits into from
Oct 10, 2023
Merged

ffi: fix makefile for file-allocate #240

merged 2 commits into from
Oct 10, 2023

Conversation

dougsland
Copy link
Collaborator

the make all was failing. this fix the issue.

@dougsland
ffi: fix makefile for file-allocate
1d78ffb 
the make all was failing. this fix the issue.

Signed-off-by: Douglas Schilling Landgraf <dougsland@redhat.com>
@dougsland
ffi: add generated bin/ for the tools in gitignore
daad276 
gcc will generat some files, let's ignore the generated bin dir.

Signed-off-by: Douglas Schilling Landgraf <dougsland@redhat.com>
@dougsland
Copy link
Collaborator Author

@Yarboa not sure what's wrong here qm container not found again?

@Yarboa
Copy link
Collaborator

Yarboa commented Oct 10, 2023 •
edited
Loading

@Yarboa not sure what's wrong here qm container not found again?

//me checking
yes running this one node1

podman exec -t qm bash -c "systemctl status bluechi-agent"

Oct 10 07:40:51 node1 bluechi-agent[49]: Starting bluechi-agent 0.6.0
Oct 10 07:40:51 node1 bluechi-agent[49]: Connecting to manager on tcp:host=10.90.0.7,port=842
Oct 10 07:40:51 node1 bluechi-agent[49]: Registering as 'qm-node1' failed: Unexpected node name
Oct 10 07:40:51 node1 bluechi-agent[49]: Initial manager connection failed, retrying
Oct 10 07:40:53 node1 bluechi-agent[49]: Trying to connect to manager (try 1)
Oct 10 07:40:53 node1 bluechi-agent[49]: Connecting to manager on tcp:host=10.90.0.7,port=842
Oct 10 07:40:53 node1 bluechi-agent[49]: Connected to manager as 'qm-node1'

It is connected, and running

Rerun the Job, could be TF issue

LGTM

@dougsland dougsland merged commit d694727 into containers:main Oct 10, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Yarboa Yarboa Yarboa approved these changes

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

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

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

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

@priyverm priyverm Awaiting requested review from priyverm priyverm 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.

None yet
2 participants
@dougsland @Yarboa