Skip to content

Conversation

@sed-i
Copy link
Contributor

@sed-i sed-i commented Mar 18, 2025

Issue

On arm, grafana-agent produces the following logs:

Could not locate cos-tool at: "cos-tool-aarch64"
Skipping injection of juju topology as label matchers
`cos-tool` unavailable. Leaving expression unchanged: ...

while the filename we have on disk is cos-tool-arm64.

Solution

Map aarch64 to arm64.

Related: #29

Context

This is only a partial fix, because the CosTool class is still replicated in:

Related:

Testing Instructions

Deploy the following bundle, and when it settles, inspect juju debug-log --replay | grep "Could not locate cos-tool"

default-base: [email protected]/stable
applications:
  ga20:
    charm: grafana-agent
    channel: latest/edge
    revision: 453
    base: [email protected]/stable
  ga22:
    charm: grafana-agent
    channel: latest/edge
    revision: 452
    base: [email protected]/stable
  ga24:
    charm: grafana-agent
    channel: latest/edge
    revision: 454
  hwo20:
    charm: hardware-observer
    channel: latest/stable
    revision: 219
    base: [email protected]/stable
    resources:
      perccli-deb: 1
      sas2ircu-bin: 1
      sas3ircu-bin: 1
      storcli-deb: 1
  hwo22:
    charm: hardware-observer
    channel: latest/stable
    revision: 221
    base: [email protected]/stable
    resources:
      perccli-deb: 1
      sas2ircu-bin: 1
      sas3ircu-bin: 1
      storcli-deb: 1
  hwo24:
    charm: hardware-observer
    channel: latest/stable
    revision: 217
    resources:
      perccli-deb: 1
      sas2ircu-bin: 1
      sas3ircu-bin: 1
      storcli-deb: 1
  ssc:
    charm: self-signed-certificates
    channel: latest/stable
    revision: 265
    base: [email protected]/stable
    num_units: 1
    to:
    - "3"
    constraints: arch=arm64
  ub20:
    charm: ubuntu
    channel: latest/stable
    revision: 25
    base: [email protected]/stable
    num_units: 1
    to:
    - "0"
    constraints: arch=arm64
    storage:
      block: loop,100M
      files: rootfs,100M
  ub22:
    charm: ubuntu
    channel: latest/stable
    revision: 25
    base: [email protected]/stable
    num_units: 1
    to:
    - "1"
    constraints: arch=arm64
    storage:
      block: loop,100M
      files: rootfs,100M
  ub24:
    charm: ubuntu
    channel: latest/stable
    revision: 25
    num_units: 1
    to:
    - "2"
    constraints: arch=arm64
    storage:
      block: loop,100M
      files: rootfs,100M
machines:
  "0":
    constraints: arch=arm64
    base: [email protected]/stable
  "1":
    constraints: arch=arm64
    base: [email protected]/stable
  "2":
    constraints: arch=arm64
  "3":
    constraints: arch=arm64
    base: [email protected]/stable
relations:
- - ga20:juju-info
  - ub20:juju-info
- - ga22:juju-info
  - ub22:juju-info
- - ga24:juju-info
  - ub24:juju-info
- - hwo20:general-info
  - ub20:juju-info
- - hwo20:cos-agent
  - ga20:cos-agent
- - hwo22:general-info
  - ub22:juju-info
- - hwo22:cos-agent
  - ga22:cos-agent
- - hwo24:general-info
  - ub24:juju-info
- - hwo24:cos-agent
  - ga24:cos-agent
- - ssc:certificates
  - ga20:certificates
- - ssc:certificates
  - ga22:certificates
- - ssc:certificates
  - ga24:certificates

@sed-i sed-i marked this pull request as ready for review March 18, 2025 03:42
@sed-i sed-i requested a review from a team as a code owner March 18, 2025 03:42
sed-i added 2 commits March 27, 2025 10:36
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.

4 participants