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

Unable to run start-build after restarting my server #1650

Closed
smarterclayton opened this issue Apr 8, 2015 · 10 comments
Closed

Unable to run start-build after restarting my server #1650

smarterclayton opened this issue Apr 8, 2015 · 10 comments
Assignees
Labels
kind/bug Categorizes issue or PR as related to a bug.
Milestone

Comments

@smarterclayton
Copy link
Contributor

I restarted my server after running a few builds, and now start-build is broken:

$ osc start-build ruby-sample-build
F0408 18:52:44.617205   24019 startbuild.go:41] Error received from API: build "ruby-sample-build-3" already exists

This should never happen

$ osc get bc -o yaml
apiVersion: v1beta1
creationTimestamp: null
items:
- apiVersion: v1beta1
  kind: BuildConfig
  lastVersion: 4
  metadata:
    creationTimestamp: 2015-04-08T16:48:25Z
    labels:
      name: ruby-sample-build
      template: application-template-stibuild
    name: ruby-sample-build
    namespace: default
    resourceVersion: "16430"
    selfLink: /osapi/v1beta1/buildConfigs/ruby-sample-build?namespace=default
    uid: 12eb6b3c-de0f-11e4-9cef-080027c5bfa9
  parameters:
    output:
      to:
        kind: ImageStream
        name: origin-ruby-sample
    source:
      git:
        uri: git://github.com/openshift/ruby-hello-world.git
      type: Git
    strategy:
      stiStrategy:
        builderImage: openshift/ruby-20-centos7
        image: openshift/ruby-20-centos7
      type: STI
  triggers:
  - github:
      secret: secret101
    type: github
  - generic:
      secret: secret101
    type: generic
  - imageChange:
      from:
        name: ruby-20-centos7
      image: openshift/ruby-20-centos7
      imageRepositoryRef:
        name: ruby-20-centos7
      lastTriggeredImageID: openshift/ruby-20-centos7:latest
      tag: latest
    type: imageChange
kind: List
resourceVersion: 16463
$ osc get builds -o yaml
apiVersion: v1beta1
creationTimestamp: null
items:
- apiVersion: v1beta1
  completionTimestamp: 2015-04-08T18:50:12Z
  config:
    kind: BuildConfig
    name: ruby-sample-build
    namespace: default
  kind: Build
  metadata:
    creationTimestamp: 2015-04-08T18:43:53Z
    labels:
      buildconfig: ruby-sample-build
      name: ruby-sample-build
      template: application-template-stibuild
    name: ruby-sample-build-3
    namespace: default
    resourceVersion: "16223"
    selfLink: /osapi/v1beta1/builds/ruby-sample-build-3?namespace=default
    uid: 34b7250c-de1f-11e4-aca4-080027c5bfa9
  parameters:
    output:
      dockerImageReference: 172.30.17.133:5000/default/origin-ruby-sample
      imageTag: default/origin-ruby-sample
      registry: 172.30.17.133:5000
      to:
        kind: ImageStream
        name: origin-ruby-sample
    source:
      git:
        uri: git://github.com/openshift/ruby-hello-world.git
      type: Git
    strategy:
      stiStrategy:
        builderImage: openshift/ruby-20-centos7:latest
        image: openshift/ruby-20-centos7:latest
      type: STI
  status: Failed
- apiVersion: v1beta1
  completionTimestamp: 2015-04-08T18:50:12Z
  config:
    kind: BuildConfig
    name: ruby-sample-build
    namespace: default
  kind: Build
  metadata:
    creationTimestamp: 2015-04-08T18:46:22Z
    labels:
      buildconfig: ruby-sample-build
      name: ruby-sample-build
      template: application-template-stibuild
    name: ruby-sample-build-4
    namespace: default
    resourceVersion: "16216"
    selfLink: /osapi/v1beta1/builds/ruby-sample-build-4?namespace=default
    uid: 8d553178-de1f-11e4-aca4-080027c5bfa9
  parameters:
    output:
      dockerImageReference: 172.30.17.133:5000/default/origin-ruby-sample
      imageTag: default/origin-ruby-sample
      registry: 172.30.17.133:5000
      to:
        kind: ImageStream
        name: origin-ruby-sample
    source:
      git:
        uri: git://github.com/openshift/ruby-hello-world.git
      type: Git
    strategy:
      stiStrategy:
        builderImage: openshift/ruby-20-centos7:latest
        image: openshift/ruby-20-centos7:latest
      type: STI
  status: Failed
kind: List
resourceVersion: 16492
@smarterclayton smarterclayton added this to the 0.5.0 (beta3) milestone Apr 8, 2015
@smarterclayton smarterclayton added the kind/bug Categorizes issue or PR as related to a bug. label Apr 8, 2015
@bparees bparees assigned soltysh and unassigned bparees Apr 8, 2015
@bparees
Copy link
Contributor

bparees commented Apr 8, 2015

that's pretty odd since the BC specifically shows LastVersion=4.

@smarterclayton
Copy link
Contributor Author

Can a controller update that? It's possible the controller was still processing on restart when I ran the command (it shouldn't have failed to start the build though).

----- Original Message -----

that's pretty odd since the BC specifically shows LastVersion=4.


Reply to this email directly or view it on GitHub:
#1650 (comment)

@bparees
Copy link
Contributor

bparees commented Apr 8, 2015

I don't think the controller should be updating it, it should be updated when we create the build which is now the responsibility of the instantiate/clone endpoint. but @soltysh is the expert here since that's all his new code.

i guess i'm wondering if you'd run start-build a second time, what would have happened. (would it have gotten "5" as i'd expect? and if so, who got "4"?)

@brenton
Copy link
Contributor

brenton commented Apr 20, 2015

@bparees, did something land in master to address this? I'm not seeing this issue in the latest OSE builds.

@bparees
Copy link
Contributor

bparees commented Apr 20, 2015

we got some etcd fixes, maybe that helped?

@bparees
Copy link
Contributor

bparees commented Apr 20, 2015

otherwise no, nothing that i know of was specifically targeted at or changed this flow.

@smarterclayton
Copy link
Contributor Author

This might have been fixed by andy's change to the trigger.

----- Original Message -----

otherwise no, nothing that i know of was specifically targeted at or changed
this flow.


Reply to this email directly or view it on GitHub:
#1650 (comment)

@ncdc
Copy link
Contributor

ncdc commented Apr 20, 2015

My change to the trigger ensures that the From image stream matches on both namespace and name, defaulting the namespace to the bc's namespace if it's not specified. Without my fix, builds would be created endlessly if you have 2 streams with the same name but in different namespaces. I'm not sure it would fix this?

@bparees
Copy link
Contributor

bparees commented Apr 20, 2015

@smarterclayton you mean the image change controller fix? what @ncdc just said as i was writing this. :)

@smarterclayton
Copy link
Contributor Author

Closing based on unreproducibility

jboyd01 pushed a commit to jboyd01/origin that referenced this issue Feb 6, 2018
…service-catalog/' changes from d969acde90..b69b4a6c80

b69b4a6c80 origin build: modify hard coded path
527fac4d02 origin build: add origin tooling
545ffdb chart changes for v0.1.5 release (openshift#1709)
4d9be8f Use userInfo for Originating-Identity so extras is correct. (openshift#1702)
f358b99 Call destroy function on each storage interface (openshift#1705)
36b5de9 refactor binding reconciliation functions (openshift#1687)
5699360 Change binding_retrievable to bindingRetrievable
0d8bcfe thread through stopCh to DestroyFunc (openshift#1671)
1c45aef Migrate from glide to dep for dependency management (openshift#1670)
1cf0dd9 Add svcat to Makefile (openshift#1683)
45b1013 make verify validates that versioned APIs contain json tags for fields, addresses openshift#1303 (openshift#1480)
0ee8398 Build the integration test binary before running any tests (openshift#1666)
0fe0aa7 Update design.md (openshift#1674)
1280d24 controller requires permission to update secrets (openshift#1663)
129d98e Contribute svcat (openshift#1664)
ff9739b Update dependencies to kubernetes-1.9.1 (openshift#1633)
9c36019 chart changes for v0.1.4 release (openshift#1669)
93319f6 move apiserver generation to script and verify (openshift#1662)
385f0da refactor service instance provision/update/poll reconciliation (openshift#1648)
e015212 run each integration test individually (openshift#1661)
412e242 Tell people whether we're checking external hrefs (openshift#1659)
ae05361 retry failed unbind requests (openshift#1653)
7eae845 doc for setting up Service Catalog with Prometheus metrics (openshift#1654)
0720cf9 minor README copy edit (openshift#1656)
8bd347d run some integration subtests in parallel (openshift#1637)
b83800c Use $ and console to indicate multi-command blocks
789c4b2 Use dynamic reaction to fix data race (openshift#1650)
f1be763 only check external hrefs on master (openshift#1652)
65c6d20 Controller-manager crash loops if API server is not available on startup (openshift#1376) (openshift#1591)
9225c92 embedded etcd is the way of the future for our tests (openshift#1651)
605c952 Fix required fields in OpenAPI schema (openshift#1602)
899ca21 Revert "Switch to wget for integration apiserver checks (openshift#1384)" (openshift#1585)
2f496ee Update code-of-conduct.md (openshift#1635)
c1c69cf Build the e2e binary in CI (openshift#1647)
4e2dcef Wait for successful API discovery (openshift#1646)
5ae6d99 Bump copyright date in generated code (openshift#1645)
8be5b05 Serve OpenAPI spec only when --serve-openapi-spec switch is enabled (openshift#1612)
19fb30e silence go-restful logging output (openshift#1622)
fdbabf0 Add walkthrough link back (openshift#1620)
7c73e9a Add link to main k8s docs on service-catalog (openshift#1627)
f59adc9 Overhauling the design document (openshift#1619)
cd7b633 Updating the install documentation (openshift#1616)
f6e5441 fix compilation error from updated util.WaitForBindingCondition() (openshift#1629) (openshift#1631)
54e57af Provide OSB Client proxy to instrument with metrics (openshift#1615)
026b86f Disable test added in 1611 that contains data race (openshift#1626)
cb735a6 Add integration tests for ServiceInstances (openshift#1611)
67dbabb Cleaning up the docs README (openshift#1618)
6bddc07 remove email from docker login during Travis deploy (openshift#1614)
a604bc3 Use ConfigMaps for leader election (openshift#1599)
c6f193a Add controller integration tests for ServiceInstance create and update (openshift#1578)
26cf23b Rename OWNERS assignees: to approvers: (openshift#1508)
1163edc expose Prometheus metrics from Controller (openshift#677) (openshift#1608)
2cd6554 Clean up docs/ folder (openshift#1609)
1d7e96d Adding Service Binding Create Integration Tests (openshift#1580)
6a4c469 Make the maximum polling backoff configurable (openshift#1607)
31bbf55 Rename the imported package to avoid name conflict (openshift#1603)
3cdd556 Add validation for broker spec to SAR admission controller (openshift#1605)
a3408ce fix docker volume mount when building with docker under SELinux (openshift#1500) (openshift#1534)
307e747 Remove unneeded vendors of vendors (openshift#1596)
770fc74 Make ups-instance.yaml in walkthrough to demonstrate good practices (openshift#1592)
9112ba1 Add links to docs/README (openshift#1589)
8902648 Add additional service to ups-broker to fix e2e (openshift#1583)
0bcbc7d move instance update logic out of reconcileServiceInstanceDelete (openshift#1584)
7ef5a3e Do not send Parameters field when there are no parameters from sourced secret (openshift#1559)
4c51b25 Remove unneeded code that sets reason for provision/update call failure (openshift#1561)
b122cb9 fix bind injection failure not being persisted in API server (openshift#1546)
66421d5 Clear out current operation when starting reconciliation of a delete (openshift#1564)
8cca70a Send an empty object for Parameters when deleting all parameters of a ServiceInstance (openshift#1555)
270426c Add controllerTest type as a helper for running controller integration tests. (openshift#1577)
e26c2d7 Ignore .vscode folder in project root (openshift#1579)
REVERT: d969acde90 Add additional service to ups-broker to fix e2e (openshift#1583)
REVERT: 1bcd53b684 origin build: add origin tooling

git-subtree-dir: cmd/service-catalog/go/src/github.com/kubernetes-incubator/service-catalog
git-subtree-split: b69b4a6c8003f25d040e3087c7b1b16d1854a9e9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

No branches or pull requests

5 participants