Skip to content

Commit

Permalink
Deprecate Log Plugins (#94)
Browse files Browse the repository at this point in the history
  • Loading branch information
EngHabu authored Jan 20, 2021
1 parent 501b648 commit 27f0eeb
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 0 deletions.
1 change: 1 addition & 0 deletions flyteidl/clients/go/coreutils/logs/cloudwatch.go
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@ func (s cloudwatchLogPlugin) GetTaskLog(podName, namespace, containerName, conta
}, nil
}

// Deprecated: Please use NewTemplateLogPlugin from github.com/lyft/flyteplugins/go/tasks/pluginmachinery/tasklog instead.
func NewCloudwatchLogPlugin(region, groupName string) LogPlugin {
return &cloudwatchLogPlugin{
region: region,
Expand Down
1 change: 1 addition & 0 deletions flyteidl/clients/go/coreutils/logs/kubernetes.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ func (s kubernetesLogPlugin) GetTaskLog(podName, namespace, containerName, conta
}, nil
}

// Deprecated: Please use NewTemplateLogPlugin from github.com/lyft/flyteplugins/go/tasks/pluginmachinery/tasklog instead.
func NewKubernetesLogPlugin(k8sURL string) LogPlugin {
return &kubernetesLogPlugin{
k8sURL: k8sURL,
Expand Down
1 change: 1 addition & 0 deletions flyteidl/clients/go/coreutils/logs/logs.go
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ import (
"github.com/lyft/flyteidl/gen/pb-go/flyteidl/core"
)

// Deprecated: Please use Plugin interface from github.com/lyft/flyteplugins/go/tasks/pluginmachinery/tasklog instead.
type LogPlugin interface {
// Generates a TaskLog object given necessary computation information
GetTaskLog(podName, namespace, containerName, containerID, logName string) (core.TaskLog, error)
Expand Down
1 change: 1 addition & 0 deletions flyteidl/clients/go/coreutils/logs/stackdriver.go
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ func (s *stackdriverLogPlugin) GetTaskLog(podName, namespace, containerName, con
}, nil
}

// Deprecated: Please use NewTemplateLogPlugin from github.com/lyft/flyteplugins/go/tasks/pluginmachinery/tasklog instead.
func NewStackdriverLogPlugin(gcpProject, logResource string) LogPlugin {
return &stackdriverLogPlugin{
gcpProject: gcpProject,
Expand Down

0 comments on commit 27f0eeb

Please sign in to comment.