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

Where to host cs.k8s.io #2182

Open
ameukam opened this issue Mar 9, 2021 · 53 comments · Fixed by #3529 or #6657
Open

Where to host cs.k8s.io #2182

ameukam opened this issue Mar 9, 2021 · 53 comments · Fixed by #3529 or #6657
Labels
lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. sig/contributor-experience Categorizes an issue or PR as relevant to SIG Contributor Experience. sig/k8s-infra Categorizes an issue or PR as relevant to SIG K8s Infra.

Comments

@ameukam
Copy link
Member

ameukam commented Mar 9, 2021

https://cs.k8s.io is running on a baremetal server provided by Equinix Metal(ex Packet) under CNCF budget and operated until now by @dims.

The question was asked about whether or not we should host CodeSearch on aaa cluster.

Ref: https://kubernetes.slack.com/archives/CCK68P2Q2/p1615204807111900?thread_ts=1615189697.108500&cid=CCK68P2Q2

Issue open to track the discussions and the consensus about this.

@nikhita
Copy link
Member

nikhita commented Mar 9, 2021

@dims where is the original source code for cs.k8s.io? 👀

@nikhita
Copy link
Member

nikhita commented Mar 9, 2021

/wg k8s-infra

@ameukam
Copy link
Member Author

ameukam commented Mar 9, 2021

/sig contributor-experience
/priority backlog

/assign @spiffxp
cc @mrbobbytables @alisondy @cblecker @munnerz

@ameukam
Copy link
Member Author

ameukam commented Mar 9, 2021

@dims where is the original source code for cs.k8s.io? eyes

@nikhita You can find the config here https://github.com/dims/k8s-code.appspot.com/

@BenTheElder
Copy link
Member

What's the argument against hosting it on AAA?

@dims
Copy link
Member

dims commented Mar 11, 2021

@BenTheElder nothing other than someone has to do it :) oh, i don't know how to wire the ingress/dns stuff

i tried a long time ago :) #96

@ameukam
Copy link
Member Author

ameukam commented Mar 11, 2021

What's the argument against hosting it on AAA?

I would say lack of artifact destined for aaa (aka no up-to-date container image for hound). We could host the image on k8s-staging-infra-tools.

@nikhita
Copy link
Member

nikhita commented Mar 24, 2021

@ameukam should this issue be migrated to the k/k8s.io repo?

@ameukam
Copy link
Member Author

ameukam commented Mar 24, 2021

@nikhita I'm not sure about the right place of this issue. Just wanted to put this under SIG Contribex TLs and Chairs radar.

@BenTheElder
Copy link
Member

it should be under k/k8s.io imho. I think we should host it on AAA fwiw.

@nikhita
Copy link
Member

nikhita commented Jun 10, 2021

Moving to k8s.io repo. slack discussion - https://kubernetes.slack.com/archives/CCK68P2Q2/p1623300972130500

@nikhita nikhita transferred this issue from kubernetes/community Jun 10, 2021
@spiffxp
Copy link
Member

spiffxp commented Aug 9, 2021

/sig contributor-experience
/wg k8s-infra

@k8s-ci-robot k8s-ci-robot added sig/contributor-experience Categorizes an issue or PR as relevant to SIG Contributor Experience. wg/k8s-infra labels Aug 9, 2021
@jimdaga
Copy link

jimdaga commented Aug 11, 2021

I took a stab at onboarding codesearch; @spiffxp could I get your input? I want to make sure I didn't miss anything.
I want to stage all the infra, and get it deployed via prow first. Then we can follow up with another PR to cut-over DNS when we are ready.

#2513
kubernetes/test-infra#23201

I could also work on adding the docker build logic after, but I haven't worked in that repo yet so I'll have to do some digging.

cc @dims

@spiffxp
Copy link
Member

spiffxp commented Aug 17, 2021

/priority important-soon
/milestone v1.23

@k8s-ci-robot k8s-ci-robot added the priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. label Aug 17, 2021
@k8s-ci-robot k8s-ci-robot added this to the v1.23 milestone Aug 17, 2021
@justaugustus
Copy link
Member

What about using https://sourcegraph.com/kubernetes to minimize the maintenance burden here?
This is something I suggested to @dims in the past, but didn't have the bandwidth to do at the time.

@dims
Copy link
Member

dims commented Aug 19, 2021

choices are:

  1. leave things where they are
  2. move to k8s wg infra
  3. redirect to cs.k8s.io to sourcegraph
  • i have been taking care of 1 already for a while with minimal downtime, so i am ok with continuing to do so
  • if someone wants to do 2, i am happy to work with help, show how things are setup and we can shut down the equinix vm
  • i personally don't like option 3, i love the the hound UX, if the consensus is we should go with 3, that is fine with me. I am happy to run a personal instance on a custom domain for myself (community is welcome to use)

if i missed any other options, please feel free to chime in.

@spiffxp
Copy link
Member

spiffxp commented Sep 17, 2021

/unassign

@Priyankasaggu11929 Priyankasaggu11929 removed their assignment Oct 4, 2022
@upodroid
Copy link
Member

Can we explore deprecating this in favour of GitHub Code Search?

https://cs.github.com/
https://github.com/features/code-search

@ameukam
Copy link
Member Author

ameukam commented Dec 16, 2022

Not really. one issue is that Github CS requires auth to use it while we have anonymous queries using cs.k8s.io (e.g. https://go.k8s.io/owners/dims).

@k8s-triage-robot
Copy link

The Kubernetes project currently lacks enough contributors to adequately respond to all issues.

This bot triages un-triaged issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue as fresh with /remove-lifecycle stale
  • Close this issue with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Mar 16, 2023
@BenTheElder
Copy link
Member

/lifecycle frozen
closing this is not helpful, unless we've also shut down the existing infra.

@k8s-ci-robot k8s-ci-robot added lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Mar 17, 2023
@ameukam ameukam removed this from the v1.26 milestone Jan 23, 2024
@ameukam ameukam removed the priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. label Jan 23, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in SIG K8S Infra Apr 10, 2024
@ameukam ameukam reopened this Apr 10, 2024
@ameukam
Copy link
Member Author

ameukam commented Apr 18, 2024

/assign @SohamChakraborty

@k8s-ci-robot
Copy link
Contributor

@ameukam: GitHub didn't allow me to assign the following users: SohamChakraborty.

Note that only kubernetes members with read permissions, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time.
For more information please see the contributor guide

In response to this:

/assign @SohamChakraborty

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.

@SohamChakraborty
Copy link
Contributor

I think this is now ready for migration from the bare metal server to aaa cluster. I spoke with Arnaud and he will decide on a path for migration.

@BenTheElder
Copy link
Member

We will need to raise this to
/priority important-soon
Given https://www.theregister.com/2024/11/18/equinix_ends_metal_iaas/

We have time, and other pressing work, but we need to be tracking moving it.

IMHO: while there are other options out there, we have a lot of links to this in issues etc, if it's cheap to operate I think we should keep it for now and just lift it over to AAA cluster or similar.

What's left for this one?

@k8s-ci-robot k8s-ci-robot added the priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. label Dec 3, 2024
@BenTheElder
Copy link
Member

https://cs-canary.k8s.io/ seems to have performance issues but also maybe isn't running the same version:

https://cs.k8s.io/?q=NodeStageVolume&i=nope&files=&excludeFiles=&repos=
Screenshot 2024-12-03 at 3 24 56 PM

https://cs-canary.k8s.io/?q=NodeStageVolume&i=nope&files=&excludeFiles=&repos=
Screenshot 2024-12-03 at 3 24 59 PM

(note the response times)

@ameukam
Copy link
Member Author

ameukam commented Dec 4, 2024

Maybe run this inside a GCP MIG or a AWS ASG ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. sig/contributor-experience Categorizes an issue or PR as relevant to SIG Contributor Experience. sig/k8s-infra Categorizes an issue or PR as relevant to SIG K8s Infra.
Projects
Status: Done
Status: Needs Triage