Feature branch sync to master #5525
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Yann Dirson <[email protected]>
Different network card (like virtual ones or USB ones) do not have a vendor:product causing networkd daemon to report a lot of errors like Error in read one line of file: /sys/class/net/eth0/device/device, exception Unix.Unix_error(Unix.ENOENT, "open", "/sys/class/net/eth0/device/device") Raised by primitive operation at Xapi_stdext_unix__Unixext.with_file in file "lib/xapi-stdext-unix/unixext.ml", line 68, characters 11-40 Called from Xapi_stdext_unix__Unixext.buffer_of_file in file "lib/xapi-stdext-unix/unixext.ml" (inlined), line 155, characters 31-83 Called from Xapi_stdext_unix__Unixext.string_of_file in file "lib/xapi-stdext-unix/unixext.ml", line 157, characters 47-73 Called from Network_utils.Sysfs.read_one_line in file "ocaml/networkd/lib/network_utils.ml", line 156, characters 6-33 Do not report these missing file as errors. Signed-off-by: Frediano Ziglio <[email protected]>
… in C# SDK Also: - Rename `README.dist` and `README-NuGet.dist` to `README.md` and `README-NuGet.md` - Change content of aforementioned READMEs to use markdown - Remove mentions of patched Json.NET library - Update copyright notice year - Do not add `.txt` extension to `LICENSE` file Signed-off-by: Danilo Del Busso <[email protected]>
Signed-off-by: Danilo Del Busso <[email protected]>
Hyphens are converted when using `ResGen.exe` but they shouldn't be used to keep a consistent style Signed-off-by: Danilo Del Busso <[email protected]>
Enables building without the use of `ResGen.exe` Signed-off-by: Danilo Del Busso <[email protected]>
Discussed here: xapi-project#5462 Signed-off-by: Benjamin Reis <[email protected]>
- New pif helper: `get_non_link_ipv6` to get the 1st non link IPv6 of a PIF - Use the helper in `migrate_receive` and `get_primary_address` used in host evacuation to have a valid IPv6 of the destination host Co-authored-by: Pau Ruiz Safont <[email protected]> Signed-off-by: Benjamin Reis <[email protected]>
Signed-off-by: Guillaume <[email protected]>
Also: - Update copyright notices - Update `README` files to use markdown - Add `.gitignore` file - Drop extension from `LICENSE.txt` - Add `.sln` file since it's needed for NuGet usage in VS22 - Add .NET 8.0 to list of target frameworks for the SDK project - Ensure all DLLs are placed in `bin/` when running `dotnet build`. Without `CopyLocalLockFileAssemblies = true` this is not the case for .NET builds. Signed-off-by: Danilo Del Busso <[email protected]>
Makes use of new composite action that sets up a XenAPI environment in a Ubuntu node. Signed-off-by: Danilo Del Busso <[email protected]>
Signed-off-by: Danilo Del Busso <[email protected]>
This can then be used by observer.py, whereas the value 'bugtool' was useless Signed-off-by: Steven Woods <[email protected]>
This enables the sampled flag (https://www.w3.org/TR/trace-context/#sampled-flag). This indicates that we may have recorded trace data. In the future if we use sampling, this flag can be set with more granularity. Signed-off-by: Steven Woods <[email protected]>
This is necessary for the python observers to set their local endpoint value. It also allows filtering traces by component Signed-off-by: Steven Woods <[email protected]>
The default_attributes are not added to the DB and so these are missing in the observer.conf. Dom0ObserverConfig currently lacks a single source of truth outside of the DB (and default_attributes is not given to the DB) and so attributes outside of the DB are lost whenever a different setter function is used. This should be reworked in future so that observer.conf is read from/updated instead of regenerated each time. Signed-off-by: Steven Woods <[email protected]>
This is intended as the start of a new directory structure. Splitting the python3-only scripts into a new directory gives a simple way to exclude them from python2 tests and coverage. Signed-off-by: Steven Woods <[email protected]>
It is not supported in scheduled runs, and fails. Use unique key for shellcheck group Signed-off-by: Edwin Török <[email protected]>
Co-authored-by: Pau Ruiz Safont <[email protected]> Signed-off-by: Török Edwin <[email protected]>
…info Add 'threads_per_core' in 'Host.cpu_info'
`xapi_xenpos.ml` -> `xapi_xenops.ml` Signed-off-by: Luca Zhang <[email protected]>
Signed-off-by: Vincent Liu <[email protected]>
Add support for clusterd to write host name and uuid to corosync.conf on each host. This is done by augmenting the address type with uuid info rather than adding a new parameter to the cluster api calls which can cause compatibility issues during pool upgrade. Signed-off-by: Vincent Liu <[email protected]>
Signed-off-by: Vincent Liu <[email protected]>
…[email protected] GitHub Action to read .env files: Update to falti/dotenv-action@v1 (fixes Node 16 warnings)
shellscripts: fix the worse complaints from shellcheck
The custom logrotation setup for xapi sometimes conflicted with the normal one. Remove the former. Install the xapi logrotate configuration has been moved to be managed as part of the normal logrotation, with the global settings removed. Duplicate settings in the configuration have been removed. Signed-off-by: Pau Ruiz Safont <[email protected]>
Also update URL in `pom.xml` Signed-off-by: Danilo Del Busso <[email protected]>
The bug is in parsing the output of 'xen-livepatch list'. The following output can't be parsed correctly: lp_4.17.3-3.11.gf717213.xs8-4.17.3-3.12.xs8| CHECKED The regex pattern doesn't exclude the '|' after 'xs8'. As a result, the vertical line '|' is parsed as part of '3.12.xs8|'. The correct one should be '3.12.xs8'. Signed-off-by: Ming Lu <[email protected]>
…389840 CA-389840: Bug in parsing output of 'xen-livepatch list'
…tate domains Signed-off-by: Lunfan Zhang <[email protected]>
Signed-off-by: Luca Zhang <[email protected]>
…CP-48430 CP-48430 Update the running_domains metrics to include the blocked state domains
[OIL] Moved some more generation code to templates.
Not used. It was never implemented. Signed-off-by: Frediano Ziglio <[email protected]>
Not used. Signed-off-by: Frediano Ziglio <[email protected]>
Before these changes, the `SaveCertificates` method relied on machines having a `SpecialFolder.MyDocuments` folder. This is true in Windows and GUI versions of some Linux distros, but it's an assumption that caused the save method to fail if the folder didn't exist. With this commit, we're storing and reading from the path where `$PROFILE` is stored, which is platform agnostic from the point of view of the SDK. Signed-off-by: Danilo Del Busso <[email protected]>
Signed-off-by: Danilo Del Busso <[email protected]>
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## feature/xs9 #5525 +/- ##
=============================================
+ Coverage 49.0% 51.8% +2.7%
=============================================
Files 18 19 +1
Lines 2319 2586 +267
=============================================
+ Hits 1138 1341 +203
- Misses 1181 1245 +64
Flags with carried forward coverage won't be shown. Click here to find out more. |
gangj
approved these changes
Mar 21, 2024
There was a problem hiding this comment.
I think the title should be: "Feature branch sync from master"? :)
The 3 commits ahead of xapi-project/xen-api:master here: https://github.com/liulinC/xen-api/tree/private/linl/dev is a little confusing, but checked the last merge commit:
gangj@gangj-OptiPlex-7050:~/Documents/dev_env/Wukong/xen-api$ git show 3d5f055b12
commit 3d5f055b12026e68dc67d5e9b440ec68ba7c286e (HEAD -> private/linl/dev, liulinC/private/linl/dev)
Merge: b11f6788d 9fcfbfc22
Author: Lin Liu <[email protected]>
Date: Thu Mar 21 01:17:34 2024 +0000
Merge branch 'master' into private/linl/dev
gangj@gangj-OptiPlex-7050:~/Documents/dev_env/Wukong/xen-api$ git log -1 3d5f055b12
commit 3d5f055b12026e68dc67d5e9b440ec68ba7c286e (HEAD -> private/linl/dev, liulinC/private/linl/dev)
Merge: b11f6788d 9fcfbfc22
Author: Lin Liu <[email protected]>
Date: Thu Mar 21 01:17:34 2024 +0000
Merge branch 'master' into private/linl/dev
gangj@gangj-OptiPlex-7050:~/Documents/dev_env/Wukong/xen-api$
It is indeed just a merge from the latest master(latest commit: 9fcfbfc) to the feature branch: xapi-project:feature/xs9 (latest commit: b11f678) with no conflict.
psafont
approved these changes
Mar 21, 2024
pytype_reporter extracted 50 problem reports from pytype output. You can check the results of the job here |
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.