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

bosh create - package spec github path issues. #67

Closed
saluru opened this issue Aug 6, 2015 · 8 comments
Closed

bosh create - package spec github path issues. #67

saluru opened this issue Aug 6, 2015 · 8 comments

Comments

@saluru
Copy link

saluru commented Aug 6, 2015

I'm trying to deploy Diego release in my local, as part of that i'm running bosh create , but some of the package spec file github path has been migrated and create release is failing.

bosh create release --force && quit
Syncing blobs...

Building DEV release

Release artifact cache: /Users/sumanaluru/.bosh/cache

Building license

Building license...
Using final version '861d82c0d9745784acfcd026ccf44e942579824a'

Building packages

Building acceptance-tests...
Using final version '2f54ad04d4f720d8e42ad802ca431677635902c8'
Downloading from blobstore (id=e2d9e686-0234-4c24-b0e9-ddf0278ab164)...

Building auctioneer...
Package 'auctioneer' has a glob that resolves to an empty file list: github.com/cloudfoundry-incubator/auctioneer/cmd/auctioneer/*.go

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this. You can view the current status of your issue at: https://www.pivotaltracker.com/story/show/100708486.

@emalm
Copy link
Member

emalm commented Aug 6, 2015

Hi, @saluru,

Thanks for the report. Are you trying to deploy from the tip of the develop branch right now? Looks like the team might have forgotten to push some of the submodules on this commit. Creating a release works for me on the current master commit, though (version 0.1407.0).

Thanks again,
Eric, CF Runtime Diego PM

@Amit-PivotalLabs
Copy link

I have a different hypothesis @saluru, what happens if you run git submodule update --init src/github.com/cloudfoundry-incubator/auctioneer from the root of the diego-release repo? If you then try to create the release, do you get a bit further (you might see a different glob complaining about resolving to an empty list).

@saluru
Copy link
Author

saluru commented Aug 6, 2015

Hi @eric,

I will try with master , but i see you have same dead links in spec file (version 0.1407.0) which are giving 404 .


name: auctioneer

dependencies:

  • golang_1.4

files:

  • github.com/cloudfoundry-incubator/auctioneer/cmd/auctioneer/*.go # gosub

@saluru
Copy link
Author

saluru commented Aug 6, 2015

Hi @Amit-PivotalLabs

unknown90840df169fe:diego-release sumanaluru$ git submodule update --init src/github.com/cloudfoundry-incubator/auctioneer

fatal: reference is not a tree: 145bc488a656a5e15d233b7c0340551dc014b4af

Unable to checkout '145bc488a656a5e15d233b7c0340551dc014b4af' in submodule path 'src/github.com/cloudfoundry-incubator/auctioneer'

Amit i see what you are trying to tell, i dont see my local src is getting cloned, let do a fresh one and retry see if the issue resolve.

@saluru
Copy link
Author

saluru commented Aug 6, 2015

Hi @eric,

I tried using master, still got the same error . I'm following the instructions provided in diego-release.
If you observe its not just one package for all.

When i access GIT url in the browser its giving 404, but when i click from diego-release repo its going to Good One.

Problem URL in spec file: https://github.com/cloudfoundry-incubator/diego-acceptance-tests/assets/
Good One: https://github.com/cloudfoundry-incubator/diego-acceptance-tests/tree/master/assets/

Building packages

Building acceptance-tests...
Package 'acceptance-tests' has a glob that resolves to an empty file list: github.com/cloudfoundry-incubator/diego-acceptance-tests/assets/*/

@Amit-PivotalLabs
Copy link

@saluru, the Unable to checkout '145bc488a656a5e15d233b7c0340551dc014b4af' in submodule path 'src/github.com/cloudfoundry-incubator/auctioneer' error message jives with what @ematpl suggested, that some of the submodules had not been pushed up, by accident. If you don't have any work in progress in the diego-release repo, you can do the following:

git fetch origin
git checkout master
git reset --hard origin/master
./scripts/update

and then create release. The ./scripts/update will hydrate all your submodules which will be empty until you clone the diego-release repo and update/init the submodules. The ./scripts/update does that for you, and takes care of doing it recursively because some submodules might have their own submodules.

This should work on master because master has had a release built from it, from scratch. It is currently failing on develop because a submodule was accidentally not pushed when diego-release was bumped to change a submodule pointer.

@saluru
Copy link
Author

saluru commented Aug 6, 2015

Thank You @Amit-PivotalLabs & @ematpl that fixed the issue.

@saluru saluru closed this as completed Aug 6, 2015
jfmyers9 pushed a commit that referenced this issue Mar 2, 2016
Submodule src/github.com/pkg/sftp cbc2879..e84cc8c:
  > Merge pull request #85 from pkg/rootdir
  > Merge pull request #81 from pkg/server-option
  > Merge pull request #80 from pkg/functional-options
  > Merge pull request #78 from pkg/server-readonly
  > Merge pull request #79 from pkg/readme-badges
  > Merge pull request #77 from pkg/namedreturns
  > Merge pull request #75 from pkg/golint
  > Merge pull request #73 from mdlayher/golint
  > Merge pull request #74 from LiterallyElvis/master
  > Merge pull request #72 from mdlayher/master
  > Merge pull request #70 from pkg/fixedbugs/69
  > Merge pull request #67 from mdlayher/master
  > Merge pull request #64 from pkg/fixedbugs/28
  > Merge pull request #63 from pkg/fixedbugs/5
  > Merge pull request #61 from pkg/add-travis
  > Merge pull request #62 from xiu/bugfix/35-cant-remove-directories-on-servu
  > Merge pull request #58 from sykesm/client-realpath
  > Merge pull request #60 from pkg/fixedbugs/57
  > updated contribution guidelines

Signed-off-by: James Myers <jmyers@pivotal.io>
robertjsullivan added a commit that referenced this issue Mar 12, 2020
Submodule src/github.com/square/certstrap b61237589..bd8b02aa8:
  > Bump Go to 1.13
  > Merge pull request #85 from square/mbyczkowski/better-versioning
  > Merge pull request #81 from square/mbyczkowski/option-for-output
  > Merge pull request #80 from square/dependabot/go_modules/github.com/urfave/cli-1.21.0
  > Merge pull request #79 from square/mbyczkowski/godoc-badge
  > Merge pull request #71 from julianedwards/extractname
  > Merge pull request #77 from julianedwards/newtemplates
  > Merge pull request #73 from defer/ft-text-fixes
  > Merge pull request #69 from square/cs/dockerfile
  > Merge pull request #68 from square/cs/update-cli
  > Merge pull request #67 from square/cs/mod

Co-authored-by: Edwin Xie <exie@pivotal.io>
mariash added a commit that referenced this issue May 23, 2023
…emitter

Submodule src/code.cloudfoundry.org/auction 20b5e4eb0..46a89db7a:
  > Support distributed tracing (#11)
Submodule src/code.cloudfoundry.org/auctioneer c72115836..be803dc1e:
  > Support distributed tracing (#13)
Submodule src/code.cloudfoundry.org/bbs 08ff19fb9..f3900dad6:
  > Merge branch 'klapkov-routing_info'
  > Support distributed tracing (#67)
Submodule src/code.cloudfoundry.org/cfdot 9c6e1b442..ef69fb66b:
  > Support distributed tracing (#7)
Submodule src/code.cloudfoundry.org/diego-ssh 62afa2597..1bdd96ca0:
  > Update bbs client calls with trace ID (#48)
Submodule src/code.cloudfoundry.org/executor b452e62e8..1c64887c7:
  > Support distributed tracing (#79)
Submodule src/code.cloudfoundry.org/inigo 3c8b1a8f8..c1bc9f5d9:
  > Run trace id test in inigo (#26)
Submodule src/code.cloudfoundry.org/rep 0de86f227..b9d5c98b7:
  > Support distributed tracing (#42)
Submodule src/code.cloudfoundry.org/route-emitter df496f44d..b6a50dc67:
  > Log trace ID when handling events
  > Pass trace ID from event to bbs client
  > Remove unneeded check
  > Update watcher tests and add new case for the fallback
  > Add fallback to old BBS endpoint
  > Use routing_info bbs endpoint when syncing
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

No branches or pull requests

4 participants