Skip to content

Fix broken CI caused by renaming project to Deskflow#7522

Merged
nbolton merged 14 commits intomasterfrom
deskflow-ci
Sep 18, 2024
Merged

Fix broken CI caused by renaming project to Deskflow#7522
nbolton merged 14 commits intomasterfrom
deskflow-ci

Conversation

@nbolton
Copy link
Copy Markdown
Member

@nbolton nbolton commented Sep 18, 2024 •
edited
Loading

Blocking: #7520, #7511

Problem: Renaming the project and moving it to a new org caused CI to break.

Solution:

  • Fixed GitHub vars to reflect project rename
  • Add paid runner for Linux ARM64 (ubuntu-24.04-2-core-arm64)
  • Create new SonarCloud org
  • Create new Docker Hub org
  • Restore some unit test original values (tests are designed around those values)
  • Fixed some syntax errors caused by the rename (fixed in fix: windows build typos #7521)

Packages (artifacts): https://github.com/deskflow/deskflow/actions/runs/10920095596?pr=7522

Note: The license/activaiton/serial related code will be moved downstream to Synergy next: #7517

@nbolton nbolton changed the title Fix CI Fix CI broken by renaming project and relocating to new org Sep 18, 2024
@nbolton nbolton requested a review from sithlord48 September 18, 2024 10:02
This was referenced Sep 18, 2024
Merged
Merged
Merged
@nbolton nbolton changed the title Fix CI broken by renaming project and relocating to new org Fix broken CI caused by renaming project Sep 18, 2024
@nbolton nbolton changed the title Fix broken CI caused by renaming project Fix broken CI caused by renaming project to Deskflow Sep 18, 2024
@nbolton nbolton enabled auto-merge (squash) September 18, 2024 10:15
@deskflow deskflow deleted a comment from github-actions bot Sep 18, 2024
@github-actions
Copy link
Copy Markdown

github-actions bot commented Sep 18, 2024 •
edited
Loading

Test results

See workflow run for test output.

Job name Unit tests Integration tests
macos-14-arm64 ✅ Pass ✅ Pass
macos-13-x64 ✅ Pass ✅ Pass
linux-ubuntu-22.04-amd64 ✅ Pass ✅ Pass
linux-rockylinux-9-amd64 ✅ Pass ✅ Pass
linux-almalinux-9-amd64 ✅ Pass ✅ Pass
linux-debian-13-amd64 ✅ Pass ✅ Pass
linux-debian-12-amd64 ✅ Pass ✅ Pass
linux-fedora-39-amd64 ✅ Pass ✅ Pass
linux-ubuntu-24.04-amd64 ✅ Pass ✅ Pass
linux-archlinux-amd64 ✅ Pass ✅ Pass
linux-fedora-40-amd64 ✅ Pass ✅ Pass
linux-manjaro-amd64 ✅ Pass ✅ Pass
macos-13-x64 ✅ Pass ✅ Pass
linux-debian-12-arm64 ✅ Pass ✅ Pass
linux-fedora-40-arm64 ✅ Pass ✅ Pass
windows-2022-x64 ✅ Pass ✅ Pass

@github-actions
Copy link
Copy Markdown

github-actions bot commented Sep 18, 2024

Valgrind summary

See workflow run for full valgrind output.

Unit tests

==3739== HEAP SUMMARY:
==3739==     in use at exit: 50,627 bytes in 331 blocks
==3739==   total heap usage: 36,671 allocs, 36,340 frees, 6,091,396 bytes allocated

==3739== LEAK SUMMARY:
==3739==    definitely lost: 334 bytes in 10 blocks
==3739==    indirectly lost: 734 bytes in 13 blocks
==3739==      possibly lost: 0 bytes in 0 blocks
==3739==    still reachable: 47,543 bytes in 287 blocks
==3739==         suppressed: 0 bytes in 0 blocks
==3739== 
==3739== ERROR SUMMARY: 28 errors from 28 contexts (suppressed: 0 from 0)

Integration tests

==3756== HEAP SUMMARY:
==3756==     in use at exit: 42,300 bytes in 324 blocks
==3756==   total heap usage: 3,039 allocs, 2,715 frees, 771,725 bytes allocated

==3756== LEAK SUMMARY:
==3756==    definitely lost: 920 bytes in 40 blocks
==3756==    indirectly lost: 80 bytes in 4 blocks
==3756==      possibly lost: 1,600 bytes in 4 blocks
==3756==    still reachable: 37,684 bytes in 255 blocks
==3756==         suppressed: 0 bytes in 0 blocks
==3756== 
==3756== ERROR SUMMARY: 45 errors from 45 contexts (suppressed: 0 from 0)

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Sep 18, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@nbolton nbolton merged commit 0220a48 into master Sep 18, 2024
@nbolton nbolton deleted the deskflow-ci branch September 18, 2024 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

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

2 participants

@nbolton @sithlord48