apache / apache/buildstream

Summary shows artifact servers which are not in use

Open
#1,929 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
140
Forks
45
Avg merge
1d 3h
Merged PRs (30d)
6

Description

I just forgot to put my certs/keys in place before a build.

I saw:

```
[--:--:--][ ][ main:core activity ] WARNING Failed to initialize remote : Failed to load credentials file: /home/tristan/work/project/certs/artifacts.key
[--:--:--][ ][ main:core activity ] WARNING Failed to initialize remote :8980: Failed to load credentials file: /home/tristan/work/project/certs/artifacts.key
```

And then in the summary I had:

```
Artifact cache servers
* :8980
push: False type: storage instance: None
loaded from: command line
* :8982
push: False type: index instance: None
loaded from: command line
```

I think this is incorrect, not sure if we should just *omit* this data for caches which we were unable to connect to, or perhaps we should just say `connected: True` or `connected: False` for clarity.

Contributor guide

Open the contributing guide

Research direction

Reproduce a build with the artifact certificates or keys missing and compare the warning with the artifact cache server summary. Trace the summary generation from that behavior, then determine whether failed servers should be omitted or explicitly marked as disconnected. Done means the summary no longer presents unavailable servers as usable and regression coverage verifies the chosen behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
build-system
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.