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

Adjustment to fit existing MS name. #121

Draft
wants to merge 34 commits into
base: master
Choose a base branch
from

Conversation

YanumVlad
Copy link

Official module has a typo: credential instead of credentials.
Credentials parameter is in use for the rest of clients in Azure API. Hence, this is a temporary fix until MS fixes core module.

xpdable and others added 30 commits October 21, 2019 10:32
Signed-off-by: Sylvain Hellegouarch <[email protected]>
Signed-off-by: Sylvain Hellegouarch <[email protected]>
Signed-off-by: Bugra Derre (415) <[email protected]>
Signed-off-by: Sylvain Hellegouarch <[email protected]>
added optional path parameter to fill_disk
…-incubator#94)

This commit will interrupt the experiment execution, when
- secrets are missing - instead of continuing the experiment execution
flow
- an invalid cloud is configured - instead of taking the public cloud

Besides that this commit introduces some refactoring:
* it paves the way for further authentication mechanisms
* it separates mixed concerns from the init module into its own auth
* it eases the tests and focuses them on the important parts of this
project module
* it logs error messages from error-prone queries from the resource
graph client

Resolves: chaostoolkit-incubator#84

Signed-off-by: Bugra Derre <[email protected]>
…list-unsupported-windows-scripts

Bug: Update list of unsupported scripts for Windows VM
…/98-offer-vmss-probes

Offer a VMSS probe that counts the number of provisioned instances

LGTM
…e/refactoring-v4

Apply the DRY principle to command module
…troduce-credential-file

Allow to load secrets from azure credential file
This commit enables the ct-azure to use the filter argument
to filter for the VMSS and in the second layer to query for
the instances that are in the filtered VMSS.

Resolves: chaostoolkit-incubator#108

Signed-off-by: Bugra <[email protected]>

b
…e/108-select-more-than-one-vmss-instance

Select more than one VMSS instance
This commit enables the ctk-azure to output results from activities. For
now the activities of concern are the aks/machine actions and the vmss
actions.

In the same commit I fixed minor issues such as the MANIFEST that was
leading to a wrong path of the compute scripts.

Resolves: chaostoolkit-incubator#112

Signed-off-by: Bugra <[email protected]>
…e/112-return-results-from-activities

Return output results for activities
Signed-off-by: Sylvain Hellegouarch <[email protected]>
Signed-off-by: Sylvain Hellegouarch <[email protected]>
Signed-off-by: Sylvain Hellegouarch <[email protected]>
Signed-off-by: Sylvain Hellegouarch <[email protected]>
Signed-off-by: Sylvain Hellegouarch <[email protected]>
Lawouach and others added 4 commits May 14, 2020 13:53
Signed-off-by: Sylvain Hellegouarch <[email protected]>
…tor#119)

Added new Managemant client function which calls WebSiteManagementClient

Signed-off-by: Hemant <[email protected]>
Official module has a typo: credential instead of credentials.
Credentials parameter is in use for the rest of clients in Azure API. Hence, this is a temporary fix until MS fixes core module.

Signed-off-by: Vlad Yanum <[email protected]>
@Lawouach
Copy link
Contributor

That is awesome but sadly I can't that PR now with the changes between the two trees :(

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.

9 participants