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

Recompile plugin with steampipe-plugin-sdk v5.10.1 #225

Merged
merged 1 commit into from
May 31, 2024

Conversation

burahimu
Copy link
Contributor

@burahimu burahimu commented May 26, 2024

By updating the sdk version the following bug fix is included.

After update, I don't have the runtime error anymore. It fixes #224

@burahimu burahimu changed the title chore(steampipe plugin): update sdk version fix(steampipe plugin): update sdk version May 26, 2024
@burahimu burahimu changed the title fix(steampipe plugin): update sdk version fix(steampipe sdk): update sdk version May 26, 2024
@burahimu burahimu changed the title fix(steampipe sdk): update sdk version fix(steampipe sdk): update version May 26, 2024
@burahimu
Copy link
Contributor Author

burahimu commented May 26, 2024

question: is there any doc that says how to run tests?

I want to include the Connection Key Column context_name, when running an SQL query, to ensure that everything works.

@ParthaI
Copy link
Contributor

ParthaI commented May 27, 2024

Hello @burahimu, I apologize for the inconvenience. There was a bug in Steampipe SDK v5.10.0 which has been fixed in v5.10.1. Thank you for submitting the PR and for your contribution.

We don't have a proper doc to test connection key quals yet.
We need to run the query as follows:

  • In your ~/.steampipe/config/kubernetes.spc file must have aggregator connection setup.
  • Please try the following query against the aggregator connection to test it out.
select
  hc.name as chart_name,
  hc.type as chart_type,
  hc.version as chart_version,
  hr.name as release_name,
  hr.status as release_status,
  hr.version as deployment_version,
  hr.first_deployed as deployed_at,
  hr.config as deployment_config
from
  agg_con.helm_chart as hc
  left join agg_con.helm_release as hr
    on hc.name = hr.chart_name
where
  hr.status = 'deployed'
  and hc.context_name = hr.context_name;

Note: agg_con is an aggregator connection for the kubernetes plugin.

@misraved misraved changed the title fix(steampipe sdk): update version Recompile plugin with steampipe-plugin-sdk v5.10.1 May 27, 2024
@burahimu
Copy link
Contributor Author

Hello @burahimu, I apologize for the inconvenience. There was a bug in Steampipe SDK v5.10.0 which has been fixed in v5.10.1. Thank you for submitting the PR and for your contribution.

We don't have a proper doc to test connection key quals yet. We need to run the query as follows:

  • In your ~/.steampipe/config/kubernetes.spc file must have aggregator connection setup.
  • Please try the following query against the aggregator connection to test it out.
select
  hc.name as chart_name,
  hc.type as chart_type,
  hc.version as chart_version,
  hr.name as release_name,
  hr.status as release_status,
  hr.version as deployment_version,
  hr.first_deployed as deployed_at,
  hr.config as deployment_config
from
  agg_con.helm_chart as hc
  left join agg_con.helm_release as hr
    on hc.name = hr.chart_name
where
  hr.status = 'deployed'
  and hc.context_name = hr.context_name;

Note: agg_con is an aggregator connection for the kubernetes plugin.

Thanks for the reply.

It's proper to the developer to have a kube context available? Is it not integrated to run a local cluster (i.e: kind)

@ParthaI
Copy link
Contributor

ParthaI commented May 30, 2024

Hi @burahimu,

  • Yes, a developer typically needs to have a kube context available to interact with any Kubernetes cluster, whether it is a local cluster or a remote one.
  • The kube context is necessary for kubectl and other Kubernetes tools to know which cluster to interact with and which credentials to use.
  • When you create a cluster with kind, it automatically updates your kubeconfig file with the context for the new cluster.
  • You can see the context details for the local cluster with the help of kubectl config get-contexts command.

I haven't experimented much with local clusters yet, but feel free to give it a try.

Thanks!

@misraved misraved merged commit df51f28 into turbot:main May 31, 2024
1 check passed
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

Successfully merging this pull request may close these issues.

Having runtime error when using context_name column
3 participants