Skip to content

Commit

Permalink
Rename service/parserprovider to service/mapprovider
Browse files Browse the repository at this point in the history
  • Loading branch information
mx-psi committed Oct 13, 2021
1 parent c114539 commit f7ec3d8
Show file tree
Hide file tree
Showing 19 changed files with 21 additions and 21 deletions.
4 changes: 2 additions & 2 deletions config/configtest/configtest.go
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ import (
"go.opentelemetry.io/collector/component"
"go.opentelemetry.io/collector/config"
"go.opentelemetry.io/collector/config/configunmarshaler"
"go.opentelemetry.io/collector/service/parserprovider"
"go.opentelemetry.io/collector/service/mapprovider"
)

// The regular expression for valid config field tag.
Expand All @@ -35,7 +35,7 @@ var configFieldTagRegExp = regexp.MustCompile("^[a-z0-9][a-z0-9_]*$")
// LoadConfig loads a config from file, and does NOT validate the configuration.
func LoadConfig(fileName string, factories component.Factories) (*config.Config, error) {
// Read yaml config from file
cp, err := parserprovider.NewExpandMapProvider(parserprovider.NewFileMapProvider(fileName)).Get(context.Background())
cp, err := mapprovider.NewExpandMapProvider(mapprovider.NewFileMapProvider(fileName)).Get(context.Background())
if err != nil {
return nil, err
}
Expand Down
10 changes: 5 additions & 5 deletions service/collector_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ import (
"go.opentelemetry.io/collector/service/defaultcomponents"
"go.opentelemetry.io/collector/service/internal/builder"
"go.opentelemetry.io/collector/service/internal/extensions"
"go.opentelemetry.io/collector/service/parserprovider"
"go.opentelemetry.io/collector/service/mapprovider"
)

const configStr = `
Expand Down Expand Up @@ -78,7 +78,7 @@ func TestCollector_StartAsGoRoutine(t *testing.T) {
set := CollectorSettings{
BuildInfo: component.NewDefaultBuildInfo(),
Factories: factories,
ConfigMapProvider: parserprovider.NewInMemoryMapProvider(strings.NewReader(configStr)),
ConfigMapProvider: mapprovider.NewInMemoryMapProvider(strings.NewReader(configStr)),
}
col, err := New(set)
require.NoError(t, err)
Expand Down Expand Up @@ -115,7 +115,7 @@ func TestCollector_Start(t *testing.T) {
col, err := New(CollectorSettings{
BuildInfo: component.NewDefaultBuildInfo(),
Factories: factories,
ConfigMapProvider: parserprovider.NewFileMapProvider("testdata/otelcol-config.yaml"),
ConfigMapProvider: mapprovider.NewFileMapProvider("testdata/otelcol-config.yaml"),
LoggingOptions: []zap.Option{zap.Hooks(hook)},
})
require.NoError(t, err)
Expand Down Expand Up @@ -182,7 +182,7 @@ func TestCollector_ReportError(t *testing.T) {
col, err := New(CollectorSettings{
BuildInfo: component.NewDefaultBuildInfo(),
Factories: factories,
ConfigMapProvider: parserprovider.NewFileMapProvider("testdata/otelcol-config.yaml"),
ConfigMapProvider: mapprovider.NewFileMapProvider("testdata/otelcol-config.yaml"),
})
require.NoError(t, err)

Expand Down Expand Up @@ -302,7 +302,7 @@ func TestCollector_reloadService(t *testing.T) {
},
{
name: "retire_service_ok_load_ok",
parserProvider: parserprovider.NewInMemoryMapProvider(strings.NewReader(configStr)),
parserProvider: mapprovider.NewInMemoryMapProvider(strings.NewReader(configStr)),
service: &service{
telemetry: componenttest.NewNopTelemetrySettings(),
builtExporters: builder.Exporters{},
Expand Down
4 changes: 2 additions & 2 deletions service/collector_windows.go
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ import (
"golang.org/x/sys/windows/svc"
"golang.org/x/sys/windows/svc/eventlog"

"go.opentelemetry.io/collector/service/parserprovider"
"go.opentelemetry.io/collector/service/mapprovider"
)

type WindowsService struct {
Expand Down Expand Up @@ -133,7 +133,7 @@ func openEventLog(serviceName string) (*eventlog.Log, error) {

func newWithWindowsEventLogCore(set CollectorSettings, elog *eventlog.Log) (*Collector, error) {
if set.ConfigMapProvider == nil {
set.ConfigMapProvider = parserprovider.NewDefaultMapProvider(getConfigFlag(), getSetFlag())
set.ConfigMapProvider = mapprovider.NewDefaultMapProvider(getConfigFlag(), getSetFlag())
}
set.LoggingOptions = append(
set.LoggingOptions,
Expand Down
4 changes: 2 additions & 2 deletions service/command.go
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ package service // import "go.opentelemetry.io/collector/service"
import (
"github.com/spf13/cobra"

"go.opentelemetry.io/collector/service/parserprovider"
"go.opentelemetry.io/collector/service/mapprovider"
)

// NewCommand constructs a new cobra.Command using the given Collector.
Expand All @@ -29,7 +29,7 @@ func NewCommand(set CollectorSettings) *cobra.Command {
SilenceUsage: true,
RunE: func(cmd *cobra.Command, args []string) error {
if set.ConfigMapProvider == nil {
set.ConfigMapProvider = parserprovider.NewDefaultMapProvider(getConfigFlag(), getSetFlag())
set.ConfigMapProvider = mapprovider.NewDefaultMapProvider(getConfigFlag(), getSetFlag())
}
col, err := New(set)
if err != nil {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
// See the License for the specific language governing permissions and
// limitations under the License.

package parserprovider // import "go.opentelemetry.io/collector/service/parserprovider"
package mapprovider // import "go.opentelemetry.io/collector/service/mapprovider"

import "go.opentelemetry.io/collector/config"

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
// See the License for the specific language governing permissions and
// limitations under the License.

package parserprovider
package mapprovider

import (
"context"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
// See the License for the specific language governing permissions and
// limitations under the License.

package parserprovider // import "go.opentelemetry.io/collector/service/parserprovider"
package mapprovider // import "go.opentelemetry.io/collector/service/mapprovider"

import (
"context"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
// See the License for the specific language governing permissions and
// limitations under the License.

package parserprovider
package mapprovider

import (
"context"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
// See the License for the specific language governing permissions and
// limitations under the License.

package parserprovider // import "go.opentelemetry.io/collector/service/parserprovider"
package mapprovider // import "go.opentelemetry.io/collector/service/mapprovider"

import (
"context"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
// See the License for the specific language governing permissions and
// limitations under the License.

package parserprovider // import "go.opentelemetry.io/collector/service/parserprovider"
package mapprovider // import "go.opentelemetry.io/collector/service/mapprovider"

import (
"context"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
// See the License for the specific language governing permissions and
// limitations under the License.

package parserprovider // import "go.opentelemetry.io/collector/service/parserprovider"
package mapprovider // import "go.opentelemetry.io/collector/service/mapprovider"

import (
"context"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
// See the License for the specific language governing permissions and
// limitations under the License.

package parserprovider
package mapprovider

import (
"context"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
// See the License for the specific language governing permissions and
// limitations under the License.

package parserprovider // import "go.opentelemetry.io/collector/service/parserprovider"
package mapprovider // import "go.opentelemetry.io/collector/service/mapprovider"

import (
"bytes"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
// See the License for the specific language governing permissions and
// limitations under the License.

package parserprovider
package mapprovider

import (
"context"
Expand Down

0 comments on commit f7ec3d8

Please sign in to comment.