Skip to content

Build with charmcraft#72

Merged
addyess merged 9 commits intocanonical:masterfrom
charmed-kubernetes:KU-2847/drop-focal-add-noble-support
Apr 3, 2025
Merged

Build with charmcraft#72
addyess merged 9 commits intocanonical:masterfrom
charmed-kubernetes:KU-2847/drop-focal-add-noble-support

Conversation

@addyess
Copy link
Copy Markdown

@addyess addyess commented Mar 25, 2025

  • Adjustments to build reactive charm using charmcraft

Details

  • negotiable drop of focal, addition of noble support.

@addyess addyess force-pushed the KU-2847/drop-focal-add-noble-support branch from 67b17ee to 0f84deb Compare March 25, 2025 15:58
Copy link
Copy Markdown
Member

@mateoflorido mateoflorido left a comment

Choose a reason for hiding this comment

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

Great, thanks for adding support for Noble!

Comment thread tests/data/bundle.yaml
series: {{ s }}
constraints: arch=amd64
{% endfor %} No newline at end of file
constraints: arch=amd64 virt-type=virtual-machine
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just out of curiosity, why the charm needs a VM now?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

it's probably just force of habit now for me to avoid lxd priviledged containers. The docker-registry charm actually runs the registry in a docker container on the machine. maybe it's not necessary... would you like me to remove this and see how things go without it?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

yep, @mateoflorido -- the reason this was there is now clear:

i had issues running in lxd on noble:

2025-04-03 16:11:07 WARNING unit.docker-registry-noble/0.install logger.go:60 docker: Error response from daemon: failed to create task for container: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error jailing process inside rootfs: pivot_root .: permission denied: unknown.
2025-04-03 16:11:07 ERROR unit.docker-registry-noble/0.juju-log server.go:326 Could not create new container: registry
2025-04-03 16:11:07 ERROR unit.docker-registry-noble/0.juju-log server.go:326 Hook error:
Traceback (most recent call last):
  File "/var/lib/juju/agents/unit-docker-registry-noble-0/.venv/lib/python3.12/site-packages/charms/reactive/__init__.py", line 74, in main
    bus.dispatch(restricted=restricted_mode)
  File "/var/lib/juju/agents/unit-docker-registry-noble-0/.venv/lib/python3.12/site-packages/charms/reactive/bus.py", line 390, in dispatch
    _invoke(other_handlers)
  File "/var/lib/juju/agents/unit-docker-registry-noble-0/.venv/lib/python3.12/site-packages/charms/reactive/bus.py", line 359, in _invoke
    handler.invoke()
  File "/var/lib/juju/agents/unit-docker-registry-noble-0/.venv/lib/python3.12/site-packages/charms/reactive/bus.py", line 181, in invoke
    self._action(*args)
  File "/var/lib/juju/agents/unit-docker-registry-noble-0/charm/reactive/docker_registry.py", line 36, in start
    layer.docker_registry.start_registry()
  File "/var/lib/juju/agents/unit-docker-registry-noble-0/charm/lib/charms/layer/docker_registry.py", line 525, in start_registry
    subprocess.check_call(cmd)
  File "/usr/lib/python3.12/subprocess.py", line 413, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['docker', 'run', '-d', '-p', '5000:5000', '-p', '5001:5001', '--restart', 'unless-stopped', '-v', '/etc/docker/registry/config.yml:/etc/docker/registry/config.yml', '-v', '/srv/registry:/var/lib/registry', '--name', 'registry', 'registry:2']' returned non-zero exit status 126.

@addyess addyess force-pushed the KU-2847/drop-focal-add-noble-support branch from 6c2ccdf to c72417f Compare April 3, 2025 16:43
@addyess addyess merged commit cc3b6c5 into canonical:master Apr 3, 2025
12 checks passed
@addyess addyess deleted the KU-2847/drop-focal-add-noble-support branch April 3, 2025 17:22
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.

2 participants