Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

WIP: Bazel #700

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

WIP: Bazel #700

wants to merge 14 commits into from

Conversation

simonswine
Copy link
Contributor

No description provided.

@jetstack-bot
Copy link
Collaborator

@simonswine: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@jetstack-bot jetstack-bot added the do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. label Jan 23, 2019
@simonswine simonswine self-assigned this Jan 23, 2019
@jetstack-bot jetstack-bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. dco-signoff: no Indicates that at least one commit in this pull request is missing the DCO sign-off message. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Jan 23, 2019
@jetstack-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: simonswine

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jetstack-bot jetstack-bot added approved Indicates a PR has been approved by an approver from all required OWNERS files. area/puppet Indicates a PR is affecting Puppet manifests area/terraform Indicates a PR is affecting Terraform configuration labels Jan 23, 2019
@simonswine simonswine closed this Jan 24, 2019
@simonswine simonswine reopened this Jan 24, 2019
@simonswine
Copy link
Contributor Author

@JoshVanL @MattiasGees Can you give that a try:

bazel build //cmd/...
./bazel-bin/cmd/tarmak/(darwin|linux)_amd64_pure_stripped/tarmak version
./bazel-bin/cmd/tarmak/(darwin|linux)_amd64_pure_stripped/tarmak cluster apply --wing-dev-mode 

/unassign
/assign @JoshVanL @MattiasGees

@JoshVanL
Copy link
Contributor

/unassign

@jetstack-bot jetstack-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 29, 2019
@jetstack-bot
Copy link
Collaborator

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.

📝 Please follow instructions in the contributing guide to update your commits with the DCO

Full details of the Developer Certificate of Origin can be found at developercertificate.org.

The list of commits missing DCO signoff:

  • 82c7351 Use bazel for building
  • d52e09c generate vendor rules
  • 63f6c8f regenerated using gazelle
  • 44628d7 Upgrade to golang 1.10.8
  • 8dbe9bc Ignore recursive symlinks in kevinburke's repo
  • f9b0f3d Ensure to include all important puppet files
  • 89d4859 Add tags file
  • 2b5147c Add another missing BUILD.bazel
  • a96087b Correct generate instructions
  • 3008687 Support building for darwin targets
  • 8f2ea98 Add GHR for publishing release to github
  • 41bd8a1 Upgrade UPX to 3.95
  • ee0fd6b Add a publish script
  • e75e386 Add missing bazel files

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jetstack-bot jetstack-bot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Jan 30, 2019
@jetstack-bot
Copy link
Collaborator

@simonswine: PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@jetstack-bot
Copy link
Collaborator

@simonswine: The following test failed, say /retest to rerun them all:

Test name Commit Details Rerun command
tarmak-puppet-module-tarmak-acceptance-1-14-centos e75e386 link /test puppet-tarmak-acceptance-centos v1.14

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. area/puppet Indicates a PR is affecting Puppet manifests area/terraform Indicates a PR is affecting Terraform configuration dco-signoff: no Indicates that at least one commit in this pull request is missing the DCO sign-off message. do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants