Merge ~athos-ribeiro/ubuntu/+source/isc-kea:dep8-smoke into ubuntu/+source/isc-kea:ubuntu/devel

Proposed by Athos Ribeiro
Status: Merged
Approved by: git-ubuntu bot
Approved revision: not available
Merged at revision: d07c115c7ddff5e14741841f3f33a8642de4ce41
Proposed branch: ~athos-ribeiro/ubuntu/+source/isc-kea:dep8-smoke
Merge into: ubuntu/+source/isc-kea:ubuntu/devel
Diff against target: 47 lines (+27/-0)
3 files modified
debian/changelog (+6/-0)
debian/tests/control (+3/-0)
debian/tests/smoke-tests (+18/-0)
Reviewer Review Type Date Requested Status
git-ubuntu bot Approve
Andreas Hasenack Approve
Canonical Server Reporter Pending
Review via email: mp+436968@code.launchpad.net

Description of the change

Introduce initial smoketests for kea, as per LP: #1863102.

PPA: https://launchpad.net/~athos-ribeiro/+archive/ubuntu/kea-dep8/+packages

To post a comment you must log in.
Revision history for this message
Andreas Hasenack (ahasenack) :
Revision history for this message
Andreas Hasenack (ahasenack) :
Revision history for this message
Athos Ribeiro (athos-ribeiro) wrote :

I addressed the pipefail comment, made sure the remaining service is installed and pushed the changes.

Tests are passing locally.

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

I didn't know you could squash the options together in one line like "set -eo pipefail", but I trust your testing :)

+1

review: Approve
Revision history for this message
git-ubuntu bot (git-ubuntu-bot) wrote :

Approvers: athos-ribeiro, ahasenack
Uploaders: athos-ribeiro, ahasenack
MP auto-approved

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/debian/changelog b/debian/changelog
index 4076e01..0b48206 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,9 @@
1isc-kea (2.2.0-1ubuntu1) lunar; urgency=medium
2
3 * d/tests: add simple DEP8 smoke tests. (LP: #1863102)
4
5 -- Athos Ribeiro <athos.ribeiro@canonical.com> Tue, 07 Feb 2023 14:15:06 -0300
6
1isc-kea (2.2.0-1build1) lunar; urgency=medium7isc-kea (2.2.0-1build1) lunar; urgency=medium
28
3 * No change rebuild for PostgreSQL 159 * No change rebuild for PostgreSQL 15
diff --git a/debian/tests/control b/debian/tests/control
4new file mode 10064410new file mode 100644
index 0000000..166bd24
--- /dev/null
+++ b/debian/tests/control
@@ -0,0 +1,3 @@
1Tests: smoke-tests
2Restrictions: needs-root
3Depends: kea-dhcp4-server, kea-dhcp6-server, kea-ctrl-agent, kea-dhcp-ddns-server, curl, jq
diff --git a/debian/tests/smoke-tests b/debian/tests/smoke-tests
0new file mode 1006444new file mode 100644
index 0000000..2077b90
--- /dev/null
+++ b/debian/tests/smoke-tests
@@ -0,0 +1,18 @@
1#!/bin/bash
2
3set -eo pipefail
4
5# Check dhcp4 server configuration file
6kea-dhcp4 -t /etc/kea/kea-dhcp4.conf
7
8# Check dhcp6 server configuration file
9kea-dhcp6 -t /etc/kea/kea-dhcp6.conf
10
11# Check control agent API
12curl -s -X POST -H "Content-Type: application/json" -d '{ "command": "version-get", "service": [ "dhcp4" ] }' 127.0.0.1:8000 | jq '.[0].text'
13
14# Check control agent API through kea-shell
15echo | kea-shell --service dhcp4 --host 127.0.0.1 --port 8000 version-get | jq '.[0].text'
16
17# Use keactrl to check if services are running
18keactrl status | grep 'DHCPv[46] server: active'

Subscribers

People subscribed via source and target branches