Skip to content

Commit c312a57

Browse files
gerrod3cursoragent
gerrod3
and
cursoragent
committed
Migrate to Django 5 STORAGES
Assisted by: cursor composer-1.5 Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent fda8d0b commit c312a57

File tree

5 files changed

+65
-48
lines changed

5 files changed

+65
-48
lines changed

‎.ci/assets/ci_constraints.txt‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
# Pulpcore versions without the openapi command do no longer work in the CI
2-
pulpcore>=3.21.30,!=3.23.*,!=3.24.*,!=3.25.*,!=3.26.*,!=3.27.*,!=3.29.*,!=3.30.*,!=3.31.*,!=3.32.*,!=3.33.*,!=3.34.*,!=3.35.*,!=3.36.*,!=3.37.*,!=3.38.*,!=3.40.*,!=3.41.*,!=3.42.*,!=3.43.*,!=3.44.*,!=3.45.*,!=3.46.*,!=3.47.*,!=3.48.*,!=3.50.*,!=3.51.*,!=3.52.*,!=3.53.*,!=3.54.*
2+
# Pulpcore versions without the django 5 storage compatibility will fail, >3.63,<3.70
3+
pulpcore>=3.21.30,!=3.23.*,!=3.24.*,!=3.25.*,!=3.26.*,!=3.27.*,!=3.29.*,!=3.30.*,!=3.31.*,!=3.32.*,!=3.33.*,!=3.34.*,!=3.35.*,!=3.36.*,!=3.37.*,!=3.38.*,!=3.40.*,!=3.41.*,!=3.42.*,!=3.43.*,!=3.44.*,!=3.45.*,!=3.46.*,!=3.47.*,!=3.48.*,!=3.50.*,!=3.51.*,!=3.52.*,!=3.53.*,!=3.54.*,!=3.64.*,!=3.65.*,!=3.66.*,!=3.67.*,!=3.68.*,!=3.69.*
34

45

56
tablib!=3.6.0

‎.ci/scripts/check_release.py‎

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,20 @@
11
#!/usr/bin/env python
2+
# /// script
3+
# requires-python = ">=3.13"
4+
# dependencies = [
5+
# "gitpython>=3.1.46,<3.2.0",
6+
# "packaging>=26.0,<26.1",
7+
# "pyyaml>=6.0.3,<6.1.0",
8+
# ]
9+
# ///
210

311
import argparse
412
import re
513
import os
614
import tomllib
7-
import yaml
815
from pathlib import Path
16+
17+
import yaml
918
from packaging.version import Version
1019
from git import Repo
1120

‎.github/workflows/ci.yml‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -94,13 +94,13 @@ jobs:
9494
run_docs: ${{ needs.check-changes.outputs.run_docs }}
9595

9696
lint:
97-
needs:
98-
- "check-changes"
99-
if: needs.check-changes.outputs.run_tests == '1'
10097
uses: "./.github/workflows/lint.yml"
10198

10299
build:
103-
needs: "lint"
100+
needs:
101+
- "check-changes"
102+
- "lint"
103+
if: needs.check-changes.outputs.run_tests == '1'
104104
uses: "./.github/workflows/build.yml"
105105

106106
test:

‎.github/workflows/scripts/install.sh‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ if [ "$TEST" = "azure" ]; then
109109
- ./azurite:/etc/pulp\
110110
command: "azurite-blob --skipApiVersionCheck --blobHost 0.0.0.0"' vars/main.yaml
111111
sed -i -e '$a azure_test: true\
112-
pulp_scenario_settings: {"AZURE_ACCOUNT_KEY": "Eby8vdM02xNOcqFlqUwJPLlmEtlCDXJ1OUzFT50uSRZ6IFsuFq2UVErCz4I6tq/K1SZFPTOtr/KBHBeksoGMGw==", "AZURE_ACCOUNT_NAME": "devstoreaccount1", "AZURE_CONNECTION_STRING": "DefaultEndpointsProtocol=http;AccountName=devstoreaccount1;AccountKey=Eby8vdM02xNOcqFlqUwJPLlmEtlCDXJ1OUzFT50uSRZ6IFsuFq2UVErCz4I6tq/K1SZFPTOtr/KBHBeksoGMGw==;BlobEndpoint=http://ci-azurite:10000/devstoreaccount1;", "AZURE_CONTAINER": "pulp-test", "AZURE_LOCATION": "pulp3", "AZURE_OVERWRITE_FILES": true, "AZURE_URL_EXPIRATION_SECS": 120, "DEFAULT_FILE_STORAGE": "storages.backends.azure_storage.AzureStorage", "MEDIA_ROOT": "", "content_origin": null, "domain_enabled": true}\
112+
pulp_scenario_settings: {"MEDIA_ROOT": "", "STORAGES": {"default": {"BACKEND": "storages.backends.azure_storage.AzureStorage", "OPTIONS": {"account_key": "Eby8vdM02xNOcqFlqUwJPLlmEtlCDXJ1OUzFT50uSRZ6IFsuFq2UVErCz4I6tq/K1SZFPTOtr/KBHBeksoGMGw==", "account_name": "devstoreaccount1", "azure_container": "pulp-test", "connection_string": "DefaultEndpointsProtocol=http;AccountName=devstoreaccount1;AccountKey=Eby8vdM02xNOcqFlqUwJPLlmEtlCDXJ1OUzFT50uSRZ6IFsuFq2UVErCz4I6tq/K1SZFPTOtr/KBHBeksoGMGw==;BlobEndpoint=http://ci-azurite:10000/devstoreaccount1;", "expiration_secs": 120, "location": "pulp3", "overwrite_files": true}}, "staticfiles": {"BACKEND": "django.contrib.staticfiles.storage.StaticFilesStorage"}}, "content_origin": null, "domain_enabled": true}\
113113
pulp_scenario_env: {}\
114114
' vars/main.yaml
115115
fi

‎template_config.yml‎

Lines changed: 48 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -2,18 +2,21 @@
22
# were not present before running plugin-template have been added with their default values.
33

44
# generated with plugin_template
5+
#
6+
# After editing this file please always reapply the plugin template before committing any changes.
57

6-
api_root: /pulp/
8+
---
9+
api_root: "/pulp/"
710
black: false
811
check_commit_message: true
912
check_gettext: true
1013
check_manifest: true
1114
check_stray_pulpcore_imports: true
12-
ci_base_image: ghcr.io/pulp/pulp-ci-centos9
15+
ci_base_image: "ghcr.io/pulp/pulp-ci-centos9"
1316
ci_env: {}
14-
ci_trigger: '{pull_request: {branches: [''*'']}}'
15-
cli_package: pulp-cli
16-
cli_repo: https://github.com/pulp/pulp-cli.git
17+
ci_trigger: "{pull_request: {branches: ['*']}}"
18+
cli_package: "pulp-cli"
19+
cli_repo: "https://github.com/pulp/pulp-cli.git"
1720
core_import_allowed: []
1821
deploy_client_to_pypi: true
1922
deploy_client_to_rubygems: true
@@ -23,68 +26,71 @@ docker_fixtures: false
2326
extra_files: []
2427
flake8: true
2528
flake8_ignore: []
26-
github_org: pulp
27-
latest_release_branch: '3.12'
29+
github_org: "pulp"
30+
latest_release_branch: "3.12"
2831
lint_requirements: true
2932
os_required_packages: []
3033
parallel_test_workers: 8
31-
plugin_app_label: python
32-
plugin_default_branch: main
33-
plugin_name: pulp_python
34+
plugin_app_label: "python"
35+
plugin_default_branch: "main"
36+
plugin_name: "pulp_python"
3437
plugins:
35-
- app_label: python
36-
name: pulp_python
37-
post_job_template: null
38-
pre_job_template: null
38+
- app_label: "python"
39+
name: "pulp_python"
3940
pulp_env: {}
4041
pulp_env_azure: {}
4142
pulp_env_gcp: {}
4243
pulp_env_s3: {}
43-
pulp_scheme: https
44+
pulp_scheme: "https"
4445
pulp_settings:
45-
allowed_export_paths: /tmp
46-
allowed_import_paths: /tmp
46+
allowed_export_paths: "/tmp"
47+
allowed_import_paths: "/tmp"
4748
orphan_protection_time: 0
48-
pypi_api_hostname: https://pulp:443
49+
pypi_api_hostname: "https://pulp:443"
4950
pulp_settings_azure:
50-
AZURE_ACCOUNT_KEY: Eby8vdM02xNOcqFlqUwJPLlmEtlCDXJ1OUzFT50uSRZ6IFsuFq2UVErCz4I6tq/K1SZFPTOtr/KBHBeksoGMGw==
51-
AZURE_ACCOUNT_NAME: devstoreaccount1
52-
AZURE_CONNECTION_STRING: DefaultEndpointsProtocol=http;AccountName=devstoreaccount1;AccountKey=Eby8vdM02xNOcqFlqUwJPLlmEtlCDXJ1OUzFT50uSRZ6IFsuFq2UVErCz4I6tq/K1SZFPTOtr/KBHBeksoGMGw==;BlobEndpoint=http://ci-azurite:10000/devstoreaccount1;
53-
AZURE_CONTAINER: pulp-test
54-
AZURE_LOCATION: pulp3
55-
AZURE_OVERWRITE_FILES: true
56-
AZURE_URL_EXPIRATION_SECS: 120
57-
DEFAULT_FILE_STORAGE: storages.backends.azure_storage.AzureStorage
58-
MEDIA_ROOT: ''
51+
MEDIA_ROOT: ""
52+
STORAGES:
53+
default:
54+
BACKEND: "storages.backends.azure_storage.AzureStorage"
55+
OPTIONS:
56+
account_key: "Eby8vdM02xNOcqFlqUwJPLlmEtlCDXJ1OUzFT50uSRZ6IFsuFq2UVErCz4I6tq/K1SZFPTOtr/KBHBeksoGMGw=="
57+
account_name: "devstoreaccount1"
58+
azure_container: "pulp-test"
59+
connection_string: "DefaultEndpointsProtocol=http;AccountName=devstoreaccount1;AccountKey=Eby8vdM02xNOcqFlqUwJPLlmEtlCDXJ1OUzFT50uSRZ6IFsuFq2UVErCz4I6tq/K1SZFPTOtr/KBHBeksoGMGw==;BlobEndpoint=http://ci-azurite:10000/devstoreaccount1;"
60+
expiration_secs: 120
61+
location: "pulp3"
62+
overwrite_files: true
63+
staticfiles:
64+
BACKEND: "django.contrib.staticfiles.storage.StaticFilesStorage"
5965
content_origin: null
6066
domain_enabled: true
6167
pulp_settings_gcp: null
6268
pulp_settings_s3:
63-
MEDIA_ROOT: ''
69+
MEDIA_ROOT: ""
6470
STORAGES:
6571
default:
66-
BACKEND: storages.backends.s3boto3.S3Boto3Storage
72+
BACKEND: "storages.backends.s3boto3.S3Boto3Storage"
6773
OPTIONS:
68-
access_key: AKIAIT2Z5TDYPX3ARJBA
69-
addressing_style: path
70-
bucket_name: pulp3
71-
default_acl: '@none'
72-
endpoint_url: http://minio:9000
73-
region_name: eu-central-1
74-
secret_key: fqRvjWaPU5o0fCqQuUWbj9Fainj2pVZtBCiDiieS
75-
signature_version: s3v4
74+
access_key: "AKIAIT2Z5TDYPX3ARJBA"
75+
addressing_style: "path"
76+
bucket_name: "pulp3"
77+
default_acl: "@none"
78+
endpoint_url: "http://minio:9000"
79+
region_name: "eu-central-1"
80+
secret_key: "fqRvjWaPU5o0fCqQuUWbj9Fainj2pVZtBCiDiieS"
81+
signature_version: "s3v4"
7682
staticfiles:
77-
BACKEND: django.contrib.staticfiles.storage.StaticFilesStorage
83+
BACKEND: "django.contrib.staticfiles.storage.StaticFilesStorage"
7884
domain_enabled: true
7985
pydocstyle: true
80-
release_email: pulp-infra@redhat.com
81-
release_user: pulpbot
86+
release_email: "pulp-infra@redhat.com"
87+
release_user: "pulpbot"
8288
stalebot: true
8389
stalebot_days_until_close: 30
8490
stalebot_days_until_stale: 90
8591
stalebot_limit_to_pulls: true
8692
supported_release_branches:
87-
- '3.11'
93+
- "3.11"
8894
sync_ci: true
8995
test_azure: true
9096
test_cli: true
@@ -95,4 +101,5 @@ test_performance: false
95101
test_reroute: true
96102
test_s3: true
97103
use_issue_template: true
104+
...
98105

0 commit comments

Comments
 (0)