-
Notifications
You must be signed in to change notification settings - Fork 303
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.
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
aws lambda - relayer observer #1483
base: main
Are you sure you want to change the base?
Conversation
kakarot_scripts/aws_lambda/relayers_observer/lambda_function.py
Outdated
Show resolved
Hide resolved
Can you add AWS CDK Python probably it is the best idea |
kakarot_scripts/aws_lambda/relayers_observer/lambda_function.py
Outdated
Show resolved
Hide resolved
kakarot_scripts/aws_lambda/relayers_observer/lambda_function.py
Outdated
Show resolved
Hide resolved
kakarot_scripts/aws_lambda/relayers_observer/lambda_function.py
Outdated
Show resolved
Hide resolved
kakarot_scripts/aws_lambda/relayers_observer/lambda_function.py
Outdated
Show resolved
Hide resolved
...lambda/relayers_observer/relayers_observer_service/relayers_observer_lambda/requirements.txt
Outdated
Show resolved
Hide resolved
# trunk-ignore-all(trivy/DS002) | ||
# trunk-ignore-all(trivy/DS026) | ||
# trunk-ignore-all(checkov/CKV_DOCKER_2) | ||
# trunk-ignore-all(checkov/CKV_DOCKER_3) | ||
# trunk-ignore-all(hadolint/DL3033) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do we need to exclude them all?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in my opinion should be excluded, these are the the lint issues:
- Ensure that HEALTHCHECK instructions have been added to container images
- Ensure that a user for the container has been created
- Specify version with `yum install -y <package>-<version>`
- Pin versions in pip. Instead of `pip install <package>` use `pip install <package>==<version>`
- Specify at least 1 USER command in Dockerfile with non-root user as argument
...da/relayers_observer/relayers_observer_service/relayers_observer_lambda/relayers_observer.py
Outdated
Show resolved
Hide resolved
...da/relayers_observer/relayers_observer_service/relayers_observer_lambda/relayers_observer.py
Outdated
Show resolved
Hide resolved
...da/relayers_observer/relayers_observer_service/relayers_observer_lambda/relayers_observer.py
Outdated
Show resolved
Hide resolved
...da/relayers_observer/relayers_observer_service/relayers_observer_lambda/relayers_observer.py
Outdated
Show resolved
Hide resolved
...da/relayers_observer/relayers_observer_service/relayers_observer_lambda/relayers_observer.py
Outdated
Show resolved
Hide resolved
...da/relayers_observer/relayers_observer_service/relayers_observer_lambda/relayers_observer.py
Outdated
Show resolved
Hide resolved
54d7138
to
7f926bb
Compare
What is the status of this @eugypalu |
Resolves #1527
Pull request type
Please check the type of change your PR introduces:
What is the new behavior?
This Lambda automates the process of maintaining relayer balances, ensuring they always have sufficient funds to operate
Balance monitoring: Periodically checks the ETH balance of the main relayer account and individual relayers.
Automatic funding: If a relayer's balance falls below a predefined threshold, the Lambda attempts to fund it automatically with a fixed amount of ETH.
Slack notifications: Sends alerts to Slack channel #relayers-observer when:
Uses AWS Secrets Manager to securely retrieve the private keys necessary for transactions.
Manual replenishment of the main account: avoid automating the withdraw call to the coinbase address to replenish the main relayer account. This decision was taken to prevent the need for an automatic call by the contract owner within the Lambda, thus maintaining greater control over significant fund replenishment operations.
This change is