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

Add table azure_security_center_jit_network_access_policy. Closes #191 #192

Merged
merged 1 commit into from
Jul 27, 2021

Conversation

Subhajit97
Copy link
Contributor

@Subhajit97 Subhajit97 commented Jul 19, 2021

Integration test logs

Logs
This resource doesn't have any terraform support, also don't have any CLI command to enable it.

Example query results

Results

List virtual machines with JIT access enabled

select
  vm.name,
  vm.id,
  jsonb_pretty(vms -> 'ports') as ports
from
  azure_security_center_jit_network_access_policy,
  jsonb_array_elements(virtual_machines) as vms,
  azure_compute_virtual_machine as vm
where
  lower(vms ->> 'id') = lower(vm.id);
+-----------------+-------------------------------------------------------------------------------------------------------------------------------------------------+---------------------------------------------+
| name            | id                                                                                                                                              | ports                                       |
+-----------------+-------------------------------------------------------------------------------------------------------------------------------------------------+---------------------------------------------+
| testjit19072021 | /subscriptions/********-****-****-****-************/resourceGroups/steampipe-test12/providers/Microsoft.Compute/virtualMachines/testjit19072021 | [                                           |
|                 |                                                                                                                                                 |     {                                       |
|                 |                                                                                                                                                 |         "number": 22,                       |
|                 |                                                                                                                                                 |         "protocol": "*",                    |
|                 |                                                                                                                                                 |         "maxRequestAccessDuration": "PT3H", |
|                 |                                                                                                                                                 |         "allowedSourceAddressPrefix": "*"   |
|                 |                                                                                                                                                 |     }                                       |
|                 |                                                                                                                                                 | ]                                           |
+-----------------+-------------------------------------------------------------------------------------------------------------------------------------------------+---------------------------------------------+

@Subhajit97 Subhajit97 self-assigned this Jul 19, 2021
@Subhajit97 Subhajit97 linked an issue Jul 19, 2021 that may be closed by this pull request
Copy link
Contributor

@bigdatasourav bigdatasourav left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@rajlearner17 rajlearner17 left a comment

Choose a reason for hiding this comment

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

LGTM

@bigdatasourav bigdatasourav merged commit a12cbb7 into main Jul 27, 2021
@bigdatasourav bigdatasourav deleted the issue-191 branch July 27, 2021 08:27
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.

Add table azure_compute_virtual_machine_configuration
3 participants