This repository has been archived by the owner on Oct 12, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 9
autogenerated-apimanagementapidiagnostic #87
Merged
Merged
Changes from 14 commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
b375c1c
autogenerated-apimanagementapidiagnostic
audevbot 076e3d2
autogenerated-apimanagementapidiagnostic
audevbot a7f5307
enable code gen for api mgmt diagnostic.
metacpp faca78b
Merge branch 'master' into autogenerated-apimanagementapidiagnostic
metacpp bb0edb2
Merge branch 'master' into autogenerated-apimanagementapidiagnostic
metacpp e7a270f
Merge branch 'master' into autogenerated-apimanagementapidiagnostic
metacpp cbe72ec
remove resources.json
metacpp ba9aa2c
add listing resources script
houkms 827c5ce
testing the script
houkms a8608f5
testing script once more
houkms df3c809
print the mmspecs' branches in pipeline
houkms afe4f42
print the branches
houkms bf73641
try diff branches once more
houkms 2e090d3
listing resources script works well now
houkms 44a5e99
add fallback in script
houkms 35efdeb
test the script
houkms a1facfb
successful test for script, remove test codes
houkms 51091c5
fix minor logic error
houkms File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,7 +4,7 @@ cd `dirname $0` | |
for file in `ls ./specs`; do | ||
work_dir=./specs/$file | ||
if [ -d $work_dir ]; then | ||
if [ -n "$(git status -s $work_dir)" ]; | ||
if [ -n "$(git diff --name-only remotes/origin/master remotes/origin/$source_branch -- $work_dir)" ]; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What if we need to run it in local environment ? Can we add a fallback logic ? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Add the fallback logic in script.
|
||
then | ||
# remove the directory prefix './specs/' | ||
expr substr "$work_dir" 9 "${#work_dir}" | ||
|
This file was deleted.
Oops, something went wrong.
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
--- !ruby/object:Provider::Azure::Ansible::Config | ||
author: audevbot | ||
version_added: "2.9" | ||
overrides: !ruby/object:Overrides::ResourceOverrides | ||
ApiDiagnostic: !ruby/object:Provider::Azure::Ansible::ResourceOverride | ||
examples: [] |
Large diffs are not rendered by default.
Oops, something went wrong.
11 changes: 11 additions & 0 deletions
11
...anagementapidiagnostic/examples/ansible/apimanagement_service_apis_diagnostics_delete.yml
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
--- !ruby/object:Provider::Ansible::Example | ||
task: !ruby/object:Provider::Ansible::Task | ||
name: azure_rm_apimanagementapidiagnostic | ||
description: ApiManagementDeleteApiDiagnostic | ||
code: | ||
resource_group: myResourceGroup | ||
service_name: myService | ||
api_id: myApi | ||
diagnostic_id: myDiagnostic | ||
state: absent | ||
|
37 changes: 37 additions & 0 deletions
37
...managementapidiagnostic/examples/ansible/apimanagement_service_apis_diagnostics_patch.yml
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,37 @@ | ||
--- !ruby/object:Provider::Ansible::Example | ||
task: !ruby/object:Provider::Ansible::Task | ||
name: azure_rm_apimanagementapidiagnostic | ||
description: ApiManagementUpdateApiDiagnostic | ||
code: | ||
resource_group: myResourceGroup | ||
service_name: myService | ||
api_id: myApi | ||
diagnostic_id: myDiagnostic | ||
always_log: allErrors | ||
logger_id: /loggers/applicationinsights | ||
sampling: | ||
sampling_type: fixed | ||
percentage: '50' | ||
frontend: | ||
request: | ||
headers: | ||
- Content-type | ||
body: | ||
bytes: '512' | ||
response: | ||
headers: | ||
- Content-type | ||
body: | ||
bytes: '512' | ||
backend: | ||
request: | ||
headers: | ||
- Content-type | ||
body: | ||
bytes: '512' | ||
response: | ||
headers: | ||
- Content-type | ||
body: | ||
bytes: '512' | ||
|
37 changes: 37 additions & 0 deletions
37
...pimanagementapidiagnostic/examples/ansible/apimanagement_service_apis_diagnostics_put.yml
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,37 @@ | ||
--- !ruby/object:Provider::Ansible::Example | ||
task: !ruby/object:Provider::Ansible::Task | ||
name: azure_rm_apimanagementapidiagnostic | ||
description: ApiManagementCreateApiDiagnostic | ||
code: | ||
resource_group: myResourceGroup | ||
service_name: myService | ||
api_id: myApi | ||
diagnostic_id: myDiagnostic | ||
always_log: allErrors | ||
logger_id: /loggers/applicationinsights | ||
sampling: | ||
sampling_type: fixed | ||
percentage: '50' | ||
frontend: | ||
request: | ||
headers: | ||
- Content-type | ||
body: | ||
bytes: '512' | ||
response: | ||
headers: | ||
- Content-type | ||
body: | ||
bytes: '512' | ||
backend: | ||
request: | ||
headers: | ||
- Content-type | ||
body: | ||
bytes: '512' | ||
response: | ||
headers: | ||
- Content-type | ||
body: | ||
bytes: '512' | ||
|
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,4 @@ | ||
--- !ruby/object:Provider::Azure::Terraform::Config | ||
overrides: !ruby/object:Overrides::ResourceOverrides | ||
ApiDiagnostic: !ruby/object:Provider::Azure::Terraform::ResourceOverride | ||
properties: {} |
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would recommend pass the
source_branch
as argument to Makefile instead of using environment variable.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK. Now the
source_branch
is passed as argument.