Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

install_builder_prerequisites failed after recent Helios upgrade #7292

Open
davepacheco opened this issue Dec 20, 2024 · 0 comments · May be fixed by #7293
Open

install_builder_prerequisites failed after recent Helios upgrade #7292

davepacheco opened this issue Dec 20, 2024 · 0 comments · May be fixed by #7293
Assignees

Comments

@davepacheco
Copy link
Collaborator

I recently updated ivanova (my dev machine) with pkg update. This is what I've got now:

dap@ivanova omicron-fix $ beadm list
BE                  Active Mountpoint Space  Policy Created
helios-13           -      -          3.16M  static 2023-05-11 14:31
heliosv2            -      -          5.95M  static 2023-07-18 11:27
heliosv2-backup-1   -      -          322K   static 2023-10-17 19:50
heliosv2-backup-2   -      -          324K   static 2023-12-05 10:28
heliosv2-1          -      -          5.38M  static 2024-01-11 09:40
heliosv2-2          -      -          5.89M  static 2024-02-21 17:20
heliosv2-2-backup-1 -      -          326K   static 2024-03-07 08:42
heliosv2-3          -      -          6.15M  static 2024-03-11 17:23
heliosv2-4          -      -          6.29M  static 2024-08-07 09:24
heliosv2-4-backup-1 -      -          323K   static 2024-08-23 11:28
heliosv2-5          NR     /          10.41G static 2024-12-19 16:02
dap@ivanova omicron-fix $ pkg info osnet-incorporation
             Name: consolidation/osnet/osnet-incorporation
          Summary: OS/Net consolidation incorporation
      Description: This incorporation constrains packages from the OS/Net
                   consolidation.
            State: Installed (Manually installed)
        Publisher: helios-dev
          Version: 0.5.11
           Branch: 2.0.23080
   Packaging Date: December 19, 2024 at 08:46:26 PM
Last Install Time: September  8, 2021 at 06:52:19 AM
 Last Update Time: December 20, 2024 at 12:02:25 AM
             Size: 0.00 B
             FMRI: pkg://helios-dev/consolidation/osnet/osnet-incorporation@0.5.11-2.0.23080:20241219T204626Z

install_builder_prerequisites failed:

$ ./tools/install_builder_prerequisites.sh -y
Creating Plan (Solver setup): /
pkg install: Package 'runtime/java/openjdk17' must be uninstalled or upgraded if the requested operation is to be performed.
  Reject:  pkg://helios-dev/runtime/java/openjdk17@17.0.13-2.11
  Reason:  No version matching 'require' dependency system/library/g++-runtime@14-2.0 can be installed
Package 'developer/illumos-tools' must be uninstalled or upgraded if the requested operation is to be performed.
  Reject:  pkg://helios-dev/developer/illumos-tools@11-2.0
  Reason:  No version matching 'require' dependency runtime/java/openjdk11 can be installed
    ----------------------------------------
    Reject:  pkg://helios-dev/runtime/java/openjdk11@11.0.25-2.9
    Reason:  No version matching 'require' dependency system/library/g++-runtime@14-2.0 can be installed
    ----------------------------------------
Package 'ooce/helios-build-tools' must be uninstalled or upgraded if the requested operation is to be performed.
  Reject:  pkg://helios-dev/ooce/helios-build-tools@2-2.0
  Reason:  No version matching 'require' dependency developer/omnios-build-tools can be installed
    ----------------------------------------
    Reject:  pkg://helios-dev/developer/omnios-build-tools@11-2.0
    Reason:  No version matching 'require' dependency developer/illumos-tools can be installed
    ----------------------------------------
Package 'developer/omnios-build-tools' must be uninstalled or upgraded if the requested operation is to be performed.
  Reject:  pkg://helios-dev/developer/omnios-build-tools@11-2.0
  Reason:  [already rejected; see above]
Package 'runtime/java/openjdk11' must be uninstalled or upgraded if the requested operation is to be performed.
  Reject:  pkg://helios-dev/runtime/java/openjdk11@11.0.25-2.9
  Reason:  [already rejected; see above]

I was able to work around this by not using -y and using n to not update these packages. Since I already had packages installed from work prior to the Helios update, things seemed to work.

This patch:

dap@ivanova omicron-fix $ git diff
diff --git a/tools/install_builder_prerequisites.sh b/tools/install_builder_prerequisites.sh
index 2dbdd0653..e67bdb4cd 100755
--- a/tools/install_builder_prerequisites.sh
+++ b/tools/install_builder_prerequisites.sh
@@ -138,7 +138,7 @@ function install_packages {
     fi
   elif [[ "${HOST_OS}" == "SunOS" ]]; then
     CLANGVER=15
-    RTVER=13
+    RTVER=14
     PGVER=13
     packages=(
       "pkg:/package/pkg"

also allowed ./install_builder_prerequisites.sh -y to complete successfully with no errors.

@citrus-it's diagnosis from chat:

So for those following along, the error was because install_builder_prerequisites was trying to downgrade the GCC/G++ runtime library packages from version 14 to 13.
and runtime/java/openjdk17 (for one) depends on version 14.
and jdk17 is required by developer/omnios-build-tools
So it found no solution to the requested plan.

citrus-it added a commit that referenced this issue Dec 20, 2024
Instead of always trying to install {gcc,g++}-runtime version 13,
update them to the latest if they are not new enough to support
the pre-compiled illumos clickhouse binaries.

Fixes #7292
citrus-it added a commit to citrus-it/omicron that referenced this issue Dec 20, 2024
Instead of always trying to install {gcc,g++}-runtime version 13,
update them to the latest if they are not new enough to support
the pre-compiled illumos clickhouse binaries.

Fixes oxidecomputer#7292
citrus-it added a commit that referenced this issue Dec 20, 2024
Instead of always trying to install {gcc,g++}-runtime version 13,
update them to the latest if they are not new enough to support
the pre-compiled illumos clickhouse binaries.

Fixes #7292
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants