tune-temporal-db-conn-config : lp:~troyanov/maas : Git : Code : MAAS

Get this branch:: git clone -b tune-temporal-db-conn-config https://git.launchpad.net/~troyanov/maas

Only Anton Troyanov can upload to this branch. If you are Anton Troyanov please log in for upload directions.

Branch merges

Merged into maas:master

MAAS Lander: Approve on 2023-11-16

Jacopo Rota: Approve on 2023-11-16

Branch information

Name:: tune-temporal-db-conn-config

Repository:: lp:~troyanov/maas

Recent commits

1df420e... by Anton Troyanov on 2023-11-16

fix: change temporal db connection params

Set maxConnLifetime to "5m" (as regiond)

It appears that maxConns and maxIdleConns are set for each service (x4)
Lower the values so we don't exhaust DB connection pool
maxConns: 5
maxIdleConns: 5

Resolves LP:2043696

f646662... by Nick De Villiers on 2023-11-16

Update maas-ui to d8c92085f
ci: bump node from v16 to v20 LTS (#5210)

Co-authored-by: Peter Makowski <email address hidden>

3cb0f90... by Anton Troyanov on 2023-11-16

refactor: do not retry failed power child workflow

There is no need to retry failed power child workflow, because there is
already a retry policy applied for the activity (to retry 5 times)

```
func execPowerActivity(ctx workflow.Context, params PowerActivityParam) workflow.Future {
  ctx = workflow.WithActivityOptions(ctx, workflow.ActivityOptions{
    StartToCloseTimeout: powerActivityDuration,
    RetryPolicy: &temporal.RetryPolicy{
      MaximumAttempts: 5,
  },
})
```

5c18576... by Adam Collard on 2023-11-16

fix: lp:2042645 add missing return

01eea1b... by Anton Troyanov on 2023-11-15

fix: webhook power driver crashes

Resolves LP:2042497

9e26832... by Anton Troyanov on 2023-11-15

fix: skip null power driver values

bbb850f... by Anton Troyanov on 2023-11-15

refactor: get_effective_power_parameters only required

This change addresses the disclosure of various power parameters to drivers that do not rely on them.

- 'mac_address' used by IPMI only, but it should be BMC address (which MAAS doesn't now), so this parameter can be simply removed
- 'power_address', 'username', 'power_pass' should be set by the driver parameters
- 'power_id' used only by 'virsh
- 'power_driver' default empty value is not used by any driver='power_off_mode' - used by 'ipmi' only
- 'boot_mode' used by 'amt' only

Resolves LP:2042645

4b08f41... by Anton Troyanov on 2023-11-15