Move shamir's into public sdk/ space, fix titles

This moves shamir's code into the public sdk/ space, encouraging the
reuse of this component by other external libraries. Also fixes the
titles of API and SDK READMEs.

Signed-off-by: Alexander Scheel <alexander.m.scheel@gmail.com>

api/README.md

@@ -1,9 +1,10 @@
-Vault API
+OpenBao API
 =================
 
-This provides the `github.com/openbao/openbao/api` package which contains code useful for interacting with a Vault server.
+This provides the `github.com/openbao/openbao/api` package which contains code useful for interacting with an OpenBao or Vault server.
 
-For examples of how to use this module, see the [vault-examples](https://github.com/hashicorp/vault-examples) repo.
-For a step-by-step walkthrough on using these client libraries, see the [developer quickstart](https://www.vaultproject.io/docs/get-started/developer-qs).
+For examples of how to use this module, see the test suite contained
+within OpenBao or [vault-examples](https://github.com/hashicorp/vault-examples)
+repo from HashiCorp.
 
-[![GoDoc](https://godoc.org/github.com/openbao/openbao/api?status.png)](https://godoc.org/github.com/openbao/openbao/api)
+[![GoDoc](https://godoc.org/github.com/openbao/openbao/api?status.png)](https://godoc.org/github.com/openbao/openbao/api)

changelog/181.txt

@@ -0,0 +1,3 @@
+```release-note:improvement
+sdk/helper/shamir: move Shamir's code into public SDK namespace to encourage external reuse
+```

sdk/README.md

@@ -1,12 +1,12 @@
-Vault SDK libs
+OpenBao SDK libs
 =================
 
 This package provides the `sdk` package which contains code useful for
-developing Vault plugins.
+developing OpenBao plugins.
 
 Although we try not to break functionality, we reserve the right to reorganize
 the code at will and may occasionally cause breaks if they are warranted. As
 such we expect the tag of this module will stay less than `v1.0.0`.
 
 For any major changes we will try to give advance notice in the CHANGES section
-of Vault's CHANGELOG.md.
+of OpenBao's CHANGELOG.md.

vault/core.go

@@ -53,10 +53,10 @@ import (
 	"github.com/openbao/openbao/sdk/helper/jsonutil"
 	"github.com/openbao/openbao/sdk/helper/logging"
 	"github.com/openbao/openbao/sdk/helper/pathmanager"
+	"github.com/openbao/openbao/sdk/helper/shamir"
 	"github.com/openbao/openbao/sdk/logical"
 	"github.com/openbao/openbao/sdk/physical"
 	sr "github.com/openbao/openbao/serviceregistration"
-	"github.com/openbao/openbao/shamir"
 	"github.com/openbao/openbao/vault/cluster"
 	"github.com/openbao/openbao/vault/eventbus"
 	"github.com/openbao/openbao/vault/quotas"

vault/generate_root.go

@@ -14,7 +14,7 @@ import (
 	"github.com/openbao/openbao/helper/pgpkeys"
 	"github.com/openbao/openbao/sdk/helper/consts"
 	"github.com/openbao/openbao/sdk/helper/roottoken"
-	"github.com/openbao/openbao/shamir"
+	"github.com/openbao/openbao/sdk/helper/shamir"
 )
 
 const coreDROperationTokenPath = "core/dr-operation-token"

vault/init.go

@@ -18,7 +18,7 @@ import (
 
 	"github.com/openbao/openbao/helper/namespace"
 	"github.com/openbao/openbao/helper/pgpkeys"
-	"github.com/openbao/openbao/shamir"
+	"github.com/openbao/openbao/sdk/helper/shamir"
 )
 
 // InitParams keeps the init function from being littered with too many

vault/rekey.go

@@ -18,9 +18,9 @@ import (
 	"github.com/openbao/openbao/helper/pgpkeys"
 	"github.com/openbao/openbao/sdk/helper/consts"
 	"github.com/openbao/openbao/sdk/helper/jsonutil"
+	"github.com/openbao/openbao/sdk/helper/shamir"
 	"github.com/openbao/openbao/sdk/logical"
 	"github.com/openbao/openbao/sdk/physical"
-	"github.com/openbao/openbao/shamir"
 	"github.com/openbao/openbao/vault/seal"
 )