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

chore(deps): update dependency ops to v2.17.0 #293

Merged
merged 1 commit into from
Oct 1, 2024

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Sep 26, 2024

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
ops (changelog) ==2.16.1 -> ==2.17.0 age adoption passing confidence

Release Notes

canonical/operator (ops)

v2.17.0

Compare Source

Features

  • Optionally install Scenario with ops[testing] and expose the names in ops.testing (#​1381)
  • Change ops.main() so that you don't need to type: ignore it (#​1345)
  • Expand the secret ID out to the full URI when only given the ID (#​1358)
  • Add a JujuVersion property for Pebble log forwarding to Loki (#​1370)
  • Pre-emptively raise InvalidStatusError instead of waiting for Juju:
    • Make it an error to call CollectStatusEvent.add_status with error or unknown (#​1386)
    • Document and validate settable status values in _ModelBackend.set_status (#​1354)

Fixes

  • Fix type of StatusBase subclasses by calling StatusBase.register in __init_subclass__ (#​1383)
  • Secret.set_info and Secret.set_content can be called in the same hook (#​1373)

Documentation

  • Add top-level intro and module-level intros (#​1320)
  • Update the links to the Pebble docs (#​1362)
  • Note about repeatedly setting secret value in Juju 3.6 (#​1366)
  • config-changed is triggered by Juju trust (#​1357)
  • Typo on CharmBase inheritance example by @​theofpa (#​1349)
  • Docs: move Pebble to a separate page (#​1392)

Continuous Integration

  • Periodically run the unit tests of all GitHub-hosted published charms (#​1365)
  • Update the TIOBE reporting for the changes in coverage calculation (#​1367)
  • Spell-check the code as part of linting (#​1388)
  • Run the smoke tests on a schedule (#​1387)

Testing

  • Fix tests that leaked environment variables (#​1385)

Refactoring

  • Move the content of ops.testing to ops._private.harness (#​1369)
  • Keep the unittest.mock names in the 'mock' namespace (#​1379)
  • Deprecate StatusBase.register decorator (#​1384)

Chores

  • Note Juju version on legacy workaround (#​1355)
  • Re-enable test now that Pebble directory permissions are fixed (#​1363)
  • Generate warnings for events that will be removed in Juju 4.0 (#​1374)

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate renovate bot requested a review from a team as a code owner September 26, 2024 07:42
Copy link
Contributor

Test coverage for 7157201

Name              Stmts   Miss Branch BrPart  Cover   Missing
-------------------------------------------------------------
src/charm.py        373     41     94     15    88%   184, 192-193, 205, 337->345, 378->383, 395, 583-585, 590-591, 603-605, 610-611, 623-625, 648-650, 692->exit, 702->exit, 751-754, 764-765, 789-790, 802-803, 830-832, 834->839, 836-837, 879-880, 896-897, 907->exit, 921
src/database.py      29      1      8      1    95%   56
-------------------------------------------------------------
TOTAL               402     42    102     16    88%

Static code analysis report

Run started:2024-09-26 08:02:16.318068

Test results:
  No issues identified.

Code scanned:
  Total lines of code: 2458
  Total lines skipped (#nosec): 2
  Total potential issues skipped due to specifically being disabled (e.g., #nosec BXXX): 0

Run metrics:
  Total issues (by severity):
  	Undefined: 0
  	Low: 0
  	Medium: 0
  	High: 0
  Total issues (by confidence):
  	Undefined: 0
  	Low: 0
  	Medium: 0
  	High: 0
Files skipped (0):

@arturo-seijas arturo-seijas merged commit c73d1c5 into main Oct 1, 2024
20 checks passed
@arturo-seijas arturo-seijas deleted the renovate/ops-2.x branch October 1, 2024 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant