Fix signature verification using the upgrade command with the --source-uri flag for fleet-managed agents#11826
Conversation
b8f90c8 to
dfc42fa
Compare
e14d737 to
e8d50e0
Compare
Fix PGP verification to use the .asc file located in the dir specified by --source-uri when upgrading a fleet-managed agent.
b1d0696 to
da13908
Compare
💛 Build succeeded, but was flaky
Failed CI Steps
History
|
|
Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane) |
|
I think you need to play the same game that you have in this PR for the |
|
Nope, the sha512 is already handled as part of the fs downloade: @ycombinator, the error you're seeing is indicating a (silent) permissions issue; the source-uri path does not match the path in the error message (it silently falls back to paths.Download if there is a permissions issue).
|
The path I specified in the As you can see, the What changes should I make to file permissions to make the issue go away so that the following command would not error out on the |
|
Using a multipass VM, i'm unable to recreate: I've even altered the permissions to what you have in order to try: But I'm not able to recreate your error at all. Can you see what's in |
I turned on debug level logging and retried the upgrade command. I see these logs: |
|
Interestingly, I don't run into any issues if I download the target version's Anyway, I'm able to test the changes in this PR now, so I'm unblocked. |
|
@Mergifyio backport 8.19 9.1 9.2 9.3 |
✅ Backports have been createdDetails
|
|
@tlee-elastic fyi |
…e-uri flag for fleet-managed agents (elastic#11826) Fix PGP verification to use the .asc file located in the dir specified by --source-uri when upgrading a fleet-managed agent.
What does this PR do?
Fix PGP verification to use the .asc file located in the dir specified by --source-uri when upgrading a fleet-managed agent.
Why is it important?
Upgrading using a local source currently requires a work-around.
Checklist
I have made corresponding changes to the documentationI have made corresponding change to the default configuration filesI have added tests that prove my fix is effective or that my feature works./changelog/fragmentsusing the changelog toolDisruptive User Impact
N/A
Related issues
file://source URI to upgrade from a local directory, Elastic Agent does not fetch the .asc from the source URI #11152