Uploaded image for project: 'Couchbase Monitoring and Observability Stack'
  1. Couchbase Monitoring and Observability Stack
  2. CMOS-158

Building should not require access to gitlab

    XMLWordPrintable

Details

    • Bug
    • Status: Done
    • Major
    • Resolution: Done
    • 0.1
    • 0.1
    • cmos
    • None

    Description

      In a recent commit:
      46fff4d Container example updates (#88)

      There is a change which a user reports now breaks their build because of a HEAD request against gitlab.

      {preformat}
      Use 'docker scan' to run Snyk tests against images to find vulnerabilities and learn how to fix them
      examples/containers/run.sh
      ~/projects/cb/observability/examples/containers ~/projects/cb/observability
      WARNING: Found orphan containers (exporter, couchbase-grafana) for this project. If you removed or renamed this service in your compose file, you can run this command with the --remove-orphans flag to clean it up.
      Pulling couchbase-cmos (registry.gitlab.com/cb-vanilla/observability-stack:latest)...
      ERROR: Head https://registry.gitlab.com/v2/cb-vanilla/observability-stack/manifests/latest: denied: access forbidden
      make: *** [example-containers] Error 1{preformat}

      As I believe we intend users without access to any of our private docker registries, I think this is just a bug with the recent changes.

      Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

          I started updating the examples to work with the official images - docker compose requires a build or pull approach unfortunately, it will not use the local cache. The examples likely should use the official images, particularly for people without access to the private repositories to build the cluster monitor, however I do agree we may need to change this for now.

          I suggest we do the following:

          • Until we release officially, always build from source.
          • Once we have an official release, switch over to using that for the examples.

          We can provide instructions to build from source or use the current compose file as a development one.

          patrick.stephens Patrick Stephens (Inactive) added a comment - I started updating the examples to work with the official images - docker compose requires a build or pull approach unfortunately, it will not use the local cache. The examples likely should use the official images, particularly for people without access to the private repositories to build the cluster monitor, however I do agree we may need to change this for now. I suggest we do the following: Until we release officially, always build from source. Once we have an official release, switch over to using that for the examples. We can provide instructions to build from source or use the current compose file as a development one.
          patrick.stephens Patrick Stephens (Inactive) added a comment - Resolved by  https://github.com/couchbaselabs/observability/pull/95

          Build couchbase-observability-stack-1.0.0-156 contains observability commit eca815b with commit message:
          CMOS-158: Update example docs (#97)

          build-team Couchbase Build Team added a comment - Build couchbase-observability-stack-1.0.0-156 contains observability commit eca815b with commit message: CMOS-158 : Update example docs (#97)

          Build couchbase-observability-stack-1.0.0-156 contains observability commit e97b8da with commit message:
          CMOS-158: Always build CMOS container for examples until 0.1 release (#95)

          build-team Couchbase Build Team added a comment - Build couchbase-observability-stack-1.0.0-156 contains observability commit e97b8da with commit message: CMOS-158 : Always build CMOS container for examples until 0.1 release (#95)

          Let me know if the latest `main` sorts it for you.

          patrick.stephens Patrick Stephens (Inactive) added a comment - Let me know if the latest `main` sorts it for you.

          People

            ingenthr Matt Ingenthron
            ingenthr Matt Ingenthron
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty