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

Drop redundant prowjob namespace #3160

Merged
merged 1 commit into from
Jun 4, 2018
Merged

Drop redundant prowjob namespace #3160

merged 1 commit into from
Jun 4, 2018

Conversation

0xmichalis
Copy link
Contributor

@0xmichalis 0xmichalis commented Jun 4, 2018

/assign CecileRobertMichon

The namespace is already defaulted to "default" but we don't use prowjobs so we don't really care about this config option.

@CecileRobertMichon
Copy link
Contributor

/lgtm
/approve

@acs-bot
Copy link

acs-bot commented Jun 4, 2018

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: CecileRobertMichon

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

@codecov
Copy link

codecov bot commented Jun 4, 2018

Codecov Report

❗ No coverage uploaded for pull request base (master@f15fbc2). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #3160   +/-   ##
=========================================
  Coverage          ?   51.87%           
=========================================
  Files             ?      102           
  Lines             ?    15199           
  Branches          ?        0           
=========================================
  Hits              ?     7884           
  Misses            ?     6591           
  Partials          ?      724

@acs-bot acs-bot merged commit 79adb4d into Azure:master Jun 4, 2018
@acs-bot
Copy link

acs-bot commented Jun 4, 2018

@Kargakis: Updated the config configmap from file .prowci/config.yaml

In response to this:

/assign CecileRobertMichon

The namespace is already defaulted to "default" but we don't use prowjobs so we don't really care about this config option.

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.

@ghost ghost removed the in progress label Jun 4, 2018
@0xmichalis
Copy link
Contributor Author

0xmichalis commented Jun 4, 2018 via email

@0xmichalis 0xmichalis deleted the prow branch June 5, 2018 07:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants