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.

Sign up for GitHub

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

Jump to bottom

Release v5.2.0 #2326

Merged
merged 126 commits into from
Dec 5, 2024
Merged

Release v5.2.0 #2326

merged 126 commits into from
Dec 5, 2024

Conversation

JimMadge
Copy link
Member

@JimMadge JimMadge commented Dec 2, 2024 •
edited
Loading

✅ Checklist

  • You have given your pull request a meaningful title (e.g. Enable foobar integration rather than 515 foobar).
  • You are targeting the appropriate branch. If you're not certain which one this is, it should be develop.
  • Your branch is up-to-date with the target branch (it probably was when you started, but it may have changed since then).

🚦 Depends on

⤴️ Summary

🌂 Related issues

🔬 Tests

  • Release and security checklists passed

craddm and others added 30 commits November 13, 2024 12:16
@craddm
linting
99cbc50
@craddm
Directly exit after credentials not confirmed
4480f50
@craddm
Check that a user belongs to the correct SHM domain when registering …
95619ec 
…with SRE
@craddm
fix linting
2943ab2
@craddm
get fqdn from stack output
fdcce90
@craddm
Add exports from SRE stack
8b89c9c
@craddm
Catch more specific exceptions rather than all exceptions
e937202
@craddm
Tell user how to remove cached Graph API credential
b2311d4
@craddm
Merge remote-tracking branch 'upstream/develop' into fix-error-message
c984eec
@JimMadge
Add log analytics workspace to gitea and hedgedoc
5149004
@craddm
remove newline from error message
b792c45
@craddm
Catch another specific error type
041b5a4
@craddm
Fix tests to reflect change in exception type
7ae72ab
@JimMadge
Add log analytics to apt proxy
ad95e22
@JimMadge
Add log analytics to clamav mirror
7e6ff1d
@JimMadge
Add log analytics to apricot container group
f21323c
@JimMadge
Add log analytics to gaucamole containers
adbe0e7
@JimMadge
Add log analytics to nexus container group
d51640b
@craddm
fix linting
2b69d97
@craddm
fix linting
111db6f
@craddm
Export sre_fqdn
52f86c1
@craddm
Add final message displaying SRE FQDN in console
bdaaf01
@JimMadge
Break management docs into sections
dad635c
@JimMadge
Change markdown linting rules
8455ab5
@JimMadge
Add docs:clean script
273ebd3
@JimMadge
Fix cross-reference
efdd9f6
@JimMadge
Add docs for container logs
1a4de89
@JimMadge
Merge remote-tracking branch 'origin/develop' into container_logs
9bdac8d
@JimMadge
Fix unresolved merge conflict
fa114c1
@JimMadge
Add missing file 🤦
90f914c
dependabot bot and others added 7 commits December 2, 2024 03:30
@dependabot @github-actions
[dependabot skip] 🔧 Update Python requirements files
04d5802
@dependabot
Bump karancode/yamllint-github-action from 2.1.1 to 3.0.0
2421388 
Bumps [karancode/yamllint-github-action](https://github.com/karancode/yamllint-github-action) from 2.1.1 to 3.0.0.
- [Release notes](https://github.com/karancode/yamllint-github-action/releases)
- [Commits](karancode/yamllint-github-action@v2.1.1...v3.0.0)

---
updated-dependencies:
- dependency-name: karancode/yamllint-github-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
@JimMadge
Merge pull request #2324 from alan-turing-institute/dependabot/github…
8902611 
…_actions/karancode/yamllint-github-action-3.0.0

Bump karancode/yamllint-github-action from 2.1.1 to 3.0.0
@JimMadge
Merge pull request #2323 from alan-turing-institute/dependabot/pip/pr…
a146756 
…oduction-dependencies-00743768ed

Bump the production-dependencies group with 9 updates
@craddm
Split multi-line error message between stdout and log
c64c7d2
@JimMadge
Update release checklist
ec958a5
@JimMadge
Update SECURITY.md
ce17321
@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 2, 2024 •
edited
Loading

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)		Lines missing
  data_safe_haven/commands
  sre.py 167
  users.py 167, 172, 277
  data_safe_haven/console
  __init__.py
  data_safe_haven/exceptions
  __init__.py
  data_safe_haven/external/api
  credentials.py 231-235
  graph_api.py 840
  data_safe_haven/infrastructure/components/composite
  __init__.py
  nfsv3_blob_container.py
  nfsv3_storage_account.py 24-30, 53-144
  data_safe_haven/infrastructure/programs
  declarative_sre.py 167, 431
  data_safe_haven/infrastructure/programs/sre
  apt_proxy_server.py 39
  clamav_mirror.py 36
  data.py 76, 430, 514, 531, 669
  desired_state.py 79, 109, 125
  firewall.py 40, 339
  gitea_server.py 60
  hedgedoc_server.py 63
  identity.py 45
  remote_desktop.py 70
  software_repositories.py 44
  user_services.py 63
Project Total  

This report was generated by python-coverage-comment-action

@JimMadge JimMadge marked this pull request as ready for review December 5, 2024 12:11
@JimMadge JimMadge requested review from a team as code owners December 5, 2024 12:11
craddm
craddm approved these changes Dec 5, 2024
View reviewed changes
Copy link
Contributor

@craddm craddm left a comment

Choose a reason for hiding this comment

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

LGTM

@JimMadge JimMadge merged commit 1f28049 into latest Dec 5, 2024
11 checks passed
@JimMadge JimMadge deleted the release-v5.2.0 branch December 5, 2024 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@craddm craddm craddm approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@JimMadge @craddm