Skip to content

Reorganize storage methods and hooks#2943

Merged
joseivanlopez merged 31 commits intoagama-project:api-v2from
joseivanlopez:adapt-storage-methods
Dec 17, 2025
Merged

Reorganize storage methods and hooks#2943
joseivanlopez merged 31 commits intoagama-project:api-v2from
joseivanlopez:adapt-storage-methods

Conversation

@joseivanlopez
Copy link
Copy Markdown
Contributor

@joseivanlopez joseivanlopez commented Dec 12, 2025 •
edited
Loading

Reorganize the code to work with the storage config model and its related hooks:

  • model/storage/config-model.ts: provides the methods to work with the data of the storage config model. The methods are organized by "namespace" under the directory model/storage/config-model/ (e.g., configModel.boot.isDefault(), configModel.volumeGroup.add(...)).
  • hooks/model/storage/config-model.ts: contains the hooks to work with the storage config model.

The storage hooks provided by the directory queries/ (deprecated) were removed.

Note: the unit tests will be adapted in a separate PR.

@joseivanlopez joseivanlopez force-pushed the adapt-storage-methods branch 2 times, most recently from bddbfd2 to 24d9d9c Compare December 17, 2025 06:21
@joseivanlopez joseivanlopez changed the title Adapt storage methods Reorganize storage methods and hooks Dec 17, 2025
@joseivanlopez joseivanlopez marked this pull request as ready for review December 17, 2025 12:13
@joseivanlopez
Fix installation stage
ee33064
@joseivanlopez
Move api-model methods
7924763
@joseivanlopez
Move methods for finding devices from collection
ef9ee70
@joseivanlopez
Move boot methods
8bac429
@joseivanlopez
Extract partitionable logic
7d7a95c
@joseivanlopez
Extract boot logic
a57e71e
@joseivanlopez
Unify partitionable logic
f603d43
@joseivanlopez
Move partition methods
8868f6a
@joseivanlopez
Unify partition logic
182cc1a
@joseivanlopez
Move volume group methods
f7b024b
@joseivanlopez
Move logical volume names
702d0a8
@joseivanlopez
Move data and utils
448a14e
@joseivanlopez
Move drive methods
e4f2417
@joseivanlopez
Move md raid methods
c60637e
@joseivanlopez
Move search methods
bbdbc7c
@joseivanlopez
Move space policy methods
18cd35c
@joseivanlopez
Move filesystem methods
13f5053
@joseivanlopez
Move device methods
993ffc8
@joseivanlopez
Move devices manager
4bd60f0
@joseivanlopez
Move config model hooks
f3c3e14
@joseivanlopez
Remove useModel hook
0423543
@joseivanlopez
Move model hooks
d2a1ac4
@joseivanlopez
Move boot hooks
89b77c5
@joseivanlopez
Move drive hooks
39b10d1
@joseivanlopez
Move md raid hooks
33ec807
@joseivanlopez
Move volume group hooks
823d862
ancorgs
ancorgs approved these changes Dec 17, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@ancorgs ancorgs left a comment

Choose a reason for hiding this comment

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

I'm approving because this generally looks good to me. But I have to admit there are too many changes to really check them all in detail.

We'll have time to catch any introduced small error in subsequent adaptations (specially of the unit tests).

@joseivanlopez joseivanlopez merged commit 6f46ad7 into agama-project:api-v2 Dec 17, 2025
1 check failed
@imobachgs imobachgs mentioned this pull request Jan 9, 2026
Merged
imobachgs added a commit that referenced this pull request Jan 10, 2026
@imobachgs
Merge the new HTTP API into master (#2998)
4353398 
Merge the new HTTP API. Each PR has been already reviewed, so it should
be safe to merge it.

* #1829
* #2508
* #2772
* #2826
* #2848
* #2860
* #2863
* #2866
* #2867
* #2869
* #2870
* #2871
* #2872
* #2873
* #2874
* #2875
* #2876
* #2877
* #2880
* #2881
* #2882
* #2884
* #2885
* #2886
* #2891
* #2892
* #2893
* #2894
* #2895
* #2896
* #2897
* #2898
* #2899
* #2900
* #2901
* #2902
* #2903
* #2904
* #2908
* #2909
* #2910
* #2912
* #2913
* #2914
* #2915
* #2916
* #2917
* #2918
* #2920
* #2921
* #2923
* #2924
* #2926
* #2928
* #2929
* #2930
* #2933
* #2934
* #2935
* #2936
* #2938
* #2939
* #2942
* #2943
* #2944
* #2945
* #2946
* #2947
* #2948
* #2950
* #2951
* #2952
* #2954
* #2955
* #2956
* #2957
* #2958
* #2959
* #2960
* #2961
* #2963
* #2964
* #2965
* #2967
* #2968
* #2969
* #2970
* #2971
* #2972
* #2974
* #2975
* #2977
* #2978
* #2980
* #2982
* #2983
* #2984
* #2988
* #2989
* #2991
* #2992
* #2993
* #2994
* #2995
* #2996
* #2997
* #2999
@imobachgs imobachgs mentioned this pull request Mar 17, 2026
Merged
imobachgs added a commit that referenced this pull request Mar 17, 2026
@imobachgs
Release version 19 (#3292)
e4e1132 
Prepare to release version 19.

* #1829
* #2508
* #2772
* #2818
* #2826
* #2848
* #2860
* #2863
* #2864
* #2866
* #2867
* #2869
* #2870
* #2871
* #2872
* #2873
* #2874
* #2875
* #2876
* #2877
* #2880
* #2881
* #2882
* #2884
* #2885
* #2886
* #2891
* #2892
* #2893
* #2894
* #2895
* #2896
* #2897
* #2898
* #2899
* #2900
* #2901
* #2902
* #2903
* #2904
* #2908
* #2909
* #2910
* #2912
* #2913
* #2914
* #2915
* #2916
* #2917
* #2918
* #2920
* #2921
* #2923
* #2924
* #2926
* #2928
* #2929
* #2930
* #2933
* #2934
* #2935
* #2936
* #2937
* #2938
* #2939
* #2942
* #2943
* #2944
* #2945
* #2946
* #2947
* #2948
* #2949
* #2950
* #2951
* #2952
* #2954
* #2955
* #2956
* #2957
* #2958
* #2959
* #2960
* #2961
* #2963
* #2964
* #2965
* #2967
* #2968
* #2969
* #2970
* #2971
* #2972
* #2974
* #2975
* #2977
* #2978
* #2980
* #2981
* #2982
* #2983
* #2984
* #2988
* #2989
* #2990
* #2991
* #2992
* #2993
* #2994
* #2995
* #2996
* #2997
* #2998
* #2999
* #3000
* #3001
* #3002
* #3004
* #3005
* #3006
* #3007
* #3008
* #3009
* #3011
* #3012
* #3013
* #3014
* #3015
* #3016
* #3018
* #3019
* #3020
* #3021
* #3022
* #3023
* #3024
* #3025
* #3026
* #3027
* #3028
* #3029
* #3030
* #3031
* #3033
* #3034
* #3035
* #3036
* #3037
* #3039
* #3040
* #3041
* #3042
* #3043
* #3044
* #3045
* #3046
* #3047
* #3048
* #3049
* #3050
* #3051
* #3052
* #3053
* #3054
* #3055
* #3056
* #3057
* #3058
* #3060
* #3061
* #3062
* #3063
* #3064
* #3065
* #3066
* #3067
* #3068
* #3069
* #3070
* #3071
* #3072
* #3073
* #3074
* #3075
* #3076
* #3077
* #3078
* #3079
* #3086
* #3087
* #3088
* #3089
* #3090
* #3091
* #3092
* #3093
* #3094
* #3095
* #3096
* #3097
* #3098
* #3099
* #3100
* #3101
* #3102
* #3103
* #3104
* #3105
* #3106
* #3107
* #3108
* #3109
* #3110
* #3112
* #3113
* #3114
* #3115
* #3116
* #3117
* #3118
* #3119
* #3120
* #3122
* #3123
* #3124
* #3127
* #3128
* #3129
* #3130
* #3131
* #3133
* #3134
* #3135
* #3136
* #3137
* #3138
* #3139
* #3140
* #3141
* #3142
* #3143
* #3144
* #3145
* #3146
* #3147
* #3148
* #3149
* #3150
* #3151
* #3152
* #3153
* #3154
* #3155
* #3157
* #3158
* #3159
* #3160
* #3161
* #3162
* #3163
* #3164
* #3165
* #3166
* #3167
* #3168
* #3169
* #3170
* #3174
* #3175
* #3176
* #3177
* #3178
* #3179
* #3181
* #3182
* #3184
* #3185
* #3186
* #3188
* #3189
* #3190
* #3191
* #3192
* #3194
* #3195
* #3196
* #3197
* #3198
* #3199
* #3200
* #3201
* #3202
* #3203
* #3205
* #3206
* #3208
* #3209
* #3210
* #3213
* #3214
* #3215
* #3216
* #3217
* #3218
* #3219
* #3220
* #3222
* #3223
* #3224
* #3225
* #3226
* #3227
* #3228
* #3229
* #3230
* #3231
* #3232
* #3233
* #3234
* #3235
* #3236
* #3237
* #3238
* #3239
* #3240
* #3241
* #3242
* #3243
* #3244
* #3246
* #3247
* #3248
* #3250
* #3251
* #3252
* #3253
* #3254
* #3255
* #3256
* #3257
* #3258
* #3259
* #3260
* #3261
* #3262
* #3263
* #3265
* #3266
* #3267
* #3268
* #3269
* #3270
* #3271
* #3272
* #3273
* #3274
* #3275
* #3276
* #3277
* #3278
* #3279
* #3280
* #3281
* #3282
* #3283
* #3284
* #3285
* #3286
* #3287
* #3288
* #3289
* #3290
* #3291
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ancorgs ancorgs ancorgs 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

@joseivanlopez @ancorgs