Skip to content

Commit 6053005

Browse files
jfmyers9andrew-edgar
authored andcommitted
Use new locket Client for Presence/Locks
[#102329896] Signed-off-by: Andrew Edgar <[email protected]>
1 parent a803fe6 commit 6053005

File tree

4 files changed

+12
-12
lines changed

4 files changed

+12
-12
lines changed

cmd/bbs/main.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,7 @@ func initializeLockMaintainer(logger lager.Logger, client *api.Client, sessionMa
203203
if err != nil {
204204
logger.Fatal("Couldn't create consul session", err)
205205
}
206-
presenceManager := locket.New(session, clock.NewClock(), logger)
206+
presenceManager := locket.NewClient(session, clock.NewClock(), logger)
207207

208208
uuid, err := uuid.NewV4()
209209
if err != nil {

cmd/bbs/master_lock_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ package main_test
22

33
import (
44
"github.com/cloudfoundry-incubator/bbs/cmd/bbs/testrunner"
5-
"github.com/cloudfoundry-incubator/runtime-schema/bbs/shared"
5+
"github.com/cloudfoundry-incubator/locket"
66
"github.com/tedsuo/ifrit/ginkgomon"
77

88
. "github.com/onsi/ginkgo"
@@ -12,7 +12,7 @@ import (
1212
var _ = Describe("MasterLock", func() {
1313
Context("when the bbs cannot obtain the bbs lock", func() {
1414
BeforeEach(func() {
15-
err := consulSession.AcquireLock(shared.LockSchemaPath("bbs_lock"), []byte{})
15+
err := consulSession.AcquireLock(locket.LockSchemaPath("bbs_lock"), []byte{})
1616
Expect(err).NotTo(HaveOccurred())
1717

1818
bbsRunner = testrunner.New(bbsBinPath, bbsArgs)

cmd/bbs/ping_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ package main_test
22

33
import (
44
"github.com/cloudfoundry-incubator/bbs/cmd/bbs/testrunner"
5-
"github.com/cloudfoundry-incubator/runtime-schema/bbs/shared"
5+
"github.com/cloudfoundry-incubator/locket"
66
"github.com/tedsuo/ifrit/ginkgomon"
77

88
. "github.com/onsi/ginkgo"
@@ -30,7 +30,7 @@ var _ = Describe("Ping API", func() {
3030

3131
Context("when the BBS Server is not the leader", func() {
3232
BeforeEach(func() {
33-
err := consulSession.AcquireLock(shared.LockSchemaPath("bbs_lock"), []byte{})
33+
err := consulSession.AcquireLock(locket.LockSchemaPath("bbs_lock"), []byte{})
3434
Expect(err).NotTo(HaveOccurred())
3535

3636
bbsRunner = testrunner.New(bbsBinPath, bbsArgs)

db/consul/cells_loader_test.go

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import (
99

1010
"github.com/cloudfoundry-incubator/bbs/db"
1111
"github.com/cloudfoundry-incubator/bbs/models"
12-
"github.com/cloudfoundry-incubator/runtime-schema/bbs/services_bbs"
12+
"github.com/cloudfoundry-incubator/locket"
1313
oldmodels "github.com/cloudfoundry-incubator/runtime-schema/models"
1414
. "github.com/onsi/ginkgo"
1515
. "github.com/onsi/gomega"
@@ -24,7 +24,7 @@ var _ = Describe("CellsLoader", func() {
2424
var (
2525
clock *fakeclock.FakeClock
2626

27-
bbs *services_bbs.ServicesBBS
27+
locketClient locket.Client
2828
presence1 ifrit.Process
2929
presence2 ifrit.Process
3030
firstCellPresence oldmodels.CellPresence
@@ -35,7 +35,7 @@ var _ = Describe("CellsLoader", func() {
3535
BeforeEach(func() {
3636
logger = lagertest.NewTestLogger("test")
3737
clock = fakeclock.NewFakeClock(time.Now())
38-
bbs = services_bbs.New(consulSession, clock, logger)
38+
locketClient = locket.NewClient(consulSession, clock, logger)
3939

4040
firstCellPresence = oldmodels.NewCellPresence("first-rep", "1.2.3.4", "the-zone", oldmodels.NewCellCapacity(128, 1024, 3), []string{}, []string{})
4141
secondCellPresence = oldmodels.NewCellPresence("second-rep", "4.5.6.7", "the-zone", oldmodels.NewCellCapacity(128, 1024, 3), []string{}, []string{})
@@ -57,10 +57,10 @@ var _ = Describe("CellsLoader", func() {
5757

5858
BeforeEach(func() {
5959
cellsLoader = consulDB.NewCellsLoader(logger)
60-
presence1 = ifrit.Invoke(bbs.NewCellPresence(firstCellPresence, retryInterval))
60+
presence1 = ifrit.Invoke(locketClient.NewCellPresence(firstCellPresence, retryInterval))
6161

6262
Eventually(func() ([]oldmodels.CellPresence, error) {
63-
return bbs.Cells()
63+
return locketClient.Cells()
6464
}).Should(HaveLen(1))
6565

6666
cells, err = cellsLoader.Cells()
@@ -74,10 +74,10 @@ var _ = Describe("CellsLoader", func() {
7474

7575
Context("when one more cell is added", func() {
7676
BeforeEach(func() {
77-
presence2 = ifrit.Invoke(bbs.NewCellPresence(secondCellPresence, retryInterval))
77+
presence2 = ifrit.Invoke(locketClient.NewCellPresence(secondCellPresence, retryInterval))
7878

7979
Eventually(func() ([]oldmodels.CellPresence, error) {
80-
return bbs.Cells()
80+
return locketClient.Cells()
8181
}).Should(HaveLen(2))
8282
})
8383

0 commit comments

Comments
 (0)