Skip to content

Commit

Permalink
fix: set image version before switching to the configuration file (el…
Browse files Browse the repository at this point in the history
…astic#565)

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 d130f53 commit 6aa420d
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 6aa420d

Please sign in to comment.