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.

Sign up for GitHub

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

Jump to bottom

fix: make monitoring optional based on license #108

Merged
merged 4 commits into from
Jul 9, 2024
Merged

fix: make monitoring optional based on license #108

merged 4 commits into from
Jul 9, 2024

Conversation

zachariahmiller
Copy link
Contributor

@zachariahmiller zachariahmiller commented Jul 8, 2024 •
edited
Loading

Description

make monitoring optional based on license due to licensing restrictions.

Related Issue

Fixes #
#103

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Other (security config, docs update, etc)

Checklist before merging

@zachariahmiller zachariahmiller requested a review from a team as a code owner July 8, 2024 21:56
@zachariahmiller zachariahmiller marked this pull request as draft July 8, 2024 21:56
@zachariahmiller zachariahmiller linked an issue Jul 9, 2024 that may be closed by this pull request
Make metrics configuration optional since it requires enterprise license #103
Closed
@zachariahmiller zachariahmiller marked this pull request as ready for review July 9, 2024 18:17
zachariahmiller
zachariahmiller commented Jul 9, 2024
View reviewed changes
README.md Outdated Show resolved Hide resolved
Racer159
Racer159 requested changes Jul 9, 2024
View reviewed changes
README.md Outdated Show resolved Hide resolved
@zachariahmiller @Racer159
Update README.md
65983ff 
Co-authored-by: Wayne Starr <Racer159@users.noreply.github.com>
@zachariahmiller zachariahmiller merged commit 783c05e into main Jul 9, 2024
13 checks passed
@zachariahmiller zachariahmiller deleted the mm_metrics_optional branch July 9, 2024 18:50
@github-actions github-actions bot mentioned this pull request Jul 9, 2024
Merged
Racer159 pushed a commit that referenced this pull request Jul 23, 2024
@github-actions
chore(main): release 9.10.1-uds.0 (#109)
6e02f36 
🤖 I have created a release *beep* *boop*
---


##
[9.10.1-uds.0](v9.9.1-uds.0...v9.10.1-uds.0)
(2024-07-23)


### Features

* native support for plugins
([#99](#99))
([1940db1](1940db1))


### Bug Fixes

* make monitoring optional based on license
([#108](#108))
([783c05e](783c05e))


### Miscellaneous

* **deps:** update mattermost package dependencies
([#106](#106))
([d84f56b](d84f56b))
* **deps:** update mattermost support dependencies
([#110](#110))
([67a5edd](67a5edd))
* **deps:** update mattermost support dependencies
([#98](#98))
([05ae7cf](05ae7cf))
* update release please config for mm plugins
([#111](#111))
([22dce0c](22dce0c))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions github-actions bot mentioned this pull request Jul 23, 2024
Merged
Racer159 added a commit that referenced this pull request Jul 23, 2024
@github-actions @Racer159
chore(main): release 9.10.1-uds.0 (#113)
c64c824 
🤖 I have created a release *beep* *boop*
---

##
[9.10.1-uds.0](v9.9.1-uds.0...v9.10.1-uds.0)
(2024-07-23)


### Features

* native support for plugins
([#99](#99))
([1940db1](1940db1))


### Bug Fixes

* make monitoring optional based on license
([#108](#108))
([783c05e](783c05e))


### Miscellaneous

* add setup buildx to tag and release flow
([#112](#112))
([cbf46e5](cbf46e5))
* **deps:** update mattermost package dependencies
([#106](#106))
([d84f56b](d84f56b))
* **deps:** update mattermost support dependencies
([#110](#110))
([67a5edd](67a5edd))
* **deps:** update mattermost support dependencies
([#98](#98))
([05ae7cf](05ae7cf))
* update release please config for mm plugins
([#111](#111))
([22dce0c](22dce0c))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Wayne Starr <me@racer159.com>
@github-actions github-actions bot mentioned this pull request Jul 23, 2024
Merged
Racer159 added a commit that referenced this pull request Jul 23, 2024
@github-actions @Racer159
chore(main): release 9.10.1-uds.0 (#115)
050affb 
🤖 I have created a release *beep* *boop*
---

##
[9.10.1-uds.0](v9.9.1-uds.0...v9.10.1-uds.0)
(2024-07-23)


### Features

* native support for plugins
([#99](#99))
([1940db1](1940db1))


### Bug Fixes

* make monitoring optional based on license
([#108](#108))
([783c05e](783c05e))


### Miscellaneous

* add setup buildx to tag and release flow
([#112](#112))
([cbf46e5](cbf46e5))
* **deps:** update mattermost package dependencies
([#106](#106))
([d84f56b](d84f56b))
* **deps:** update mattermost support dependencies
([#110](#110))
([67a5edd](67a5edd))
* **deps:** update mattermost support dependencies
([#98](#98))
([05ae7cf](05ae7cf))
* setup buildx on test too and --load
([#114](#114))
([2c48609](2c48609))
* update release please config for mm plugins
([#111](#111))
([22dce0c](22dce0c))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Wayne Starr <me@racer159.com>
@github-actions github-actions bot mentioned this pull request Jul 23, 2024
Merged
Racer159 added a commit that referenced this pull request Jul 23, 2024
@github-actions @Racer159
chore(main): release 9.10.1-uds.0 (#116)
0db0bf5 
🤖 I have created a release *beep* *boop*
---


##
[9.10.1-uds.0](v9.9.1-uds.0...v9.10.1-uds.0)
(2024-07-23)


### Features

* native support for plugins
([#99](#99))
([1940db1](1940db1))


### Bug Fixes

* make monitoring optional based on license
([#108](#108))
([783c05e](783c05e))


### Miscellaneous

* add setup buildx to tag and release flow
([#112](#112))
([cbf46e5](cbf46e5))
* **deps:** update mattermost package dependencies
([#106](#106))
([d84f56b](d84f56b))
* **deps:** update mattermost support dependencies
([#110](#110))
([67a5edd](67a5edd))
* **deps:** update mattermost support dependencies
([#98](#98))
([05ae7cf](05ae7cf))
* setup buildx on test too and --load
([#114](#114))
([2c48609](2c48609))
* update release please config for mm plugins
([#111](#111))
([22dce0c](22dce0c))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Wayne Starr <me@racer159.com>
@github-actions github-actions bot mentioned this pull request Jul 23, 2024
Merged
Racer159 added a commit that referenced this pull request Jul 23, 2024
@github-actions @Racer159
chore(main): release 9.10.1-uds.0 (#117)
a9eb22b 
🤖 I have created a release *beep* *boop*
---


##
[9.10.1-uds.0](v9.9.1-uds.0...v9.10.1-uds.0)
(2024-07-23)


### Features

* native support for plugins
([#99](#99))
([1940db1](1940db1))


### Bug Fixes

* make monitoring optional based on license
([#108](#108))
([783c05e](783c05e))


### Miscellaneous

* add setup buildx to tag and release flow
([#112](#112))
([cbf46e5](cbf46e5))
* **deps:** update mattermost package dependencies
([#106](#106))
([d84f56b](d84f56b))
* **deps:** update mattermost support dependencies
([#110](#110))
([67a5edd](67a5edd))
* **deps:** update mattermost support dependencies
([#98](#98))
([05ae7cf](05ae7cf))
* setup buildx on test too and --load
([#114](#114))
([2c48609](2c48609))
* update release please config for mm plugins
([#111](#111))
([22dce0c](22dce0c))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Wayne Starr <me@racer159.com>
@github-actions github-actions bot mentioned this pull request Jul 23, 2024
Merged
Racer159 added a commit that referenced this pull request Jul 23, 2024
@github-actions @Racer159
chore(main): release 9.10.1-uds.0 (#119)
289b48f 
🤖 I have created a release *beep* *boop*
---


##
[9.10.1-uds.0](v9.9.1-uds.0...v9.10.1-uds.0)
(2024-07-23)


### Features

* native support for plugins
([#99](#99))
([1940db1](1940db1))


### Bug Fixes

* make monitoring optional based on license
([#108](#108))
([783c05e](783c05e))


### Miscellaneous

* add setup buildx to tag and release flow
([#112](#112))
([cbf46e5](cbf46e5))
* **deps:** update mattermost package dependencies
([#106](#106))
([d84f56b](d84f56b))
* **deps:** update mattermost support dependencies
([#110](#110))
([67a5edd](67a5edd))
* **deps:** update mattermost support dependencies
([#98](#98))
([05ae7cf](05ae7cf))
* fix disk pressure on the runner during release
([#118](#118))
([a765a2b](a765a2b))
* setup buildx on test too and --load
([#114](#114))
([2c48609](2c48609))
* update release please config for mm plugins
([#111](#111))
([22dce0c](22dce0c))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Wayne Starr <me@racer159.com>
@github-actions github-actions bot mentioned this pull request Aug 7, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Racer159 Racer159 Racer159 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Make metrics configuration optional since it requires enterprise license
2 participants
@zachariahmiller @Racer159