Skip to content

Fix the initialization in single product scenarios#3096

Merged
imobachgs merged 4 commits intomasterfrom
single-product-init
Jan 29, 2026
Merged

Fix the initialization in single product scenarios#3096
imobachgs merged 4 commits intomasterfrom
single-product-init

Conversation

@imobachgs
Copy link
Copy Markdown
Contributor

@imobachgs imobachgs commented Jan 29, 2026

Problem

On single product scenarios, we have a race condition that prevents Agama to start:

  • The manager service sends the SetConfig message to the storage service and awaits for the answer. It happens even before the web server is started.
  • The storage service tries to contact the manager service to set the resolvables.

Solution

  • Do not wait for the storage service when setting the configuration.
  • Additionally, retry when trying to contact the web server. Nothing can guarantee that the server is running when the storage service needs to contact it.

@imobachgs imobachgs force-pushed the single-product-init branch from 14807b1 to d542e44 Compare January 29, 2026 10:10
@imobachgs imobachgs force-pushed the single-product-init branch from d542e44 to 9eb056e Compare January 29, 2026 10:24
ancorgs
ancorgs approved these changes Jan 29, 2026
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.

LGTM

joseivanlopez
joseivanlopez approved these changes Jan 29, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@joseivanlopez joseivanlopez left a comment

Choose a reason for hiding this comment

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

LGTM

@imobachgs imobachgs merged commit f491562 into master Jan 29, 2026
20 checks passed
@imobachgs imobachgs deleted the single-product-init branch January 29, 2026 10:55
bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request Jan 30, 2026
@bmwiedemann
Update rubygem-agama-yast to version 19.pre.devel1272.5cc4683a6 / rev…
aaedde5 
… 35 via SR 1329814

https://build.opensuse.org/request/show/1329814
by user IGonzalezSosa + anag_factory
- Retry requests to the web server if the connection failed
  (gh#agama-project/agama#3096, related to bsc#1257067).

- Configure storage and iSCSI only if needed
  (gh#agama-project/agama#3064).
bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request Jan 30, 2026
@bmwiedemann
Update agama to version 0 / rev 34 via SR 1330000
00f2ba2 
https://build.opensuse.org/request/show/1330000
by user IGonzalezSosa + anag_factory
- Do not export neither network/connections nor network/state when
  they are None (related to bsc#1257400).

- Add support for "onlyRequired" key in "software" section
  using new libzypp zypp.conf.d directory.
  (gh#agama-project/agama#3100)

- Fix "agama config edit" subcommand to update the configuration
  (bsc#1257198).

- Do not wait for the storage service when setting the configuration
  (gh#agama-project/agama#3096, related to bsc#1257067).

- Expose the product configuration even in the software service is busy
  (bsc#1257443).

- Fix answers validation (gh#agama-project/agama#3094).

- Import the security settings correctly (bsc#1257290).

- Use the first product mode if none is given (related to jsc#PED-14307).

- Add agama-iscsi service and integrate iSCSI into the new HTTP API
@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

@joseivanlopez joseivanlopez joseivanlopez approved these changes

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

3 participants

@imobachgs @joseivanlopez @ancorgs