coop-cloud/abra
coop-cloud
/
abra
5
2
Fork 14
Code Issues Pull Requests 4 Packages Releases 36 Activity

feat: abra app logs shows task errors #395

Merged
p4u1 merged 1 commits from p4u1/abra:logs-task-error into main 2023-12-14 13:15:24 +00:00
Conversation 2 Commits 1 Files Changed 2 +102 -100
p4u1 commented 2023-12-12 16:55:32 +00:00
Member
Copy Link

The log command now checks for the ready state in the task list. If it is not ready. It shows the task errors. This might look like this:

ERRO[0000] Service abra-test-recipe_default_app: State rejected: No such image: ngaaaax:1.21.0 
ERRO[0000] Service abra-test-recipe_default_app: State preparing:  
ERRO[0000] Service abra-test-recipe_default_app: State rejected: No such image: ngaaaax:1.21.0 
ERRO[0000] Service abra-test-recipe_default_app: State rejected: No such image: ngaaaax:1.21.0 
ERRO[0000] Service abra-test-recipe_default_app: State rejected: No such image: ngaaaax:1.21.0

Feel free to suggest a better error ouptput.

Closes coop-cloud/organising#518

The log command now checks for the ready state in the task list. If it is not ready. It shows the task errors. This might look like this: ``` ERRO[0000] Service abra-test-recipe_default_app: State rejected: No such image: ngaaaax:1.21.0 ERRO[0000] Service abra-test-recipe_default_app: State preparing: ERRO[0000] Service abra-test-recipe_default_app: State rejected: No such image: ngaaaax:1.21.0 ERRO[0000] Service abra-test-recipe_default_app: State rejected: No such image: ngaaaax:1.21.0 ERRO[0000] Service abra-test-recipe_default_app: State rejected: No such image: ngaaaax:1.21.0 ``` Feel free to suggest a better error ouptput. Closes https://git.coopcloud.tech/coop-cloud/organising/issues/518
p4u1 added 1 commit 2023-12-12 16:55:32 +00:00
continuous-integration/drone/pr Build is passing Details
e40d99bfe2 feat: abra app logs shows task errors
p4u1 force-pushed logs-task-error from e40d99bfe2 to 98671a9f92 2023-12-12 17:01:12 +00:00 Compare
decentral1se reviewed 2023-12-12 21:06:17 +00:00
decentral1se left a comment
Owner
Copy Link

Amazin' 💯

Amazin' 💯
👍 1
cli/app/logs.go Outdated
@ -147,0 +99,4 @@
filters.Add("name", service.Spec.Name)
tasks, err := cl.TaskList(context.Background(), types.TaskListOptions{Filters: f})
if err != nil {
logrus.Fatal(err)
decentral1se commented 2023-12-12 21:02:06 +00:00
Owner
Copy Link

Slightly confusing that this function does return err and also logrus.Fatal(err). Maybe this is handier?

Slightly confusing that this function does `return err` and also `logrus.Fatal(err)`. Maybe this is handier?
p4u1 commented 2023-12-13 08:26:52 +00:00
Author
Member
Copy Link

Good catch! Changed it to return err. I did not change the logrus.Fatal(err) in the goroutine below because otherwise I would need to create an error channel.

Good catch! Changed it to `return err`. I did not change the `logrus.Fatal(err)` in the goroutine below because otherwise I would need to create an error channel.
👍 1
decentral1se marked this conversation as resolved
pkg/config/app.go Outdated
@ -120,3 +110,4 @@
return filters, nil
}
func fmtFilter(appendMode bool, exact bool, stack string, service string) string {
decentral1se commented 2023-12-12 21:04:37 +00:00
Owner
Copy Link

Missing a docstring? Pretty cryptic function otherwise. Can be taken from the Filters docstring.

Missing a docstring? Pretty cryptic function otherwise. Can be taken from the `Filters` docstring.
p4u1 commented 2023-12-13 08:27:44 +00:00
Author
Member
Copy Link

I added a docstring and simplified the function by moving the appendMode into the tailLogs function.

I added a docstring and simplified the function by moving the `appendMode` into the `tailLogs` function.
❤️ 1
decentral1se marked this conversation as resolved
pkg/config/app.go Outdated
@ -603,3 +606,3 @@
// GetTimeoutFromLabel reads the timeout value from docker label "coop-cloud.${STACK_NAME}.TIMEOUT" and returns 50 as default value
func GetTimeoutFromLabel(compose *composetypes.Config, stackName string) (int, error) {
var timeout = 50 // Default Timeout
timeout := 50 // Default Timeout
decentral1se commented 2023-12-12 21:05:49 +00:00
Owner
Copy Link

Should we maybe abstract to pkg/config/env.go as a constant?

Should we maybe abstract to `pkg/config/env.go` as a constant?
p4u1 commented 2023-12-13 08:30:13 +00:00
Author
Member
Copy Link

This was only changed because I formatted the whole file. I would prefer to make a change like this in a separate pr.

This was only changed because I formatted the whole file. I would prefer to make a change like this in a separate pr.
👍 1
decentral1se marked this conversation as resolved
p4u1 force-pushed logs-task-error from 98671a9f92 to 10e8bf0c7e 2023-12-13 08:24:36 +00:00 Compare
decentral1se approved these changes 2023-12-14 10:46:51 +00:00
decentral1se left a comment
Owner
Copy Link

LGTM! TYSM! 🤘

LGTM! TYSM! 🤘
p4u1 merged commit d4727db8f9 into main 2023-12-14 13:15:24 +00:00
p4u1 deleted branch logs-task-error 2023-12-14 13:15:25 +00:00
moritz commented 2023-12-21 00:25:20 +00:00
Member
Copy Link

Unfortunately this broke the secret handling completely. Most of the secrets can't be found anymore. Its because of the modified Filters function, that is used by the secrets.

Unfortunately this broke the secret handling completely. Most of the secrets can't be found anymore. Its because of the modified `Filters` function, that is used by the secrets.
p4u1 commented 2023-12-21 08:06:55 +00:00
Author
Member
Copy Link

Very sorry about that! Can you be a bit more specific on how to reproduce the problem? Because for me it still works and also the integration tests for secrets still pass.

Very sorry about that! Can you be a bit more specific on how to reproduce the problem? Because for me it still works and also the integration tests for secrets still pass.
Sign in to join this conversation.
Reviewers
No reviewers
decentral1se
Labels
Clear labels   
bug

Something is not working

  
build

go build related issues

  
ci/cd

Building things with CI/CD

  
contributing

Dealing with contributing

  
design

UI/UX

  
documentation

Documenting all the things

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
meta

relating to the project or repo itself

  
question

More information is needed

  
security

Security related

  
wontfix

This won't be fixed

No Label
bug
build
ci/cd
contributing
design
documentation
duplicate
enhancement
help wanted
invalid
meta
question
security
wontfix
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: coop-cloud/abra#395
No description provided.
Delete Branch "p4u1/abra:logs-task-error"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?