Merge ~mthaddon/charm-k8s-bind/+git/charm-k8s-bind:end-user into charm-k8s-bind:master

Proposed by Tom Haddon
Status: Merged
Approved by: Tom Haddon
Approved revision: 3adff97e50bd5b4317d7a91584ce71ee64a363f2
Merged at revision: 0a8dd267a40755311d723c4cb8458a59db4f0b45
Proposed branch: ~mthaddon/charm-k8s-bind/+git/charm-k8s-bind:end-user
Merge into: charm-k8s-bind:master
Diff against target: 115 lines (+25/-49)
2 files modified
README.md (+19/-46)
metadata.yaml (+6/-3)
Reviewer Review Type Date Requested Status
Laurent Sesquès Approve
Canonical IS Reviewers Pending
Review via email: mp+395302@code.launchpad.net

Commit message

Update README and metadata.yaml to be more end-user focused

Description of the change

Update README and metadata.yaml to be more end-user focused

To post a comment you must log in.
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
Laurent Sesquès (sajoupa) :
review: Approve
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision 0a8dd267a40755311d723c4cb8458a59db4f0b45

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/README.md b/README.md
2index 9fc2862..d53b3a5 100644
3--- a/README.md
4+++ b/README.md
5@@ -1,70 +1,37 @@
6-# Bind charm
7+# Bind Operator
8
9 A Juju charm deploying Bind, configurable to use a git repository for its configuration files.
10
11 ## Overview
12
13-This is a k8s workload charm and can only be deployed to a Juju k8s cloud,
14-attached to a controller using `juju add-k8s`.
15+BIND 9 has evolved to be a very flexible, full-featured DNS system. Whatever
16+your application is, BIND 9 probably has the required features. As the first,
17+oldest, and most commonly deployed solution, there are more network engineers
18+who are already familiar with BIND 9 than with any other system.
19
20-This charm is not currently ready for production due to issues with providing
21-an egress to route TCP and UDP traffic to the pods. See:
22+## Usage
23
24-https://bugs.launchpad.net/charm-k8s-bind/+bug/1889746
25-
26-https://bugs.launchpad.net/juju/+bug/1889703
27-
28-## Getting Started
29-
30-Notes for deploying a test setup locally using microk8s, the setup of which is
31-documented here: https://juju.is/docs/microk8s-cloud
32-
33-```
34-juju add-model bind-test
35-```
36-
37-1. Either deploy with the default image from Dockerhub:
38+For details on using Kubernetes with Juju [see here](https://juju.is/docs/kubernetes), and for
39+details on using Juju with MicroK8s for easy local testing [see here](https://juju.is/docs/microk8s-cloud).
40
41+To deploy this charm in a juju k8s model:
42 ```
43 juju deploy cs:~bind-charmers/bind
44 ```
45-2. Or optionally build your own image locally and deploy with that
46-
47-```
48-sudo snap install charmcraft
49-git clone https://git.launchpad.net/charm-k8s-bind
50-cd charm-k8s-bind
51-make image-build
52-docker push localhost:32000/bind
53-juju deploy cs:~bind-charmers/bind --config bind_image_path=localhost:32000/bind:latest bind
54-
55-juju wait
56-juju status
57-```
58-
59-Assuming you're using the image from dockerhub, or an image built locally from
60-this repo without changes, the charm will deploy bind with its stock Ubuntu
61-package configuration, which will forward all queries to root name servers.
62+The charm will deploy bind with its stock Ubuntu package configuration, which
63+will forward all queries to root name servers.
64
65 DNSSEC is also enabled by default.
66
67 Custom config can be deployed by setting the `custom_config_repo` option to
68 point to a Git repository containing a valid set of configuration files with
69-which to populate the /etc/bind/ directory within the pod(s).
70+which to populate the `/etc/bind/` directory within the pod(s).
71
72 Since bind won't allow recursion from external clients by default, to allow
73-easy testing, I've set up a repo with a copy of the stock config, modified to
74-allow recursion from all RFC1918 networks:
75-
76-https://code.launchpad.net/~barryprice/junk/+git/etc-bind-stock-config-with-all-rfc1918-nets-allowed
77-
78-To configure a local microk8s-deployed bind application to use this config,
79-simply run:
80-
81+easy testing, you can use [a repo with a copy of the stock config, modified to allow recursion from all RFC1918 networks](https://code.launchpad.net/~barryprice/junk/+git/etc-bind-stock-config-with-all-rfc1918-nets-allowed):
82 ```
83 juju config bind custom_config_repo=https://git.launchpad.net/~barryprice/junk/+git/etc-bind-stock-config-with-all-rfc1918-nets-allowed
84 ```
85-
86 Once the application is ready, you should be able to query the App's IP address
87 (not the Unit addresses) directly from your local machine:
88
89@@ -84,3 +51,9 @@ $ dig +short example.com @10.152.183.246
90 93.184.216.34
91 $
92 ```
93+
94+## Known Issues
95+
96+This charm is unable to provide an ingress to route TCP and UDP traffic to the
97+pods due to [LP#1889703](https://bugs.launchpad.net/juju/+bug/1889703). This is
98+also being tracked as [a bug against the charm itself](https://bugs.launchpad.net/charm-k8s-bind/+bug/1889746).
99diff --git a/metadata.yaml b/metadata.yaml
100index 12075b3..7b1d7b2 100644
101--- a/metadata.yaml
102+++ b/metadata.yaml
103@@ -1,6 +1,9 @@
104-name: "bind"
105-summary: "Bind"
106-description: "The original, complete open source DNS implementation"
107+name: bind
108+summary: The original, complete open source DNS implementation.
109+docs: https://discourse.charmhub.io/t/bind-documentation-overview/3973
110+description: |
111+ The original, complete open source DNS implementation.
112+ https://www.isc.org/bind/
113 min-juju-version: 2.8.0
114 maintainers:
115 - https://launchpad.net/~bind-charmers <bind-charmers@lists.launchpad.net>

Subscribers

People subscribed via source and target branches

to all changes: