Skip to content

Commit 0ec1d80

Browse files
committed
Updated new code with new struct names
Signed-off-by: Shmuel Kallner <[email protected]>
1 parent a425e3b commit 0ec1d80

File tree

5 files changed

+10
-10
lines changed

5 files changed

+10
-10
lines changed

pkg/epp/datalayer/endpoint.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -50,17 +50,17 @@ type ModelServer struct {
5050
}
5151

5252
// NewEndpoint returns a new ModelServer with the given PodInfo and Metrics.
53-
func NewEndpoint(pod *PodInfo, metrics *Metrics) *ModelServer {
54-
if pod == nil {
55-
pod = &PodInfo{}
53+
func NewEndpoint(meta *EndpointMetadata, metrics *Metrics) *ModelServer {
54+
if meta == nil {
55+
meta = &EndpointMetadata{}
5656
}
5757
if metrics == nil {
5858
metrics = NewMetrics()
5959
}
6060
ep := &ModelServer{
6161
attributes: NewAttributes(),
6262
}
63-
ep.UpdatePod(pod)
63+
ep.UpdateMetadata(meta)
6464
ep.UpdateMetrics(metrics)
6565
return ep
6666
}

pkg/epp/datalayer/factory_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ func TestFactory(t *testing.T) {
3131
source := &DummySource{}
3232
factory := NewEndpointFactory([]DataSource{source}, 100*time.Millisecond)
3333

34-
pod1 := &PodInfo{
34+
pod1 := &EndpointMetadata{
3535
NamespacedName: types.NamespacedName{
3636
Name: "pod1",
3737
Namespace: "default",
@@ -44,7 +44,7 @@ func TestFactory(t *testing.T) {
4444
dup := factory.NewEndpoint(context.Background(), pod1, nil)
4545
assert.Nil(t, dup, "expected to fail to create a duplicate collector")
4646

47-
pod2 := &PodInfo{
47+
pod2 := &EndpointMetadata{
4848
NamespacedName: types.NamespacedName{
4949
Name: "pod2",
5050
Namespace: "default",

pkg/epp/datalayer/metrics/datasource_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ func TestDatasource(t *testing.T) {
5050

5151
ctx := context.Background()
5252
factory := datalayer.NewEndpointFactory([]datalayer.DataSource{source}, 100*time.Millisecond)
53-
pod := &datalayer.PodInfo{
53+
pod := &datalayer.EndpointMetadata{
5454
NamespacedName: types.NamespacedName{
5555
Name: "pod1",
5656
Namespace: "default",

pkg/epp/datalayer/metrics/extractor.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@ func (ext *Extractor) Extract(ctx context.Context, data any, ep datalayer.Endpoi
138138
}
139139
}
140140

141-
logger := log.FromContext(ctx).WithValues("pod", ep.GetPod().NamespacedName)
141+
logger := log.FromContext(ctx).WithValues("pod", ep.GetMetadata().NamespacedName)
142142
if updated {
143143
clone.UpdateTime = time.Now()
144144
logger.V(logutil.TRACE).Info("Refreshed metrics", "updated", clone)

pkg/epp/datalayer/metrics/logger_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,14 +78,14 @@ func TestLogger(t *testing.T) {
7878
assert.Contains(t, logOutput, "\"Stale metrics\": \"[]\"")
7979
}
8080

81-
var pod1 = &datalayer.PodInfo{
81+
var pod1 = &datalayer.EndpointMetadata{
8282
NamespacedName: types.NamespacedName{
8383
Name: "pod1",
8484
Namespace: "default",
8585
},
8686
Address: "1.2.3.4:5678",
8787
}
88-
var pod2 = &datalayer.PodInfo{
88+
var pod2 = &datalayer.EndpointMetadata{
8989
NamespacedName: types.NamespacedName{
9090
Name: "pod2",
9191
Namespace: "default",

0 commit comments

Comments
 (0)