Commit e3ea129
Cherry pick for Spring Cloud Azure 6.3.0 (#48908)
* Fix: Skip App Configuration validation when feature is disabled (#47588)
(cherry picked from commit 0d08d06)
* Collection monitoring (#47648)
* start collection monitoring
* Updating collection monitoring
* Updating State
* Updated docs
* cleaning up refresh
* Apply suggestions from code review
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* fixing tests
* New tests
* More new tests
* Update AzureAppConfigDataLoader.java
* Fixing Formatting
* Update AppConfigurationReplicaClient.java
* Code Review comments
* review comments
* code review items
* Format fixing
* fixing tests
---------
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
(cherry picked from commit f6267fc)
* App Configuration Provider - Tag filters (#47985)
* Tag Filter + Updated JavaDocs
* Fixing tests
* Update CHANGELOG.md
* Apply suggestions from code review
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* review comments
* Tag Filter util file + tests
* Update ValidationUtilTest.java
* assertsame
---------
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
(cherry picked from commit e43ce23)
* Load balance bug (#48121)
* Fixes Load Balancing Bug
* fixes
* new version
* Apply suggestion from @Copilot
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Update sdk/spring/spring-cloud-azure-appconfiguration-config/src/main/java/com/azure/spring/cloud/appconfiguration/config/implementation/ConnectionManager.java
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Update sdk/spring/spring-cloud-azure-appconfiguration-config/src/test/java/com/azure/spring/cloud/appconfiguration/config/implementation/ConnectionManagerTest.java
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
---------
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
(cherry picked from commit c237703)
* fix(spring-messaging-servicebus): evict stale processors from cache (#48062)
* fix(spring-messaging-servicebus): evict stale processors from cache (#48030)
* chore: revert CHANGELOG entry per reviewer feedback
* Update sdk/spring/spring-messaging-azure-servicebus/src/test/java/com/azure/spring/messaging/servicebus/core/DefaultServiceBusNamespaceProcessorFactoryTests.java
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* fix: separate try-catch for listener notification and stale processor close
Ensures stale.close() always executes even if listener notification
throws, preventing a potential resource leak.
* refactor: extract buildProcessorName to local variable in stale eviction block
Avoids recomputing buildProcessorName(key) multiple times and ensures
consistent value across debug log, listener callback, and warn logs.
* test: add mock-based tests for running-to-not-running processor transition
- Add testRunningProcessorReturnedFromCache: verifies running processor
stays in cache without eviction
- Add testProcessorEvictedAfterTransitionToNotRunning: verifies stale
(non-running) processor is evicted, closed, and replaced
- Update comments on existing stale tests to clarify they test
never-started processors; reference new mock tests for transition
- Add reflection helper getProcessorMap() for injecting mocks into cache
---------
Co-authored-by: Eldert Grootenboer (from Dev Box) <egrootenboer@microsoft.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
(cherry picked from commit 1c16e17)
* Fix: KeyVaultJcaProvider registered at highest JCA priority breaks mTLS with standard keystores (#48198)
* Initial plan
* Fix: KeyVaultJcaProvider registered at highest priority breaks mTLS with standard keystores
Co-authored-by: rujche <171773178+rujche@users.noreply.github.com>
* Revert spring-cloud-azure-autoconfigure/CHANGELOG.md - keep aligned with other versions
Co-authored-by: rujche <171773178+rujche@users.noreply.github.com>
* Update CHANGELOG issue link to #48183
Co-authored-by: rujche <171773178+rujche@users.noreply.github.com>
* Apply PR review suggestions: remove unused param and use PROVIDER_NAME constant
Co-authored-by: rujche <171773178+rujche@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: rujche <171773178+rujche@users.noreply.github.com>
(cherry picked from commit 7f9adaf)
* update changelog
* App Config Spring - Json content type fix (#48448)
* fixing semi colon support
* Update CHANGELOG.md
* bug fix
* review comments
* Apply suggestions from code review
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
* remove release notes
* Update JsonConfigurationParser.java
* Update JsonConfigurationParser.java
* Update JsonConfigurationParser.java
* Update JsonConfigurationParser.java
---------
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
(cherry picked from commit c0c405a)
* update changelog
* App Config - Startup retry (#47857)
* Refactor + Startup Retry
* Update AzureAppConfigDataLoader.java
* Adding Tests
* Updating readme, correct location
* Apply suggestions from code review
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* interval change
* fixes after merge
* Apply suggestions from code review
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Update AzureAppConfigDataLoader.java
* review comments
* Apply suggestions from code review
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
---------
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
(cherry picked from commit 02491f7)
* App Config Spring - Refresh Refactor (#47877)
* Refactor Refresh
* review fixes
* Apply suggestions from code review
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Update StateHolderTest.java
* Update AppConfigurationRefreshUtil.java
* Update AppConfigurationRefreshUtil.java
* Update sdk/spring/spring-cloud-azure-appconfiguration-config/src/main/java/com/azure/spring/cloud/appconfiguration/config/implementation/StateHolder.java
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Update AzureAppConfigBootstrapRegistrar.java
* Update AppConfigurationRefreshUtilTest.java
* Update ConnectionManager.java
* Update AppConfigurationRefreshUtilTest.java
* fixing after merge
* Update AppConfigurationRefreshUtilTest.java
* fixing tests
* fixing merge issue
* Update RecurrenceEvaluator.java
* Update RecurrenceEvaluator.java
* better fix
* new fix
* Update AppConfigurationWatchAutoConfiguration.java
* Update AppConfigurationPullRefresh.java
* Update AppConfigurationPullRefresh.java
---------
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
(cherry picked from commit dcd90d0)
* Fixing Load Balancing shuffle (#48274)
* Fixing Load Balancing shuffle
* Update ConnectionManager.java
* Update ConnectionManager.java
(cherry picked from commit bb97593)
* Bug 47910.count query text block (#47911)
* RED: isCountQuery() should return true for queries in text blocks
* GREEN: isCountQuery() should return true for queries in text blocks
* RED: isSumQuery() should return true for sum query in java text block
* GREEN: isSumQuery() should return true for sum query in java text block
---------
Co-authored-by: Kushagra Thapar <kuthapar@microsoft.com>
(cherry picked from commit d3f381d)
* Fix flaky AddressRepositoryIT tests due to non-deterministic ordering and ambiguous IDs (#48071)
* Initial plan
* Fix flaky AddressRepositoryIT tests - order-sensitive assertions and ambiguous findById
Co-authored-by: kushagraThapar <14034156+kushagraThapar@users.noreply.github.com>
* Remove unused TEST_ADDRESS_TEMP variables from testFindAllByStreetNotNull and testCountByStreetNotNull
Co-authored-by: kushagraThapar <14034156+kushagraThapar@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: kushagraThapar <14034156+kushagraThapar@users.noreply.github.com>
Co-authored-by: Rujun Chen <rujche@microsoft.com>
(cherry picked from commit 6cf1688)
* Startup Bug Fixes (#48278)
* Startup Bug Fixes
* Update README.md
* Update AppConfigurationProperties.java
---------
Co-authored-by: Nishant Routray <nroutray@microsoft.com>
(cherry picked from commit 35d4a6c)
* Daylight saving time fix (#48363)
* Daylight saving time fix
* dst tests
* test updates
* Update RecurrenceEvaluator.java
* test update
* Update RecurrenceValidatorDSTTest.java
* copilot comments
* Apply suggestions from code review
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
---------
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
(cherry picked from commit c57f2d2)
* update changelog
* update changelog
* update changelog
* fix
* sync eng/pipelines
* sync eng/common
---------
Co-authored-by: Copilot <198982749+copilot@users.noreply.github.com>
Co-authored-by: Matthew Metcalf <mrm9084@gmail.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Eldert Grootenboer <eldert@eldert.net>
Co-authored-by: Jason Bodnar <jason.bodnar@blackbaud.com>
Co-authored-by: Kushagra Thapar <kuthapar@microsoft.com>
Co-authored-by: kushagraThapar <14034156+kushagraThapar@users.noreply.github.com>
Co-authored-by: Rujun Chen <rujche@microsoft.com>1 parent 6265f61 commit e3ea129
92 files changed
Lines changed: 5529 additions & 1860 deletions
File tree
- eng
- common
- instructions/azsdk-tools
- mcp
- pipelines/templates
- jobs
- steps
- scripts
- Helpers
- spelling
- tsp-client
- pipelines
- templates
- jobs
- stages
- steps
- sdk/spring
- azure-spring-data-cosmos
- src
- main/java/com/azure/spring/data/cosmos/repository/support
- test/java/com/azure/spring/data/cosmos/repository
- integration
- support
- spring-cloud-azure-appconfiguration-config/src
- main/java/com/azure/spring/cloud/appconfiguration/config
- implementation
- configuration
- feature
- properties
- test/java/com/azure/spring/cloud/appconfiguration/config
- implementation
- properties
- spring-cloud-azure-autoconfigure/src
- main/java/com/azure/spring/cloud/autoconfigure/implementation/keyvault/jca
- test/java/com/azure/spring/cloud/autoconfigure/implementation/keyvault/jca
- spring-cloud-azure-feature-management/src
- main/java/com/azure/spring/cloud/feature/management/implementation/timewindow/recurrence
- test/java/com/azure/spring/cloud/feature/management/filters/recurrence
- spring-cloud-azure-starter-appconfiguration-config
- spring-messaging-azure-servicebus/src
- main/java/com/azure/spring/messaging/servicebus/core
- test/java/com/azure/spring/messaging/servicebus/core
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
Lines changed: 2 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
29 | 29 | | |
30 | 30 | | |
31 | 31 | | |
32 | | - | |
33 | 32 | | |
34 | | - | |
35 | | - | |
| 33 | + | |
| 34 | + | |
36 | 35 | | |
37 | 36 | | |
38 | 37 | | |
| |||
Lines changed: 48 additions & 10 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
11 | | - | |
12 | | - | |
13 | | - | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
14 | 16 | | |
15 | 17 | | |
16 | 18 | | |
| |||
21 | 23 | | |
22 | 24 | | |
23 | 25 | | |
24 | | - | |
| 26 | + | |
25 | 27 | | |
26 | 28 | | |
27 | 29 | | |
| |||
30 | 32 | | |
31 | 33 | | |
32 | 34 | | |
33 | | - | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
34 | 40 | | |
35 | 41 | | |
36 | 42 | | |
| |||
132 | 138 | | |
133 | 139 | | |
134 | 140 | | |
135 | | - | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
| 147 | + | |
| 148 | + | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
136 | 158 | | |
137 | 159 | | |
138 | 160 | | |
139 | 161 | | |
140 | 162 | | |
141 | 163 | | |
142 | 164 | | |
143 | | - | |
| 165 | + | |
144 | 166 | | |
145 | 167 | | |
146 | 168 | | |
147 | 169 | | |
148 | 170 | | |
149 | 171 | | |
150 | 172 | | |
151 | | - | |
| 173 | + | |
152 | 174 | | |
153 | 175 | | |
154 | 176 | | |
| 177 | + | |
155 | 178 | | |
156 | | - | |
157 | | - | |
| 179 | + | |
| 180 | + | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
| 184 | + | |
| 185 | + | |
| 186 | + | |
| 187 | + | |
| 188 | + | |
| 189 | + | |
| 190 | + | |
| 191 | + | |
| 192 | + | |
| 193 | + | |
| 194 | + | |
| 195 | + | |
Lines changed: 10 additions & 40 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | | - | |
| 2 | + | |
3 | 3 | | |
4 | | - | |
5 | | - | |
6 | | - | |
7 | | - | |
8 | | - | |
9 | | - | |
10 | | - | |
11 | | - | |
12 | | - | |
13 | | - | |
14 | | - | |
15 | | - | |
16 | | - | |
17 | | - | |
18 | | - | |
19 | | - | |
20 | | - | |
21 | | - | |
22 | | - | |
23 | | - | |
24 | | - | |
25 | | - | |
26 | | - | |
27 | | - | |
28 | | - | |
29 | | - | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
30 | 10 | | |
31 | 11 | | |
32 | 12 | | |
| |||
36 | 16 | | |
37 | 17 | | |
38 | 18 | | |
39 | | - | |
40 | | - | |
41 | | - | |
42 | | - | |
43 | | - | |
44 | | - | |
45 | | - | |
46 | | - | |
47 | | - | |
48 | | - | |
49 | | - | |
50 | | - | |
| 19 | + | |
| 20 | + | |
51 | 21 | | |
52 | | - | |
| 22 | + | |
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
44 | 44 | | |
45 | 45 | | |
46 | 46 | | |
47 | | - | |
| 47 | + | |
48 | 48 | | |
49 | 49 | | |
50 | 50 | | |
| |||
Lines changed: 0 additions & 74 deletions
This file was deleted.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
7 | | - | |
8 | | - | |
9 | 7 | | |
10 | 8 | | |
11 | 9 | | |
| |||
15 | 13 | | |
16 | 14 | | |
17 | 15 | | |
18 | | - | |
| 16 | + | |
19 | 17 | | |
20 | 18 | | |
21 | 19 | | |
22 | 20 | | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
23 | 24 | | |
24 | 25 | | |
25 | 26 | | |
| |||
103 | 104 | | |
104 | 105 | | |
105 | 106 | | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
| 125 | + | |
| 126 | + | |
| 127 | + | |
| 128 | + | |
| 129 | + | |
| 130 | + | |
| 131 | + | |
| 132 | + | |
106 | 133 | | |
107 | 134 | | |
108 | 135 | | |
109 | 136 | | |
110 | 137 | | |
111 | 138 | | |
112 | | - | |
113 | | - | |
| 139 | + | |
| 140 | + | |
114 | 141 | | |
115 | 142 | | |
116 | 143 | | |
| |||
126 | 153 | | |
127 | 154 | | |
128 | 155 | | |
129 | | - | |
130 | | - | |
| 156 | + | |
| 157 | + | |
131 | 158 | | |
132 | 159 | | |
133 | 160 | | |
| |||
164 | 191 | | |
165 | 192 | | |
166 | 193 | | |
167 | | - | |
| 194 | + | |
168 | 195 | | |
169 | 196 | | |
170 | 197 | | |
| |||
175 | 202 | | |
176 | 203 | | |
177 | 204 | | |
178 | | - | |
| 205 | + | |
| 206 | + | |
179 | 207 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
107 | 107 | | |
108 | 108 | | |
109 | 109 | | |
110 | | - | |
111 | 110 | | |
| 111 | + | |
112 | 112 | | |
113 | 113 | | |
114 | 114 | | |
| |||
Lines changed: 0 additions & 11 deletions
This file was deleted.
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
24 | | - | |
| 24 | + | |
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
| |||
Lines changed: 1 addition & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
| 28 | + | |
28 | 29 | | |
29 | 30 | | |
30 | 31 | | |
| |||
0 commit comments