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

Use consistent image #384

Merged
merged 3 commits into from
Aug 24, 2021
Merged

Use consistent image #384

merged 3 commits into from
Aug 24, 2021

Conversation

metanerd
Copy link
Contributor

Summary

Ticket Link

@metanerd
Copy link
Contributor Author

Breaking build step is known to the introducer: https://github.com/mattermost/mmctl/commits/master
I think this could be merged already, even before the build is fixed.

@metanerd metanerd requested a review from streamer45 August 24, 2021 09:36
@@ -25,7 +25,7 @@ commands:
parameters:
version:
type: string
default: 1.31.0
default: 1.42.0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this needed or just updating while we are here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just updating while we are here.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright, as long as it doesn't cause new errors we should be good :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are right, might need a cache key bump.

@metanerd metanerd merged commit e879285 into master Aug 24, 2021
@metanerd metanerd deleted the use-consistent-image branch August 24, 2021 10:12
metanerd added a commit that referenced this pull request Aug 24, 2021
* Use consistent and newer images.

* Bump cache key
metanerd added a commit that referenced this pull request Aug 24, 2021
* Lock CI from /release-5.38 (#369)

Co-authored-by: Mmbot <mmbot@mattermost>

* [Build] Support go 1.17 (#381)

* Build on 1.17.0 for #381 (#383)

* Build on 1.17.0 for #381

* Use consistent image (#384)

* Use consistent and newer images.

* Bump cache key

Co-authored-by: Mattermod <mattermod@users.noreply.github.com>
Co-authored-by: Mmbot <mmbot@mattermost>
Co-authored-by: Roopak Venkatakrishnan <roopak.v@gmail.com>
metanerd added a commit that referenced this pull request Aug 26, 2021
* [Build] Support go 1.17 (#381)

* Build on 1.17.0 for #381 (#383)

* Build on 1.17.0 for #381

* Use consistent image (#384)

* Use consistent and newer images.

* Bump cache key

Co-authored-by: Roopak Venkatakrishnan <roopak.v@gmail.com>
metanerd added a commit that referenced this pull request Aug 26, 2021
* [Build] Support go 1.17 (#381)

* Build on 1.17.0 for #381 (#383)

* Build on 1.17.0 for #381

* Use consistent image (#384)

* Use consistent and newer images.

* Bump cache key

Co-authored-by: Roopak Venkatakrishnan <roopak.v@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants