|
| 1 | +## Version 1.3.1 Release Notes |
| 2 | + |
| 3 | +- **[Branding] Allow for SSL setup failures (#1414)(#1417)** |
| 4 | + |
| 5 | + [Kawika Avilla](mailto:kavilla414@gmail.com) |
| 6 | + |
| 7 | + Setup HTTP Agent in the render portion when it did not need to be |
| 8 | + it just needed a one time setup for the life time of the server. |
| 9 | + |
| 10 | + Also if this fails to read the keys then it would fail. But it's |
| 11 | + only used for custom branding. We shouldn't failed for custom branding |
| 12 | + just rely on default branding. |
| 13 | + |
| 14 | + Issue Resolved: |
| 15 | + https://discuss.opendistrocommunity.dev/t/is-opensearch-dashboard-server-certificate-and-key-required-to-be-reloaded-everytime-when-gui-is-accessed/9069/13 |
| 16 | + |
| 17 | +* **[CI][build] Use BUILD_NUMBER for building bundles (#1371)(#1407)** |
| 18 | + |
| 19 | + [Kawika Avilla](mailto:kavilla414@gmail.com) |
| 20 | + |
| 21 | + When running a release build for example: |
| 22 | + |
| 23 | + ``` |
| 24 | + yarn build-platform --linux --skip-os-packages --release |
| 25 | + ``` |
| 26 | + |
| 27 | + The build task runs through get_build_number and |
| 28 | + checks how many commits you have locally and determines the |
| 29 | + build number. From there, and this is the value that |
| 30 | + is used to as a cache busting mechanism. |
| 31 | + |
| 32 | + However, in the release build repo |
| 33 | + https://github.com/opensearch-project/opensearch-build |
| 34 | + |
| 35 | + When this gets packaged and verified it actually pulls |
| 36 | + from the specified branch and only retrieves the HEAD |
| 37 | + commit. Thus making the count of commits locally equal |
| 38 | + to `1` and get_build_number always return `1` for releases |
| 39 | + essentially breaking the cache buster. |
| 40 | + |
| 41 | + The build repo however, sets an env variable of `BUILD_NUMBER` |
| 42 | + so if this value is available it will use it instead of commit |
| 43 | + count. |
| 44 | + |
| 45 | + The CI runs the unit tests and only gets the latest commit as well |
| 46 | + so instead of setting a env build number and basically creating |
| 47 | + the same unit test only check this locally. |
| 48 | + |
| 49 | + Issues resolved: |
| 50 | + |
| 51 | + - opensearch-project/opensearch-build#1769 |
| 52 | + - #1363 |
| 53 | + |
| 54 | +* **[Improvement] node healthcheck optimization (#1386)** |
| 55 | + |
| 56 | + [Kawika Avilla](mailto:kavilla414@gmail.com) |
| 57 | + |
| 58 | + Health check when enabled checked the entire cluster to get |
| 59 | + the value set from the nodeId. Switching the health check to |
| 60 | + query /\_cluster/state/nodes should provide the same data needed |
| 61 | + for the health check without transmitting the rest of the |
| 62 | + cluster state data over the network. |
| 63 | + |
| 64 | + Issue: |
| 65 | + #1354 |
0 commit comments