Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PMM-12339 remove go tip #94

Merged
merged 4 commits into from
Jul 20, 2023
Merged

PMM-12339 remove go tip #94

merged 4 commits into from
Jul 20, 2023

Conversation

ademidoff
Copy link
Member

@ademidoff ademidoff marked this pull request as ready for review July 19, 2023 11:33
@ademidoff
Copy link
Member Author

ademidoff commented Jul 19, 2023

@percona/pmm-admins Could you please configure the github actions to run the tests?

Please configure also the reviewers to be assigned by default.

Please configure also the auto-merge.

@ademidoff ademidoff requested review from a team, BupycHuk and JiriCtvrtka and removed request for a team July 19, 2023 11:36
JiriCtvrtka
JiriCtvrtka previously approved these changes Jul 19, 2023
BupycHuk
BupycHuk previously approved these changes Jul 19, 2023
@ademidoff
Copy link
Member Author

The test failures are unrelated to this PR, tests have been failing since long. We need to fix them in a separate PR IMO.

@artemgavrilov
Copy link

@BupycHuk please setup required rewiewers and set auto-merge (I can't find where to configure it)

@artemgavrilov
Copy link

@atymchuk I think it makes sense to change trigger from on: [push, pull_request] to push on main and pull_request on everything else. (same as in other repos). Otherwise tests triggered twice in PRs.

@ademidoff
Copy link
Member Author

ademidoff commented Jul 19, 2023

@atymchuk I think it makes sense to change trigger from on: [push, pull_request] to push on main and pull_request on everything else. (same as in other repos). Otherwise tests triggered twice in PRs.

I fixed this one, but the other one doesn't seem to suffer from the same problem. percona/postgres_exporter#142

Could you please remove the need to approve the workflow runs on every push? That seems to be too annoying.

Screenshot 2023-07-19 at 15 06 27

@BupycHuk BupycHuk merged commit d206d79 into main Jul 20, 2023
1 of 2 checks passed
@BupycHuk BupycHuk deleted the PMM-12339-remove-go-tip branch July 20, 2023 16:35
mprokopov pushed a commit to ozean12/rds_exporter that referenced this pull request Jul 9, 2024
* PMM-12339 remove go tip

PMM-12339

https://goreleaser.com/deprecations/?h=rm+dist#-rm-dist

* PMM-12339 refactor go workflow

* PMM-12339 Hix CI

* PMM-12339 avoid triggering tests twice

---------

Co-authored-by: Artem Gavrilov <artem.gavrilov@percona.com>
mprokopov added a commit to ozean12/rds_exporter that referenced this pull request Jul 9, 2024
* Create rds_exporter-component.yaml

* PMM-6120 Use go 1.14 (percona#45)

* PMM-6213 Remove go 1.13 from travis. (percona#46)

* PMM-6213 Remove go 1.13 from travis.

* PMM-6213 Add go 1.15.x

* PMM-6213 Add go tip.

* PMM-6756 Added go 1.15.x. (percona#48)

* [PMM-6756] Added go 1.15.x.

* PMM-6756 Update

* PMM-6756 Update

* PMM-6756 Revert

* Dummy

* PMM-6756 Update golden files

Co-authored-by: Andrii Skomorokhov <andrii.skomorokhov@percona.com>
Co-authored-by: Alexey Palazhchenko <alexey.palazhchenko@gmail.com>

* Overcome the limitation of 100 dbinstances (percona#47)

* ci: rotate integration keys (percona#52)

* Delete .travis.yml

* PMM-7602 Add GitHub Action (percona#53)

* PMM-7602 Add GitHub Action

* PMM-7602 Point actions to forked repo.

* PMM-7602 Removed travis.

* PMM-7602 Use go modules.

* PMM-7602 Fix check.

* PMM-7602 Fix mod.

* PMM-7602 Fix setup.

* PMM-7602 Fix setup.

* PMM-7602 Fix setup.

* add environment

* PMM-7602 Fix tests.

* PMM-7602 Fix codecov

* PMM-7602 defines what to use go get or go install

* PMM-7602 Improve go get/install

* PMM-7602 Improve get promu.

* PMM-7602 Use tools module for tooling.

* PMM-7602 Fix mods.

* PMM-7602 Fix path

* PMM-7602 Fix gocoverutil

Co-authored-by: Denys Kondratenko <denisok@users.noreply.github.com>

* PPMM-7961 Updated exporter_shared (percona#59)

* PMM-7902 Remove Go 1.15 (percona#60)

* Add docker multistage build (percona#61)

* bring back maxInterval=60 sec (percona#63)

Co-authored-by: Volodymyr Linevych <vlin@hellofresh.com>
Co-authored-by: Volodymyr Linevych <4980689+vlinevych@users.noreply.github.com>

* PMM-8207 Cleanup. (percona#57)

Co-authored-by: Jiri Ctvrtka <jiri.ctvrtka.sde@gmail.com>

* add the ReplicaLag metric (percona#51)

* Update README.md (percona#65)

Remove mentioning upstream as we reworked the project entirely. This statement can confuse people

* PMM-6013 change default branch to main (percona#67)

* Support assume role (percona#70)

* assume role

* move credentials into session

* adding .goreleaser.yml (percona#72)

* adding .goreleaser.yml

Signed-off-by: Max Knee <Max_Knee@comcast.com>

* adding new line

Signed-off-by: Max Knee <Max_Knee@comcast.com>

* Apply suggestions from code review

Co-authored-by: Denys Kondratenko <denisok@users.noreply.github.com>

* Update .goreleaser.yml

* adding releaser action

Signed-off-by: Max Knee <Max_Knee@comcast.com>

* Update release.yml

* Update .goreleaser.yml

* maxknee test docker build

Signed-off-by: Max Knee <Max_Knee@comcast.com>

* adding go 1.16

Signed-off-by: Max Knee <Max_Knee@comcast.com>

* debugging action

Signed-off-by: Max Knee <Max_Knee@comcast.com>

* updating dockerfile

Signed-off-by: Max Knee <Max_Knee@comcast.com>

* reverting docker file and adding new releaserfile

Signed-off-by: Max Knee <Max_Knee@comcast.com>

Co-authored-by: Denys Kondratenko <denisok@users.noreply.github.com>

* Update Dockerfile.releaser

* Update .goreleaser.yml

* Update .goreleaser.yml

* Update release.yml

* Update release.yml

* cost exclaimer  (percona#73)

* cost exclaimer and docker push

* Update Makefile

Remove change of makefile

* PMM-9337 Update go 1.17 (percona#77)

* PMM-9337 Update go 1.17

* Fix codecov

Co-authored-by: JiriCtvrtka <62988319+JiriCtvrtka@users.noreply.github.com>

* Fix incorrect version (percona#76)

* Fix incorrect version.

* FIx for releaser itself.

* Removing promu.

* Delete another file related to promu.

* update ldflags

update ldflags to fill correct values for version

* Remove make build.

* Tidy.

* Revert "Tidy."

This reverts commit 472ca26.

* add prometheus again

* add required packages

* Remove make tarball from CI.

* Remove make tarball

* trigger

Co-authored-by: Denys Kondratenko <denisok@users.noreply.github.com>

* PMM-11049 Typo present in enhanced metrics scraper for rds_exporter (percona#82)

Co-authored-by: cezmunsta <github@incoming-email.co.uk>

* PMM-11312 --version (percona#84)

* PMM-11312 --version

* PMM-11312 make feature-build

* PMM-11312 newline

* PMM-11312 rename command

* PMM-11312 just update version

* PMM-7 Add dependabot (percona#85)

* PMM-11497 Update dependencies. (percona#87)

* PMM-11632 rds_exporter ignores DisableEnhancedMetrics (percona#91)

* Discord link has been removed (percona#92)

* PMM-12339 remove go tip (percona#94)

* PMM-12339 remove go tip

PMM-12339

https://goreleaser.com/deprecations/?h=rm+dist#-rm-dist

* PMM-12339 refactor go workflow

* PMM-12339 Hix CI

* PMM-12339 avoid triggering tests twice

---------

Co-authored-by: Artem Gavrilov <artem.gavrilov@percona.com>

* Rename build to builder

* golang 1.19

* Patch dist

---------

Co-authored-by: Eduardo Rodrigues Semensati <24209506+EduSemensati@users.noreply.github.com>
Co-authored-by: Andrii Skomorokhov <askomorokhov@users.noreply.github.com>
Co-authored-by: Jaroslav Musil <49949480+yareach@users.noreply.github.com>
Co-authored-by: Andrii Skomorokhov <andrii.skomorokhov@percona.com>
Co-authored-by: Alexey Palazhchenko <alexey.palazhchenko@gmail.com>
Co-authored-by: Volodymyr Linevych <4980689+vlinevych@users.noreply.github.com>
Co-authored-by: Alex Tymchuk <alexander.tymchuk@percona.com>
Co-authored-by: Denys Kondratenko <denisok@users.noreply.github.com>
Co-authored-by: Carlos Salguero <carlos.salguero@percona.com>
Co-authored-by: Nurlan Moldomurov <nurlan.moldomurov@percona.com>
Co-authored-by: Volodymyr Linevych <vlin@hellofresh.com>
Co-authored-by: Jiri Ctvrtka <jiri.ctvrtka.sde@gmail.com>
Co-authored-by: ben lovett <blovett@users.noreply.github.com>
Co-authored-by: Roma Novikov <roma.novikov@percona.com>
Co-authored-by: Nikita Beletskii <nikita.beletskii@percona.com>
Co-authored-by: Yonah Dissen <47282577+yonahd@users.noreply.github.com>
Co-authored-by: Max Knee <maxknee@gmail.com>
Co-authored-by: JiriCtvrtka <62988319+JiriCtvrtka@users.noreply.github.com>
Co-authored-by: cezmunsta <github@incoming-email.co.uk>
Co-authored-by: Yaroslav Podorvanov <63663261+YaroslavPodorvanov@users.noreply.github.com>
Co-authored-by: Artem Gavrilov <charlieblackwood7@gmail.com>
Co-authored-by: Przemyslaw Kadej <przemyslaw@kadej.org>
Co-authored-by: Daniil Bazhenov <mail@dbazhenov.ru>
Co-authored-by: Artem Gavrilov <artem.gavrilov@percona.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants