cartography icon indicating copy to clipboard operation
cartography copied to clipboard

chore: update azure-mgmt-sql requirement from <=1.0.0 to <=3.0.1

Open dependabot[bot] opened this issue 10 months ago • 3 comments

Updates the requirements on azure-mgmt-sql to permit the latest version.

Commits

You can trigger a rebase of this PR by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Note Automatic rebases have been disabled on this pull request as it has been open for over 30 days.

dependabot[bot] avatar Feb 05 '25 23:02 dependabot[bot]

I'm not a user of the Azure module (yet) so a major version upgrade makes me a bit nervous

achantavy avatar Feb 07 '25 07:02 achantavy

Hi @achantavy! I'm currently using azure-mgmt-sql v3.0.1, and I can't integrate Cartography on it because of the dependency mismatch. What do you think what would be the best way to upgrade it in Cartography?

josemazo avatar Nov 11 '25 17:11 josemazo

Hi @achantavy and team! First of all, thanks for the beautiful job you guys do at Cartography. We want to use the main Cartography package but due to the Azure conflict with our version in Prowler we need to fork the repo until this is sorted out (updated). I wanted to let you guys know since it is not company strategy (to fork it whatsoever) but pure engineering and package management. Please let us know when the package version is updated. Thank you!

toniblyx avatar Dec 15 '25 12:12 toniblyx

Hi @achantavy following the above comment from Toni, we created https://github.com/cartography-cncf/cartography/pull/2154 to bump azure-mgmt-sql and also modified the code with the breaking changes included. We run a scan for Azure and everything worked as expected. Thanks!

jfagoagas avatar Dec 19 '25 07:12 jfagoagas