Migrate Ubuntu 18 to 22 directly in the matricses #4442
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
|
/azp run cpp - core - ci |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run cpp - core |
|
Azure Pipelines successfully started running 1 pipeline(s). |
danieljurek
force-pushed
the
djurek/migrate-ubuntu18
branch
from
65206fd
to
141972a
Compare
|
/azp run cpp - core |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run cpp - core |
|
Azure Pipelines successfully started running 1 pipeline(s). |
danieljurek
changed the title
benbp
reviewed
Mar 21, 2023
benbp
reviewed
Mar 21, 2023
|
@gearama -- I'm seeing a build error when running other pipelines with our new matrix: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2648585&view=logs&j=ee76c233-a59e-5872-c34c-b9843bcc71ea&t=7df4bab8-9439-5dda-1896-ba94748419f5&l=110 |
danieljurek
force-pushed
the
djurek/migrate-ubuntu18
branch
from
969d54c
to
1611b9b
Compare
14 tasks
Co-authored-by: Ben Broderick Phillips <[email protected]>
danieljurek
force-pushed
the
djurek/migrate-ubuntu18
branch
from
1611b9b
to
3861290
Compare
benbp
approved these changes
Mar 31, 2023
danieljurek
requested review from
ahsonkhan,
antkmsft,
vhvb1989,
gearama and
LarryOsterman
as code owners
|
Failure from code coverage being |
LarryOsterman
added a commit
that referenced
this pull request
Apr 3, 2023
* Sync eng/common directory with azure-sdk-tools for PR 5608 (#4411) * ongoing * Move get-codeowners scripts and tests to their own dirs * address PR remarks * Fix CodeOwnerFileLocation path, fix casing, and dedup param defaults * fix param names * add todos on needed changes in cpp repo * Add CodeownersFileLocation to Get-Codeowners and use $null for default param values * move get-codeowners back to scripts/ and rename -functions to .lib * fix: use empty string as defaults instead of $nulls, to fix invocation * fix bug with invocation of Get-Codeowners + add support for passing IncludeNonUserAliases as switch * fix path iin Metadata-Helpers.ps1 * fix typo * Update archetype-cpp-release.yml --------- Co-authored-by: Konrad Jamrozik <[email protected]> * First attempt at fixing nlohmann doctest compilation error (#4499) * First attempt at fixing nlohmann doctest compilation error * fixed compilation issue * Fixed ApiView generation for azure core * update doctest lib to 2.4.11 (#4501) * Migrate Ubuntu 18 to 22 directly in the matricses (#4442) * First cut at migrating Ubuntu 18 to 22 directly in the matricses * Try 20.04 * Remove azure.list from apt configuration on 22.04 * Parens * gpp-8 on Ubuntu 20.04 * Remove g++-5 * Move config changes to steps/ * Remove g++-5 from live tests * Update eng/pipelines/templates/steps/fix-linux-1es-configs.yml Co-authored-by: Ben Broderick Phillips <[email protected]> * Review feedback * Finish renaming * Setting line coverage target to 91% --------- Co-authored-by: Ben Broderick Phillips <[email protected]> * Initial population of azure-core-amqp * cspell * AMQP specific cspell dictionary * CI pipeline fixes * azure-uamqp-c cannot work on UWP builds * Final cspell error; added amqp package to artifacts * clang-format; added copyright notices to everything * Added required files * more CI fixes * Renamed some files * Renamed some files * fixed case of amqp directory * Renamed models directory * Renamed network directory * Most filenames are now lower case * Hopefully fix CI pipeline failures * Sort lines in cmakelists.txt * Fixed name of message_target.cp * Fixed name of transfer_instance.cpp * Added preset for AMQP builds; reverted change from cmakesettings.json * Add AMQP to doxygen generation; fixed clang-format issue * Case folding fixes * Correctly set up for DLL builds; removed diagnostic code * Fixed socket listener to initialize platform * clang-format * clang-format * clang-format; clang compilation fix * Moved message sender implementation to shared pointer to enable copy and move semantics * Don't forward declare enums with underlying type * MOved MessageReceiver into an implementation class * A couple of clang compiler fixes * Moved all connection and session code to shared pointers * Converted network callbacks to use events; restructured network to use shared pointer * clang-format fixes * case sensitivity * clang fixes 4 random port for socket listener echo tests clang-format Random port for two more tests Null event handler before destroying messagereceiver clang-format Random port for ReceiverOpenClose Random port for LinkAttachDetach Use random port for session tests gcc warnings again 3 gcc warnings again 2 gcc warnings again Print errno on socket listener start error gcc warnings; enabled code coverage for amqp clang-format Missed file clang fixes 8 clang fixes 7 clang fixes 6 clang fixes 5 clang fixes 3 clang fixes 2 clang fixes * Catch exception thrown from worker thread * Sample fixes * Added context to wait for polled result * clang build fix clang build fix * wait until listener thread completes before starting message receiver * wait until listener thread completes before starting message receiver 2 * more thread cleanup changes * more linux diagnostics 5 more linux diagnostics 13 more linux diagnostics 12 more linux diagnostics 11 more linux diagnostics 10 more linux diagnostics 9 more linux diagnostics 8 more linux diagnostics 7 more linux diagnostics 6 more linux diagnostics 4 more linux diagnostics 3 more linux diagnostics 2 more linux diagnostics * Search for ports at 5000, not 0x5000; clang-format changes * Fixed AV in MessageSenderReceiver when test is cancelled; * Diagnostics for FindAvailableSOcket * Diagnostics for FindAvailableSOcket * Diagnostics for FindAvailableSOcket * pull request feedback * Added cbs test * Finished snake case rename * Fixed copyright and license text * Undid accidental checkin * Moved common and models types to _internal terminal namespace * Basic CBS authentication test * removed test resource name * Fixed SAS token samples * Sanitized token writer sample * CI pipeline fixes * Clear event callback when destroying message sender; close sender and receiver when done with test thread * ApiView fixes; clang-format * Fixed a coupe of CI pipeline issues * Moved credenial type to a member variable not a virtual method returnign a constant * Full mock implementation of claims based security * Renamed ClaimBasedSecurity to ClaimsBasedSecurity; clang-format; clang fixes * Improved code coverage. * clang fix * unit test fix * Improved code coverage for connection string tests * Added test for authenticated sender and receiver * Added message source and target tests Use better constructor for tokencredential Use better constructor for tokencredential crlf at the end of every file; made ClaimBasedSecurity class a _detail class More code coverage fixes Improved coe coverage for amqp message Message->Value conversion 3 Message->Value conversion 2 Try casting message to value Try casting message to value Reverted attestation chagnes 2 Added more tests Fixed doxygen issue Don't use sender unless sender is set Removed incorrect test * More code coverage fixes * Removed 2 param ctor * More code coverage * clang-format * Hopefully final code coverage fixes * clang-format * Moved all amqp types to _internal terminal namespace * Undid attestation change * clang-format fixes * Renamed Azure::Core::Amqp::Value to Azure::Core::Amqp::AmqpValue because Value is too generic a type name when you have 'using namespace' directives * clang-format * clang-format2 * turn crash into failure * Try using async send rather than sync send for responding to CBS operations * It helps to build the code you write * Trigger CI --------- Co-authored-by: Azure SDK Bot <[email protected]> Co-authored-by: Konrad Jamrozik <[email protected]> Co-authored-by: Victor Vazquez <[email protected]> Co-authored-by: Daniel Jurek <[email protected]> Co-authored-by: Ben Broderick Phillips <[email protected]>
antkmsft
pushed a commit
that referenced
this pull request
Apr 5, 2023
* First cut at migrating Ubuntu 18 to 22 directly in the matricses * Try 20.04 * Remove azure.list from apt configuration on 22.04 * Parens * gpp-8 on Ubuntu 20.04 * Remove g++-5 * Move config changes to steps/ * Remove g++-5 from live tests * Update eng/pipelines/templates/steps/fix-linux-1es-configs.yml Co-authored-by: Ben Broderick Phillips <[email protected]> * Review feedback * Finish renaming * Setting line coverage target to 91% --------- Co-authored-by: Ben Broderick Phillips <[email protected]>
antkmsft
pushed a commit
that referenced
this pull request
Apr 5, 2023
* First cut at migrating Ubuntu 18 to 22 directly in the matricses * Try 20.04 * Remove azure.list from apt configuration on 22.04 * Parens * gpp-8 on Ubuntu 20.04 * Remove g++-5 * Move config changes to steps/ * Remove g++-5 from live tests * Update eng/pipelines/templates/steps/fix-linux-1es-configs.yml Co-authored-by: Ben Broderick Phillips <[email protected]> * Review feedback * Finish renaming * Setting line coverage target to 91% --------- Co-authored-by: Ben Broderick Phillips <[email protected]>
benbp
added a commit
that referenced
this pull request
Apr 6, 2023
* Sync eng/common directory with azure-sdk-tools for PR 5608 (#4411) * ongoing * Move get-codeowners scripts and tests to their own dirs * address PR remarks * Fix CodeOwnerFileLocation path, fix casing, and dedup param defaults * fix param names * add todos on needed changes in cpp repo * Add CodeownersFileLocation to Get-Codeowners and use $null for default param values * move get-codeowners back to scripts/ and rename -functions to .lib * fix: use empty string as defaults instead of $nulls, to fix invocation * fix bug with invocation of Get-Codeowners + add support for passing IncludeNonUserAliases as switch * fix path iin Metadata-Helpers.ps1 * fix typo * Update archetype-cpp-release.yml --------- Co-authored-by: Konrad Jamrozik <[email protected]> * First attempt at fixing nlohmann doctest compilation error (#4499) * First attempt at fixing nlohmann doctest compilation error * fixed compilation issue * Fixed ApiView generation for azure core * update doctest lib to 2.4.11 (#4501) * Migrate Ubuntu 18 to 22 directly in the matricses (#4442) * First cut at migrating Ubuntu 18 to 22 directly in the matricses * Try 20.04 * Remove azure.list from apt configuration on 22.04 * Parens * gpp-8 on Ubuntu 20.04 * Remove g++-5 * Move config changes to steps/ * Remove g++-5 from live tests * Update eng/pipelines/templates/steps/fix-linux-1es-configs.yml Co-authored-by: Ben Broderick Phillips <[email protected]> * Review feedback * Finish renaming * Setting line coverage target to 91% --------- Co-authored-by: Ben Broderick Phillips <[email protected]> * Initial population of azure-core-amqp * cspell * AMQP specific cspell dictionary * CI pipeline fixes * azure-uamqp-c cannot work on UWP builds * Final cspell error; added amqp package to artifacts * clang-format; added copyright notices to everything * Added required files * more CI fixes * Renamed some files * Renamed some files * fixed case of amqp directory * Renamed models directory * Renamed network directory * Most filenames are now lower case * Hopefully fix CI pipeline failures * Sort lines in cmakelists.txt * Fixed name of message_target.cp * Fixed name of transfer_instance.cpp * Added preset for AMQP builds; reverted change from cmakesettings.json * Add AMQP to doxygen generation; fixed clang-format issue * Case folding fixes * Correctly set up for DLL builds; removed diagnostic code * Fixed socket listener to initialize platform * clang-format * clang-format * clang-format; clang compilation fix * Moved message sender implementation to shared pointer to enable copy and move semantics * Don't forward declare enums with underlying type * MOved MessageReceiver into an implementation class * A couple of clang compiler fixes * Moved all connection and session code to shared pointers * Converted network callbacks to use events; restructured network to use shared pointer * clang-format fixes * case sensitivity * clang fixes 4 random port for socket listener echo tests clang-format Random port for two more tests Null event handler before destroying messagereceiver clang-format Random port for ReceiverOpenClose Random port for LinkAttachDetach Use random port for session tests gcc warnings again 3 gcc warnings again 2 gcc warnings again Print errno on socket listener start error gcc warnings; enabled code coverage for amqp clang-format Missed file clang fixes 8 clang fixes 7 clang fixes 6 clang fixes 5 clang fixes 3 clang fixes 2 clang fixes * Catch exception thrown from worker thread * Sample fixes * Added context to wait for polled result * clang build fix clang build fix * wait until listener thread completes before starting message receiver * wait until listener thread completes before starting message receiver 2 * more thread cleanup changes * more linux diagnostics 5 more linux diagnostics 13 more linux diagnostics 12 more linux diagnostics 11 more linux diagnostics 10 more linux diagnostics 9 more linux diagnostics 8 more linux diagnostics 7 more linux diagnostics 6 more linux diagnostics 4 more linux diagnostics 3 more linux diagnostics 2 more linux diagnostics * Search for ports at 5000, not 0x5000; clang-format changes * Fixed AV in MessageSenderReceiver when test is cancelled; * Diagnostics for FindAvailableSOcket * Diagnostics for FindAvailableSOcket * Diagnostics for FindAvailableSOcket * pull request feedback * Added cbs test * Finished snake case rename * Fixed copyright and license text * Undid accidental checkin * Moved common and models types to _internal terminal namespace * Basic CBS authentication test * removed test resource name * Fixed SAS token samples * Sanitized token writer sample * CI pipeline fixes * Clear event callback when destroying message sender; close sender and receiver when done with test thread * ApiView fixes; clang-format * Fixed a coupe of CI pipeline issues * Moved credenial type to a member variable not a virtual method returnign a constant * Full mock implementation of claims based security * Renamed ClaimBasedSecurity to ClaimsBasedSecurity; clang-format; clang fixes * Improved code coverage. * clang fix * unit test fix * Improved code coverage for connection string tests * Added test for authenticated sender and receiver * Added message source and target tests Use better constructor for tokencredential Use better constructor for tokencredential crlf at the end of every file; made ClaimBasedSecurity class a _detail class More code coverage fixes Improved coe coverage for amqp message Message->Value conversion 3 Message->Value conversion 2 Try casting message to value Try casting message to value Reverted attestation chagnes 2 Added more tests Fixed doxygen issue Don't use sender unless sender is set Removed incorrect test * More code coverage fixes * Removed 2 param ctor * More code coverage * clang-format * Hopefully final code coverage fixes * clang-format * Moved all amqp types to _internal terminal namespace * Undid attestation change * clang-format fixes * Renamed Azure::Core::Amqp::Value to Azure::Core::Amqp::AmqpValue because Value is too generic a type name when you have 'using namespace' directives * clang-format * clang-format2 * turn crash into failure * Try using async send rather than sync send for responding to CBS operations * It helps to build the code you write * Trigger CI --------- Co-authored-by: Azure SDK Bot <[email protected]> Co-authored-by: Konrad Jamrozik <[email protected]> Co-authored-by: Victor Vazquez <[email protected]> Co-authored-by: Daniel Jurek <[email protected]> Co-authored-by: Ben Broderick Phillips <[email protected]>
benbp
added a commit
that referenced
this pull request
Apr 6, 2023
* Sync eng/common directory with azure-sdk-tools for PR 5608 (#4411) * ongoing * Move get-codeowners scripts and tests to their own dirs * address PR remarks * Fix CodeOwnerFileLocation path, fix casing, and dedup param defaults * fix param names * add todos on needed changes in cpp repo * Add CodeownersFileLocation to Get-Codeowners and use $null for default param values * move get-codeowners back to scripts/ and rename -functions to .lib * fix: use empty string as defaults instead of $nulls, to fix invocation * fix bug with invocation of Get-Codeowners + add support for passing IncludeNonUserAliases as switch * fix path iin Metadata-Helpers.ps1 * fix typo * Update archetype-cpp-release.yml --------- Co-authored-by: Konrad Jamrozik <[email protected]> * First attempt at fixing nlohmann doctest compilation error (#4499) * First attempt at fixing nlohmann doctest compilation error * fixed compilation issue * Fixed ApiView generation for azure core * update doctest lib to 2.4.11 (#4501) * Migrate Ubuntu 18 to 22 directly in the matricses (#4442) * First cut at migrating Ubuntu 18 to 22 directly in the matricses * Try 20.04 * Remove azure.list from apt configuration on 22.04 * Parens * gpp-8 on Ubuntu 20.04 * Remove g++-5 * Move config changes to steps/ * Remove g++-5 from live tests * Update eng/pipelines/templates/steps/fix-linux-1es-configs.yml Co-authored-by: Ben Broderick Phillips <[email protected]> * Review feedback * Finish renaming * Setting line coverage target to 91% --------- Co-authored-by: Ben Broderick Phillips <[email protected]> * Initial population of azure-core-amqp * cspell * AMQP specific cspell dictionary * CI pipeline fixes * azure-uamqp-c cannot work on UWP builds * Final cspell error; added amqp package to artifacts * clang-format; added copyright notices to everything * Added required files * more CI fixes * Renamed some files * Renamed some files * fixed case of amqp directory * Renamed models directory * Renamed network directory * Most filenames are now lower case * Hopefully fix CI pipeline failures * Sort lines in cmakelists.txt * Fixed name of message_target.cp * Fixed name of transfer_instance.cpp * Added preset for AMQP builds; reverted change from cmakesettings.json * Add AMQP to doxygen generation; fixed clang-format issue * Case folding fixes * Correctly set up for DLL builds; removed diagnostic code * Fixed socket listener to initialize platform * clang-format * clang-format * clang-format; clang compilation fix * Moved message sender implementation to shared pointer to enable copy and move semantics * Don't forward declare enums with underlying type * MOved MessageReceiver into an implementation class * A couple of clang compiler fixes * Moved all connection and session code to shared pointers * Converted network callbacks to use events; restructured network to use shared pointer * clang-format fixes * case sensitivity * clang fixes 4 random port for socket listener echo tests clang-format Random port for two more tests Null event handler before destroying messagereceiver clang-format Random port for ReceiverOpenClose Random port for LinkAttachDetach Use random port for session tests gcc warnings again 3 gcc warnings again 2 gcc warnings again Print errno on socket listener start error gcc warnings; enabled code coverage for amqp clang-format Missed file clang fixes 8 clang fixes 7 clang fixes 6 clang fixes 5 clang fixes 3 clang fixes 2 clang fixes * Catch exception thrown from worker thread * Sample fixes * Added context to wait for polled result * clang build fix clang build fix * wait until listener thread completes before starting message receiver * wait until listener thread completes before starting message receiver 2 * more thread cleanup changes * more linux diagnostics 5 more linux diagnostics 13 more linux diagnostics 12 more linux diagnostics 11 more linux diagnostics 10 more linux diagnostics 9 more linux diagnostics 8 more linux diagnostics 7 more linux diagnostics 6 more linux diagnostics 4 more linux diagnostics 3 more linux diagnostics 2 more linux diagnostics * Search for ports at 5000, not 0x5000; clang-format changes * Fixed AV in MessageSenderReceiver when test is cancelled; * Diagnostics for FindAvailableSOcket * Diagnostics for FindAvailableSOcket * Diagnostics for FindAvailableSOcket * pull request feedback * Added cbs test * Finished snake case rename * Fixed copyright and license text * Undid accidental checkin * Moved common and models types to _internal terminal namespace * Basic CBS authentication test * removed test resource name * Fixed SAS token samples * Sanitized token writer sample * CI pipeline fixes * Clear event callback when destroying message sender; close sender and receiver when done with test thread * ApiView fixes; clang-format * Fixed a coupe of CI pipeline issues * Moved credenial type to a member variable not a virtual method returnign a constant * Full mock implementation of claims based security * Renamed ClaimBasedSecurity to ClaimsBasedSecurity; clang-format; clang fixes * Improved code coverage. * clang fix * unit test fix * Improved code coverage for connection string tests * Added test for authenticated sender and receiver * Added message source and target tests Use better constructor for tokencredential Use better constructor for tokencredential crlf at the end of every file; made ClaimBasedSecurity class a _detail class More code coverage fixes Improved coe coverage for amqp message Message->Value conversion 3 Message->Value conversion 2 Try casting message to value Try casting message to value Reverted attestation chagnes 2 Added more tests Fixed doxygen issue Don't use sender unless sender is set Removed incorrect test * More code coverage fixes * Removed 2 param ctor * More code coverage * clang-format * Hopefully final code coverage fixes * clang-format * Moved all amqp types to _internal terminal namespace * Undid attestation change * clang-format fixes * Renamed Azure::Core::Amqp::Value to Azure::Core::Amqp::AmqpValue because Value is too generic a type name when you have 'using namespace' directives * clang-format * clang-format2 * turn crash into failure * Try using async send rather than sync send for responding to CBS operations * It helps to build the code you write * Trigger CI --------- Co-authored-by: Azure SDK Bot <[email protected]> Co-authored-by: Konrad Jamrozik <[email protected]> Co-authored-by: Victor Vazquez <[email protected]> Co-authored-by: Daniel Jurek <[email protected]> Co-authored-by: Ben Broderick Phillips <[email protected]>
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.
@gearama -- This PR fails on coverage check
https://dev.azure.com/azure-sdk/public/_build/results?buildId=2642752&view=logs&j=a04ecbde-cdd5-5770-d3ea-d981d9ab3758&t=8f180a1d-b523-54fe-0960-b987b67b53fc
https://dev.azure.com/azure-sdk/public/_build/results?buildId=2642752&view=logs&jobId=a04ecbde-cdd5-5770-d3ea-d981d9ab3758&j=a04ecbde-cdd5-5770-d3ea-d981d9ab3758&t=e5237235-bbf0-549e-c248-3cbd5cab6435
We might need to move the coverage value down, or find a way to get a few more lines evaluated for coverage.
Line coverage for Ubuntu 18 ( with included compiler:
https://dev.azure.com/azure-sdk/public/_build/results?buildId=2642898&view=logs&j=ca6c228c-7ba2-55ec-3482-b320fca07ebb&t=e455decc-d55c-5dc8-2f85-0f43f7e884da&l=83
https://dev.azure.com/azure-sdk/public/_build/results?buildId=2642898&view=logs&j=ca6c228c-7ba2-55ec-3482-b320fca07ebb&t=21bcdd56-e632-5327-2ccf-6437d5fae715