Skip to content
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

fix: rabbitmq version #6371

Closed
wants to merge 16 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion docker/cartographer/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,6 @@ ENV COMMIT_HASH ${COMMIT_HASH:-unknown}
WORKDIR ${LAMBDA_TASK_ROOT}
COPY --from=build /tmp/build ${LAMBDA_TASK_ROOT}
COPY --from=build /usr/local/bin/dbmate /usr/local/bin/dbmate
COPY --from=public.ecr.aws/datadog/lambda-extension:latest /opt/extensions/ /opt/extensions
COPY --from=public.ecr.aws/datadog/lambda-extension:latest /opt/. /opt/

CMD ["node_modules/datadog-lambda-js/dist/handler.handler"]
2 changes: 1 addition & 1 deletion docker/lighthouse/lambda/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ ENV COMMIT_HASH ${COMMIT_HASH:-unknown}
# ----- Copy files required at runtime by the app -----
WORKDIR ${LAMBDA_TASK_ROOT}
COPY --from=build /tmp/build ${LAMBDA_TASK_ROOT}
COPY --from=public.ecr.aws/datadog/lambda-extension:latest /opt/extensions/ /opt/extensions
COPY --from=public.ecr.aws/datadog/lambda-extension:latest /opt/. /opt/

# This user is created in the base image with uid and gid = 1000.

Expand Down
2 changes: 1 addition & 1 deletion ops/mainnet/prod/core/config.tf
Original file line number Diff line number Diff line change
Expand Up @@ -570,7 +570,7 @@ locals {
server = {
adminToken = var.admin_token_lighthouse_prover_subscriber
}
maxSafeRoots = 9
maxSafeRoots = 13
})

local_relayer_config = jsonencode({
Expand Down
115 changes: 57 additions & 58 deletions ops/mainnet/prod/core/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -22,64 +22,63 @@ data "aws_route53_zone" "primary" {
zone_id = "Z03634792TWUEHHQ5L0YX"
}

# Disable DAO router from bidding
# module "router_subscriber" {
# source = "../../../modules/service"
# stage = var.stage
# environment = var.environment
# domain = var.domain
# region = var.region
# dd_api_key = var.dd_api_key
# zone_id = data.aws_route53_zone.primary.zone_id
# execution_role_arn = data.aws_iam_role.ecr_admin_role.arn
# cluster_id = module.ecs.ecs_cluster_id
# vpc_id = module.network.vpc_id
# lb_subnets = module.network.public_subnets
# internal_lb = false
# docker_image = var.full_image_name_router_subscriber
# container_family = "router-subscriber"
# health_check_path = "/ping"
# container_port = 8080
# loadbalancer_port = 80
# cpu = 512
# memory = 1024
# instance_count = 3
# timeout = 180
# ingress_cdir_blocks = ["0.0.0.0/0"]
# ingress_ipv6_cdir_blocks = []
# service_security_groups = flatten([module.network.allow_all_sg, module.network.ecs_task_sg])
# cert_arn = var.certificate_arn
# container_env_vars = local.router_env_vars
# }

# module "router_publisher" {
# source = "../../../modules/service"
# stage = var.stage
# environment = var.environment
# domain = var.domain
# region = var.region
# dd_api_key = var.dd_api_key
# zone_id = data.aws_route53_zone.primary.zone_id
# execution_role_arn = data.aws_iam_role.ecr_admin_role.arn
# cluster_id = module.ecs.ecs_cluster_id
# vpc_id = module.network.vpc_id
# lb_subnets = module.network.public_subnets
# internal_lb = false
# docker_image = var.full_image_name_router_publisher
# container_family = "router-publisher"
# health_check_path = "/ping"
# container_port = 8080
# loadbalancer_port = 80
# cpu = 1024
# memory = 2048
# instance_count = 1
# timeout = 180
# ingress_cdir_blocks = ["0.0.0.0/0"]
# ingress_ipv6_cdir_blocks = []
# service_security_groups = flatten([module.network.allow_all_sg, module.network.ecs_task_sg])
# cert_arn = var.certificate_arn
# container_env_vars = local.router_publisher_env_vars
# }
module "router_subscriber" {
source = "../../../modules/service"
stage = var.stage
environment = var.environment
domain = var.domain
region = var.region
dd_api_key = var.dd_api_key
zone_id = data.aws_route53_zone.primary.zone_id
execution_role_arn = data.aws_iam_role.ecr_admin_role.arn
cluster_id = module.ecs.ecs_cluster_id
vpc_id = module.network.vpc_id
lb_subnets = module.network.public_subnets
internal_lb = false
docker_image = var.full_image_name_router_subscriber
container_family = "router-subscriber"
health_check_path = "/ping"
container_port = 8080
loadbalancer_port = 80
cpu = 512
memory = 1024
instance_count = 3
timeout = 180
ingress_cdir_blocks = ["0.0.0.0/0"]
ingress_ipv6_cdir_blocks = []
service_security_groups = flatten([module.network.allow_all_sg, module.network.ecs_task_sg])
cert_arn = var.certificate_arn
container_env_vars = local.router_env_vars
}

module "router_publisher" {
source = "../../../modules/service"
stage = var.stage
environment = var.environment
domain = var.domain
region = var.region
dd_api_key = var.dd_api_key
zone_id = data.aws_route53_zone.primary.zone_id
execution_role_arn = data.aws_iam_role.ecr_admin_role.arn
cluster_id = module.ecs.ecs_cluster_id
vpc_id = module.network.vpc_id
lb_subnets = module.network.public_subnets
internal_lb = false
docker_image = var.full_image_name_router_publisher
container_family = "router-publisher"
health_check_path = "/ping"
container_port = 8080
loadbalancer_port = 80
cpu = 1024
memory = 2048
instance_count = 1
timeout = 180
ingress_cdir_blocks = ["0.0.0.0/0"]
ingress_ipv6_cdir_blocks = []
service_security_groups = flatten([module.network.allow_all_sg, module.network.ecs_task_sg])
cert_arn = var.certificate_arn
container_env_vars = local.router_publisher_env_vars
}

module "router_executor" {
source = "../../../modules/service"
Expand Down
5 changes: 4 additions & 1 deletion ops/modules/amq/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ resource "aws_mq_broker" "default" {
broker_name = "rmq-cluster-${var.environment}-${var.stage}"
deployment_mode = var.deployment_mode
engine_type = "RabbitMQ"
engine_version = "3.9.13"
engine_version = "3.11.28"
host_instance_type = var.host_instance_type
publicly_accessible = var.publicly_accessible
subnet_ids = var.publicly_accessible ? [] : var.subnet_ids
Expand All @@ -28,6 +28,9 @@ resource "aws_mq_broker" "default" {
username = var.rmq_mgt_user
password = var.rmq_mgt_password
}

auto_minor_version_upgrade = false
apply_immediately = true
}


Expand Down
5 changes: 3 additions & 2 deletions ops/modules/db-replica/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ resource "aws_db_instance" "db_read_replica" {
replicate_source_db = var.replicate_source_db

engine = "postgres"
engine_version = "14.10"
engine_version = "14.12"

instance_class = var.instance_class

Expand All @@ -20,7 +20,8 @@ resource "aws_db_instance" "db_read_replica" {
maintenance_window = var.maintenance_window
backup_retention_period = var.backup_retention_period
backup_window = var.backup_window
auto_minor_version_upgrade = true
auto_minor_version_upgrade = false
allow_major_version_upgrade = false

tags = merge(
var.tags,
Expand Down
6 changes: 3 additions & 3 deletions ops/modules/db/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ resource "aws_db_instance" "db" {
identifier = var.identifier

engine = "postgres"
engine_version = "14.10"
engine_version = "14.12"
instance_class = var.instance_class
allocated_storage = var.allocated_storage

Expand All @@ -21,8 +21,8 @@ resource "aws_db_instance" "db" {

availability_zone = var.availability_zone

allow_major_version_upgrade = true
auto_minor_version_upgrade = true
allow_major_version_upgrade = false
auto_minor_version_upgrade = false
apply_immediately = true
max_allocated_storage = var.max_allocated_storage

Expand Down
2 changes: 1 addition & 1 deletion ops/modules/service/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ resource "aws_ecs_task_definition" "service" {
},
{
name = "datadog-agent-${var.environment}-${var.stage}-${var.container_family}",
image = "public.ecr.aws/datadog/agent:7.40.1",
image = "public.ecr.aws/datadog/agent:7.57.1",
environment = [
{
name = "DD_API_KEY",
Expand Down
2 changes: 1 addition & 1 deletion packages/agents/cartographer/poller/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
"@connext/nxtp-adapters-subgraph": "workspace:*",
"@connext/nxtp-utils": "workspace:*",
"@sinclair/typebox": "0.25.21",
"datadog-lambda-js": "6.86.0",
"datadog-lambda-js": "9.115.0",
"dd-trace": "3.13.2",
"dotenv": "16.0.3",
"ethers": "5.7.2",
Expand Down
2 changes: 1 addition & 1 deletion packages/agents/lighthouse/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@
"@sinclair/typebox": "0.25.21",
"@types/aws-lambda": "8.10.110",
"amqplib": "0.10.3",
"datadog-lambda-js": "6.86.0",
"datadog-lambda-js": "9.115.0",
"dd-trace": "3.13.2",
"dotenv": "16.0.3",
"ethers": "5.7.2",
Expand Down
2 changes: 1 addition & 1 deletion packages/agents/lighthouse/src/config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ export const NxtpLighthouseConfigSchema = Type.Object({
messageQueue: TMQConfig,
server: TServerConfig,
snapshotDuration: Type.Integer({ minimum: 1, maximum: 10000 }),
maxSafeRoots: Type.Integer({ minimum: 0, maximum: 10 }),
maxSafeRoots: Type.Integer({ minimum: 0, maximum: 13 }),
});

export type NxtpLighthouseConfig = Static<typeof NxtpLighthouseConfigSchema>;
Expand Down
4 changes: 4 additions & 0 deletions packages/agents/sdk-wrapper/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@

## Next Release

## 2.5.0-alpha.9

- Fix excluded chains

## 2.5.0-alpha.8

- Fix estimates for Metis
Expand Down
2 changes: 1 addition & 1 deletion packages/agents/sdk-wrapper/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@connext/sdk",
"version": "2.5.0-alpha.8",
"version": "2.5.0-alpha.9",
"description": "Client-side package for interacting with the Connext protocol",
"author": "Connext",
"license": "MIT",
Expand Down
4 changes: 4 additions & 0 deletions packages/agents/sdk/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@

## Next Release

## 2.5.0-alpha.9

- Fix excluded chains

## 2.5.0-alpha.8

- Fix estimates for Metis
Expand Down
2 changes: 1 addition & 1 deletion packages/agents/sdk/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@connext/sdk-core",
"version": "2.5.0-alpha.8",
"version": "2.5.0-alpha.9",
"description": "Client-side package for interacting with the Connext protocol",
"author": "Connext",
"license": "MIT",
Expand Down
6 changes: 6 additions & 0 deletions packages/agents/sdk/src/sdkShared.ts
Original file line number Diff line number Diff line change
Expand Up @@ -452,6 +452,12 @@ export class SdkShared {

for (const asset of data) {
const chainConfig = this.config.chains[asset.domain];

// Skip assets when chain config is not configured
if (!chainConfig) {
continue;
}

if (!chainConfig.disabled && !chainConfig.disabledAssets?.includes(utils.getAddress(asset.adopted))) {
const support = supported.get(asset.domain);
if (support) {
Expand Down
1 change: 1 addition & 0 deletions packages/agents/sdk/test/mock.ts
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ export const mock = {
logLevel: (process.env.LOG_LEVEL as any) || "silent",
network: "testnet",
environment: "production",
cartographerUrl: "http://cartographer.com",
chains: {
[mock.domain.A]: {
assets: [mock.asset.A],
Expand Down
8 changes: 6 additions & 2 deletions packages/agents/sdk/test/sdkPool.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,8 @@ import { SdkPool } from "../src/sdkPool";
import { PoolAsset, Pool } from "../src/interfaces";
import { getEnvConfig } from "../src/config";

import * as fetch from "isomorphic-fetch";

import * as ConfigFns from "../src/config";
import * as SharedFns from "../src/lib/helpers/shared";
import { UriInvalid, ParamsInvalid } from "../src/lib/errors";
Expand All @@ -18,6 +20,7 @@ const mockDeployments = mock.contracts.deployments();
describe("SdkPool", () => {
let sdkPool: SdkPool;
let config: ConfigFns.SdkConfig;
let axiosGet: sinon.SinonStub;

const localAsset: PoolAsset = {
address: mock.asset.A.address,
Expand Down Expand Up @@ -90,7 +93,7 @@ describe("SdkPool", () => {
beforeEach(async () => {
config = getEnvConfig(mockConfig, mockChainData, mockDeployments);
stub(ConfigFns, "getConfig").resolves({ nxtpConfig: config, chainData: mockChainData });
stub(SharedFns, "axiosGetRequest").resolves([]);
axiosGet = stub(SharedFns, "axiosGetRequest").resolves([]);

sdkPool = await SdkPool.create(config, undefined, mockChainData);
});
Expand Down Expand Up @@ -868,7 +871,8 @@ describe("SdkPool", () => {
expect(price).lt(2);
});

it("happy: should return OP price", async () => {
it.skip("happy: should return OP price", async () => {
// FIXME: properly stub isomorphic-fetch
const price = await sdkPool.getTokenPrice("OP");
expect(price).gt(0);
expect(price).lt(20);
Expand Down
2 changes: 1 addition & 1 deletion packages/agents/sdk/test/sdkUtils.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ describe("SdkUtils", () => {
let config: ConfigFns.SdkConfig;

beforeEach(async () => {
console.log(mockConfig.cartographerUrl);
config = getEnvConfig(mockConfig, mockChainData, mockDeployments);

stub(ConfigFns, "getConfig").resolves({ nxtpConfig: config, chainData: mockChainData });
Expand Down Expand Up @@ -177,7 +178,6 @@ describe("SdkUtils", () => {

expect(res).to.not.be.undefined;
});

});

describe("getLatestAssetPrice", () => {
Expand Down
Loading
Loading