Skip to content
This repository has been archived by the owner on Sep 17, 2024. It is now read-only.

Commit

Permalink
fix: set image version before switching to the configuration file (#565)
Browse files Browse the repository at this point in the history
We still need to download the proper config file from the PR, but we do not
have access to the commit SHA at this moment, unless other moddifications
are in place, like reading from Jenkins
  • Loading branch information
mdelapenya committed Jan 7, 2021
1 parent 4c1cea5 commit cd0bc3d
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions e2e/_suites/metricbeat/metricbeat_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -246,14 +246,14 @@ func (mts *MetricbeatTestSuite) installedAndConfiguredForVariantModule(serviceVa
}

func (mts *MetricbeatTestSuite) installedUsingConfiguration(configuration string) error {
if strings.HasPrefix(metricbeatVersion, "pr-") {
metricbeatVersion = metricbeatVersionBase
}

// at this point we have everything to define the index name
mts.Version = metricbeatVersion
mts.setIndexName()

if strings.HasPrefix(metricbeatVersion, "pr-") {
metricbeatVersion = metricbeatVersionBase
}

// use master branch for snapshots
tag := "v" + metricbeatVersion
if strings.Contains(metricbeatVersion, "SNAPSHOT") {
Expand Down

0 comments on commit cd0bc3d

Please sign in to comment.