Skip to content

Add PPA for dotnet backport during image building#222

Merged
yhaliaw merged 8 commits into
mainfrom
fix/dotnet-backport
May 27, 2026
Merged

Add PPA for dotnet backport during image building#222
yhaliaw merged 8 commits into
mainfrom
fix/dotnet-backport

Conversation

@yhaliaw
Copy link
Copy Markdown
Contributor

@yhaliaw yhaliaw commented May 22, 2026

Applicable spec:

Overview

Add the PPA for dotnet backports.

Rationale

Needed to install additional dotnet version for different ubuntu base.
E.g., jammy with dotnet 10, resolute with dotnet 8.

Checklist

Copy link
Copy Markdown
Contributor

Copilot AI commented May 22, 2026

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • canonical-bos01.cdn.snapcraftcontent.com
    • Triggering command: /usr/lib/snapd/snapd /usr/lib/snapd/snapd (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR aims to enable installing additional .NET versions during OpenStack runner image builds by adding the ppa:dotnet/backports repository to the cloud-init provisioning flow. It also includes an unrelated update to the vendored cos_agent charm library affecting scrape config/job naming determinism.

Changes:

  • Add ppa:dotnet/backports via add-apt-repository during image build (cloud-init template + unit test expectation).
  • Document the change in docs/changelog.md.
  • Update vendored lib/charms/grafana_agent/v0/cos_agent.py to generate deterministic scrape config job names and deepcopy generic alert rules (LIBPATCH bump).

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
lib/charms/grafana_agent/v0/cos_agent.py Adds deterministic scrape config job naming + deep-copies alert rule templates; bumps library patch.
docs/changelog.md Adds a changelog entry for the dotnet backports PPA change.
app/tests/unit/test_openstack_builder.py Updates expected rendered cloud-init script to include adding the dotnet backports PPA.
app/src/github_runner_image_builder/templates/cloud-init.sh.j2 Adds dotnet backports PPA setup during apt package installation in cloud-init.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread app/src/github_runner_image_builder/templates/cloud-init.sh.j2 Outdated
Comment thread lib/charms/grafana_agent/v0/cos_agent.py
Comment thread lib/charms/grafana_agent/v0/cos_agent.py
@yhaliaw yhaliaw enabled auto-merge (squash) May 25, 2026 06:47
@yhaliaw yhaliaw merged commit 652665c into main May 27, 2026
44 checks passed
@yhaliaw yhaliaw deleted the fix/dotnet-backport branch May 27, 2026 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants