forked from grafana/grafana
-
Notifications
You must be signed in to change notification settings - Fork 2
Alerting: Update Swagger spec #1
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
Open
github-actions
wants to merge
1
commit into
main
Choose a base branch
from
update-alerting-swagger-spec
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
eb5d919 to
657e728
Compare
657e728 to
34404c9
Compare
0c6b2cd to
b3d6b1c
Compare
asmyasnikov
pushed a commit
that referenced
this pull request
Oct 9, 2023
* use grafana/public instead * parameterise script (still need to fix + run drone locally) * make drone --------- Co-authored-by: Ashley Harrison <[email protected]>
38009f7 to
cd1a70d
Compare
cd1a70d to
95d071b
Compare
95d071b to
5dbb0c0
Compare
5dbb0c0 to
b834316
Compare
f3fdfce to
d333b50
Compare
d333b50 to
63104f2
Compare
63104f2 to
5122939
Compare
asmyasnikov
pushed a commit
that referenced
this pull request
Dec 11, 2023
…rafana#75393) * chore: move over initial changes from sofia's pr grafana#58029 * chore: remove go_image * chore: begin removing edition, remove unused imports * chore: remove edition from swagger_gen.star and generate .drone.yml * chore: regen drone.yml * fix: fix order of load statements * fix: try #2 fix order of load statements * linter fixes * chore: add doc comment explaining purpose of new clone_pr_branch step * fix: add placeholder documentation for ver_mode arg * attempt #1 to import and use clone_enterprise_step_pr * Update scripts/drone/pipelines/swagger_gen.star Co-authored-by: Sofia Papagiannaki <[email protected]> * attempt #2 to import and use clone_enterprise_step_pr * fix: fix drone lint err invalid or unknown step dependency * fix: regen hmac to make drone run * fix: fix drone lint err * fix: update swagger-clean cmd in step * attempt to return non zero exit code * test to see if pipeline fails * fix: add git to clone pr branch step * fix: add git and make to swagger-gen step * try to rerun drone * debug: figure out why cannot find make swagger-clean * debug: see if cd grafana/grafana fixes things * debug: undo cd grafana/grafana * debug: add more logging statements * debug: try and remove cd grafana in swagger-gen * debug: removed grafana after clone statement; add debug before cloning * fix: remove disable clone * regen specs to see if swagger-gen step passes now * add descriptive error message to swagger-gen step * remove api-spec.json from .gitignore * revert backend change, regen spec * add back backend change, regen specs * Update scripts/drone/pipelines/swagger_gen.star Co-authored-by: Sofia Papagiannaki <[email protected]> * Update scripts/drone/pipelines/swagger_gen.star Co-authored-by: Sofia Papagiannaki <[email protected]> * revert gitignore change, set enterprise source to drone source branch * chore: remove unused variable, change committish var name to source * testing functionality: make a new BE change without gen spec, pipeline should fail * test functionality: does removing err msg cause step to fail properly * test functionality: add back err msg and && after * chore: remove debug statements from swagger gen step * chore: remove debug lines from clone_pr_branch step * test functionality: regen specs, swagger_gen step should pass * test funcionality: regen specs again ???? * chore: update swagger-gen step err msg * test functionality: make BE change dont regen spec, swagger gen should fail * test functionality: regen the specs, swagger-gen should pass * chore: revert test BE change, regen spec * chore: remove unused clone step * chore: regen drone.yml --------- Co-authored-by: Timur Olzhabayev <[email protected]> Co-authored-by: Sofia Papagiannaki <[email protected]>
e3fd2a5 to
953c7ab
Compare
bfc6e75 to
5c1b889
Compare
5c1b889 to
1b95980
Compare
771d779 to
68a4e08
Compare
c29c3df to
69b01d6
Compare
69b01d6 to
008aee6
Compare
264ce32 to
171cfb5
Compare
9eebf75 to
8f310f3
Compare
8f310f3 to
4dd7299
Compare
4dd7299 to
5328aea
Compare
58a598b to
3118803
Compare
3118803 to
0050bcb
Compare
91562d3 to
e7c2f12
Compare
e7c2f12 to
c3e6758
Compare
c3e6758 to
9860e0f
Compare
9860e0f to
7297900
Compare
7297900 to
bd94e0b
Compare
bd94e0b to
76f43d8
Compare
76f43d8 to
8366a2d
Compare
da521b3 to
5261b4b
Compare
ab60c15 to
668a971
Compare
0c1b9fc to
9393aab
Compare
9393aab to
4e1cdd0
Compare
98d2810 to
94f37b5
Compare
94f37b5 to
4060326
Compare
4060326 to
f49c6af
Compare
f49c6af to
4f87854
Compare
4f87854 to
1980ce6
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is an automated pull request to update the alerting swagger spec.
Please review and merge.