Skip to content

Conversation

@hieblmi
Copy link
Collaborator

@hieblmi hieblmi commented Mar 26, 2025

New loop features like instant loop-out and static loop-in depend on aperture.
This PR adds a aperture docker instance to allow the loop client to acquire a L402 which is precondition for the instant and static service.

@hieblmi hieblmi self-assigned this Mar 26, 2025
@hieblmi hieblmi marked this pull request as draft March 26, 2025 14:32
@hieblmi hieblmi force-pushed the static-regtest branch 2 times, most recently from 7612078 to 557cc15 Compare March 28, 2025 11:16
@hieblmi hieblmi requested review from bhandras, levmi and sputn1ck March 28, 2025 11:17
@hieblmi hieblmi marked this pull request as ready for review March 28, 2025 11:18
@sputn1ck
Copy link
Member

sputn1ck commented Mar 31, 2025

I'm getting the following error when running ./regtest.sh start

Error response from daemon: Could not find the file /root/.aperture/tls.cert in container aperture
chmod: cannot access '/tmp/aperture-tls.cert': No such file or directory
lstat /tmp/aperture-tls.cert: no such file or directory

this is the error in my aperture container

unable to parse config file: [6:15] cannot unmarshal uint64 into Go struct field Config.WriteTimeout of type time.Duration
   3 | servestatic: true
   4 | debuglevel: trace
   5 | insecure: false
>  6 | writetimeout: 0
                     ^
   8 | servername: aperture
   9 | autocert: false
  10 | 
  11 | 

@hieblmi hieblmi force-pushed the static-regtest branch 2 times, most recently from 9c69bda to 2269b20 Compare March 31, 2025 10:01
@hieblmi hieblmi requested a review from sputn1ck March 31, 2025 10:02
Copy link
Member

@sputn1ck sputn1ck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tACK! Great job 💯

command:
- "--configfile=/root/.aperture/aperture.yaml"


Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we also bump the client to 0.18.5 pls.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, I updated it.

@hieblmi hieblmi merged commit c3fa12e into lightninglabs:master Mar 31, 2025
4 checks passed
@hieblmi hieblmi deleted the static-regtest branch March 31, 2025 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants