Merge ~bladernr/plainbox-provider-certification-server:1767255-add-ccs-deprecated-script into plainbox-provider-certification-server:master

Proposed by Jeff Lane 
Status: Merged
Approved by: Jeff Lane 
Approved revision: 62e2d0063f7b6f46644d0833845c91cfe84bec1a
Merged at revision: c4b4d132b82fd017ca70e73917f983d100f6fd3e
Proposed branch: ~bladernr/plainbox-provider-certification-server:1767255-add-ccs-deprecated-script
Merge into: plainbox-provider-certification-server:master
Diff against target: 30 lines (+24/-0)
1 file modified
tools/canonical-certification-server (+24/-0)
Reviewer Review Type Date Requested Status
Pierre Equoy Approve
Rod Smith Approve
Jeff Lane  Needs Resubmitting
Review via email: mp+344772@code.launchpad.net

Commit message

Add a script to replace the deprecated CCS command to inform users that they need to run something else instead.

Description of the change

Add a script to replace the deprecated CCS command to inform users that they need to run something else instead.

To post a comment you must log in.
Revision history for this message
Jeff Lane  (bladernr) wrote :
Revision history for this message
Rod Smith (rodsmith) wrote :

This looks good to me, except for one duplicated word in the message (see inline comment).

review: Needs Fixing
Revision history for this message
Jeff Lane  (bladernr) wrote :

Whoops, missed that when I was re-formatting the output lines.

Thanks. Fixed.

review: Needs Resubmitting
Revision history for this message
Rod Smith (rodsmith) wrote :

LGTM

review: Approve
Revision history for this message
Pierre Equoy (pieq) wrote :

Nice user experience touch!

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/tools/canonical-certification-server b/tools/canonical-certification-server
2new file mode 100755
3index 0000000..4ec9d95
4--- /dev/null
5+++ b/tools/canonical-certification-server
6@@ -0,0 +1,24 @@
7+#!/bin/bash
8+
9+usage () {
10+ echo "DEPRECATED: $0"
11+ echo
12+ echo "*** ATTENTION! ***"
13+ echo "This command is deprecated and no longer functions"
14+ echo "For certification testing, you should now use the version specific"
15+ echo "commands that initiate certification runs."
16+ echo
17+ echo "certify-18.04: launches the 18.04 certification suite"
18+ echo "certify-16.04: launches the 16.04 certification suite"
19+ echo
20+ echo "Additionally there are other commands for running the smaller test"
21+ echo "plans for targeted testing of Network, Storage, USB, and other"
22+ echo "areas."
23+ echo
24+ echo "Please refer to the current Self-Test Guide for more information:"
25+ echo "https://certification.canonical.com/"
26+ echo
27+}
28+
29+usage
30+exit 1

Subscribers

People subscribed via source and target branches